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

Reply via email to