Ext.EventManager.removeFromSpecialCache() iterating past end of array error

[cscomkl]

Tested version: Ext JS 3.4.0 and 3.4.1
Browsers: All

        removeFromSpecialCache: function(o) {
            var i = 0,
                len = specialElCache.length;
                
            for (; i < len; ++i) {
                if (specialElCache[i].el == o) {
                    specialElCache.splice(i, 1); 
                }
            }
        },

When "specialElCache.splice(i, 1);" removes an item from the "specialElCache" array, array length is no longer "len" and causes "TypeError: specialElCache[i] is undefined" (Firefox) when iterating past the end of the array.

I am now using temporary fix:

        removeFromSpecialCache: function(o) {
            var i = 0,
                len = specialElCache.length,
                a=[];
                
            for (; i < len; ++i) {
                if (specialElCache[i].el != o) {
                    a.push(specialElCache[i]);
                }
            }
            specialElCache=a;
        },

[mitchellsimoens]

Thanks for the report! I have opened a bug in our bug tracker.

[padraig]

Any update on this bug? I'm getting it when I call destroy() on an Ext.Window component.

[Sergeith]

I have the same bug on FireFox (Extjs 3.4.1)

[padraig]

For me it happens when I've got 2 htmleditor components on the window and I call destroy() on the window. With only 1 htmleditor on the window, the error does not occur.

It happens on the 2nd call to

Ext.EventManager.removeFromSpecialCache(doc);

in the beforeDestroy() function of Ext.form.HtmlEditor()