Opened 9 years ago

Closed 9 years ago

#12344 closed defect (fixed)

IE < 8: dojox.form.Uploader test pages throw error

Reported by: Kenneth G. Franqueiro Owned by: Mike Wilcox
Priority: high Milestone: 1.6
Component: DojoX Form Version: 1.6.0rc1
Keywords: ie uploader Cc:
Blocked By: Blocking:

Description

The test pages for the new uploader, http://archive.dojotoolkit.org/nightly/dojotoolkit/dojox/form/tests/test_Uploader.html and http://archive.dojotoolkit.org/nightly/dojotoolkit/dojox/form/tests/test_UploaderAll.html, throw mostly harmless errors in IE 6 and 7. This is because both these files dojo.require('dojo._firebug.firebug') manually which shouldn't be done - this is done automatically as necessary if the isDebug flag is turned on in djConfig.

It's easy enough to simply remove the offending line in each test file; I'm wondering if the intent was to include isDebug: true in djConfig all along, or if we should just remove the lines and that's it?

Change History (3)

comment:1 Changed 9 years ago by Mike Wilcox

Resolution: fixed
Status: newclosed

(In [23884]) Fixes #12344 - fixed crazy debug code in tests

comment:2 Changed 9 years ago by dante

Resolution: fixed
Status: closedreopened

comment:3 Changed 9 years ago by dante

Milestone: tbd1.6
Resolution: fixed
Status: reopenedclosed

please set milestones in the future.

Note: See TracTickets for help on using tickets.