diff --git a/odoo_openupgrade_wizard/cli_estimate_workload.py b/odoo_openupgrade_wizard/cli_estimate_workload.py
index 33aef3a..5a72bfe 100644
--- a/odoo_openupgrade_wizard/cli_estimate_workload.py
+++ b/odoo_openupgrade_wizard/cli_estimate_workload.py
@@ -46,7 +46,6 @@ def estimate_workload(ctx, analysis_file_path, extra_modules_list):
analysis.modules = sorted(analysis.modules)
# Render html file
- # TODO, make
ensure_file_exists_from_template(
Path(analysis_file_path),
templates.ANALYSIS_HTML_TEMPLATE,
diff --git a/odoo_openupgrade_wizard/templates.py b/odoo_openupgrade_wizard/templates.py
index 4b503f3..882734b 100644
--- a/odoo_openupgrade_wizard/templates.py
+++ b/odoo_openupgrade_wizard/templates.py
@@ -276,7 +276,7 @@ ANALYSIS_HTML_TEMPLATE = """
| {{odoo_module.name}}
|
- {% for version in odoo_module.analyse.all_version %}
+ {% for version in odoo_module.analyse.all_versions %}
{% set module_version = odoo_module.get_module_version(version) %}
{% if module_version %}
{% set size_text = module_version.get_size_text() %}
diff --git a/odoo_openupgrade_wizard/tools_odoo_module.py b/odoo_openupgrade_wizard/tools_odoo_module.py
index 57495a3..b6ab621 100644
--- a/odoo_openupgrade_wizard/tools_odoo_module.py
+++ b/odoo_openupgrade_wizard/tools_odoo_module.py
@@ -475,6 +475,9 @@ class OdooModuleVersion(object):
if self.openupgrade_state and (
self.openupgrade_state.lower().startswith("done")
or self.openupgrade_state.lower().startswith("nothing to do")
+ or self.openupgrade_state.lower().startswith(
+ "no db layout changes"
+ )
):
return
else:
@@ -646,6 +649,9 @@ class OdooModuleVersion(object):
or self.openupgrade_state.lower().startswith(
"nothing to do"
)
+ or self.openupgrade_state.lower().startswith(
+ "no db layout changes"
+ )
):
return "lightgreen"
else:
@@ -673,15 +679,10 @@ class OdooModuleVersion(object):
self.odoo_module.module_type == "odoo"
and self.version != self.odoo_module.analyse.initial_version
):
- if self.openupgrade_state.lower().startswith(
- "done"
- ) or self.openupgrade_state.lower().startswith(
- "nothing to do"
- ):
+ if self.openupgrade_state:
return self.openupgrade_state
else:
return "To analyse"
- return ""
else:
if self.state == "merged":
return "Merged into %s" % self.target_module