Merge lp:~colin-zaber/openobject-addons/colin-bugfix_product_purchase into lp:openobject-addons

Proposed by Colin Ligertwood
Status: Needs review
Proposed branch: lp:~colin-zaber/openobject-addons/colin-bugfix_product_purchase
Merge into: lp:openobject-addons
To merge this branch: bzr merge lp:~colin-zaber/openobject-addons/colin-bugfix_product_purchase
Reviewer Review Type Date Requested Status
OpenERP Core Team Pending
Review via email: mp+181646@code.launchpad.net

Description of the change

In order to fix a price calculation bug in purchase order creation, in product/pricelist.py: I've reversed the order of two arguments to a call to ._compute_qty() in price_get_multi() to what seems the correct order.

Before this change, unit of measure conversions would be performed backwards here, resulting in fractions of one unit being added to purchase orders.

To post a comment you must log in.

Unmerged revisions

7244. By Colin Ligertwood

[FIX] change order of arguments to product_uom_obj._compute_qty in product.pricelist.price_get_multi to what seems the correct order.
 - qty_in_product_uom = product_uom_obj._compute_qty(cr, uid, product_default_uom, qty, to_uom_id=seller_uom)
 + qty_in_product_uom = product_uom_obj._compute_qty(cr, uid, seller_uom, qty, to_uom_id=product_default_uom)

7243. By Launchpad Translations on behalf of openerp

Launchpad automatic translations update.

7242. By Launchpad Translations on behalf of openerp

Launchpad automatic translations update.

7241. By Launchpad Translations on behalf of openerp

Launchpad automatic translations update.

7240. By Launchpad Translations on behalf of openerp

Launchpad automatic translations update.

7239. By Launchpad Translations on behalf of openerp

Launchpad automatic translations update.

7238. By Launchpad Translations on behalf of openerp

Launchpad automatic translations update.

7237. By Launchpad Translations on behalf of openerp

Launchpad automatic translations update.

7236. By Launchpad Translations on behalf of openerp

Launchpad automatic translations update.

7235. By Launchpad Translations on behalf of openerp

Launchpad automatic translations update.

Subscribers

People subscribed via source and target branches

to all changes: