Review: Needs Fixing
26 + if uom and prod:
27 + if uom <> prod_uom_po:
28 + uom = prod_uom_po
combine these 2 if
37 + if 'uom_change' in context:
38 + uom = context['uom_change']
can be done like context.get('uom_change', uom)
its good if we keep all uom assignation in one section of code instead of
spreading those code in different lines
--
https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-782149-ron/+merge/61519
Your team OpenERP R&D Team is subscribed to branch
lp:~openerp-dev/openobject-addons/trunk-bug-782149-ron.
_______________________________________________
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