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#863 — IE7 gives JS error on navigating away from Manager

Attached to Project — MODx
Opened by Nick Crossland (ncrossland) - Wednesday, 16 May 2007, 10:45AM
Last edited by Jason Coward (opengeek) - Thursday, 15 May 2008, 11:45AM
Task Type Bug Report
Category Core Distribution
Status Closed
Assigned To Jason Coward (opengeek)
Operating System All
Severity Medium
Priority Normal
Reported Version 0.9.5
Due in Version 0.9.6.2
Due Date Undecided
Percent Complete 100%

Details

IE7 gives a javascript error on navigating away from the manager.

A runtime error has occured.
Line: 494
Object doesn't support this property or method

This happens every time. It appears that the unload event (showing the "working" panel) is using the Moo $ function to try and find an element that has already been unloaded.


This task depends upon

This task blocks these from closing
Closed by  Brunaud (coroico)
Sunday, 18 May 2008, 04:53AM
Reason for closing:  Fixed
Comment by Stefan Scholl (stesch) - Monday, 30 July 2007, 09:05AM
I get an error, too. Logging out of MODx 0.9.6 results in the attached error.

Translated details:

Line: 127
Char: 9
Error: object doesn't support property or method
Code: 0
URL: .../manager/indes.php?a=1&f=menu
  Error_logout_modx096_ie7.png
Comment by Tony (sparkiwi) - Thursday, 17 January 2008, 07:59PM
I was getting this error too - till I realised that it may have been the fact that I had a site manager open for two different sites in tabs on my (avant) browser. When i closed the first one the problem vanished.

Comment by Adam Kirk (Crash1hd) - Thursday, 15 May 2008, 11:40AM
I had the same error but I was also getting it in FF2 if I had tabs open, I have broken it down and resolved the issue to being line 16 in header.inc.php was this

window.addEvent('unload', document_onunload);

changed it to this

window.addEvent('beforeunload', document_onunload);

and the error goes away :)

Comment by Jason Coward (opengeek) - Thursday, 15 May 2008, 11:45AM
  • Field changed: Status (Implementing → Requires testing)
  • Field changed: Percent Complete (0% → 100%)
Committed fix at revision 3705 (for 0.9.6.2 inclusion)

Comment by Brunaud (coroico) - Sunday, 18 May 2008, 04:53AM
with SVN3709. Logout of manager under IE7 is now OK. No js error.