openerp-dev-web team mailing list archive

["Nehal Panchal \(Open ERP\)" <nep@xxxxxxxxxxx>]

Nehal Panchal (Open ERP) has proposed merging lp:~openerp-dev/openobject-addons/5.0-bug-751966-imp-nep into lp:openobject-addons/5.0.

Requested reviews:
  Anup(OpenERP) (ach-openerp)
Related bugs:
  Bug #751966 in OpenERP Addons: "Bad factor calculation en mrp_production action_compute"
  https://bugs.launchpad.net/openobject-addons/+bug/751966

For more details, see:
https://code.launchpad.net/~openerp-dev/openobject-addons/5.0-bug-751966-imp-nep/+merge/62633

Hello,

  This fixes the wrong calculation of UoM factor in MRP.

Thanks.
-- 
https://code.launchpad.net/~openerp-dev/openobject-addons/5.0-bug-751966-imp-nep/+merge/62633
Your team OpenERP R&D Team is subscribed to branch lp:~openerp-dev/openobject-addons/5.0-bug-751966-imp-nep.

=== modified file 'mrp/mrp.py'
--- mrp/mrp.py	2011-03-31 13:47:44 +0000
+++ mrp/mrp.py	2011-05-27 09:07:23 +0000
@@ -471,7 +471,7 @@
             #if bom_point.routing_id and bom_point.routing_id.location_id:
             #   self.write(cr, uid, [production.id], {'location_src_id': bom_point.routing_id.location_id.id})
 
-            factor = production.product_qty * production.product_uom.factor_inv / bom_point.product_uom.factor
+            factor = production.product_qty * production.product_uom.factor_inv * bom_point.product_uom.factor
             res = self.pool.get('mrp.bom')._bom_explode(cr, uid, bom_point, factor / bom_point.product_qty, properties)
             results = res[0]
             results2 = res[1]