Opened 13 years ago

Closed 12 years ago

Last modified 11 years ago

#2192 closed defect (wontfix)

dojo is not defined on line 3429 in dojo.js

Reported by: sirber@… Owned by: bill
Priority: high Milestone:
Component: Events Version: 0.4.1
Keywords: Cc: Adam Peller
Blocked By: Blocking:

Description

line 3429: if((x==0)&&(_31e)&&(dojo.event.browser.isEvent(arguments[x]))){ in dojo.js

only on firefox, no bug on IE, trigger on page load a second time. The page does nothing fancy, just: DatePicker?, Button, Menu2, Clock.

Nothing blocks, everything works as expected, just getting this error.

Attachments (2)

bug.php (875 bytes) - added by guest 13 years ago.
frameset emulator and bug repro
iframetest.html (230 bytes) - added by Douglas Hays 12 years ago.
simple file with iframe

Download all attachments as: .zip

Change History (21)

comment:1 Changed 13 years ago by liucougar

Owner: changed from alex to liucougar

a test case please

if you use firebug, try to disable it

Changed 13 years ago by guest

Attachment: bug.php added

frameset emulator and bug repro

comment:2 Changed 13 years ago by sirber@…

Online version:

http://www.webernic.com/tests/bug.php

Play with "Link 1" and "Link 2". Errors will arise after few clicks.

comment:3 Changed 13 years ago by liucougar

which build do you use?

comment:4 Changed 13 years ago by sirber@…

Dojo: 0.4.1 Firefox: 1.5.0.8 (Flock 0.7.8)

comment:5 Changed 13 years ago by liucougar

I meant which prebuilt dojo you use?

comment:6 Changed 13 years ago by liucougar

and what if you do not include clock in your page? do you still see that error?

comment:7 Changed 13 years ago by sirber@…

I have: dojo-0.4.1-ajax (out of the box)

and yes, errors whitout the clock too. use the same link above to see.

comment:8 Changed 13 years ago by Adam Peller

Try putting your dojo.require's in <HEAD>

comment:9 Changed 13 years ago by jkuhnert

I think this might be a windown.onunload bug. Google has a similar bug that tries to run a GC() function that drives me nuts.

We'd have to either ensure the unload stuff runs before unload (somehow) or possibly rewrite that function to handle "built in" javascript functions only to get its work done.

comment:10 Changed 13 years ago by dylan

Milestone: 0.9

comment:11 Changed 12 years ago by Adam Peller

Resolution: invalid
Status: newclosed

comment:12 Changed 12 years ago by Adam Peller

please re-open if you can provide a running testcase, preferably a simple attachment

comment:13 Changed 12 years ago by Douglas Hays

Milestone: 0.90.4.4
Priority: normalhigh
Resolution: invalid
Status: closedreopened

This needs to be fixed in the 0.4.x trunk. Several customers are running into this. It can be easily recreated by loading nearly any of the test_*html files into an iframe, and then reloading the iframe contents. I will attach a simple html file with the iframe created and recreate steps.

Changed 12 years ago by Douglas Hays

Attachment: iframetest.html added

simple file with iframe

comment:14 Changed 12 years ago by Douglas Hays

On my system, I ran the following:

file:///c:/tmp/iframetest.html?file://c:/ajax/04x/tests/widget/test_SelectNLS.html

After it loaded OK, I clicked the Reload button and the error occurs. I could not recreate this on 0.9, but it is affecting 0.4 customers.

comment:15 Changed 12 years ago by Douglas Hays

Cc: Adam Peller added

comment:16 Changed 12 years ago by Douglas Hays

The error occurs after the 2nd click of the Reload button but before the page actually reloads the 2nd time.

comment:17 Changed 12 years ago by Douglas Hays

Owner: changed from liucougar to bill
Status: reopenednew

I believe this line in PopupContainer?.js

targetWindow = dojo.html.getDocumentWindow(window.top && window.top.document
window.document);

should be replaced with:

targetWindow = dojo.html.getDocumentWindow(dojo.doc());

otherwise dojo may be undefined for the targetWindow.

comment:18 Changed 12 years ago by dylan

Resolution: wontfix
Status: newclosed

comment:19 Changed 11 years ago by (none)

Milestone: 0.4.4

Milestone 0.4.4 deleted

Note: See TracTickets for help on using tickets.