?

User tests: Successful: Unsuccessful:

avatar betweenbrain betweenbrain - open - 4 Oct 2013
avatar spignataro
spignataro - comment - 5 Oct 2013

I just realized that the .xml file fix was not part of the change either. Please see my full request here:

https://github.com/joomla/joomla-cms/pull/230/files

avatar betweenbrain
betweenbrain - comment - 11 Oct 2013

@spignataro in your tracker item, you stated

Go to that article in the Content Manager and choose the layout override
I interpreted that, and tested it, as using the existing Alternative Layout field. That is how I understood the issue and patched accordingly. That field wasn't working for me before I made the changes that I did. I don't think we need to add a field. Can you verify, testing against 2.5.14?
avatar spignataro
spignataro - comment - 11 Oct 2013

It looks like you may be right - I will perform a test next week when I get a chance to fully review. But it looks like you are correct by reviewing the ocde.

avatar betweenbrain
betweenbrain - comment - 5 Dec 2013

Any update on this PR?

avatar spignataro
spignataro - comment - 5 Dec 2013

You are correct - we don't need to add a field. Sorry I forgot to post back to this. Thanks for the follow up.

avatar Bakual
Bakual - comment - 5 Dec 2013

Are you sure this is a bug? Because when you set a menu item to show a single article, the article options are not relevant. Those from the menu item will override it.
Thus if you want an alternate layout, you need to set it in the menu item instead, not in the article. Menu item takes precedence over article item in this case.
It's another case if the menu item points to a different view, for example the category view. Then the article parameter will be in effect and the layout set in the article should be chosen instead.

avatar brianteeman brianteeman - change - 21 Aug 2014
Status New Pending
avatar nicksavov nicksavov - change - 21 Aug 2014
Labels Removed: ?
avatar betweenbrain betweenbrain - close - 16 Sep 2014
avatar betweenbrain betweenbrain - close - 16 Sep 2014
avatar betweenbrain betweenbrain - head_ref_deleted - 16 Sep 2014
avatar betweenbrain betweenbrain - change - 16 Sep 2014
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2014-09-16 09:07:22

Add a Comment

Login with GitHub to post a comment