From dd9bced5ef51a15c34a6fce3fa5c714bf252414e Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Alexandre=20D=2E=20D=C3=ADaz?=
 <alexandre.diaz@tecnativa.com>
Date: Mon, 1 Mar 2021 23:00:26 +0100
Subject: [PATCH] [FIX] mail_tracking: Store cc

---
 mail_tracking/__manifest__.py             |  2 +-
 mail_tracking/models/mail_thread.py       | 13 ++++++++-----
 mail_tracking/tests/test_mail_tracking.py |  4 ++--
 3 files changed, 11 insertions(+), 8 deletions(-)

diff --git a/mail_tracking/__manifest__.py b/mail_tracking/__manifest__.py
index 69f41b2e8..175e13a12 100644
--- a/mail_tracking/__manifest__.py
+++ b/mail_tracking/__manifest__.py
@@ -7,7 +7,7 @@
 {
     "name": "Email tracking",
     "summary": "Email tracking system for all mails sent",
-    "version": "13.0.1.0.7",
+    "version": "13.0.1.0.8",
     "category": "Social Network",
     "website": "https://github.com/OCA/social",
     "author": ("Tecnativa, " "Odoo Community Association (OCA)"),
diff --git a/mail_tracking/models/mail_thread.py b/mail_tracking/models/mail_thread.py
index c2718cc4a..cc10100b2 100644
--- a/mail_tracking/models/mail_thread.py
+++ b/mail_tracking/models/mail_thread.py
@@ -29,17 +29,20 @@ class MailThread(models.AbstractModel):
             ("mail_tracking_ids.state", "in", list(failed_states)),
         ]
 
-    @api.returns("self", lambda value: value.id)
-    def message_post(self, *args, **kwargs):
+    @api.model
+    def _message_route_process(self, message, message_dict, routes):
         """Adds CC recipient to the message.
 
         Because Odoo implementation avoid store 'from, to, cc' recipients we
         ensure that this information its written into the mail.message record.
         """
-        kwargs.update(
-            {"email_cc": kwargs.get("cc", False), "email_to": kwargs.get("to", False)}
+        message_dict.update(
+            {
+                "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):
         """Adds email 'extra' recipients as suggested recipients.
diff --git a/mail_tracking/tests/test_mail_tracking.py b/mail_tracking/tests/test_mail_tracking.py
index 2411e8a04..3ebda7658 100644
--- a/mail_tracking/tests/test_mail_tracking.py
+++ b/mail_tracking/tests/test_mail_tracking.py
@@ -182,7 +182,7 @@ class TestMailTracking(TransactionCase):
         # pylint: disable=C8107
         message = self.recipient.with_user(sender_user).message_post(
             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
         recipients = self.recipient._message_get_suggested_recipients()
@@ -235,7 +235,7 @@ class TestMailTracking(TransactionCase):
         # pylint: disable=C8107
         message = self.recipient.with_user(sender_user).message_post(
             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
         recipients = self.recipient._message_get_suggested_recipients()