Success

User tests: Successful: Unsuccessful:

avatar Bakual
Bakual
16 Jul 2013

Changing $this->input->get('layout') from the default "CMD" filter to "STRING" since the layout is allowed to contain a ":". This is in case of an additional layout in a template where the layout is stored as "templatename:customlayout".

This is currently an issue if you want to create an alternate layout for the categories view in com_content.
The bug doesn't appear in all views because there are checks for the current active menu item layout built-in.

However it never works if you use the URL parameter "&layout=templatename:customlayout".

Tracker: http://joomlacode.org/gf/project/joomla/tracker/?action=TrackerItemEdit&tracker_item_id=31494

avatar Bakual Bakual - open - 16 Jul 2013
avatar Bakual
Bakual - comment - 24 Oct 2013

Merged upstream master

avatar babsgosgens
babsgosgens - comment - 3 Dec 2013

Why is this not merged in yet? Also see: #2637

avatar Bakual
Bakual - comment - 3 Dec 2013

Because unfortunately nobody cared to test it :disappointed:
We usually need two good tests logged in the associated tracker item. If nobody cares to test, nothing will get fixed.

So you would be a big help if you can test it and mark a successful test in the tracker.

avatar mbabker mbabker - reference | - 6 Dec 13
avatar Bakual Bakual - close - 6 Dec 2013
avatar Bakual Bakual - change - 6 Dec 2013
Status New Closed
Closed_Date 0000-00-00 00:00:00 2013-12-06 17:27:51
avatar Bakual Bakual - close - 6 Dec 2013
avatar Bakual Bakual - head_ref_deleted - 6 Dec 2013
avatar babsgosgens
babsgosgens - comment - 9 Dec 2013

Awesome, great team work!

avatar Bakual
Bakual - comment - 9 Dec 2013

You were a big help pushing that forward. Thanks :+1:

Add a Comment

Login with GitHub to post a comment