Rifakat (OpenERP) has proposed merging 
lp:~openerp-dev/openobject-addons/6.1-opw-578768-rha into 
lp:openobject-addons/6.1.

Requested reviews:
  Naresh(OpenERP) (nch-openerp)
Related bugs:
  Bug #898498 in OpenERP Addons: "[6.0.3] Physical inventory - import inventory 
action broken"
  https://bugs.launchpad.net/openobject-addons/+bug/898498
  Bug #1045334 in OpenERP Addons: "import inventory"
  https://bugs.launchpad.net/openobject-addons/+bug/1045334

For more details, see:
https://code.launchpad.net/~openerp-dev/openobject-addons/6.1-opw-578768-rha/+merge/122627

Hello,

Fixed problem of 'Fill Inventory' wizards which adds qty for dummy moves also,
Ex. Stock -> Stock

If we try to fill inventory for location 'Stock',
It search for all the stock movements with location 'Stock' as Source or 
Destination
which includes dummy movement too and adds qty for dummy movement which results 
into
posting wrong move for inventory.

Detail steps are provided in lp:1045334

Please review this fix.

Regards,
Rifakat
-- 
https://code.launchpad.net/~openerp-dev/openobject-addons/6.1-opw-578768-rha/+merge/122627
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-addons/6.1-opw-578768-rha.
=== modified file 'stock/wizard/stock_fill_inventory.py'
--- stock/wizard/stock_fill_inventory.py	2011-12-08 05:58:41 +0000
+++ stock/wizard/stock_fill_inventory.py	2012-09-04 08:00:28 +0000
@@ -67,8 +67,6 @@
 
         inventory_line_obj = self.pool.get('stock.inventory.line')
         location_obj = self.pool.get('stock.location')
-        product_obj = self.pool.get('product.product')
-        stock_location_obj = self.pool.get('stock.location')
         move_obj = self.pool.get('stock.move')
         uom_obj = self.pool.get('product.uom')
         if ids and len(ids):
@@ -93,8 +91,9 @@
             datas = {}
             res[location] = {}
             move_ids = move_obj.search(cr, uid, ['|',('location_dest_id','=',location),('location_id','=',location),('state','=','done')], context=context)
+            moves = [move for move in move_obj.browse(cr, uid, move_ids, context=context) if move.location_id != move.location_dest_id]
 
-            for move in move_obj.browse(cr, uid, move_ids, context=context):
+            for move in moves:
                 lot_id = move.prodlot_id.id
                 prod_id = move.product_id.id
                 if move.location_dest_id.id == location:

_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-gtk
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openerp-dev-gtk
More help   : https://help.launchpad.net/ListHelp

Reply via email to