[FIX] mail, *: fix opens and redirects, adapt messaging menu close

task-2244279
task-2262862
task-2274993
task-2277537
task-2282238

closes odoo/odoo#56745

X-original-commit: 109f202
Related: odoo/enterprise#12761
Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
Signed-off-by: Sébastien Theys (seb) <seb@odoo.com>

1 files changed, 29 insertions, 24 deletions

diff --git a/addons/snailmail/static/src/models/notification_group/notification_group.js b/addons/snailmail/static/src/models/notification_group/notification_group.js
index c63ebe4c066..3fff33cbb01 100644
--- a/addons/snailmail/static/src/models/notification_group/notification_group.js
+++ b/addons/snailmail/static/src/models/notification_group/notification_group.js
@@ -15,18 +15,18 @@ registerInstancePatchModel('mail.notification_group', 'snailmail/static/src/mode
      * @override
      */
     openCancelAction() {
-        if (this.notification_type === 'snail') {
-            this.env.bus.trigger('do-action', {
-                action: 'snailmail.snailmail_letter_cancel_action',
-                options: {
-                    additional_context: {
-                        default_model: this.res_model,
-                        unread_counter: this.notifications.length,
-                    },
-                },
-            });
+        if (this.notification_type !== 'snail') {
+            return this._super(...arguments);
         }
-        return this._super(...arguments);
+        this.env.bus.trigger('do-action', {
+            action: 'snailmail.snailmail_letter_cancel_action',
+            options: {
+                additional_context: {
+                    default_model: this.res_model,
+                    unread_counter: this.notifications.length,
+                },
+            },
+        });
     },
 
     //--------------------------------------------------------------------------
@@ -37,20 +37,25 @@ registerInstancePatchModel('mail.notification_group', 'snailmail/static/src/mode
      * @override
      */
     _openDocuments() {
-        if (this.notification_type === 'snail') {
-            this.env.bus.trigger('do-action', {
-                action: {
-                    name: this.env._t("Snailmail Failures"),
-                    type: 'ir.actions.act_window',
-                    view_mode: 'kanban,list,form',
-                    views: [[false, 'kanban'], [false, 'list'], [false, 'form']],
-                    target: 'current',
-                    res_model: this.res_model,
-                    domain: [['message_ids.snailmail_error', '=', true]],
-                },
-            });
+        if (this.notification_type !== 'snail') {
+            return this._super(...arguments);
+        }
+        this.env.bus.trigger('do-action', {
+            action: {
+                name: this.env._t("Snailmail Failures"),
+                type: 'ir.actions.act_window',
+                view_mode: 'kanban,list,form',
+                views: [[false, 'kanban'], [false, 'list'], [false, 'form']],
+                target: 'current',
+                res_model: this.res_model,
+                domain: [['message_ids.snailmail_error', '=', true]],
+            },
+        });
+        if (this.env.messaging.device.isMobile) {
+            // messaging menu has a higher z-index than views so it must
+            // be closed to ensure the visibility of the view
+            this.env.messaging.messagingMenu.close();
         }
-        return this._super(...arguments);
     },
 });