?
avatar bnisevic
bnisevic
31 May 2014

Steps to reproduce the issue

  1. Go to Global Configuration (com_config)
  2. Change some settings. For example Error reproting or Site Offline.
  3. Click Save

Expected result

Selected/Changed values should be displayed.

Actual result

Old value is displayed instead of new one. It will display previously saved value like nothing is changed. If you reload the page saved value will be displayed correctly. It is not displayed correctly only on first load after Save.

System information (as much as possible)

It is localhost on Ubuntu, tried in Google Chrome 35.0.1916.114 and Firefox 29.

Additional comments

Votes

# of Users Experiencing Issue
1/2
Average Importance Score
3.50

avatar bnisevic bnisevic - open - 31 May 2014
avatar bnisevic
bnisevic - comment - 31 May 2014

This should be check on Ubuntu. On Windows it can not be reproduced.
You may blame the J!Tracker Application for transmitting this comment.

avatar bnisevic
bnisevic - comment - 31 May 2014

I recreated this bug on OS X Mavericks ans Safari 7.
You may blame the J!Tracker Application for transmitting this comment.

avatar hans2103
hans2103 - comment - 31 May 2014

@test
cannot reproduce on Chrome with Mac
(fresh installed Joomla 3.3. staging)

avatar zanardigit
zanardigit - comment - 31 May 2014

@test
Cannot reproduce on Google Chrome 35 on Ubuntu 12.04. Config values are displayed correctly after save.
What's the "cache" configuration to reproduce the error?

avatar hans2103
hans2103 - comment - 31 May 2014

@test
cannot reproduce on Firefox on a Mac

avatar pe7er
pe7er - comment - 31 May 2014

@test
I can not reproduce the error at Chromium Version 34.0.1847 (Debian 7.5)
You may blame the J!Tracker Application for transmitting this comment.

avatar phproberto
phproberto - comment - 1 Jun 2014

I can confirm the issue on Ubuntu 14.04 + Firefox v29.0

Changes are saved but not shown properly if you don't reload the page.


You may blame the J!Tracker Application for transmitting this comment.

avatar phproberto phproberto - reference | - 1 Jun 14
avatar phproberto
phproberto - comment - 1 Jun 2014

This fixes the issue: #3701

avatar phproberto
phproberto - comment - 11 Jul 2014
avatar phproberto phproberto - reference | - 11 Jul 14
avatar brianteeman
brianteeman - comment - 11 Jul 2014
avatar phproberto
phproberto - comment - 11 Jul 2014

My cache is OFF. You had the problem in Safari so I think that discards a linux or firefox problem. Seeing the patch you can see that is obvious that the form data is not saved in the session except if the form has an error.

avatar Bakual
Bakual - comment - 11 Jul 2014

Imho, we usually don't save the data into the session on success. It's not needed because in that case the data should be reloaded from the file/database.

We need to store the data into the session on failure so we don't loose the changed data the user changed.

So imho the current code there should actually be correct. But I have no clue why it fails in your cases.

avatar JoomliC
JoomliC - comment - 11 Jul 2014

I can confirm the issue on mac OS X 10.6.8 + Firefox v30.0 + Joomla 3.3.1

Patch #3701 from Roberto is fixing this issue! Well done! :+1:

avatar phproberto
phproberto - comment - 12 Jul 2014

@Bakual I would also prefer to force a fresh reload each time but it doesn't work even forcing the session com_config.config.global.data destruction to be sure nothing is pre-loaded from session. The config loaded is the previous to the save even after a page redirection on save (of course with cache disabled).

I have also spent hours playing with php.ini settings to see if there is a server setting that can cause that issue and checking if it could be related to the BOM fix applied to avoid configuration errors.

The only thing I know is that the issue exists and that storing the session data does not cause any issue. So applying JM's philosophy on this: "if somebody finds a better fix for this in the future apply it. Meanwhile fix it".

avatar Bakual
Bakual - comment - 12 Jul 2014

interesting bug ;)

avatar zero-24 zero-24 - close - 27 Jul 2014
avatar brianteeman brianteeman - change - 27 Jul 2014
Status New Closed
Closed_Date 0000-00-00 00:00:00 2014-07-27 13:50:45
avatar brianteeman brianteeman - close - 27 Jul 2014
avatar zero-24 zero-24 - change - 7 Jul 2015
Labels Added: ?

Add a Comment

Login with GitHub to post a comment