Merge lp:~openerp-dev/openobject-addons/trunk-opw-572997-port-mma into lp:openobject-addons

[Mayur Maheshwari(OpenERP)]

Hello,

[FIX] stock : Fixing a uos quantity problem

Steps:
1). Create a product with default_UOM = g, quantity = 4000
2). Create sale order with above product with Quantity(UOM) = 4000 g, Quantity(UOS)=4 kg
3). Confirm the sale order, Open the related delivery order
4). Process it with Quantity = 500 g
5). Open the current delivery order line you will see the Quantity(UOM) = 3500 g and
Quantity(UOS) = 3500 kg, Quantity(UOS) should be 3.5 kg
6). Open the related backorder same issue here.

=> code is forward port from 6.1

Thanks,
Mayur

Unmerged revisions

8147. By Xavier ALT (OpenERP) on 2012-11-28

[FIX]stock: correctly convert partial picking/move product UoS quantity based on product UoM quantity

=== modified file 'stock/stock.py'
--- stock/stock.py	2012-11-26 18:28:11 +0000
+++ stock/stock.py	2012-11-28 09:14:20 +0000
@@ -25,6 +25,7 @@
 import time
 from operator import itemgetter
 from itertools import groupby
+from functools import partial
 
 from osv import fields, osv
 from tools.translate import _
@@ -1290,6 +1291,8 @@
 
 
             for move in too_few:
+                convert_to_uos = partial(uom_obj._compute_qty_obj, cr, uid, move.product_uom, context=context)
+                to_uos = move.product_uos or move.product_uom
                 product_qty = move_product_qty[move.id]
                 if not new_picking:
                     new_picking_name = pick.name
@@ -1306,7 +1309,7 @@
                 if product_qty != 0:
                     defaults = {
                             'product_qty' : product_qty,
-                            'product_uos_qty': product_qty, #TODO: put correct uos_qty
+                            'product_uos_qty': convert_to_uos(product_qty, to_uos),
                             'picking_id' : new_picking,
                             'state': 'assigned',
                             'move_dest_id': False,
@@ -1320,7 +1323,7 @@
                 move_obj.write(cr, uid, [move.id],
                         {
                             'product_qty': move.product_qty - partial_qty[move.id],
-                            'product_uos_qty': move.product_qty - partial_qty[move.id], #TODO: put correct uos_qty
+                            'product_uos_qty': convert_to_uos(move.product_qty - partial_qty[move.id], to_uos),
                             'prodlot_id': False,
                             'tracking_id': False,
                         })
@@ -1336,7 +1339,7 @@
                 product_qty = move_product_qty[move.id]
                 defaults = {
                     'product_qty' : product_qty,
-                    'product_uos_qty': product_qty, #TODO: put correct uos_qty
+                    'product_uos_qty': convert_to_uos(product_qty, to_uos),
                     'product_uom': product_uoms[move.id]
                 }
                 prodlot_id = prodlot_ids.get(move.id)
@@ -2758,11 +2761,13 @@
                                 })
 
         for move in too_few:
+            convert_to_uos = partial(uom_obj._compute_qty_obj, cr, uid, move.product_uom, context=context)
+            to_uos = move.product_uos or move.product_uom
             product_qty = move_product_qty[move.id]
             if product_qty != 0:
                 defaults = {
                             'product_qty' : product_qty,
-                            'product_uos_qty': product_qty,
+                            'product_uos_qty': convert_to_uos(product_qty, to_uos),
                             'picking_id' : move.picking_id.id,
                             'state': 'assigned',
                             'move_dest_id': False,
@@ -2776,17 +2781,19 @@
             self.write(cr, uid, [move.id],
                     {
                         'product_qty': move.product_qty - product_qty,
-                        'product_uos_qty': move.product_qty - product_qty,
+                        'product_uos_qty': convert_to_uos(move.product_qty - product_qty, to_uos),
                         'prodlot_id': False,
                         'tracking_id': False,
                     })
 
 
         for move in too_many:
+            convert_to_uos = partial(uom_obj._compute_qty_obj, cr, uid, move.product_uom, context=context)
+            to_uos = move.product_uos or move.product_uom
             self.write(cr, uid, [move.id],
                     {
                         'product_qty': move.product_qty,
-                        'product_uos_qty': move.product_qty,
+                        'product_uos_qty': convert_to_uos(move.product_qty, to_uos),
                     })
             complete.append(move)