?
avatar n3t
n3t
22 Apr 2017

Steps to reproduce the issue

  1. Define some custom fields for articles
  2. Go to com_content configuration and disable option "Show Article Options" (set to No)
  3. Edit any article

Expected result

Custom fields page should be visible

Actual result

Custom fields page is not visible

Additional comments

IMO fields should be displayed, even when Options are hidden. Fields are not Options, it is more Content then options.

avatar n3t n3t - open - 22 Apr 2017
avatar joomla-cms-bot joomla-cms-bot - labeled - 22 Apr 2017
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 22 Apr 2017

Confirmed: Set in "Articles: Options" > Tab "Editing Layout" "Show Article Options" to "Hide" and in Article Tab "Field" is also hidden.

avatar franz-wohlkoenig franz-wohlkoenig - change - 22 Apr 2017
Status New Confirmed
avatar brianteeman
brianteeman - comment - 25 Apr 2017

I can confirm this but I dont see where the code is for the fields to be injected - @laoneo can you point me in the right direction and I can have a go at fixing it

avatar n3t
n3t - comment - 25 Apr 2017

Should be layout joomla.edit.params - condition if ($displayData->get('show_options', 1)).
Value show_options is set in edit view of article.

There are two possible solutions IMO:

  • fieldset name options should be rendered separately, influenced by condition above, and all othe fieldsets in foreach cycle should not be influenced by the condition.
  • Or do not set show_options value but ignore_fieldsets in article edit view
avatar laoneo
laoneo - comment - 26 Apr 2017

@n3t has right. I think it doesn't sound like a easy one line fix. Needs some toughts. Basically it needs a different way of loading.

avatar Bakual
Bakual - comment - 26 Apr 2017

I think just adding the respective fieldsets to the ignore_fieldsets if "show_options" is set would actually be the correct way.
But the way it is currently done (the parameter is checked in the JLayout) I fear this isn't really possible to do in a B/C way. The intention of this parameter is to actually hide all options rendered by that JLayout, which includes fields.

avatar zero-24
zero-24 - comment - 26 Apr 2017

The intention of this parameter is to actually hide all options rendered by that JLayout, which includes fields.

But is this than not expected behavior? As the same is true for 3rd party extensions that adds tabs. If you disable that option they are not showed.

avatar brianteeman
brianteeman - comment - 26 Apr 2017

@zero-24 the expected behaviour is that the tab called "Article Options" is controlled by that option and no other tab

avatar AlexRed
AlexRed - comment - 19 May 2017

I can confirm the problem on Joomla! 3.7.2-Dev (nightly Build)


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/15473.

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 8 Jun 2017

Struggled again about this Issue.

avatar laoneo
laoneo - comment - 13 Jun 2017

Can you guys please test #16661 carefully if there are no side effects. Because when this one gets in, only the fieldsets with the name basic are not loaded anymore.

avatar brianteeman
brianteeman - comment - 13 Jun 2017

Closed as we have a pr for testing

avatar brianteeman brianteeman - change - 13 Jun 2017
Status Confirmed Closed
Closed_Date 0000-00-00 00:00:00 2017-06-13 09:03:48
Closed_By brianteeman
avatar brianteeman brianteeman - close - 13 Jun 2017

Add a Comment

Login with GitHub to post a comment