From 8f65d05ac9afacfc2e11470d86fc3da321f89be6 Mon Sep 17 00:00:00 2001
From: "elias.mendez" <elias.mendez@openpyme.mx>
Date: Thu, 12 Jan 2023 16:43:21 -0600
Subject: [PATCH] fix(l10n.mx.facturae): se agrego un if para revisar si existe
 columna

se agrego un if para revisar si existe columna
---
 .../migrations/3.1.0/post-migration.py        | 22 ++++++++++++-------
 .../migrations/3.1.0/pre-migration.py         |  2 +-
 2 files changed, 15 insertions(+), 9 deletions(-)

diff --git a/l10n_mx_facturae/migrations/3.1.0/post-migration.py b/l10n_mx_facturae/migrations/3.1.0/post-migration.py
index c19574731d..45ea7186bd 100644
--- a/l10n_mx_facturae/migrations/3.1.0/post-migration.py
+++ b/l10n_mx_facturae/migrations/3.1.0/post-migration.py
@@ -16,6 +16,13 @@ def migrate_res_partner(env):
             "cfdi_adenda_ids",
             legacy_name,
         )
+        # Clear helper fields used for migrate data
+        tools.drop_columns(
+            env.cr,
+            [
+                ("res_partner", tools.get_legacy_name("cfdi_adenda")),
+            ],
+        )
 
 
 def migrate_account_invoice(env):
@@ -29,17 +36,16 @@ def migrate_account_invoice(env):
             "cfdi_adenda_ids",
             legacy_name,
         )
+        # Clear helper fields used for migrate data
+        tools.drop_columns(
+            env.cr,
+            [
+                ("account_invoice", tools.get_legacy_name("cfdi_adenda_id")),
+            ],
+        )
 
 
 @tools.migrate(use_env=True, uid=SUPERUSER_ID)
 def migrate(env, installed_version):
     migrate_res_partner(env)
     migrate_account_invoice(env)
-    # Clear helper fields used for migrate data
-    tools.drop_columns(
-        env.cr,
-        [
-            ("res_partner", tools.get_legacy_name("cfdi_adenda")),
-            ("account_invoice", tools.get_legacy_name("cfdi_adenda_id")),
-        ],
-    )
diff --git a/l10n_mx_facturae/migrations/3.1.0/pre-migration.py b/l10n_mx_facturae/migrations/3.1.0/pre-migration.py
index cbe82d84ef..1554366a89 100644
--- a/l10n_mx_facturae/migrations/3.1.0/pre-migration.py
+++ b/l10n_mx_facturae/migrations/3.1.0/pre-migration.py
@@ -13,5 +13,5 @@ column_renames = {
 
 @tools.migrate(use_env=True, uid=SUPERUSER_ID)
 def migrate(env, installed_version):
-    if column_renames:
+    if tools.column_exists(env.cr, "res_partner", "cfdi_adenda_id"):
         tools.rename_columns(env.cr, column_renames)
-- 
GitLab