Merge lp:~camptocamp/stock-logistic-flows/7.0-picking_dispatch-duplicate-move-1281471 into lp:stock-logistic-flows

Proposed by Guewen Baconnier @ Camptocamp
Status: Merged
Approved by: Alexandre Fayolle - camptocamp
Approved revision: 44
Merged at revision: 50
Proposed branch: lp:~camptocamp/stock-logistic-flows/7.0-picking_dispatch-duplicate-move-1281471
Merge into: lp:stock-logistic-flows
Diff against target: 18 lines (+8/-0)
1 file modified
picking_dispatch/picking_dispatch.py (+8/-0)
To merge this branch: bzr merge lp:~camptocamp/stock-logistic-flows/7.0-picking_dispatch-duplicate-move-1281471
Reviewer Review Type Date Requested Status
Yannick Vaucher @ Camptocamp code review Approve
Alexandre Fayolle - camptocamp code review, no test Approve
Pedro Manuel Baeza code review Approve
Review via email: mp+206866@code.launchpad.net

Commit message

[FIX] do not copy the dispatch_id when duplicating moves

Description of the change

Fixes: lp:1281471

Erase the dispatch_id from duplicated stock moves

To post a comment you must log in.
Revision history for this message
Pedro Manuel Baeza (pedro.baeza) wrote :

Makes sense. Thank you!

review: Approve (code review)
Revision history for this message
Alexandre Fayolle - camptocamp (alexandre-fayolle-c2c) wrote :

LGTM

review: Approve (code review, no test)
Revision history for this message
Yannick Vaucher @ Camptocamp (yvaucher-c2c) wrote :

LGTM

review: Approve (code review)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'picking_dispatch/picking_dispatch.py'
2--- picking_dispatch/picking_dispatch.py 2012-12-10 10:51:23 +0000
3+++ picking_dispatch/picking_dispatch.py 2014-02-18 08:58:37 +0000
4@@ -157,6 +157,14 @@
5 help='who this move is dispatched to'),
6 }
7
8+ def copy_data(self, cr, uid, id, default=None, context=None):
9+ if default is None:
10+ default = {}
11+ default = default.copy()
12+ default['dispatch_id'] = False
13+ return super(StockMove, self).\
14+ copy_data(cr, uid, id, default=default, context=context)
15+
16 def do_partial(self, cr, uid, ids, partial_datas, context=None):
17 """
18 in addition to what the original method does, create backorder

Subscribers

People subscribed via source and target branches