Review: Needs Fixing

yeks... it doesn't sound good to me.

that's really weird that you changed in the module stock for a problem related 
to mrp. As the function action_consume is available also for stock features, 
you probably have introduced side effects such as the one that obliged you to 
modify the yaml test.

i think that a best idea would be to add an optional paramater in 
action_consume that would be the destination location: if it is filled, we use 
this one, otherwise we keep the current behavior. Then, in mrp you just have to 
make sure the destination location is correctly passed when calling this 
function.



-- 
https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-925351-mma/+merge/91642
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-addons/trunk-bug-925351-mma.

_______________________________________________
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