? Success

User tests: Successful: Unsuccessful:

avatar Bakual
Bakual
4 Feb 2014

Same as PR #2686, but against staging.

Issue

Fieldset descriptions don't show up. This is because the code is currently in the layout joomla.edit.fieldset but we don't have this description information there. It looks for a property of the (assumed) object $fieldSet while in fact $fieldSet is an array of fields.
This is the case everywhere except for the component parameters itself. So in modules, plugins, menus and templates parameters and maybe also other places.

Solution

Moving the code to the joomla.edit.params layout where this information (the object) is present.

Testing Instructions

Try to add a description to a fieldset. For example in modules/mod_articles_news/mod_articles_news.xml change

<fieldset
    name="advanced">

to

<fieldset
    name="advanced" description="test">

Currently, the description isn't shown, after the patch is applied it will show.

Known Limitations

This patch only covers "custom" fieldsets. The basic ones (like basic, description, request) are threated differently in the code and thus need to be taken care seperately.

Tracker

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

avatar Bakual Bakual - open - 4 Feb 2014
avatar losedk
losedk - comment - 7 Apr 2014

Tested. Works as expected

avatar Bakual Bakual - change - 12 May 2014
Title
Show fieldset descriptions in modules, menus, plugins parameters
[#32958] Show fieldset descriptions in modules, menus, plugins parameters
avatar Bakual Bakual - change - 12 May 2014
Title
Show fieldset descriptions in modules, menus, plugins parameters
[#32958] Show fieldset descriptions in modules, menus, plugins parameters
avatar infograf768 infograf768 - reference | 08560a3 - 30 May 14
avatar infograf768 infograf768 - merge - 30 May 2014
avatar infograf768 infograf768 - close - 30 May 2014
avatar infograf768 infograf768 - change - 30 May 2014
Status New Closed
Closed_Date 0000-00-00 00:00:00 2014-05-30 09:23:39
avatar infograf768 infograf768 - close - 30 May 2014
avatar Bakual Bakual - head_ref_deleted - 30 May 2014

Add a Comment

Login with GitHub to post a comment