Odoo Addons (MOVED TO GITHUB)

Merge lp:~openerp-dev/openobject-addons/5.0-bug-751966-imp-nep into lp:openobject-addons/5.0

  1. 5.0-bug-751966-imp-nep
  2. Merge into 5.0
Proposed by Nehal Panchal (OpenERP)
Status: Merged
Merged at revision: 2910
Proposed branch: lp:~openerp-dev/openobject-addons/5.0-bug-751966-imp-nep
Merge into: lp:openobject-addons/5.0
Diff against target: 12 lines (+1/-1)
1 file modified
mrp/mrp.py (+1/-1)
To merge this branch: bzr merge lp:~openerp-dev/openobject-addons/5.0-bug-751966-imp-nep
Reviewer Review Type Date Requested Status
Anup(SerpentCS) (community) Approve
Review via email: mp+62633@code.launchpad.net

Description of the change

Hello,

  This fixes the wrong calculation of UoM factor in MRP.

Thanks.

To post a comment you must log in.
Revision history for this message
Anup(SerpentCS) (anup-serpent) wrote :

Hello Nehal,

   It's not working when I am having My BoM in 'm' and my Production Order in 'cm'. It gives wrongly calculated quantities.

  Kindly fix this.

Thanks.

review: Needs Fixing
Revision history for this message
Anup(SerpentCS) (anup-serpent) wrote :

Hello Nehal,

  It was a mis-configuration. It's working properly.

Thanks for the fix.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'mrp/mrp.py'
2--- mrp/mrp.py 2011-03-31 13:47:44 +0000
3+++ mrp/mrp.py 2011-05-27 09:07:23 +0000
4@@ -471,7 +471,7 @@
5 #if bom_point.routing_id and bom_point.routing_id.location_id:
6 # self.write(cr, uid, [production.id], {'location_src_id': bom_point.routing_id.location_id.id})
7
8- factor = production.product_qty * production.product_uom.factor_inv / bom_point.product_uom.factor
9+ factor = production.product_qty * production.product_uom.factor_inv * bom_point.product_uom.factor
10 res = self.pool.get('mrp.bom')._bom_explode(cr, uid, bom_point, factor / bom_point.product_qty, properties)
11 results = res[0]
12 results2 = res[1]