[IMP] account_tax_balance: use rename_fields instead of rename_columns

The rename_fields method is cleaner, because it handles ir_model_fields and translations.
pull/837/head
mreficent 2021-10-04 19:44:21 +02:00 committed by Valentin Vinagre Urteaga
parent 899a25a43c
commit 54a210ad30
1 changed files with 11 additions and 5 deletions

View File

@ -16,16 +16,22 @@ def migrate(env, version):
""" """
old_move_type_column = "move_type" old_move_type_column = "move_type"
new_move_type_column = "financial_type" new_move_type_column = "financial_type"
move_model_name = "account.move"
move_table_name = "account_move" move_table_name = "account_move"
enterprise_move_type_rename = "move_type_custom" enterprise_move_type_rename = "move_type_custom"
ou_move_type_rename = openupgrade.get_legacy_name(old_move_type_column) ou_move_type_rename = openupgrade.get_legacy_name(old_move_type_column)
for move_type_rename in (enterprise_move_type_rename, ou_move_type_rename): for move_type_rename in (enterprise_move_type_rename, ou_move_type_rename):
if openupgrade.column_exists(env.cr, move_table_name, move_type_rename): if openupgrade.column_exists(env.cr, move_table_name, move_type_rename):
openupgrade.rename_columns( openupgrade.rename_fields(
env.cr, env,
{ [
move_table_name: [(move_type_rename, new_move_type_column)], (
}, move_model_name,
move_table_name,
move_type_rename,
new_move_type_column,
),
],
) )
break break