Thank you for reporting this bug. We will make it our priority to review this report.
  1. #1
    Ext User
    Join Date
    Feb 2010
    Posts
    1
    Vote Rating
    0
    Dmitry Ovsyanko is on a distinguished road

      0  

    Default [CLOSED]An Ext.grid.GridPanel.processEvent fix proposal

    [CLOSED]An Ext.grid.GridPanel.processEvent fix proposal


    Hello,

    Sorry if it was already considered here.

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

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

    Code:
        
        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).

  2. #2
    Sencha - Ext JS Dev Team evant's Avatar
    Join Date
    Apr 2007
    Location
    Sydney, Australia
    Posts
    16,655
    Vote Rating
    583
    evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute

      0  

    Default


    A similar fix has been added into SVN (the view now does all the event processing).
    Evan Trimboli
    Sencha Developer
    Twitter - @evantrimboli
    Don't be afraid of the source code!

Thread Participants: 1