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

[Mayur Maheshwari(OpenERP)]

Hello,

step to reproduce:

1] There are 2 UOM
  -kg ( reference unit)
 - gm (Smaller than the reference UoM and ratio 1000)
2] Make a product like "test product" with purchase unit = g and uom = g
3] From incoming shipment process 500 gram "test product" to stock location and make lot like "00001"
4] Now from internal move change location stock to self -1 and apply lot as already create "00001"
   so here raise a warning message

    Insufficient Stock in Lot !
    You are moving 500gm products but only 0.50 gm available in this lot.

For better understanding of the issue, please refer to the video attached to the comment #3 of lp:1068152.

This branch fixes this issue
code is forward port from 6.1

Thanks,
Mayur

[Fabien (Open ERP) (fp-tinyerp)]

The fix is not clean. There reference UoM may not exist. It's not a good practice to search on this. You should just multiply or divise by the rate of the product UoM.

review: Disapprove

[Mayur Maheshwari(OpenERP) (mma-openerp)]

Hello ,

    I have Improve onchange and make qty of multiply with its factor

Thanks

review: Needs Resubmitting

Unmerged revisions

7965. By Mayur Maheshwari(OpenERP) on 2012-12-06

[IMP]stock: improve a code of onchage and make qty of multiply with its factor

7964. By Ravi Gohil (OpenERP) on 2012-11-07

[FIX]account_voucher : 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

=== modified file 'stock/stock.py'
--- stock/stock.py	2012-11-06 15:04:31 +0000
+++ stock/stock.py	2012-11-07 12:23:24 +0000
@@ -1881,11 +1881,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_id = 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_id, 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 = {