Thank you for reporting this bug. We will make it our priority to review this report.
  1. #1
    Sencha Premium Member tobiu's Avatar
    Join Date
    May 2007
    Location
    Munich (Germany)
    Posts
    2,695
    Vote Rating
    114
    tobiu is a name known to all tobiu is a name known to all tobiu is a name known to all tobiu is a name known to all tobiu is a name known to all tobiu is a name known to all

      0  

    Default [CLOSED][3.3.0-rc]Ext.grid.ColumnModel -> destroy

    [CLOSED][3.3.0-rc]Ext.grid.ColumnModel -> destroy


    hi team,

    this method seems to have changed in 3.3:

    Code:
        destroy : function() {
            var length = this.config.length,
                i = 0;
    
            for (; i < length; i++){
                this.config[i].destroy(); // Column's destroy encapsulates all cleanup.
            }
            delete this.config;
            delete this.lookup;
            this.purgeListeners();
        }
    i sometimes get the error in firebug:

    this.config[i].destroy is not a function
    [Break on this error] this.config[i].destroy();

    i guess it happens when a column or the cm itself is not yet rendered when calling destroy().
    adding a check for this should help.


    kind regards
    tobiu
    Best regards
    Tobias Uhlig
    __________

    S-CIRCLES Social Network Engine

  2. #2
    Sencha Premium Member tobiu's Avatar
    Join Date
    May 2007
    Location
    Munich (Germany)
    Posts
    2,695
    Vote Rating
    114
    tobiu is a name known to all tobiu is a name known to all tobiu is a name known to all tobiu is a name known to all tobiu is a name known to all tobiu is a name known to all

      0  

    Default


    update: the theory was wrong. the error occured when using sakis rowactions-plugin with 3.3 (the column does not have a destroy-method).

    so, this is not really an ext bug. i added a check if destroy exists, just in case.

    Code:
    Ext.override(Ext.grid.ColumnModel, {
        destroy : function() {
            var length = this.config.length,
                i = 0;
    
            for (; i < length; i++){
                if(typeof this.config[i].destroy == 'function'){
                    this.config[i].destroy(); // Column's destroy encapsulates all cleanup.
                }
            }
            delete this.config;
            delete this.lookup;
            this.purgeListeners();
        }
    });
    Best regards
    Tobias Uhlig
    __________

    S-CIRCLES Social Network Engine

  3. #3
    Sencha - Ext JS Dev Team Animal's Avatar
    Join Date
    Mar 2007
    Location
    Notts/Redwood City
    Posts
    30,508
    Vote Rating
    58
    Animal has a spectacular aura about Animal has a spectacular aura about Animal has a spectacular aura about

      0  

Similar Threads

  1. Replies: 11
    Last Post: 3 Jan 2011, 12:31 PM
  2. Replies: 0
    Last Post: 13 Mar 2009, 12:53 AM
  3. [2.01b1][CLOSED] Ext.grid.ColumnModel config: hidden
    By mjlecomte in forum Ext 2.x: Bugs
    Replies: 2
    Last Post: 4 Nov 2007, 10:06 AM
  4. [Solved] Ext 1.1: How to set column style (css) in Ext.grid.ColumnModel config?
    By Phenothiasine in forum Ext 1.x: Help & Discussion
    Replies: 5
    Last Post: 10 Aug 2007, 9:28 AM
  5. [Ext.BasicDialog].destroy(true) doesn't destroy shadow
    By diegovilar in forum Ext 1.x: Bugs
    Replies: 1
    Last Post: 25 Mar 2007, 12:42 PM

Thread Participants: 1