Wizard "Fill inventory" creates wrong lines for moves where source=destination
Bug #1228156 reported by
Numérigraphe
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Odoo Addons (MOVED TO GITHUB) |
Fix Released
|
Undecided
|
OpenERP R&D Addons Team 2 | ||
Bug Description
When fixing bug #797772, one corner case was not taken into account : stock moves where source=location.
Indeed, the program searches for moves in the location, and then if the location is the destination the quantity is added, otherwise it is subtracted.
But when the location is both the source AND the destination of the move, it should be ignored.
Lionel Sausin
Related branches
lp:~numerigraphe/openobject-addons/6.0-inventory-dummy-moves
- OpenERP Core Team: Pending requested
-
Diff: 80 lines (+54/-0)3 files modifiedstock/__openerp__.py (+1/-0)
stock/test/stock_inventory_dummy_moves.yml (+51/-0)
stock/wizard/stock_fill_inventory.py (+2/-0)
lp:~numerigraphe-team/ocb-addons/7.0-inventory-dummy-moves-test
- Holger Brunn (Therp): Disapprove
- Stefan Rijnhart (Opener): Needs Information
- Loïc Bellier - Numérigraphe (community): Approve (code review)
-
Diff: 63 lines (+48/-0)2 files modifiedstock/__openerp__.py (+1/-0)
stock/test/stock_inventory_dummy_moves.yml (+47/-0)
lp:~numerigraphe-team/openobject-addons/trunk-wms-inventory-dummy-moves-test
- qdp (OpenERP): Pending requested
- OpenERP Core Team: Pending requested
-
Diff: 50 lines (+35/-0)2 files modifiedstock/__openerp__.py (+1/-0)
stock/test/stock_inventory_dummy_moves.yml (+34/-0)
Revision history for this message
| Numérigraphe (numerigraphe) wrote | #1 |
| Changed in openobject-addons: | |
| assignee: | nobody → Numérigraphe (numerigraphe) |
Revision history for this message
| Numérigraphe (numerigraphe) wrote | #2 |
| Changed in openobject-addons: | |
| status: | New → Fix Committed |
| assignee: | Numérigraphe (numerigraphe) → nobody |
| Changed in openobject-addons: | |
| assignee: | nobody → OpenERP R&D Addons Team 2 (openerp-dev-addons2) |
Revision history for this message
| Numérigraphe (numerigraphe) wrote | #3 |
Revision history for this message
| Numérigraphe (numerigraphe) wrote | #4 |
| Changed in openobject-addons: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
This can be fixed simply checking that the location is not the source before adding the quantity.
We'll be posting a branch with a fix and a regression test ASAP.
Lionel