From 024440e30553dc9265f299152b87a9ba656c0fcf Mon Sep 17 00:00:00 2001
From: sergio-teruel <sergio.teruel@tecnativa.com>
Date: Fri, 13 Sep 2024 18:59:39 +0200
Subject: [PATCH] [FIX] sale_order_type: Empty analytic account when sale order
 is duplicated.

---
 sale_order_type/models/sale.py                |  7 +++++++
 sale_order_type/tests/test_sale_order_type.py | 21 +++++++++++++++++++
 2 files changed, 28 insertions(+)

diff --git a/sale_order_type/models/sale.py b/sale_order_type/models/sale.py
index c44cabe0fb5..f5615d232ac 100644
--- a/sale_order_type/models/sale.py
+++ b/sale_order_type/models/sale.py
@@ -147,6 +147,13 @@ def _prepare_invoice(self):
             res["sale_type_id"] = self.type_id.id
         return res
 
+    def copy_data(self, default=None):
+        vals_list = super().copy_data(default)
+        if self.type_id.analytic_account_id:
+            for vals in vals_list:
+                vals["analytic_account_id"] = self.type_id.analytic_account_id.id
+        return vals_list
+
 
 class SaleOrderLine(models.Model):
     _inherit = "sale.order.line"
diff --git a/sale_order_type/tests/test_sale_order_type.py b/sale_order_type/tests/test_sale_order_type.py
index 3de1dba61e1..5800f6b31b0 100644
--- a/sale_order_type/tests/test_sale_order_type.py
+++ b/sale_order_type/tests/test_sale_order_type.py
@@ -57,6 +57,14 @@ def setUp(self):
         self.immediate_payment = self.env.ref("account.account_payment_term_immediate")
         self.sale_pricelist = self.env.ref("product.list0")
         self.free_carrier = self.env.ref("account.incoterm_FCA")
+        self.analytic_account = self.env["account.analytic.account"].create(
+            {
+                "name": "Test AA",
+                "code": "TESTSALE_REINVOICE",
+                "company_id": self.partner_child_1.company_id.id,
+                "partner_id": self.partner_child_1.id,
+            }
+        )
         self.sale_type = self.sale_type_model.create(
             {
                 "name": "Test Sale Order Type",
@@ -68,6 +76,7 @@ def setUp(self):
                 "pricelist_id": self.sale_pricelist.id,
                 "incoterm_id": self.free_carrier.id,
                 "quotation_validity_days": 10,
+                "analytic_account_id": self.analytic_account.id,
             }
         )
         self.sale_type_quot = self.sale_type_model.create(
@@ -265,3 +274,15 @@ def test_sequence_default(self):
         name = order.name
         order.type_id = self.sale_type_sequence_default
         self.assertEqual(name, order.name, "The sequence shouldn't change!")
+
+    def test_sale_copy_function(self):
+        """
+        Test when duplicating the sale order the account analytic account is set.
+        """
+        order = self.create_sale_order()
+        order.onchange_partner_id()
+        order.onchange_type_id()
+        new_order = order.copy()
+        self.assertEqual(
+            new_order.analytic_account_id.id, order.type_id.analytic_account_id.id
+        )