[FIX] upgrade_analysis: migration errors

OCA · Dec 13, 2024 · 3a4fbdd · 3a4fbdd
1 parent 855b13c
commit 3a4fbdd
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/upgrade_analysis/compare.py b/upgrade_analysis/compare.py
@@ -480,7 +480,8 @@ def compare_model_sets(old_records, new_records):
                         text += f" [{column['model_type']}]"
                     reprs[module_map(column["module"])].append(text)
                     reprs["general"].append(
-                        f"obsolete model {model} [module module_map(column['module'])]"
+                        f"obsolete model {model} "
+                        f"[module {module_map(column['module'])}]"
                     )
                 else:
                     moved_module = ""
@@ -492,7 +493,7 @@ def compare_model_sets(old_records, new_records):
                         moved_module,
                     )
                     if column["model_type"]:
-                        text += " [column['model_type']]"
+                        text += f" [{column['model_type']}]"
                     reprs[module_map(column["module"])].append(text)
                     reprs["general"].append(
                         f"obsolete model {model} (renamed to {model_map(model)}) "
@@ -502,11 +503,11 @@ def compare_model_sets(old_records, new_records):
                 if module_map(column["module"]) != new_models[model]:
                     text = f"model {model} (moved to {new_models[model]})"
                     if column["model_type"]:
-                        text += " [column['model_type']]"
+                        text += f" [{column['model_type']}]"
                     reprs[module_map(column["module"])].append(text)
                     text = f"model {model} (moved from {old_models[model]})"
                     if column["model_type"]:
-                        text += " [column['model_type']]"
+                        text += f" [{column['model_type']}]"
 
     for column in copy.copy(new_records):
         model = column["model"]

diff --git a/upgrade_analysis/upgrade_log.py b/upgrade_analysis/upgrade_log.py
@@ -213,7 +213,7 @@ def log_xml_id(cr, module, xml_id):
     if "." not in xml_id:
         xml_id = f"{module}.{xml_id}"
     cr.execute(
-        "SELECT model FROM ir_model_data " "WHERE module = %s AND name = %s",
+        "SELECT model FROM ir_model_data WHERE module = %s AND name = %s",
         xml_id.split("."),
     )
     record = cr.fetchone()