From c8148fab79dd75cf10f631de56b31fe6fdcbc881 Mon Sep 17 00:00:00 2001 From: "Pedro M. Baeza" Date: Sat, 22 Oct 2022 11:30:43 +0200 Subject: [PATCH] [OU-FIX] account_statement_import_online: Conditional renaming + filling As the changes for having online raw data have been made only on v12 and v14, people coming from v13 won't have this column populated on the DB, so we need to be tolerant with that circumstance. --- .../migrations/14.0.3.0.0/post-migration.py | 14 ++++++++------ .../migrations/14.0.3.0.0/pre-migration.py | 7 ++++--- 2 files changed, 12 insertions(+), 9 deletions(-) diff --git a/account_statement_import_online/migrations/14.0.3.0.0/post-migration.py b/account_statement_import_online/migrations/14.0.3.0.0/post-migration.py index 7ea77570..d0b52831 100644 --- a/account_statement_import_online/migrations/14.0.3.0.0/post-migration.py +++ b/account_statement_import_online/migrations/14.0.3.0.0/post-migration.py @@ -7,9 +7,11 @@ from openupgradelib import openupgrade @openupgrade.migrate() def migrate(env, version): - openupgrade.logged_query( - env.cr, - "UPDATE account_bank_statement_line SET raw_data={online_raw_data}".format( - online_raw_data=openupgrade.get_legacy_name("online_raw_data") - ), - ) + column = openupgrade.get_legacy_name("online_raw_data") + if openupgrade.column_exists(env.cr, "account_bank_statement_line", column): + openupgrade.logged_query( + env.cr, + "UPDATE account_bank_statement_line SET raw_data={online_raw_data}".format( + online_raw_data=column, + ), + ) diff --git a/account_statement_import_online/migrations/14.0.3.0.0/pre-migration.py b/account_statement_import_online/migrations/14.0.3.0.0/pre-migration.py index d68537d4..8eb0fd85 100644 --- a/account_statement_import_online/migrations/14.0.3.0.0/pre-migration.py +++ b/account_statement_import_online/migrations/14.0.3.0.0/pre-migration.py @@ -13,6 +13,7 @@ _column_renames = { @openupgrade.migrate() def migrate(env, version): - if not version: - return - openupgrade.rename_columns(env.cr, _column_renames) + if openupgrade.column_exists( + env.cr, "account_bank_statement_line", "online_raw_data" + ): + openupgrade.rename_columns(env.cr, _column_renames)