From ff2657e5df1f37834a4dbbe356530031b1469922 Mon Sep 17 00:00:00 2001
From: AaronHForgeFlow <ahenriquez@eficent.com>
Date: Thu, 1 Dec 2022 11:09:00 +0100
Subject: [PATCH] [13.0][FIX] account_move_line_sale_info: add the sale line
 information to credit notes

---
 .../models/account_move.py                    |  6 +++
 .../tests/test_account_move_line_sale_info.py | 39 ++++++++++++++++++-
 2 files changed, 43 insertions(+), 2 deletions(-)

diff --git a/account_move_line_sale_info/models/account_move.py b/account_move_line_sale_info/models/account_move.py
index 6e8cf814..92611c2b 100644
--- a/account_move_line_sale_info/models/account_move.py
+++ b/account_move_line_sale_info/models/account_move.py
@@ -47,3 +47,9 @@ class AccountMoveLine(models.Model):
         index=True,
         copy=False,
     )
+
+    def _copy_data_extend_business_fields(self, values):
+        # Same way Odoo standard does for purchase_line_id field
+        res = super(AccountMoveLine, self)._copy_data_extend_business_fields(values)
+        values["sale_line_id"] = self.sale_line_id.id
+        return res
diff --git a/account_move_line_sale_info/tests/test_account_move_line_sale_info.py b/account_move_line_sale_info/tests/test_account_move_line_sale_info.py
index 000310c5..a33c42e6 100644
--- a/account_move_line_sale_info/tests/test_account_move_line_sale_info.py
+++ b/account_move_line_sale_info/tests/test_account_move_line_sale_info.py
@@ -170,7 +170,15 @@ class TestAccountMoveLineSaleInfo(common.TransactionCase):
                 % (str(expected_balance), sale_line.name),
             )
 
-    def test_sale_invoice(self):
+    def move_reversal_wiz(self, move):
+        wizard = (
+            self.env["account.move.reversal"]
+            .with_context(active_model="account.move", active_ids=[move.id])
+            .create({"journal_id": move.journal_id.id})
+        )
+        return wizard
+
+    def test_01_sale_invoice(self):
         """Test that the po line moves from the sale order to the
         account move line and to the invoice line.
         """
@@ -214,7 +222,7 @@ class TestAccountMoveLineSaleInfo(common.TransactionCase):
                     "from the invoice to the account move line.",
                 )
 
-    def test_name_get(self):
+    def test_02_name_get(self):
         sale = self._create_sale([(self.product, 1)])
         so_line = sale.order_line[0]
         name_get = so_line.with_context(**{"so_line_info": True}).name_get()
@@ -242,3 +250,30 @@ class TestAccountMoveLineSaleInfo(common.TransactionCase):
                 )
             ],
         )
+
+    def test_03_credit_note(self):
+        """Test the credit note is linked to the sale order line"""
+        sale = self._create_sale([(self.product, 1)])
+        so_line = False
+        for line in sale.order_line:
+            so_line = line
+            break
+        sale.action_confirm()
+        picking = sale.picking_ids[0]
+        picking.move_lines.write({"quantity_done": 1.0})
+        picking.button_validate()
+        sale._create_invoices()
+        invoice = sale.invoice_ids[0]
+        invoice._post()
+        reversal_wizard = self.move_reversal_wiz(invoice)
+        credit_note = self.env["account.move"].browse(
+            reversal_wizard.reverse_moves()["res_id"]
+        )
+        for aml in credit_note.line_ids:
+            if aml.product_id == so_line.product_id and aml.move_id:
+                self.assertEqual(
+                    aml.sale_line_id,
+                    so_line,
+                    "sale Order line has not been copied "
+                    "from the invoice to the credit note.",
+                )