Merge lp:~openerp-dev/openobject-addons/6.0-opw-572997-ado into lp:openobject-addons/6.0

[Amit Dodiya (OpenERP)]

Hello,

[FIX] stock : Fixing a uos qty 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 Back-ported from trunk.

Regards,
Amit

[Naresh(OpenERP) (nch-openerp)]

review: Approve

[lp:~openerp-dev/openobject-addons/6.0-opw-572997-ado]

5153. By Xavier ALT on 2012-04-27

[FIX] stock: fix partial picking UoS convertion - missing part

5154. By Xavier ALT on 2012-04-30

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

[Numérigraphe (numerigraphe)]

Has this been merged yet ?

Unmerged revisions

5154. By Xavier ALT on 2012-04-30

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

5153. By Xavier ALT on 2012-04-27

[FIX] stock: fix partial picking UoS convertion - missing part

5152. By Amit Dodiya (OpenERP) on 2012-03-21

    [FIX] stock : Fixing a uos qty problem

=== modified file 'stock/stock.py'
--- stock/stock.py	2012-04-30 14:27:39 +0000
+++ stock/stock.py	2012-04-30 16:27:21 +0000
@@ -23,6 +23,7 @@
 import time
 from operator import itemgetter
 from itertools import groupby
+from functools import partial
 
 from osv import fields, osv
 from tools.translate import _
@@ -1239,6 +1240,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:
@@ -1251,7 +1254,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,
@@ -1264,7 +1267,7 @@
                 move_obj.write(cr, uid, [move.id],
                         {
                             'product_qty' : move.product_qty - product_qty,
-                            'product_uos_qty':move.product_qty - product_qty, #TODO: put correct uos_qty
+                            'product_uos_qty': convert_to_uos(move.product_qty - product_qty, to_uos),
                         })
 
             if new_picking:
@@ -1273,10 +1276,12 @@
                 if prodlot_ids.get(move.id):
                     move_obj.write(cr, uid, [move.id], {'prodlot_id': prodlot_ids[move.id]})
             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
                 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),
                 }
                 prodlot_id = prodlot_ids.get(move.id)
                 if prodlot_ids.get(move.id):
@@ -2479,11 +2484,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,
@@ -2497,15 +2504,17 @@
             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),
                     })
 
 
         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)