Skip to content

Commit

Permalink
[FIX] mail_show_follower: Use company from the document to render mes…
Browse files Browse the repository at this point in the history
…sages correctly

In a multi-company environment, if one company uses the show_followers_partner_format by partner_name and another company uses partner_email, a user with permissions in both companies may experience issues. If a document belongs to Company A, but the current company is set to Company B, the message format is taken from the current company (Company B) instead of the document’s company (Company A). This fix ensures that the message format is rendered based on the document's company.
  • Loading branch information
carlos-lopez-tecnativa committed Oct 10, 2024
1 parent 3eeb62c commit a8af850
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions mail_show_follower/models/mail_mail.py
Original file line number Diff line number Diff line change
Expand Up @@ -78,8 +78,10 @@ def _send(self, auto_commit=False, raise_exception=False, smtp_session=None):
or mail.author_id.lang
or company.partner_id.lang
)
final_cc = mail.with_context(lang=langs[0])._build_cc_text(
show_in_cc_recipients
final_cc = (

Check warning on line 81 in mail_show_follower/models/mail_mail.py

View check run for this annotation

Codecov / codecov/patch

mail_show_follower/models/mail_mail.py#L81

Added line #L81 was not covered by tests
mail.with_context(lang=langs[0])
.with_company(company)
._build_cc_text(show_in_cc_recipients)
)
mail.body_html = final_cc + mail.body_html

Check warning on line 86 in mail_show_follower/models/mail_mail.py

View check run for this annotation

Codecov / codecov/patch

mail_show_follower/models/mail_mail.py#L86

Added line #L86 was not covered by tests
return super()._send(
Expand Down

0 comments on commit a8af850

Please sign in to comment.