Review: Approve
I would expect the defaults for empty o2m to be False rather than an empty list 
or tuple, but I guess it's better to take care of this case too.
The set() method of fields.one2many will handle False or [] properly, so we're 
safe there too.
Ship it :-)
-- 
https://code.launchpad.net/~openerp-dev/openobject-server/6.0-opw-4364-jvo/+merge/52076
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-server/6.0-opw-4364-jvo.

_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-web
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openerp-dev-web
More help   : https://help.launchpad.net/ListHelp

Reply via email to