-
Notifications
You must be signed in to change notification settings - Fork 42
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
"Ext.util.Observable#addEvents" is deprecated. #26
Comments
You are right. That piece of code is there since the |
What do I need to do in order to get this to work with Ext JS 5? |
I'd be very interested in seeing this project modified to work with Ext JS 5. |
im working on it. soon it will be availible ... about Ext.util.Observable#addEvents" is deprecated. ... just remove it - in ExtJS 5 - Events added automatically |
@Grey2k, @ivan-novakov any chances there will be ExtJS5 version in near future? |
so, all i should have to do is remove addEvents from all files? |
yep ) |
Gyus - its almost done ) Today i start tested it in production and soon make the resease. its will work only on ExtJS > 5.0.2 cause this bug selModel : Ext.create('Ext.selection.CheckboxModel', {
checkOnly: true, // not working
}); |
Using Ext 5.0.1, you will get the error: "Ext.util.Observable#addEvents" is deprecated.
See this ticket (http://www.sencha.com/forum/showthread.php?198969-Observable-deprecate-addEvents)
"You don't need the addEvents call. The event will get added the first time it's fired, or the first time a listener is added. Whichever comes first."
I just commented out the addEvent calls to make it work.
You can follow their suggested method of documenting the events like so:
The text was updated successfully, but these errors were encountered: