Success! Looks like we've fixed this one. According to our records the fix was applied for TOUCH-2616 in a recent build.
  1. #1
    Sencha Premium Member
    Join Date
    Apr 2009
    Posts
    193
    Vote Rating
    8
    jhoweaa is on a distinguished road

      0  

    Default CompositeElement and Ext.EventManager

    CompositeElement and Ext.EventManager


    I've been looking at some code which uses a CompositeElement and noticed that the code won't run unless I use the compatibility library. When using the regular library I get an error when some code in CompositeElementLite tries to use Ext.EventManager. The code in question from CompositeElementLite looks like this:

    Code:
    // fixes scope with flyweight
    addListener: function(eventName, handler, scope, opt) {
         var els = this.elements,
               len = els.length,
               i, e;
         for (i = 0; i < len; i++) {
                e = els[i];
                if (e) {
                    Ext.EventManager.on(e, eventName, handler, scope || e, opt);
                }
        }
        return this;
    }
    
    According to the documentation, Ext.EventManager is deprecated and shouldn't be used. If so, it doesn't seem like a base Sencha Touch class should be using it either.

    Did this one slip through?

    Thanks.

    Jim

  2. #2
    Sencha - Senior Forum Manager mitchellsimoens's Avatar
    Join Date
    Mar 2007
    Location
    Gainesville, FL
    Posts
    37,525
    Vote Rating
    871
    mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute

      0  

    Default


    Appears it has.
    Mitchell Simoens @SenchaMitch
    Sencha Inc, Senior Forum Manager
    ________________
    Check out my GitHub, lots of nice things for Ext JS 4 and Sencha Touch 2
    https://github.com/mitchellsimoens

    Think my support is good? Get more personalized support via a support subscription. https://www.sencha.com/store/

    Need more help with your app? Hire Sencha Services services@sencha.com

    Want to learn Sencha Touch 2? Check out Sencha Touch in Action that is in print!

    When posting code, please use BBCode's CODE tags.

  3. #3
    Sencha - Sencha Touch Dev Team rdougan's Avatar
    Join Date
    Oct 2008
    Posts
    1,158
    Vote Rating
    6
    rdougan will become famous soon enough

      0  

    Default


    Fixed for the next release. Thanks.
    Sencha Inc.
    Robert Dougan - @rdougan
    Sencha Touch 2 and Ext JS 4 Core Team Member, SASS/Theming Wizard.

Thread Participants: 2