?
avatar infograf768
infograf768
4 Aug 2017

Steps to reproduce the issue

Create a multingual site with staging

Expected result (test on a 3.7.4)

screen shot 2017-08-04 at 10 52 30

Actual result (on staging)

screen shot 2017-08-04 at 10 58 11

@izharaazmi
Could it be a consequence of one of your PRs?

avatar infograf768 infograf768 - open - 4 Aug 2017
avatar joomla-cms-bot joomla-cms-bot - change - 4 Aug 2017
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 4 Aug 2017
avatar franz-wohlkoenig franz-wohlkoenig - change - 4 Aug 2017
Category com_associations com_menus Multilanguage
avatar izharaazmi
izharaazmi - comment - 4 Aug 2017

Not sure but as I remember, I didn't add support for icons display yet.
This can be added now.

avatar infograf768
infograf768 - comment - 4 Aug 2017

This is what we had before in enabled:

if ($menuType->home == 0)
		{
			$titleicon = '';
		}
		elseif ($menuType->home == 1 && $menuType->language == '*')
		{
			$titleicon = ' <span class="icon-home" aria-hidden="true"></span>';
		}
		elseif ($menuType->home > 1)
		{
			$titleicon = ' <span>'
				. JHtml::_('image', 'mod_languages/icon-16-language.png', $menuType->home, array('title' => JText::_('MOD_MENU_HOME_MULTIPLE')), true)
				. '</span>';
		}
		elseif ($menuType->image && JHtml::_('image', 'mod_languages/' . $menuType->image . '.gif', null, null, true, true))
		{
			$titleicon = ' <span>' .
				JHtml::_('image', 'mod_languages/' . $menuType->image . '.gif', $alt, array('title' => $menuType->title_native), true) . '</span>';
		}
		else
		{
			$titleicon = ' <span class="label" title="' . $menuType->title_native . '">' . $menuType->sef . '</span>';
		}

avatar izharaazmi
izharaazmi - comment - 4 Aug 2017

Well okay. I'll try to look into it this weekend.

avatar franz-wohlkoenig franz-wohlkoenig - change - 4 Aug 2017
Status New Discussion
avatar infograf768
infograf768 - comment - 12 Aug 2017

@izharaazmi

We really can't release 3.8 with this bug.
Honestly I prefer reverting #16451 than go without the flags.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/17406.

avatar izharaazmi
izharaazmi - comment - 12 Aug 2017

Sorry. I'll look at this today.

avatar izharaazmi
izharaazmi - comment - 12 Aug 2017

@infograf768 I have done the fixes. I was trying to test with multiple menu
items but stuck. I am not able to create default menu item for different
languages.

I tried looking at the code but I don't understand it at –
libraries/src/Table/Menu.php:236

The entire if block (if ($this->home == '1')) does not look like doing what
it says. Can you please look and confirm. Also please advise if I might be
missing on something with multilingual menu homes.

avatar infograf768
infograf768 - comment - 13 Aug 2017

Closing as we have a patch
#17511

avatar infograf768 infograf768 - change - 13 Aug 2017
Status Discussion Closed
Closed_Date 0000-00-00 00:00:00 2017-08-13 06:03:02
Closed_By infograf768
avatar infograf768 infograf768 - close - 13 Aug 2017

Add a Comment

Login with GitHub to post a comment