From db99bb6801e038e84d6dcf353cd45049de115083 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?C=C3=A9dric=20Krier?= <ced@b2ck.com>
Date: Tue, 1 Oct 2024 18:32:18 +0200
Subject: [PATCH] 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
---
 modules/stock/shipment.py | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/modules/stock/shipment.py b/modules/stock/shipment.py
index a3d6212a6a..c96a96e7ac 100644
--- a/modules/stock/shipment.py
+++ b/modules/stock/shipment.py
@@ -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(