Merge lp:~acsone-openerp/stock-logistic-warehouse/7.0-inventory-hierarchical-fill-lga into lp:~numerigraphe-team/stock-logistic-warehouse/7.0-inventory-hierarchical
Status: | Superseded |
---|---|
Proposed branch: | lp:~acsone-openerp/stock-logistic-warehouse/7.0-inventory-hierarchical-fill-lga |
Merge into: | lp:~numerigraphe-team/stock-logistic-warehouse/7.0-inventory-hierarchical |
Diff against target: |
343 lines (+259/-14) 8 files modified
stock_inventory_hierarchical_location/__openerp__.py (+1/-1) stock_inventory_hierarchical_location/inventory_hierarchical_location.py (+19/-9) stock_inventory_hierarchical_location/inventory_hierarchical_location_view.xml (+1/-1) stock_inventory_hierarchical_location/tests/__init__.py (+39/-0) stock_inventory_hierarchical_location/tests/fill_inventory_test.py (+117/-0) stock_inventory_hierarchical_location/wizard/__init__.py (+1/-0) stock_inventory_hierarchical_location/wizard/stock_fill_location_inventory.py (+76/-0) stock_inventory_location/stock_inventory_location.py (+5/-3) |
To merge this branch: | bzr merge lp:~acsone-openerp/stock-logistic-warehouse/7.0-inventory-hierarchical-fill-lga |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Lionel Sausin - Initiatives/Numérigraphe | has conflicts | Needs Fixing | |
Review via email: mp+222965@code.launchpad.net |
This proposal has been superseded by a proposal from 2014-06-12.
Description of the change
I make some change in the module stock_inventory
The goal is to take account of sub-inventories in case of an exhaustive inventory in two case :
-confirm inventory
-fill inventory
Unmerged revisions
- 39. By Laetitia Gangloff (Acsone)
-
stock_inventory
_hierarchical_ location: to fill inventory, check if it is an exhaustive fill to define if location should be exclude - 38. By Laetitia Gangloff (Acsone)
-
stock_inventory
_hierarchical_ location: use sub-inventories to determine missing locations and to fill inventory with the fill_inventory wizard - move test in tests and add some python test for the previous issue / stock_inventory _location: correct a mysterious problem that hapenned when many inventory ids are passed
Thanks for your help.
I've just merged our own fixes of the day, will you please merge them to avoid conflicts ?
I think I understand what you're trying to do in fill_inventory and it's not exactly how we had it in v6 - which is broken right now in v7, and would require a patch on the code addons.
Your patch completely excludes the locations of sub-inventories, whereas we used to only exclude the generated inventory lines. That would allow us to "catch" the lines that could have been missing from sub-inventories, but since they're exhaustive anyway there should not be any.
So I guess your approach is correct.
I'll let Loïc decide, he's working on the problem too.