From 34b4536cafbd8f7097b6c0156109b60f6ddb181a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?St=C3=A9phane=20Laporte?= <stephane.laporte@enercoop.org>
Date: Tue, 15 Apr 2025 13:55:08 +0200
Subject: [PATCH] [FIX] api.depends, store total_share and counter_percentage

---
 models/acc_priority_group.py         | 4 ++--
 models/acc_priority_group_counter.py | 6 ++----
 2 files changed, 4 insertions(+), 6 deletions(-)

diff --git a/models/acc_priority_group.py b/models/acc_priority_group.py
index e4f4148..40bd1e8 100644
--- a/models/acc_priority_group.py
+++ b/models/acc_priority_group.py
@@ -33,7 +33,7 @@ class AccPriorityGroup(models.Model):
     )
 
     counter_datas = fields.Json(compute="_compute_counter_datas")
-    total_share = fields.Float(compute="_compute_total_share", store=False)
+    total_share = fields.Float(compute="_compute_total_share", store=True)
     # ------------------------------------------------------
     # SQL Constraints
     # ------------------------------------------------------
@@ -45,7 +45,7 @@ class AccPriorityGroup(models.Model):
     # ------------------------------------------------------
     # Computed fields / Search Fields
     # ------------------------------------------------------
-    @api.depends("acc_priority_group_counter_ids")
+    @api.depends("acc_priority_group_counter_ids", "acc_priority_group_counter_ids.acc_counter_share")
     def _compute_total_share(self):
         for record in self:
             record.total_share = sum(
diff --git a/models/acc_priority_group_counter.py b/models/acc_priority_group_counter.py
index d3774ee..718c47b 100644
--- a/models/acc_priority_group_counter.py
+++ b/models/acc_priority_group_counter.py
@@ -47,7 +47,7 @@ class AccPriorityGroupCounter(models.Model):
         related="acc_priority_group_id.type_algo", readonly=True
     )
     acc_counter_percentage = fields.Float(
-        compute="_compute_acc_counter_percentage", store=False
+        compute="_compute_acc_counter_percentage", store=True
     )
 
     # ------------------------------------------------------
@@ -62,9 +62,7 @@ class AccPriorityGroupCounter(models.Model):
     # Computed fields / Search Fields
     # ------------------------------------------------------
 
-    @api.depends(
-        "acc_priority_group_id.acc_priority_group_counter_ids", "acc_counter_share"
-    )
+    @api.depends("acc_priority_group_id.total_share")
     def _compute_acc_counter_percentage(self):
         for record in self:
             total_share = record.acc_priority_group_id.total_share
-- 
GitLab