[#18159] - Allow alternate menuitems being selected as alternate layouts
- Fixed in Code Base
- 9 Oct 2017
- Medium
- Build: staging
- # 18159
- Diff
- Bakual:FixAlternateMenuitems
User tests: Successful: Unsuccessful:
Pull Request for Issue #15547.
Summary of Changes
Removing the code which unsets alternative layouts if there is an accompagning XML file (making it an alternative menuitem)
Testing Instructions
- Create an alternative layout (just the php file) and an alternative menuitem (php file plus xml file).
- Test that they both can be selected as alternative layout in the global com_content config and in the article.
- Test that the selected layout is shown in frontend.
Expected result
Alternative menuitems are available also as alternative layouts
Actual result
They're not shown
Documentation Changes Required
Not sure if that is documented anywhere
Caveat
As you can see in the comments, the current behavior seems to be intended. I just don't know why as I can't imagine as side effect.
The code was introduced years ago with https://developer.joomla.org/joomlacode-archive/issue-23054.html
joomla-cms-bot
- | Category | ⇒ | Libraries |
| Status | New | ⇒ | Pending |
| Labels |
Added:
?
|
||
I have tested this item
Works like a charme, like this behaviour better, Thanks Thomas
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/18159.
I have tested this item
Thanks for the patch.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/18159.
| Status | Pending | ⇒ | Ready to Commit |
RTC after two successful tests.
mbabker
- | Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2017-10-09 23:27:59 |
| Closed_By | ⇒ | mbabker | |
| Labels |
Added:
?
|
||
From the comment in line 184, the part where it says " or that have XML files" should also be removed.