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

Reply via email to