Merge pull request #4383 from Tecnativa/16.0-ou_imp-delivery-carrier_…

…name

[16.0][OU-IMP] delivery: Quick carrier_name population

openupgrade_scripts/scripts/delivery/16.0.1.0/pre-migration.py

@@ -0,0 +1,27 @@
+# SPDX-FileCopyrightText: 2024 Tecnativa - Pedro M. Baeza
+# SPDX-License-Identifier: AGPL-3.0-or-later
+from openupgradelib import openupgrade
+
+
+def _precreate_stock_move_line_carrier_name(env):
+    """It seems Odoo doesn't handle with a SQL related fields with more than one path
+    (several dots), so let's do it here for optimizing times.
+    """
+    openupgrade.logged_query(
+        env.cr, "ALTER TABLE stock_move_line ADD carrier_name VARCHAR"
+    )
+    openupgrade.logged_query(
+        env.cr,
+        """
+        UPDATE stock_move_line sml
+        SET carrier_name = dc.name
+        FROM stock_picking sp
+        JOIN delivery_carrier dc ON dc.id = sp.carrier_id
+        WHERE sml.picking_id = sp.id
+        """,
+    )
+
+
+@openupgrade.migrate()
+def migrate(env, version):
+    _precreate_stock_move_line_carrier_name(env)

openupgrade_scripts/scripts/delivery/16.0.1.0/upgrade_analysis_work.txt

@@ -17,7 +17,7 @@ delivery     / product.template         / country_of_origin (many2one)  : NEW re
 # NOTHING TO DO: New field for Intrastat. May should be filled from OCA Intrastat modules
 
 delivery     / stock.move.line          / carrier_name (char)           : NEW isrelated: related, stored
-# NOTHING TO DO: Related stored already handled in a fast way by ORM.
+# DONE: pre-migration: Pre-create it and fast fill it, as ORM seems to not handle related fields with more than 1 path.
 
 ---XML records in module 'delivery'---
 NEW ir.actions.act_window: delivery.action_delivery_zip_prefix_list