Review: Needs Fixing

Hello Amit,

There is a scope to improve here...

        +            if not uom:
        +                uom = product_obj.uom_id and product_obj.uom_id.id or 
False

        - result['product_uom'] = product_obj.uom_id and product_obj.uom_id.id 
or False
        + result['product_uom'] = uom

then no need of this change too
-                    result.update({'price_unit': price, 'price_subtotal': 
price*product_uom_qty})
19      -
20      +                    result.update({'price_unit': price, 
'price_subtotal': price*product_uom_qty, 'product_uom': uom})


Thanks,
-- 
https://code.launchpad.net/~openerp-dev/openobject-addons/6.0-opw-57629-ado/+merge/87919
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-addons/6.0-opw-57629-ado.

_______________________________________________
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