? ? Pending

User tests: Successful: Unsuccessful:

avatar Bakual
Bakual
6 Mar 2017

The article edit form has some rather complex code just to override some display options based on item attributes. Our JRegistry class could handle that much more simpler.

Summary of Changes

Replaces the rather complex code with proper Registry calls.

Testing Instructions

  • Make sure the article options in the "Configure Edit Screen" behave as expected. You can ignor ethe "Frontend Images and Links" one as that one isn't used in the backend form (for obvious reasons).

Expected result

Setting one of the options to "No" should hide the respective tab even if global options are set to "Yes" and vice versa. Article settings take precedence over global settings.

Actual result

Same behavior.

Documentation Changes Required

None

avatar Bakual Bakual - open - 6 Mar 2017
avatar Bakual Bakual - change - 6 Mar 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 6 Mar 2017
Category Administration com_content
avatar franz-wohlkoenig franz-wohlkoenig - test_item - 7 Mar 2017 - Tested successfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 7 Mar 2017

I have tested this item successfully on 688f408


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

avatar alikon alikon - test_item - 7 Mar 2017 - Tested successfully
avatar alikon
alikon - comment - 7 Mar 2017

I have tested this item successfully on 688f408


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

avatar jeckodevelopment jeckodevelopment - change - 7 Mar 2017
Status Pending Ready to Commit
avatar jeckodevelopment
jeckodevelopment - comment - 7 Mar 2017

RTC


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

avatar wilsonge wilsonge - change - 8 Mar 2017
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-03-08 15:16:53
Closed_By wilsonge
Labels Added: ? ?
avatar wilsonge wilsonge - close - 8 Mar 2017
avatar wilsonge wilsonge - merge - 8 Mar 2017

Add a Comment

Login with GitHub to post a comment