1. #1
    Sencha - Ext JS Dev Team Animal's Avatar
    Join Date
    Mar 2007
    Location
    Notts/Redwood City
    Posts
    30,570
    Vote Rating
    53
    Animal will become famous soon enough Animal will become famous soon enough

      0  

    Default [2.0.1] Observable.addEvents does not add properties to the events object

    The code does not handle being passed multiple string event names as parameters.

    The current code is:

    Code:
        addEvents : function(o){
            if(!this.events){
                this.events = {};
            }
            if(typeof o == 'string'){
                for(var i = 0, a = arguments, v; v = a[i]; i++){
                    if(!this.events[a[i]]){
                        o[a[i]] = true;
                    }
                }
            }else{
                Ext.applyIf(this.events, o);
            }
        },
    It should be

    Code:
        addEvents : function(o){
            if(!this.events){
                this.events = {};
            }
            if(typeof o == 'string'){
                for(var i = 0, a = arguments, v = o; v = a[i]; i++){
                    if(!this.events[v]){
                        this.events[v] = true;
                    }
                }
            }else{
                Ext.applyIf(this.events, o);
            }
        },

  2. #2
    Sencha - Community Support Team Condor's Avatar
    Join Date
    Mar 2007
    Location
    The Netherlands
    Posts
    24,246
    Vote Rating
    111
    Condor has much to be proud of Condor has much to be proud of Condor has much to be proud of Condor has much to be proud of Condor has much to be proud of Condor has much to be proud of Condor has much to be proud of Condor has much to be proud of Condor has much to be proud of

      0  

    Default DUP

    This has been reported before.

Thread Participants: 1