Review: Needs Fixing

well this fix seems some what complicated when a user tries to update the field 
externally either through say xmlrpc script.

you check the category ID for old and new along with if the move exist or not 
 product_move = self.pool.get('stock.move').search(cr, uid, [('product_id', 
'=', ids[0])])

but you do the move check for just the first product and not all the product 
that are to be written.

say for eg: I want product with ID 4,5 to change there default UOM from PCE to 
KM and I have stock move for the product 5. so actually this should not allow 
me to update but in your case it does.

can you please recheck.

Thanks,
Naresh

-- 
https://code.launchpad.net/~openerp-dev/openobject-addons/6.1-opw-576033-rgo/+merge/111150
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-addons/6.1-opw-576033-rgo.

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

Reply via email to