?
Referenced as Pull Request for: # 12629
avatar infograf768
infograf768
28 Oct 2016

Steps to reproduce the issue

Create a contact for the super user
Create some articles
Create a menu item to display one or multiple articles, do NOT make it the home page.
Create a menu item displaying the Contact category to which belongs the super user contact: alias is english-contacts in this case.
Do NOT create a menu item displaying the single contact.

Display the article in frontend.
screen shot 2016-10-28 at 08 14 06

Click on the Super User link in the article intro

Expected result

We should imho get an url containing the alias of the menu item for the contact category and using its Itemid.
Something in my case like:
/english-contacts/1.html

Testing that url DOES display the contact correctly.

Actual result

When using legacy router for the contact component, we get:

/component/contact/contact/1.html?Itemid=103

When using the Modern router (Ids off), we get

/component/contact/contact/1-contact.html?Itemid=103

In both cases
1.html or 1-contact.html correspond to the id of the contact
Itemid= 103 corresponds to the Home page Itemid, not even to the Article menu item...

Note: the example here is from a multilingual site but we get the same on a monolanguage one.

avatar infograf768 infograf768 - open - 28 Oct 2016
avatar infograf768
infograf768 - comment - 28 Oct 2016

@Hackwar and other routers specialists

avatar Hackwar
Hackwar - comment - 28 Oct 2016

I agree, the URL should look the way you wrote. Can you make sure that menu item and category have the right permissions and that they are published? I will try to replicate this on my end.

avatar zero-24 zero-24 - change - 28 Oct 2016
Labels Added: ?
avatar infograf768
infograf768 - comment - 28 Oct 2016

yes, correct permissions and published.

avatar brianteeman brianteeman - change - 28 Oct 2016
Category Router / SEF
avatar Hackwar
Hackwar - comment - 29 Oct 2016

Found the issue and fixed it. See #12616

avatar brianteeman
brianteeman - comment - 29 Oct 2016

Closed as we have a PR for testing #12616 thanks @Hackwar hackwar

avatar brianteeman brianteeman - change - 29 Oct 2016
Status New Closed
Closed_Date 0000-00-00 00:00:00 2016-10-29 09:50:30
Closed_By brianteeman
avatar brianteeman brianteeman - close - 29 Oct 2016

Add a Comment

Login with GitHub to post a comment