Do not copy origin of stock moves in each shipment
The ShipmentMixin is also used by the production which does not have a moves field. Closes #13555
This commit is contained in:
@@ -146,7 +146,6 @@ class ShipmentMixin:
|
||||
def copy(cls, shipments, default=None):
|
||||
default = default.copy() if default is not None else {}
|
||||
default.setdefault('number')
|
||||
default.setdefault('moves.origin', None)
|
||||
return super().copy(shipments, default=default)
|
||||
|
||||
|
||||
@@ -643,6 +642,7 @@ class ShipmentIn(
|
||||
default = {}
|
||||
else:
|
||||
default = default.copy()
|
||||
default.setdefault('moves.origin', None)
|
||||
default.setdefault('received_by', None)
|
||||
default.setdefault('done_by', None)
|
||||
return super(ShipmentIn, cls).copy(shipments, default=default)
|
||||
@@ -1002,6 +1002,7 @@ class ShipmentInReturn(ShipmentAssignMixin, Workflow, ModelSQL, ModelView):
|
||||
default = {}
|
||||
else:
|
||||
default = default.copy()
|
||||
default.setdefault('moves.origin', None)
|
||||
default.setdefault('assigned_by', None)
|
||||
default.setdefault('done_by', None)
|
||||
return super(ShipmentInReturn, cls).copy(shipments, default=default)
|
||||
@@ -1781,6 +1782,7 @@ class ShipmentOut(
|
||||
default = {}
|
||||
else:
|
||||
default = default.copy()
|
||||
default.setdefault('moves.origin', None)
|
||||
default.setdefault('picked_by', None)
|
||||
default.setdefault('packed_by', None)
|
||||
default.setdefault('done_by', None)
|
||||
@@ -2169,6 +2171,7 @@ class ShipmentOutReturn(
|
||||
if default is None:
|
||||
default = {}
|
||||
default = default.copy()
|
||||
default.setdefault('moves.origin', None)
|
||||
default.setdefault('received_by', None)
|
||||
default.setdefault('done_by', None)
|
||||
return super(ShipmentOutReturn, cls).copy(shipments, default=default)
|
||||
@@ -2668,6 +2671,7 @@ class ShipmentInternal(
|
||||
else:
|
||||
default = default.copy()
|
||||
default.setdefault('moves', outgoing_moves)
|
||||
default.setdefault('moves.origin', None)
|
||||
default.setdefault('moves.from_location', partial(
|
||||
shipment_field, name='from_location'))
|
||||
default.setdefault('moves.to_location', partial(
|
||||
|
||||
Reference in New Issue
Block a user