[#16089] - Prevent double amp encoding
- Fixed in Code Base
- 17 May 2017
- Medium
- Build: staging
- # 16089
- Diff
- PhilETaylor:doubleamp
User tests: Successful: Unsuccessful:
Pull Request for Issue #16088 .
Summary of Changes
fix the double amp issue
Steps to reproduce the issue
Updated from 3.7.0. to 3.7.1
Turn SEF off in Joomla config.
Create menu items to category blog
Expected result
Something like
index.php?option=com_content&view=categories&id=50&Itemid=288
Actual result
double ampsands
index.php?option=com_content&view=categories&id=50& amp;Itemid=288
Votes
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Modules Front End |
The double ampersands make the menu items behave unexpectedly. Sidebar modules are not displayed when menu item is called with the double ampersands.
Workaround: move the menu items to a non displaying menu. Create new menu items, type external link, with manually constructed link not containing the double ampersand.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/16089.
The double ampersands make the menu items behave unexpectedly. Sidebar modules are not displayed when menu item is called with the double ampersands.
Yes this I know, and this is fixed with this PR
The workaround IS THIS PR - you're meant to be testing it, not commenting on what the original issue is and a scary way of avoiding it.
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/16089.
Sorry. New guy.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/16089.
RTC anyone?
| Status | Pending | ⇒ | Ready to Commit |
RTC
What does RTC mean please?
What does RTC mean please?
Ready to Commit.
Ready to Commit.
?
Ah ha! Cool. Thanks. Thought it might have been that, but it seemed more like that should have been ready to merge.
wilsonge
- | Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2017-05-17 22:47:55 |
| Closed_By | ⇒ | wilsonge | |
| Labels |
Added:
?
|
||
@PhilETaylor Can this issue couse generating backend links as SEF links?
Thanks @PhilETaylor then I have wird issue to debug.
I have tested this item✅ successfully on 7928a9b
After applying patch the URL turn back to expected like index.php?option=com_content&view=article&id=8&Itemid=116
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/16089.