openerp-dev-web team mailing list archive
-
openerp-dev-web team
-
Mailing list archive
-
Message #06036
Re: [Merge] lp:~openerp-dev/openobject-server/trunk-temporal-db-ksa into lp:~openerp-dev/openobject-server/trunk-temporal-db
Review: Needs Fixing
same story again...
*either* write each record.id separately inside for loop
*or* return write for all ids
vals is updated inside for loop, but not used there,
can be moved outside,
also updation of context can be checked, its updated with a same value for every record,
but take some time to test the behavior + code
--
https://code.launchpad.net/~openerp-dev/openobject-server/trunk-temporal-db-ksa/+merge/59035
Your team OpenERP R&D Team is subscribed to branch lp:~openerp-dev/openobject-server/trunk-temporal-db.
References