From 84b70b73c532b0d7af23b049c7885da129b6e559 Mon Sep 17 00:00:00 2001
From: laurentbarontini <l.barontini@open-squared.ch>
Date: Mon, 2 Feb 2026 17:51:26 +0100
Subject: [PATCH] 02.02.26

---
 modules/automation/cron.py      |  2 +-
 modules/purchase_trade/stock.py | 72 ++++++++++++++++-----------------
 2 files changed, 37 insertions(+), 37 deletions(-)

diff --git a/modules/automation/cron.py b/modules/automation/cron.py
index 0f321bc..1c5c491 100644
--- a/modules/automation/cron.py
+++ b/modules/automation/cron.py
@@ -310,7 +310,7 @@ class AutomationCron(ModelSQL, ModelView):
                     ShipmentIn.save([shipment])
                     shipment._create_lots_from_fintrade()
                     shipment.controller = shipment.get_controller()
-                    shipment.instructions = shipment.get_instructions()
+                    shipment.instructions = shipment.get_instructions_html()
                     ShipmentIn.save([shipment])
                     trans_shipment.commit()
                     successful_shipments += 1
diff --git a/modules/purchase_trade/stock.py b/modules/purchase_trade/stock.py
index 22f2b0f..6d70b8d 100755
--- a/modules/purchase_trade/stock.py
+++ b/modules/purchase_trade/stock.py
@@ -466,45 +466,45 @@ class ShipmentIn(metaclass=PoolMeta):
                 if c.party.IsAvailableForControl(self):
                     return c.party
     
-    # def get_instructions_html(self):
-    #     lines = [
-    #         "<p>Hi,</p>",
-    #         "<p>Please find details below for the requested control</p>",
-    #         f"<p><strong>BL number:</strong> {self.bl_number}</p>",
-    #     ]
-
-    #     if self.incoming_moves:
-    #         for m in self.incoming_moves:
-    #             if m.lot:
-    #                 lines.append(
-    #                     "<p>"
-    #                     f"<strong>Lot nb:</strong> {m.lot.lot_name} | "
-    #                     f"<strong>Net Qt:</strong> {m.lot.get_current_quantity()} {m.lot.lot_unit_line.symbol} | "
-    #                     f"<strong>Gross Qt:</strong> {m.lot.get_current_gross_quantity()} {m.lot.lot_unit_line.symbol}"
-    #                     "</p>"
-    #                 )
-
-    #     return "".join(lines)
-
-    def get_instructions(self):
+    def get_instructions_html(self):
         lines = [
-            "Hi,",
-            "",
-            "Please find details below for the requested control",
-            f"BL number: {self.bl_number}",
-            ""
+            "<p>Hi,</p>",
+            "<p>Please find details below for the requested control</p>",
+            f"<p><strong>BL number:</strong> {self.bl_number}</p>",
         ]
 
         if self.incoming_moves:
             for m in self.incoming_moves:
                 if m.lot:
                     lines.append(
-                        f"Lot nb: {m.lot.lot_name} | "
-                        f"Net Qt: {m.lot.get_current_quantity()} {m.lot.lot_unit.symbol} | "
-                        f"Gross Qt: {m.lot.get_current_gross_quantity()} {m.lot.lot_unit.symbol}"
+                        "<p>"
+                        f"<strong>Lot nb:</strong> {m.lot.lot_name} | "
+                        f"<strong>Net Qt:</strong> {m.lot.get_current_quantity()} {m.lot.lot_unit_line.symbol} | "
+                        f"<strong>Gross Qt:</strong> {m.lot.get_current_gross_quantity()} {m.lot.lot_unit_line.symbol}"
+                        "</p>"
                     )
 
-        return "\n".join(lines)
+        return "".join(lines)
+
+    # def get_instructions(self):
+    #     lines = [
+    #         "Hi,",
+    #         "",
+    #         "Please find details below for the requested control",
+    #         f"BL number: {self.bl_number}",
+    #         ""
+    #     ]
+
+    #     if self.incoming_moves:
+    #         for m in self.incoming_moves:
+    #             if m.lot:
+    #                 lines.append(
+    #                     f"Lot nb: {m.lot.lot_name} | "
+    #                     f"Net Qt: {m.lot.get_current_quantity()} {m.lot.lot_unit.symbol} | "
+    #                     f"Gross Qt: {m.lot.get_current_gross_quantity()} {m.lot.lot_unit.symbol}"
+    #                 )
+
+    #     return "\n".join(lines)
             
     def _create_lots_from_fintrade(self):
         t = Table('freight_booking_lots')
@@ -676,11 +676,11 @@ class ShipmentIn(metaclass=PoolMeta):
 
             return sale_line
 
-    # def html_to_text(self,html_content):
-    #     text = re.sub(r"<br\s*/?>", "\n", html_content, flags=re.IGNORECASE)
-    #     text = re.sub(r"</p\s*>", "\n\n", text, flags=re.IGNORECASE)
-    #     text = re.sub(r"<[^>]+>", "", text)
-    #     return html.unescape(text).strip()
+    def html_to_text(self,html_content):
+        text = re.sub(r"<br\s*/?>", "\n", html_content, flags=re.IGNORECASE)
+        text = re.sub(r"</p\s*>", "\n\n", text, flags=re.IGNORECASE)
+        text = re.sub(r"<[^>]+>", "", text)
+        return html.unescape(text).strip()
 
     @classmethod
     @ModelView.button
@@ -697,7 +697,7 @@ class ShipmentIn(metaclass=PoolMeta):
                     payload = {
                         "to": [contact[0].value],
                         "subject": "Request for control",
-                        "body": sh.instructions,
+                        "body": sh.html_to_text(sh.instructions),
                         # "attachments": [
                         #     {
                         #         "filename": "instruction.pdf",