Merge lp:~openerp-dev/openobject-addons/6.1-opw-581120-rgo into lp:openobject-addons/6.1

[Ravi Gohil (OpenERP)]

Hello,

To reproduce/for better understanding of the issue, please refer to the video attached to the comment #3 of lp:1068152.

Currently, the code in onchange_lot_id() stock/stock.py does conversion from product's default UoM to the UoM selected in stock move, but it should be between UoM selected in stock move and reference UoM of that product, as the quantity value for lot is stored after converting it to reference UoM.

I have fixed this issue by doing the correct conversion. Kindly review the fix.

Thanks.

[Vinay Rana (OpenERP) (vra-openerp)]

The propose fix will solve the issue.

review: Approve

[Naresh(OpenERP) (nch-openerp)]

Hello,

This bug was qualified as Confirmed on Trunk (means still existing and reproducible). A Merge Proposal for trunk was created to fix it. Here is the link to follow the MP on Launchpad https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-opw-581120-port-mma/+merge/133230 and be informed once it's been merged in trunk: ... If this Merge Proposal could not be merged in v6.1 at the release of v7.0, it will be closed.

Thanks,
Naresh Soni

Unmerged revisions

7057. By Ravi Gohil (OpenERP) on 2012-11-05

[FIX] stock: If we have a product configured with a default UoM whose UoM type is set to other than reference uom, and when creating production lot for this product, it shows QTY in reference UoM and not in default UoM, this causes wrong warning message when assigning lot while creating picking : (Maintenance Case : 581120)

=== modified file 'stock/stock.py'
--- stock/stock.py	2012-10-05 07:47:57 +0000
+++ stock/stock.py	2012-11-05 11:33:31 +0000
@@ -1740,11 +1740,13 @@
         ctx.update({'raise-exception': True})
         uom_obj = self.pool.get('product.uom')
         product_obj = self.pool.get('product.product')
-        product_uom = product_obj.browse(cr, uid, product_id, context=ctx).uom_id
+        product_uom_categ = product_obj.browse(cr, uid, product_id, context=ctx).uom_id.category_id.id
+        product_ref_uom = uom_obj.search(cr, uid, [('category_id', '=', product_uom_categ), ('uom_type', '=', 'reference')], context=ctx)[0]
+        product_ref_uom = uom_obj.browse(cr, uid, product_ref_uom, context=ctx)
         prodlot = self.pool.get('stock.production.lot').browse(cr, uid, prodlot_id, context=ctx)
         location = self.pool.get('stock.location').browse(cr, uid, loc_id, context=ctx)
         uom = uom_obj.browse(cr, uid, uom_id, context=ctx)
-        amount_actual = uom_obj._compute_qty_obj(cr, uid, product_uom, prodlot.stock_available, uom, context=ctx)
+        amount_actual = uom_obj._compute_qty_obj(cr, uid, product_ref_uom, prodlot.stock_available, uom, context=ctx)
         warning = {}
         if (location.usage == 'internal') and (product_qty > (amount_actual or 0.0)):
             warning = {