From 910ac4caea27ba926b35aa92829b36a3b282ac74 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?S=C3=A9bastien=20Theys?= <seb@odoo.com>
Date: Wed, 8 Jun 2022 14:01:25 +0000
Subject: [PATCH] [IMP] mail: move hasParentReloadOnAttachmentsChanged code in
 models

Remove code from legacy form in preparation of Owl form.

Part of task-2871070

Part-of: odoo/odoo#93273
---
 .../mail/static/src/models/attachment_delete_confirm_view.js  | 4 ++--
 addons/mail/static/src/widgets/form_renderer/form_renderer.js | 4 ----
 2 files changed, 2 insertions(+), 6 deletions(-)

diff --git a/addons/mail/static/src/models/attachment_delete_confirm_view.js b/addons/mail/static/src/models/attachment_delete_confirm_view.js
index c5838bbd7e3..566302a59d2 100644
--- a/addons/mail/static/src/models/attachment_delete_confirm_view.js
+++ b/addons/mail/static/src/models/attachment_delete_confirm_view.js
@@ -24,8 +24,8 @@ registerModel({
         async onClickOk() {
             const chatter = this.chatter;
             await this.attachment.remove();
-            if (chatter && chatter.component) {
-                chatter.component.trigger('o-attachments-changed');
+            if (chatter && chatter.exists() && chatter.hasParentReloadOnAttachmentsChanged) {
+                chatter.reloadParentView();
             }
         },
         /**
diff --git a/addons/mail/static/src/widgets/form_renderer/form_renderer.js b/addons/mail/static/src/widgets/form_renderer/form_renderer.js
index 4f9fea1139f..14a16506ca2 100644
--- a/addons/mail/static/src/widgets/form_renderer/form_renderer.js
+++ b/addons/mail/static/src/widgets/form_renderer/form_renderer.js
@@ -42,7 +42,6 @@ FormRenderer.include({
     destroy() {
         this._super(...arguments);
         this._chatterContainerComponent = undefined;
-        this.off('o_attachments_changed', this);
         this.off('o_chatter_rendered', this);
         this.off('o_message_posted', this);
         Component.env.bus.off('Thread:promptAddFollower-closed', this);
@@ -80,9 +79,6 @@ FormRenderer.include({
                 this.trigger_up('reload', { keepChanges: true });
             });
         }
-        if (this.chatterFields.hasRecordReloadOnAttachmentsChanged) {
-            this.on('o_attachments_changed', this, ev => this.trigger_up('reload', { keepChanges: true }));
-        }
         if (this.chatterFields.hasRecordReloadOnFollowersUpdate) {
             Component.env.bus.on('Thread:promptAddFollower-closed', this, ev => this.trigger_up('reload', { keepChanges: true }));
         }