From c3a73a1fddcec4d160468203b033040d32fcaefd Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?S=C3=A9bastien=20Theys?= <seb@odoo.com>
Date: Tue, 18 Jul 2023 16:50:29 +0000
Subject: [PATCH] [REF] mail: move mail.message/toggle_star to new service

Part of task-3265211

Part-of: odoo/odoo#129583
---
 .../static/src/core/common/mail_core_common_service.js    | 7 +++++++
 addons/mail/static/src/core/common/messaging_service.js   | 8 --------
 2 files changed, 7 insertions(+), 8 deletions(-)

diff --git a/addons/mail/static/src/core/common/mail_core_common_service.js b/addons/mail/static/src/core/common/mail_core_common_service.js
index 8c922cc5943..76d5b81913b 100644
--- a/addons/mail/static/src/core/common/mail_core_common_service.js
+++ b/addons/mail/static/src/core/common/mail_core_common_service.js
@@ -55,6 +55,13 @@ export class MailCoreCommon {
                     this.env.bus.trigger("mail.message/delete", { message });
                 }
             });
+            this.busService.subscribe("mail.message/toggle_star", (payload) => {
+                const { message_ids: messageIds, starred } = payload;
+                for (const messageId of messageIds) {
+                    const message = this.messageService.insert({ id: messageId });
+                    this.messageService.updateStarred(message, starred);
+                }
+            });
             this.busService.subscribe("mail.record/insert", (payload) => {
                 if (payload.Thread) {
                     this.threadService.insert(payload.Thread);
diff --git a/addons/mail/static/src/core/common/messaging_service.js b/addons/mail/static/src/core/common/messaging_service.js
index 8341325054f..34c465639bc 100644
--- a/addons/mail/static/src/core/common/messaging_service.js
+++ b/addons/mail/static/src/core/common/messaging_service.js
@@ -128,14 +128,6 @@ export class Messaging {
     handleNotification(notifications) {
         for (const notif of notifications) {
             switch (notif.type) {
-                case "mail.message/toggle_star": {
-                    const { message_ids: messageIds, starred } = notif.payload;
-                    for (const messageId of messageIds) {
-                        const message = this.messageService.insert({ id: messageId });
-                        this.messageService.updateStarred(message, starred);
-                    }
-                    break;
-                }
                 case "mail.message/notification_update":
                     {
                         notif.payload.elements.map((message) => {