Ticket #5091 (closed defect: fixed)

Opened 13 months ago

Last modified 11 months ago

debugAtAllCosts: true makes modules loaded twice

Reported by: jfcunat Owned by: jburke
Priority: high Milestone: 1.1
Component: Core Version: 1.0
Severity: normal Keywords:
Cc:

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

Changed 13 months ago by bill

  • owner changed from anonymous to jburke
  • milestone changed from 1.0.1 to 1.1

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.

Changed 13 months 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.

Changed 13 months 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.

Changed 11 months ago by jburke

  • status changed from new to closed
  • resolution set to fixed

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

Note: See TracTickets for help on using tickets.