PDA

View Full Version : [CLOSED]An Ext.grid.GridPanel.processEvent fix proposal



Dmitry Ovsyanko
10 Mar 2010, 6:42 AM
Hello,

Sorry if it was already considered here.

GridPanel.js in v.3.1.1 contains the following fragment:



processEvent : function(name, e){

//....
this.view.processEvent(name, e);
},
I've changed it to



processEvent : function(name, e){

//....
if (this.view) this.view.processEvent(name, e);
},
and some js errors are gone (IE8, strict mode).

It seems that processEvent can be called when the grid is already destroyed (in my case, a js code erasing the top panel was loaded on cell click).

evant
10 Mar 2010, 7:01 AM
A similar fix has been added into SVN (the view now does all the event processing).