Review: Approve
In its current state, this completely breaks the listview code (it uses events
a lot, and since event methods have been renamed they're now all broken…)
Also,
* The behavioral change of the do_ and on_ methods (and the split, they used to
do the same thing but now they don't at all) needs to be documented
* 'far's I can tell, the old binding issue (which prevents overriding a method
on an instance via its prototype for e.g. webclient or connection) is still
present in this right?
* Why go the long way round to write, in effect,
callback.add_last = callback.add_first = callback.add
?
--
https://code.launchpad.net/~openerp-dev/openerp-web/trunk-event-refactoring-al/+merge/91377
Your team OpenERP R&D Team is subscribed to branch
lp:~openerp-dev/openerp-web/trunk-event-refactoring-al.
_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-gtk
Post to : [email protected]
Unsubscribe : https://launchpad.net/~openerp-dev-gtk
More help : https://help.launchpad.net/ListHelp