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/trunk-bug-894648-amp/+merge/103980
Your team OpenERP R&D Team is subscribed to branch
lp:~openerp-dev/openobject-addons/trunk-bug-894648-amp.
_______________________________________________
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
