[FIX] mail_tracking: allowed trackings compute

In the forward port of the permission check, a column was dropped from
the tuple, but the following code wasn't adapted to the new tuple
length. For clarity sake we also name the tuple unpacking into proper
variables.

TT43453
pull/1149/head
David 2023-05-22 10:42:35 +02:00
parent c6c34c2dad
commit 4ea4152ddd
1 changed files with 6 additions and 6 deletions

View File

@ -164,13 +164,13 @@ class MailTrackingEmail(models.Model):
[("id", "in", [x for x in partner_ids if x])] [("id", "in", [x for x in partner_ids if x])]
) )
return [ return [
x[0] track_id
for x in msg_linked for track_id, mail_msg_id, partner_id in msg_linked
if (x[1] in msg_ids) # We can read the linked message if (mail_msg_id in msg_ids) # We can read the linked message
or ( or (
not any({x[1], x[2]}) and x[3] in partner_ids not mail_msg_id and partner_id in partner_ids
) # No linked msg/mail but we can read the linked partner ) # No linked mail.message but we can read the linked partner
or (not any({x[1], x[2], x[3]})) # No linked record or (not any({mail_msg_id, partner_id})) # No linked record
] ]
@api.model @api.model