← Back to team overview

openerp-dev-web team mailing list archive

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