Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#13072 closed defect (fixed)

dojo.destroy broken under IE since rev. 25179

Reported by: Patrick Ruzand Owned by: Kris Zyp
Priority: blocker Milestone: 1.7
Component: Core Version:
Keywords: Cc: pruzand@…
Blocked By: Blocking:

Description

it seems dojo.destroy (= dojo._destroyElement) has been broken recently (rev. 25179) under IE: an exception is thrown while evaluating the _destroyContainer variable:

l.130                   if(!_destroyContainer || _destroyDoc != doc){
                               _destroyContainer = doc.createElement("div");
                               _destroyDoc = doc;
                       }

_destroyContainer being undefined, it throws an error under IE.

From what I see, it is due to the now missing _destroyContainer variable initialization to null. (was there previously in 1.6).

Change History (3)

comment:1 Changed 8 years ago by Douglas Hays

Owner: set to Kris Zyp

Refs #13065

comment:2 Changed 8 years ago by Kris Zyp

Resolution: fixed
Status: newclosed

(In [25191]) Revert rest of dojo.destroy changes, hopefully fix memory leak, refs #13065, fixes #13072 !strict

comment:3 Changed 8 years ago by bill

(In [25195]) Put back unload require, it's needed again. Refs #13065, #13072, #12672, #12790 !strict.

Note: See TracTickets for help on using tickets.