? Success

User tests: Successful: Unsuccessful:

avatar infograf768
infograf768
10 Dec 2016

Issue: #13142

See description there.

Summary of Changes

Adding a special case when associating com_users menu items (login. logout, password reset, reminder, registration, etc.)
EDIT: as we only need the itemid and lang, simplified code.

Testing Instructions

Install a multilingual site with associations.
Make sure Add Alternate Meta Tags is set to Yes in the languagefilter parameter.
Set SEF to on.
Create various menu items with different aliases using com_users menu item in each language and associate them.
Display these menu items in frontend.
Check the source of the page.

Before patch, the alternate will always display the same url in the alternate for all languages.
(I have also set here one of the login menu item as child of another different menu item)

screen shot 2016-12-10 at 08 33 51

After patch
Now the alternate are correct

screen shot 2016-12-10 at 08 33 05

Documentation Changes Required

None. This is solving a bug due, I guess, to the com_users router

@ioweb-gr @mbabker @andrepereiradasilva

Thank you for testing.

avatar infograf768 infograf768 - open - 10 Dec 2016
avatar infograf768 infograf768 - change - 10 Dec 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 10 Dec 2016
Category Front End Plugins
avatar infograf768 infograf768 - change - 10 Dec 2016
Labels Added: ?
avatar infograf768 infograf768 - edited - 10 Dec 2016
avatar infograf768 infograf768 - change - 10 Dec 2016
Category Front End Plugins Front End Multilanguage Plugins
avatar gwsdesk
gwsdesk - comment - 10 Dec 2016

Trying to understand the testing instructions. Don't sure that I understand them so is here a more 'practical' explanation so me dumb blond can understand it and test? I simply do not know what to look for at present guidelines. Thanks JM

avatar infograf768 infograf768 - change - 10 Dec 2016
The description was changed
avatar infograf768 infograf768 - change - 10 Dec 2016
The description was changed
avatar infograf768 infograf768 - edited - 10 Dec 2016
avatar infograf768
infograf768 - comment - 10 Dec 2016

@gwsdesk Forgot to say that SEF should be on to test this.
Also the menu items alias should be different depending on language.

avatar AlexRed
AlexRed - comment - 10 Dec 2016

I have tested this item successfully on 9a67290

Patch ok.


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

avatar AlexRed AlexRed - test_item - 10 Dec 2016 - Tested successfully
avatar infograf768 infograf768 - change - 10 Dec 2016
The description was changed
avatar infograf768 infograf768 - edited - 10 Dec 2016
avatar gwsdesk
gwsdesk - comment - 10 Dec 2016

I have tested this item successfully on 9a67290

"Also the menu items alias should be different depending on language" Did it for me...sorry for being blond.

Thanks for clarifying. Works4me JM as well now.

Thanks for the patch !


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

avatar gwsdesk gwsdesk - test_item - 10 Dec 2016 - Tested successfully
avatar roland-d roland-d - change - 10 Dec 2016
Status Pending Ready to Commit
avatar roland-d
roland-d - comment - 10 Dec 2016

RTC as we have 2 successful tests


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

avatar zero-24 zero-24 - change - 10 Dec 2016
Milestone Added:
avatar rdeutz rdeutz - change - 11 Dec 2016
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-12-11 10:49:01
Closed_By rdeutz
Labels Added: ?
avatar rdeutz rdeutz - close - 11 Dec 2016
avatar rdeutz rdeutz - merge - 11 Dec 2016
avatar rdeutz rdeutz - reference | 8e20e66 - 11 Dec 16
avatar rdeutz rdeutz - merge - 11 Dec 2016
avatar rdeutz rdeutz - close - 11 Dec 2016
avatar infograf768 infograf768 - head_ref_deleted - 11 Dec 2016
avatar cpfeifer cpfeifer - reference | fb3e6ba - 22 Dec 16

Add a Comment

Login with GitHub to post a comment