From 6200e1ac63a2f8a2438f9f02dcf1a33ebed86636 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?J=C3=A9rome=20Maes?= Date: Mon, 24 Oct 2016 17:13:49 +0200 Subject: [PATCH] [FIX] rating_project_issue: all issue are not taken into account when recompute project satisfaction 'issue_ids' has a domain, so all the issue a folded stage are not taken into account for statisfaction percentage. So replacing the field with a search call solve the problem. --- addons/rating_project_issue/models/project_issue.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/addons/rating_project_issue/models/project_issue.py b/addons/rating_project_issue/models/project_issue.py index 15c29839449..26c23e1fb06 100644 --- a/addons/rating_project_issue/models/project_issue.py +++ b/addons/rating_project_issue/models/project_issue.py @@ -42,7 +42,7 @@ class Project(models.Model): def _send_rating_mail(self): super(Project, self)._send_rating_mail() for project in self: - project.issue_ids._send_issue_rating_mail() + self.env['project.issue'].search([('project_id', '=', project.id)])._send_issue_rating_mail() @api.depends('percentage_satisfaction_task', 'percentage_satisfaction_issue') def _compute_percentage_satisfaction_project(self): @@ -55,7 +55,7 @@ class Project(models.Model): activity_great = activity_task['great'] activity_sum = sum(activity_task.values()) if project.use_issues: - activity_issue = project.issue_ids.rating_get_grades(domain) + activity_issue = self.env['project.issue'].search([('project_id', '=', project.id)]).rating_get_grades(domain) activity_great += activity_issue['great'] activity_sum += sum(activity_issue.values()) project.percentage_satisfaction_project = activity_great * 100 / activity_sum if activity_sum else -1