Success

User tests: Successful: Unsuccessful:

avatar radiant-tech
radiant-tech
24 Feb 2014

'show_tags' is incorrectly taken from $displayData instead of $params
It is forced to always be true since it doesn't exist in $displayData ($displayData->get('show_tags', 1)).

Can be tested by setting Content Category param for show_tags to Hide. They are always displayed.
Also cannot override the component-level setting in the menu item as the field does not exist there.

Patch corrects the layout file and adds the show_tags field to content category xml file.
Tracker: http://joomlacode.org/gf/project/joomla/tracker/?action=TrackerItemEdit&tracker_item_id=33337

avatar radiant-tech radiant-tech - open - 24 Feb 2014
avatar wilsonge
wilsonge - comment - 26 Feb 2014

@test both issues confirmed and fixed with this PR :) Nice find!

avatar phproberto phproberto - reference | 60749a3 - 26 Feb 14
avatar phproberto phproberto - change - 26 Feb 2014
Status New Closed
Closed_Date 0000-00-00 00:00:00 2014-02-26 06:16:54
avatar phproberto phproberto - close - 26 Feb 2014
avatar phproberto phproberto - close - 26 Feb 2014
avatar radiant-tech radiant-tech - head_ref_deleted - 26 Feb 2014
avatar Bakual Bakual - reference | 8474094 - 27 Feb 14
avatar Bakual Bakual - reference | 3a066a8 - 12 May 14

Add a Comment

Login with GitHub to post a comment