[FIXED-514][3.1] Grid does not clean up editors

[zachext]

Ext version tested:

Ext 3.1


Adapter used:

ext


css used:

only default ext-all.css




Browser versions tested against:


IE8
FF3.5





Description:

Objects specified as editors in column configurations are left behind when an EditorGridPanel is destroyed.





Steps to reproduce the problem:

Goto: http://www.extjs.com/deploy/dev/examples/grid/edit-grid.html


In the Firebug or IE script console:

console.log(Ext.ComponentMgr.all.length)
//-> 20
Ext.getCmp('ext-comp-1005').destroy()
console.log(Ext.ComponentMgr.all.length)
// -> 7



Reload the page and click a cell in the 'Common Name' column.

'ext-comp-1019' is added to the ComponentMgr

console.log(Ext.ComponentMgr.all.length)
// -> 21
Ext.getCmp('ext-comp-1005').destroy()
console.log(Ext.ComponentMgr.all.length)
// -> 8



The result that was expected:

'ext-comp-1019' should have been removed from the ComponentMgr along with the EditorGridPanel.


The result that occurs instead:

The component remains.

[evant]

A fix for this was added in rev 5982.

[meroy]

A fix for this was added in rev 5982.

This breaks the grid row-editor example under IE 6 and FF 3.6 (not tested with other browsers).

The grid row-editor example was working with rev 5981.