Merge lp:~unifield-team/unifield-wm/us-493 into lp:unifield-wm

Proposed by jftempo
Status: Needs review
Proposed branch: lp:~unifield-team/unifield-wm/us-493
Merge into: lp:unifield-wm
Diff against target: 15 lines (+4/-1)
1 file modified
msf_outgoing/msf_outgoing.py (+4/-1)
To merge this branch: bzr merge lp:~unifield-team/unifield-wm/us-493
Reviewer Review Type Date Requested Status
UniField Reviewer Team Pending
Review via email: mp+268604@code.launchpad.net
To post a comment you must log in.

Unmerged revisions

2578. By Quentin THEURET @Amaris

US-493 [FIX] Don't take care of the canceled PACK to validate a shipment. Canceled PACK are always empty

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'msf_outgoing/msf_outgoing.py'
2--- msf_outgoing/msf_outgoing.py 2015-08-04 11:45:10 +0000
3+++ msf_outgoing/msf_outgoing.py 2015-08-20 14:25:37 +0000
4@@ -1506,7 +1506,10 @@
5 # all object of a given picking object, he is set to Done and still belong to the same shipment_id
6 # another possibility would be to unlink the picking object from the shipment, set shipment_id to False
7 # but in this case the returned pack families would not be displayed anymore in the shipment
8- packing_ids = pick_obj.search(cr, uid, [('shipment_id', '=', shipment.id), ('state', '!=', 'done'), ], context=context)
9+ packing_ids = pick_obj.search(cr, uid, [
10+ ('shipment_id', '=', shipment.id),
11+ ('state', 'not in', ['done', 'cancel']),
12+ ], context=context)
13
14 for packing in pick_obj.browse(cr, uid, packing_ids, context=context):
15 assert packing.subtype == 'packing' and packing.state == 'assigned'

Subscribers

People subscribed via source and target branches