Opened 12 years ago

Closed 12 years ago

#5091 closed defect (fixed)

debugAtAllCosts: true makes modules loaded twice

Reported by: jfcunat Owned by: James Burke
Priority: high Milestone: 1.1
Component: Core Version: 1.0
Keywords: Cc:
Blocked By: Blocking:

Description

When you add debugAtAllCosts true in a djConfig, some modules are loaded twice and it causes some problems. Try dijit/tests/test_ColorPalette.html with IE6 with a non builded Dojo, and add a debug in dojo.provide function, you will see that we go twice in many modules.

It makes a problem with templated Widget because function called by dojo.addOnUnload is executed twice and makes an error the second time. file _Templated.js line 253 It causes a problem on the _destroyElement on IE 6

Change History (4)

comment:1 Changed 12 years ago by bill

Milestone: 1.0.11.1
Owner: changed from anonymous to James Burke

This doesn't need to be fixed for 1.0.1 as there is a simple workaround (don't use debugAtAllCosts: true), but passing on to James to see if he can do anything. I suspect this is just how debugAtAllCosts works though.

comment:2 Changed 12 years ago by bill

(In [11493]) Fixes #5060 on 1.0 branch: ColorPalette? inefficient code and memory leak on IE6. Also makes _Templated.js's dojo.addOnUnload() call itempotent since it executes twice when debugAtAllCosts is true (refs #5091).

Thanks to Jean Cunat (Orange-ftgroup CCLA on file) for patch.

comment:3 Changed 12 years ago by bill

(In [11494]) Fixes #5060 on trunk: ColorPalette? inefficient code and memory leak on IE6. Also makes _Templated.js's dojo.addOnUnload() call itempotent since it executes twice when debugAtAllCosts is true (refs #5091).

Thanks to Jean Cunat (Orange-ftgroup CCLA on file) for patch.

comment:4 Changed 12 years ago by James Burke

Resolution: fixed
Status: newclosed

(In [11910]) Fixes #5091: debugAtAllCosts including files twice in the head.

Note: See TracTickets for help on using tickets.