? ? Pending

User tests: Successful: Unsuccessful:

avatar ggppdk
ggppdk
28 May 2018

Pull Request for Issue #20588

Stale session data should not override configuration file when editing global configuration

Make it consistent with all other controllers

Summary of Changes

  1. Only save data (of global configuration) into session when an error occurs
  • validation error
  • saving error
  1. Clear session data on successful execution of the task

Testing Instructions

  1. Load global config and set debug to ON and save and close
  2. Manually edit $debug = '0'; in /configuration.php, thus setting debug to OFF
  3. Click to re-edit global config

Expected result

The DEBUG setting is shown as OFF

Actual result

The DEBUG setting is shown as ON

Documentation Changes Required

None

avatar ggppdk ggppdk - open - 28 May 2018
avatar ggppdk ggppdk - change - 28 May 2018
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 28 May 2018
Category Administration com_config
avatar ggppdk ggppdk - change - 28 May 2018
Labels Added: ?
avatar carlitorweb carlitorweb - test_item - 7 Jun 2018 - Tested successfully
avatar carlitorweb
carlitorweb - comment - 7 Jun 2018

I have tested this item successfully on 72ad71b


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

avatar Quy Quy - test_item - 8 Jun 2018 - Tested successfully
avatar Quy
Quy - comment - 8 Jun 2018

I have tested this item successfully on 72ad71b


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

avatar Quy Quy - change - 8 Jun 2018
Status Pending Ready to Commit
avatar Quy
Quy - comment - 8 Jun 2018

RTC


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

avatar mbabker mbabker - close - 12 Jun 2018
avatar mbabker mbabker - merge - 12 Jun 2018
avatar mbabker mbabker - change - 12 Jun 2018
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2018-06-12 22:45:25
Closed_By mbabker
Labels Added: ?
avatar ggppdk ggppdk - change - 13 Jun 2018
The description was changed
avatar ggppdk ggppdk - edited - 13 Jun 2018

Add a Comment

Login with GitHub to post a comment