[6.1][7.0][trunk][stock] Move destination and partial picking
Affects | Status | Importance | Assigned to | Milestone | ||
---|---|---|---|---|---|---|
Odoo Addons (MOVED TO GITHUB) |
Fix Committed
|
Low
|
OpenERP R&D Addons Team 2 | |||
OpenERP Community Backports (Addons) | Status tracked in 7.0 | |||||
6.1 |
Fix Released
|
Low
|
Yann Papouin | |||
7.0 |
Fix Committed
|
Low
|
Yann Papouin |
Bug Description
When doing a partial picking, if the quantity received/sent is lesser than the expected one, then a new move with the set quantity is created and the move is set to done by workflow.
The corresponding code is:
if product_qty != 0:
defaults = {
}
I think that the "move_dest_id" value should be kept from the original move for easier traceability.
if product_qty != 0:
defaults = {
}
Related branches
- OpenERP Core Team: Pending requested
-
Diff: 21 lines (+2/-2)1 file modifiedstock/stock.py (+2/-2)
- OpenERP Core Team: Pending requested
-
Diff: 21 lines (+2/-2)1 file modifiedstock/stock.py (+2/-2)
- Omar (Pexego): Approve (code review)
- Guewen Baconnier @ Camptocamp: Approve
- Holger Brunn (Therp): Approve (code review)
-
Diff: 21 lines (+2/-2)1 file modifiedstock/stock.py (+2/-2)
- OpenERP Core Team: Pending requested
-
Diff: 21 lines (+2/-2)1 file modifiedstock/stock.py (+2/-2)
- Sandy Carter (http://www.savoirfairelinux.com): Approve (code review, pep8)
- Yannick Vaucher @ Camptocamp: Approve (code review, no tests)
- Holger Brunn (Therp): Approve (code review)
-
Diff: 21 lines (+2/-2)1 file modifiedstock/stock.py (+2/-2)
Changed in openobject-addons: | |
assignee: | nobody → OpenERP R&D Addons Team 2 (openerp-dev-addons2) |
importance: | Undecided → Low |
status: | New → Confirmed |
Changed in openobject-addons: | |
status: | Confirmed → In Progress |
Changed in ocb-addons: | |
assignee: | nobody → Yann Papouin (yann-papouin) |
importance: | Undecided → Low |
Changed in ocb-addons: | |
status: | New → Fix Committed |
Please note that this code is partially duplicated in the same file, so the code change should be applied in both parts:
in: osv.osv) :
class stock_picking(
# FIXME: needs refactoring, this code is partially duplicated in stock_move. do_partial( )!
def do_partial(self, cr, uid, ids, partial_datas, context=None):
and: osv.osv) :
class stock_move(
# FIXME: needs refactoring, this code is partially duplicated in stock_picking. do_partial( )!
def do_partial(self, cr, uid, ids, partial_datas, context=None):