Fixed alternate follower uid bug
This commit is contained in:
parent
21cc63df0b
commit
f6cdd94f5f
1 changed files with 17 additions and 5 deletions
|
|
@ -67,12 +67,18 @@ class MailThread(models.AbstractModel):
|
||||||
f"Adding {leave.alternate_follower_id.partner_id.name} as follower for {employee.name} "
|
f"Adding {leave.alternate_follower_id.partner_id.name} as follower for {employee.name} "
|
||||||
f"while on time off."
|
f"while on time off."
|
||||||
)
|
)
|
||||||
|
# Determine the user linked to the alternate follower partner
|
||||||
|
alt_partner = leave.alternate_follower_id.partner_id
|
||||||
|
alt_user = self.env["res.users"].search([("partner_id", "=", alt_partner.id)], limit=1)
|
||||||
recipients.append(
|
recipients.append(
|
||||||
{
|
{
|
||||||
"id": leave.alternate_follower_id.partner_id.id,
|
"id": leave.alternate_follower_id.partner_id.id,
|
||||||
"active": True,
|
"active": True,
|
||||||
"share": False,
|
"share": False,
|
||||||
"groups": leave.alternate_follower_id.groups_id.ids,
|
# groups should come from the user if it exists
|
||||||
|
"groups": alt_user.groups_id.ids if alt_user else [],
|
||||||
|
# required by core: used in mail_thread._notify_thread to cache partner_id
|
||||||
|
"uid": alt_user.id if alt_user else False,
|
||||||
"notif": "inbox",
|
"notif": "inbox",
|
||||||
"type": "user",
|
"type": "user",
|
||||||
}
|
}
|
||||||
|
|
@ -84,10 +90,16 @@ class MailThread(models.AbstractModel):
|
||||||
f"follower addition."
|
f"follower addition."
|
||||||
)
|
)
|
||||||
else:
|
else:
|
||||||
_logger.info(
|
# Either no alternate follower is set, or already a recipient
|
||||||
f"Not adding {leave.alternate_follower_id.partner_id.name} for {employee.name}, All ready "
|
if leave.alternate_follower_id:
|
||||||
f"a follower."
|
_logger.info(
|
||||||
)
|
f"Not adding {leave.alternate_follower_id.partner_id.name} for {employee.name}, already "
|
||||||
|
f"a recipient or no need to add."
|
||||||
|
)
|
||||||
|
else:
|
||||||
|
_logger.info(
|
||||||
|
f"No alternate follower configured for {employee.name}."
|
||||||
|
)
|
||||||
|
|
||||||
# Return the updated recipients list
|
# Return the updated recipients list
|
||||||
return recipients
|
return recipients
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue