Login!
Lost password?
 

MODx Bug/Feature Tracker and Feature Requests

Welcome to the MODx CMS Tracker. Please choose the appropriate project from the drop down menu and provide as much information as possible regarding your server environment and browser. Thanks!

FS#972 — Endless Refresh Loop on System Configuration Save

Attached to Project — MODx
Opened by Michael McWilliam (michaelcm) - Thursday, 01 November 2007, 12:43PM
Last edited by Garry Nutting (garryn) - Monday, 19 November 2007, 06:32PM
Task Type Bug Report
Category Core Distribution
Status Closed
Assigned To No-one
Operating System All
Severity Medium
Priority High
Reported Version 0.9.6
Due in Version 0.9.6.2
Due Date Undecided
Percent Complete 100%

Details

Prelude:
I have MODxCMS installed on a few different servers, there is a persistent problem on all installs when using Safari 3.0.0 to 3.0.4. It is likely a browser issue, but one I would like to see fixed as Safari is much faster than Firefox on Mac right now.

The Problem:
After clicking the 'Save' button from editing any panel of the 'System Configuration', the browser goes into an endless refresh loop.

Expected Behaviour:
The same thing that happens in Firefox where the entire manager is reloaded once, then went back to the manger home page.

Possible fix:
Instead of relying on recurring javascript to refresh the site then return to the home page, I was able to stop the issue in Safari, and still have the manager refresh once by added the following code after line 135 of manager/frames/menu.php

if(rFrame==10) {
window.parent.parent.location.href = "../manager";
}


Thanks,
Michael
This task depends upon

This task blocks these from closing
Closed by  Ryan Thrash (rthrash)
Wednesday, 23 January 2008, 12:20PM
Reason for closing:  Fixed
Comment by Xef Ned (xefned) - Sunday, 04 November 2007, 10:24AM
Michael, thanks for posting this.
I had no issues until I upgraded to Leopard which has Safari 3.0 as standard.

I urge the developers to pay attention to this issue because it will inevitably affect many as more and more users upgrade to Leopard.

Thanks! Ned

Comment by Garry Nutting (garryn) - Monday, 19 November 2007, 06:32PM
  • Field changed: Status (Unconfirmed → Requires testing)
  • Field changed: Due in Version (Undecided → 0.9.6.2)
  • Field changed: Severity (Low → Medium)
  • Field changed: Priority (Normal → High)
  • Field changed: Percent Complete (0% → 100%)
  • Field changed: Due Date (Undecided → Undecided)
Should be fixed in SVN rev 3156.