From 49dfed790f1f7da7aa30917c43903c1585387ab7 Mon Sep 17 00:00:00 2001 From: Kevin Baptiste Date: Thu, 22 Oct 2020 11:32:33 +0000 Subject: [PATCH] [FIX] rating: return correct last rating value Before this commit, the last rating value returned was not the last consumed one. closes odoo/odoo#61868 Taskid: 2260685 X-original-commit: 5fd11c46a4a12ffb977b15511e6eaf744cd8e0b6 Signed-off-by: Yannick Tivisse (yti) Signed-off-by: Kevin Baptiste --- addons/rating/models/rating_mixin.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/addons/rating/models/rating_mixin.py b/addons/rating/models/rating_mixin.py index 6825083554e..182d3d08b14 100644 --- a/addons/rating/models/rating_mixin.py +++ b/addons/rating/models/rating_mixin.py @@ -59,10 +59,10 @@ class RatingMixin(models.AbstractModel): rating_count = fields.Integer('Rating count', compute="_compute_rating_stats", compute_sudo=True) rating_avg = fields.Float("Rating Average", compute='_compute_rating_stats', compute_sudo=True) - @api.depends('rating_ids.rating') + @api.depends('rating_ids.rating', 'rating_ids.consumed') def _compute_rating_last_value(self): for record in self: - ratings = self.env['rating.rating'].search([('res_model', '=', self._name), ('res_id', '=', record.id)], limit=1) + ratings = self.env['rating.rating'].search([('res_model', '=', self._name), ('res_id', '=', record.id), ('consumed', '=', True)], limit=1) record.rating_last_value = ratings and ratings.rating or 0 @api.depends('rating_ids.res_id', 'rating_ids.rating')