Skip to content

2.5.x Multilingual: Remove language code in alternate link#4425

Merged
infograf768 merged 2 commits intojoomla:2.5.xfrom
infograf768:multilang_alternate_25x
Oct 11, 2014
Merged

2.5.x Multilingual: Remove language code in alternate link#4425
infograf768 merged 2 commits intojoomla:2.5.xfrom
infograf768:multilang_alternate_25x

Conversation

@infograf768
Copy link
Copy Markdown
Member

This ports to 2.5.x #4221

@infograf768
Copy link
Copy Markdown
Member Author

See also #4215

@smanzi
Copy link
Copy Markdown

smanzi commented Oct 8, 2014

Negative test (see http://dev.smz.it/)

in http://dev.smz.it/it/articolo-in-italiano the alternate link should be:
http://dev.smz.it/english-article
while instead it is generated as:
http://dev.smz.itglish-article

on the other hand for
http://dev.smz.it/it/un-altro-articolo
the alternate link is correctly set to
http://dev.smz.it/another-article

So... it seems we have a problem with menu items starting with /en (and probably /whatever-alternate-language-code)

I'm wondering if this issue exists also for the (already committed) J3.x version of this PR: I'l make some test ASAP.

@smanzi
Copy link
Copy Markdown

smanzi commented Oct 8, 2014

More comments and possible solution in #4221

@infograf768
Copy link
Copy Markdown
Member Author

Issue confirmed. Updating PR.

@smanzi
Copy link
Copy Markdown

smanzi commented Oct 10, 2014

@test
Success!

infograf768 added a commit that referenced this pull request Oct 11, 2014
2.5.x Multilingual: Remove language code in alternate link
@infograf768 infograf768 merged commit 58b19a4 into joomla:2.5.x Oct 11, 2014
@infograf768 infograf768 deleted the multilang_alternate_25x branch November 12, 2015 12:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants