[4.1.0-RC2] Error defining mixins that themselves use Ext.util.Observable

[n3rd]

REQUIRED INFORMATION


Ext version tested:

• Ext 4.1.0-RC2 (broken)
• Ext 4.0.7 (works)

Browser versions tested against:

• Chrome 18
• IE9
• FF11

DOCTYPE tested against:

• does not apply

Description:

• defining mixins that themselves use the Observable mixin does not work

Steps to reproduce the problem:

• define a mixin M that itself has the mixin Ext.util.Observable
• define a class C that has mixin M

The result that was expected:

• when instantiated, class C should have both mixin M and Ext.util.Observable

The result that occurs instead:

• Ext bombs out with "this.prepareClass is not a function" in Method Ext.util.Observable.onClassMixedIn() when calling Ext.define('C', ...)

Test Case:

Ext.define('M', {
	mixins: {
		observable: 'Ext.util.Observable'
	},

	mixinMethod: function () {
		// do something
	}
});

Ext.define('C', {
	mixins: {
		m: 'M'
	},

	someMethod: function () {
		this.mixinMethod();
	}
});

HELPFUL INFORMATION


Screenshot or Video:

• none

See this URL for live test case: http://jsfiddle.net/yLACs/ (working version with Ext 4.0.7)


Debugging already done:

• none

Possible fix:

• not provided

Additional CSS used:

• does not apply

Operating System:

• Windows 7 Enterprise N

[scottmartin]

This has been assigned. Thank you for the report.

Regards,
Scott.

[n3rd]

This has not been fixed in 4.1, apparently. I do not think we are going to keep using Ext JS in the future.

[evant]

The issue has been resolved, for some reason the status hasn't been updated:

Ext.require('*');

Ext.onReady(function() {

    Ext.define('M', {
        mixins: {
            observable: 'Ext.util.Observable'
        },

        mixinMethod: function() {
            // do something
        }
    });

    Ext.define('C', {
        mixins: {
            m: 'M'
        },

        someMethod: function() {
            this.mixinMethod();
        }
    });
    
    var c = new C();
    console.log(c.fireEvent, c.mixinMethod);

});

[n3rd]

I still doesn't work for us. For some reason the core seems to be an RC still:

>> Ext.versions
Object { core=4.1.0rc, extjs=4.1.0}

Anyway, we'll stick to 4.0.7 for the time being. Other developers had poor experience with the support from Sencha as well and I think we're done with Ext JS anyway.

[evant]

Sounds like you don't have the latest version then (or perhaps a caching issue), you can grab 4.1 final from here: http://www.sencha.com/products/extjs/download/

[n3rd]

Not sure how it happened, but you were right. I've hotlinked http://dev.sencha.com/deploy/ext-4.1...t-all-debug.js for testing purposes, but it still doesn't work.

me.hasListeners is undefined (ext-all-debug.js (line 24600))

[evant]

I don't see that against 4.1.0 final. What happens if you go to:

http://dev.sencha.com/deploy/ext-4.1...rray-grid.html

Then drop this in the console:

Ext.define('M', {
        mixins: {
            observable: 'Ext.util.Observable'
        },

        mixinMethod: function() {
            // do something
        }
    });

    Ext.define('C', {
        mixins: {
            m: 'M'
        },

        someMethod: function() {
            this.mixinMethod();
        }
    });
    
    var c = new C();
    console.log(c.fireEvent, c.mixinMethod);

Also, when you say "doesn't work", at what point does it crash?

[n3rd]

It may work with the simplistic code snippet I provided initially, but it doesn't work in our real-life application.

Thanks for your effort, but don't bother. We're done with Ext JS, we've had too many BC-breaks even in minor releases.