From 1120bffb66000649cdfd4dbb4d63faa1b9f23835 Mon Sep 17 00:00:00 2001 From: XavierDo Date: Wed, 5 Sep 2018 15:06:18 +0200 Subject: [PATCH] [FIX] mail: only display errors linked to message of current user The search on message_has_error (used when clicking on a failure notification) was displaying the failure of all users making it difficult to find your own mail failure. This fix solve the problem by adding a criteria on message author_id. --- addons/mail/models/mail_message.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/addons/mail/models/mail_message.py b/addons/mail/models/mail_message.py index d9ac037b936..d581a42e1ee 100644 --- a/addons/mail/models/mail_message.py +++ b/addons/mail/models/mail_message.py @@ -151,8 +151,8 @@ class Message(models.Model): @api.multi def _search_has_error(self, operator, operand): if operator == '=' and operand: - return [('notification_ids.email_status', 'in', ('bounce', 'exception'))] - return ['!', ('notification_ids.email_status', 'in', ('bounce', 'exception'))] # this wont work and will be equivalent to "not in" beacause of orm restrictions. Dont use "has_error = False" + return ['&', ('notification_ids.email_status', 'in', ('bounce', 'exception')), ('author_id', '=', self.env.user.partner_id.id)] + return ['!', '&', ('notification_ids.email_status', 'in', ('bounce', 'exception')), ('author_id', '=', self.env.user.partner_id.id)] # this wont work and will be equivalent to "not in" beacause of orm restrictions. Dont use "has_error = False" @api.depends('starred_partner_ids') def _get_starred(self):