openerp-dev-web team mailing list archive

Thread
Date

Re: [Merge] lp:~openerp-dev/openobject-server/6.0-opw-4364-jvo into lp:openobject-server/6.0

To: "Jay Vora \(OpenERP\)" <jvo@xxxxxxxxxxx>
From: "Olivier Dony \(OpenERP\)" <odo@xxxxxxxxxxx>
Date: Fri, 04 Mar 2011 14:09:52 -0000
In-reply-to: <20110303143828.675.61933.launchpad@loganberry.canonical.com>
Reply-to: mp+52076@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

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.

References

[Merge] lp:~openerp-dev/openobject-server/6.0-opw-4364-jvo into lp:openobject-server/6.0
From: Jay Vora (OpenERP), 2011-03-03