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

Proposed by Mayur Maheshwari(OpenERP)
Status: Rejected
Rejected by: Fabien (Open ERP)
Proposed branch: lp:~openerp-dev/openobject-addons/trunk-opw-578768-port-mma
Merge into: lp:openobject-addons
Diff against target: 25 lines (+5/-3)
1 file modified
stock/wizard/stock_fill_inventory.py (+5/-3)
To merge this branch: bzr merge lp:~openerp-dev/openobject-addons/trunk-opw-578768-port-mma
Reviewer Review Type Date Requested Status
OpenERP Core Team Pending
Review via email: mp+132541@code.launchpad.net

Description of the change

Hello,

    I have 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.

   Code is forward port from 6.1

Thanks
Mayur

To post a comment you must log in.
Revision history for this message
Fabien (Open ERP) (fp-tinyerp) wrote :

was already fixed in another way

Unmerged revisions

7905. By Rifakat (OpenERP)

[FIX]stock : fixed import invertory wizard problem, restricted dummy moves qty to be added into total

7904. By Launchpad Translations on behalf of openerp

Launchpad automatic translations update.

7903. By Stephane Wirtel (OpenERP)

[MERGE] point_of_sale: Fix some bugs

7902. By Vo Minh Thu

[FIX] base_gengo: fixed RST warnings in module description.

7901. By Quentin (OpenERP) <email address hidden>

[MERGE] merged the branch that adds yaml test on account voucher

7900. By Cedric Snauwaert (OpenERP)

[FIX]module mrp_repair: remove duplicate cancel button uppon creation of a repair order

7899. By Vo Minh Thu

[FIX] sale, hr: fixed RST warnings in module descriptions.

7898. By Launchpad Translations on behalf of openerp

Launchpad automatic translations update.

7897. By Olivier Dony (Odoo)

[FIX] account: currency_id must be passed to the chart installation wizard

Otherwise the installation of a chart of account
via the settings page fails miserably.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'stock/wizard/stock_fill_inventory.py'
2--- stock/wizard/stock_fill_inventory.py 2012-09-29 10:51:04 +0000
3+++ stock/wizard/stock_fill_inventory.py 2012-11-01 12:55:26 +0000
4@@ -79,8 +79,6 @@
5
6 inventory_line_obj = self.pool.get('stock.inventory.line')
7 location_obj = self.pool.get('stock.location')
8- product_obj = self.pool.get('product.product')
9- stock_location_obj = self.pool.get('stock.location')
10 move_obj = self.pool.get('stock.move')
11 uom_obj = self.pool.get('product.uom')
12 if ids and len(ids):
13@@ -104,7 +102,11 @@
14 for location in location_ids:
15 datas = {}
16 res[location] = {}
17- move_ids = move_obj.search(cr, uid, ['|',('location_dest_id','=',location),('location_id','=',location),('state','=','done')], context=context)
18+ move_ids = move_obj.search(cr, uid, ['|', '&', ('location_id', '!=', location),
19+ ('location_dest_id', '=', location),
20+ '&', ('location_id', '=', location),
21+ ('location_dest_id', '!=', location),
22+ ('state', '=', 'done')], context=context)
23
24 for move in move_obj.browse(cr, uid, move_ids, context=context):
25 lot_id = move.prodlot_id.id

Subscribers

People subscribed via source and target branches

to all changes: