?
avatar waader
waader
6 Jan 2017

Steps to reproduce the issue

With current staging go to Content > Articles > Options and you will get this notice error:

Notice: Undefined property: stdClass::$showon in ...joomla37\administrator\components\com_config\view\component\tmpl\default.php on line 60

Expected result

Actual result

System information (as much as possible)

Additional comments

avatar waader waader - open - 6 Jan 2017
avatar joomla-cms-bot joomla-cms-bot - change - 6 Jan 2017
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 6 Jan 2017
avatar Bakual
Bakual - comment - 6 Jan 2017
avatar joomdonation
joomdonation - comment - 6 Jan 2017

Yes. it is related. Reverted that PR and the notice is gone. Didn't test that yesterday, sorry

avatar joomdonation
joomdonation - comment - 6 Jan 2017

@andrepereiradasilva Changes the code in this line https://github.com/joomla/joomla-cms/blob/staging/administrator/components/com_config/view/component/tmpl/default.php#L60 to

<?php if (!empty($fieldSet->showon)) : ?>

seems solve the issue

However, do we really need this block of code https://github.com/joomla/joomla-cms/blob/staging/administrator/components/com_config/view/component/tmpl/default.php#L60-L64 ? Are we support show on for fieldsets, too ?

avatar andrepereiradasilva
andrepereiradasilva - comment - 6 Jan 2017

See #13490

avatar jeckodevelopment
jeckodevelopment - comment - 6 Jan 2017

Closing as we have a PR for that.
Please test #13490

avatar jeckodevelopment jeckodevelopment - change - 6 Jan 2017
Status New Closed
Closed_Date 0000-00-00 00:00:00 2017-01-06 14:41:17
Closed_By jeckodevelopment
avatar jeckodevelopment jeckodevelopment - close - 6 Jan 2017

Add a Comment

Login with GitHub to post a comment