Review: Approve code review, no test OK, sorry about the misunderstanding. Nevertheless, I would prefer to see some assertions, gets and tries as this function looks likely to throw an IndexError. But I see you've take them into account (removal of redundant checks, values checking, etc) -- https://code.launchpad.net/~therp-nl/server-env-tools/6.1-mass_editing-fix_dataloss/+merge/201321 Your team OpenERP Community is subscribed to branch lp:~openerp-community/server-env-tools/6.1-mass_editing.
_______________________________________________ Mailing list: https://launchpad.net/~openerp-community Post to : openerp-community@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-community More help : https://help.launchpad.net/ListHelp