Merge pull request #54 from hbrunn/14.0-316-percentage-rounding

[FIX] #316 ps_planning: percentage rounding
tosccolors · Nov 14, 2024 · b94557e · b94557e
2 parents be7344d + a95cf6a
commit b94557e
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 4 deletions.
diff --git a/ps_planning/__manifest__.py b/ps_planning/__manifest__.py
@@ -4,7 +4,7 @@
 {
     "name": "PS planning",
     "summary": "Planning tool for professional services",
-    "version": "14.0.1.0.0",
+    "version": "14.0.1.0.1",
     "development_status": "Alpha",
     "category": "Planning",
     "website": "http://www.tosc.nl",

diff --git a/ps_planning/migrations/14.0.1.0.1/pre-migration.py b/ps_planning/migrations/14.0.1.0.1/pre-migration.py
@@ -0,0 +1,4 @@
+def migrate(cr, version=None):
+    cr.execute(
+        "alter table ps_planning_report_wizard_line drop column budget_utilization"
+    )
diff --git a/ps_planning/wizards/ps_planning_report_wizard.py b/ps_planning/wizards/ps_planning_report_wizard.py
@@ -103,8 +103,9 @@ def action_open_report(self):
                 - vals["days_planned_full_month"] * mtd_fraction,
                 days_actual_commercial_mtd=vals["days_actual_mtd"]
                 - vals["days_commercial_full_month"] * mtd_fraction,
-                budget_utilization=vals["days_actual_mtd"]
-                / vals["days_commercial_full_month"]
+                budget_utilization=(
+                    vals["days_actual_mtd"] / vals["days_commercial_full_month"]
+                )
                 if vals["days_commercial_full_month"]
                 else 1,
             )
@@ -142,6 +143,6 @@ class PsPlanningReportWizardLine(models.TransientModel):
     days_actual_mtd = fields.Integer("MTD Actual MD")
     days_actual_planned_mtd = fields.Integer("MTD Actual - Planned MD")
     days_actual_commercial_mtd = fields.Integer("MTD Actual - Commercial MD")
-    budget_utilization = fields.Integer("KPI % MTD")
+    budget_utilization = fields.Float("KPI % MTD", digits=(16, 2))
     actual_commercial_ytm = fields.Integer("Actual Commercial YTM")
     manager_name = fields.Char("Project Manager")