? Pending

User tests: Successful: Unsuccessful:

avatar chmst
chmst
4 Feb 2020

Summary of Changes

Global configuration has a fieldset cache. This field set is confusing.
If the cache is off, there are fields to be filled and two of them are twice in the xml file.
If cache is set on, there appears a new input field on top of the fieldset.
The field for path is always visible, due to it's showon which is always true, also if caching is off.

I have made changes as I think it makes sense. This requires a review by system specialists,.

Testing Instructions

Look at the field set cache, set the caching on and of.

Expected result

After patch

cache-imaybe

Actual result

cache-is

Documentation Changes Required

yes

avatar chmst chmst - open - 4 Feb 2020
avatar chmst chmst - change - 4 Feb 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 4 Feb 2020
Category Administration com_config
avatar brianteeman
brianteeman - comment - 4 Feb 2020

makes sense to me


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

avatar brianteeman brianteeman - test_item - 4 Feb 2020 - Tested successfully
avatar brianteeman
brianteeman - comment - 4 Feb 2020

I have tested this item successfully on 9107d00


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

avatar jwaisner jwaisner - test_item - 5 Feb 2020 - Tested successfully
avatar jwaisner
jwaisner - comment - 5 Feb 2020

I have tested this item successfully on 9107d00


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

avatar alikon alikon - change - 5 Feb 2020
Status Pending Ready to Commit
avatar alikon
alikon - comment - 5 Feb 2020

RTC


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

avatar rdeutz rdeutz - change - 5 Feb 2020
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-02-05 08:42:57
Closed_By rdeutz
Labels Added: ?
avatar rdeutz rdeutz - close - 5 Feb 2020
avatar rdeutz rdeutz - merge - 5 Feb 2020
avatar chmst
chmst - comment - 5 Feb 2020

thank you, fastests PR ever :)

Add a Comment

Login with GitHub to post a comment