Write() : ambiguity on variable IDS which is being used for write itself and for store computation!

[David Janssens]

The loop that calls _store_set_values at the end of orm.write method overwrites the "ids" variable.
This corrupted "ids" variable is used later in the method for the calls to workflow trg_write.

[xrg (xrg)]

On Friday 14 January 2011, you wrote:
> Public bug reported:
>
> The loop that calls _store_set_values at the end of orm.write method
> overwrites the "ids" variable. This corrupted "ids" variable is used later
> in the method for the calls to workflow trg_write.
>

Can you please tell us the line number (in v6.0, latest trunk) where this
happens?

[Olivier Dony (Odoo) (odo-openerp)]

@xrg: the problem is the loop declaration itself, i.e.: http://bazaar.launchpad.net/~openerp/openobject-server/trunk/annotate/head:/bin/osv/orm.py#L3463

[Olivier Dony (Odoo) (odo-openerp)]

The fix landed in server revision 3266 <email address hidden>.
Will assign to maintenance team so they fix it in 5.0 too.

Thanks for reporting!

[Jay Vora (Serpent Consulting Services) (jayvora)]

Thanks Guys,
It has been fixed on 5.0 as well by revision 2170 <email address hidden>.