From 2e34bd22c6e91b0cb12da70ea830ae719c08dff1 Mon Sep 17 00:00:00 2001
From: Maxime Richez <maxime.richez@saluc.com>
Date: Thu, 20 Oct 2022 19:39:10 +0200
Subject: [PATCH] Do not copy before carriages and after carriages

The carriages field is already copying the values.

issue11820
review418461003
---
 sale.py  | 10 ++++++++++
 stock.py | 10 ++++++++++
 2 files changed, 20 insertions(+)

diff --git a/sale.py b/sale.py
index ab5ce034b3..e401a58059 100644
--- a/sale.py
+++ b/sale.py
@@ -88,6 +88,16 @@ class Sale(metaclass=PoolMeta):
                 pattern['after_carrier'] = 'buyer'
         return pattern
 
+    @classmethod
+    def copy(cls, sales, default=None):
+        if default is None:
+            default = {}
+        else:
+            default = default.copy()
+        default.setdefault('before_carriages')
+        default.setdefault('after_carriages')
+        return super().copy(sales, default=default)
+
 
 class Line(metaclass=PoolMeta):
     __name__ = 'sale.line'
diff --git a/stock.py b/stock.py
index 9ed9f5f0c1..e11b266bf5 100644
--- a/stock.py
+++ b/stock.py
@@ -126,6 +126,16 @@ class _ShipmentMixin:
         super().delete(shipments)
         Carriage.delete(carriages)
 
+    @classmethod
+    def copy(cls, shipments, default=None):
+        if default is None:
+            default = {}
+        else:
+            default = default.copy()
+        default.setdefault('before_carriages')
+        default.setdefault('after_carriages')
+        return super().copy(shipments, default=default)
+
 
 class Carriage_Sale(ShipmentCostSaleMixin, metaclass=PoolMeta):
     __name__ = 'stock.shipment.carriage'