[FIX] l10n_it_vat_payability: replace script migration by pre_init_hook

In this way we fix OCA#2220 (comment) Note: OCA#2220 (comment) does not apply as in the case at hand there aren't models involved in the migration.
TheMule71 · Nov 2, 2022 · 67ffeaf · 67ffeaf
1 parent d64906a
commit 67ffeaf
Show file tree

Hide file tree

Showing 6 changed files with 31 additions and 23 deletions.
diff --git a/l10n_it_vat_payability/README.rst b/l10n_it_vat_payability/README.rst
@@ -63,6 +63,7 @@ Contributors
 * Lorenzo Battistini <lorenzo.battistini@agilebg.com>
 * Gianmarco Conte <gconte@dinamicheaziendali.it>
 * Giuseppe Borruso - Dinamiche Aziendali srl <gborruso@dinamicheaziendali.it>
+* Alex Comba <alex.comba@agilebg.com>
 
 Maintainers
 ~~~~~~~~~~~

diff --git a/l10n_it_vat_payability/__init__.py b/l10n_it_vat_payability/__init__.py
@@ -1,2 +1,29 @@
 # License LGPL-3.0 or later (http://www.gnu.org/licenses/lgpl).
+# Copyright 2021 Alex Comba - Agile Business Group
+
 from . import models
+
+from openupgradelib import openupgrade
+
+
+def rename_old_italian_module(cr):
+
+    if not openupgrade.is_module_installed(cr, "l10n_it_esigibilita_iva"):
+        return
+
+    openupgrade.rename_xmlids(
+        cr,
+        [
+            (
+                "l10n_it_esigibilita_iva.view_tax_code_esigibilita_form",
+                "l10n_it_vat_payability.view_tax_code_payability_form",
+            ),
+        ],
+    )
+    openupgrade.update_module_names(
+        cr,
+        [
+            ("l10n_it_esigibilita_iva", "l10n_it_vat_payability"),
+        ],
+        merge_modules=True,
+    )
diff --git a/l10n_it_vat_payability/__manifest__.py b/l10n_it_vat_payability/__manifest__.py
@@ -17,4 +17,5 @@
         "views/account_view.xml",
     ],
     "installable": True,
+    "pre_init_hook": "rename_old_italian_module",
 }
diff --git a/l10n_it_vat_payability/migrations/14.0.1.0.0/pre-migrate.py b/l10n_it_vat_payability/migrations/14.0.1.0.0/pre-migrate.py
diff --git a/l10n_it_vat_payability/readme/CONTRIBUTORS.rst b/l10n_it_vat_payability/readme/CONTRIBUTORS.rst
@@ -2,3 +2,4 @@
 * Lorenzo Battistini <lorenzo.battistini@agilebg.com>
 * Gianmarco Conte <gconte@dinamicheaziendali.it>
 * Giuseppe Borruso - Dinamiche Aziendali srl <gborruso@dinamicheaziendali.it>
+* Alex Comba <alex.comba@agilebg.com>
diff --git a/l10n_it_vat_payability/static/description/index.html b/l10n_it_vat_payability/static/description/index.html
@@ -407,6 +407,7 @@ <h2><a class="toc-backref" href="#id4">Contributors</a></h2>
 <li>Lorenzo Battistini &lt;<a class="reference external" href="mailto:lorenzo.battistini&#64;agilebg.com">lorenzo.battistini&#64;agilebg.com</a>&gt;</li>
 <li>Gianmarco Conte &lt;<a class="reference external" href="mailto:gconte&#64;dinamicheaziendali.it">gconte&#64;dinamicheaziendali.it</a>&gt;</li>
 <li>Giuseppe Borruso - Dinamiche Aziendali srl &lt;<a class="reference external" href="mailto:gborruso&#64;dinamicheaziendali.it">gborruso&#64;dinamicheaziendali.it</a>&gt;</li>
+<li>Alex Comba &lt;<a class="reference external" href="mailto:alex.comba&#64;agilebg.com">alex.comba&#64;agilebg.com</a>&gt;</li>
 </ul>
 </div>
 <div class="section" id="maintainers">