Review: Needs Fixing code review

@Stefan: I thin you missed the point of vals[model_field]:
There is a chance of a KeyError if model_field not in val and a chance of index 
error if len(vals[model_field]) < 1 or len(vals[model_field][0]) < 3
-- 
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

Reply via email to