Opened 14 years ago

Closed 13 years ago

Last modified 12 years ago

#718 closed defect (fixed)

spurious event errors on IE7

Reported by: bill Owned by: alex
Priority: high Milestone:
Component: General Version: 0.3
Keywords: Cc:
Blocked By: Blocking:

Description

Noticed on the demo site but maybe other places also. Every 1 out of 5 times, IE gets an exception bringing up the top page.

dj_addNodeEvtHdlr(ary, "load",fp,_80) is called, and aryonload? seems to be "" (rather than null or a real function), thus causing the problem.

Obviously I can update this code to check for "" but... any comments?

Change History (7)

comment:1 Changed 14 years ago by James Burke

I can't seem to find a code reference for this specific call:

dj_addNodeEvtHdlr(ary, "load",fp,_80)

Where is this in the code?

comment:2 Changed 14 years ago by Tom Trenka

jburke:

hostenv_browser.js, line 242.

comment:3 Changed 14 years ago by James Burke

hostenv_browser.js, line 242, is the definition of the function, but I don't see a call that matches what Bill describes above, particularly with an ary and "load" parameters. There is a call to that function in hostenv_browser.js, but it only passes 3 parameters:

dj_addNodeEvtHdlr(window, "load", function(){ /* function goo */});

If Bill is referencing the function definition, then why is the second parameter "load" instead of evtName? It seemed like perhaps the code that he is referencing is either out of date, or some other code is calling dj_addNodeEvtHdlr. But no other code should be calling that function (and when I did a search on trunk, I could not find any other references).

comment:4 Changed 14 years ago by bill

Nope, it's the newest code. Remember that dojo.js is compressed, so the parameters are actually _7d, _7e, etc., with a local variable called _81 (IIRC). What I meant in the bug description is that the second argument to addNodeEvtHdlr has the value of "load". (That's why it's in quotes)

comment:5 Changed 14 years ago by alex

Owner: changed from anonymous to alex
Status: newassigned

comment:6 Changed 13 years ago by bill

Resolution: fixed
Status: assignedclosed

This doesn't reproduce for me anymore so I'm going to close the bug. If were lucky, it's been fixed. If not, we'll get another bug report soon, I guess.

comment:7 Changed 12 years ago by (none)

Milestone: 0.3.1

Milestone 0.3.1 deleted

Note: See TracTickets for help on using tickets.