[FIX] mail_tracking: Store cc

pull/1367/head
Alexandre D. Díaz 2021-03-01 23:00:26 +01:00 committed by Pedro M. Baeza
parent 1cedb804a4
commit dd9bced5ef
3 changed files with 11 additions and 8 deletions

View File

@ -7,7 +7,7 @@
{ {
"name": "Email tracking", "name": "Email tracking",
"summary": "Email tracking system for all mails sent", "summary": "Email tracking system for all mails sent",
"version": "13.0.1.0.7", "version": "13.0.1.0.8",
"category": "Social Network", "category": "Social Network",
"website": "https://github.com/OCA/social", "website": "https://github.com/OCA/social",
"author": ("Tecnativa, " "Odoo Community Association (OCA)"), "author": ("Tecnativa, " "Odoo Community Association (OCA)"),

View File

@ -29,17 +29,20 @@ class MailThread(models.AbstractModel):
("mail_tracking_ids.state", "in", list(failed_states)), ("mail_tracking_ids.state", "in", list(failed_states)),
] ]
@api.returns("self", lambda value: value.id) @api.model
def message_post(self, *args, **kwargs): def _message_route_process(self, message, message_dict, routes):
"""Adds CC recipient to the message. """Adds CC recipient to the message.
Because Odoo implementation avoid store 'from, to, cc' recipients we Because Odoo implementation avoid store 'from, to, cc' recipients we
ensure that this information its written into the mail.message record. ensure that this information its written into the mail.message record.
""" """
kwargs.update( message_dict.update(
{"email_cc": kwargs.get("cc", False), "email_to": kwargs.get("to", False)} {
"email_cc": message_dict.get("cc", False),
"email_to": message_dict.get("to", False),
}
) )
return super().message_post(*args, **kwargs) return super()._message_route_process(message, message_dict, routes)
def _message_get_suggested_recipients(self): def _message_get_suggested_recipients(self):
"""Adds email 'extra' recipients as suggested recipients. """Adds email 'extra' recipients as suggested recipients.

View File

@ -182,7 +182,7 @@ class TestMailTracking(TransactionCase):
# pylint: disable=C8107 # pylint: disable=C8107
message = self.recipient.with_user(sender_user).message_post( message = self.recipient.with_user(sender_user).message_post(
body="<p>This is a test message</p>", body="<p>This is a test message</p>",
cc="Dominique Pinon <unnamed@test.com>, sender@example.com", email_cc="Dominique Pinon <unnamed@test.com>, sender@example.com",
) )
# suggested recipients # suggested recipients
recipients = self.recipient._message_get_suggested_recipients() recipients = self.recipient._message_get_suggested_recipients()
@ -235,7 +235,7 @@ class TestMailTracking(TransactionCase):
# pylint: disable=C8107 # pylint: disable=C8107
message = self.recipient.with_user(sender_user).message_post( message = self.recipient.with_user(sender_user).message_post(
body="<p>This is a test message</p>", body="<p>This is a test message</p>",
to="Dominique Pinon <support+unnamed@test.com>, sender@example.com", email_to="Dominique Pinon <support+unnamed@test.com>, sender@example.com",
) )
# suggested recipients # suggested recipients
recipients = self.recipient._message_get_suggested_recipients() recipients = self.recipient._message_get_suggested_recipients()