From 671bd703bf7a6c76d4ab01bc2a5771f401f37a09 Mon Sep 17 00:00:00 2001 From: Nicolas Lempereur Date: Thu, 13 Apr 2017 13:55:43 +0200 Subject: [PATCH] [FIX] mail: error notification safari iphone 7 On safari iphone 7 (and possibly other browsers) we could have an error by referencing to an undeclared Notification variable. opw-726084 --- addons/mail/static/src/js/client_action.js | 2 +- addons/mail/static/src/js/utils.js | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/addons/mail/static/src/js/client_action.js b/addons/mail/static/src/js/client_action.js index 29331ff746e..0e3985c3e1b 100644 --- a/addons/mail/static/src/js/client_action.js +++ b/addons/mail/static/src/js/client_action.js @@ -121,7 +121,7 @@ var ChatAction = Widget.extend(ControlPanelMixin, { "click .o_mail_request_permission": function (event) { event.preventDefault(); this.$(".o_mail_annoying_notification_bar").slideUp(); - var def = window.Notification.requestPermission(); + var def = window.Notification && window.Notification.requestPermission(); if (def) { def.then(function (value) { if (value === 'denied') { diff --git a/addons/mail/static/src/js/utils.js b/addons/mail/static/src/js/utils.js index 3bc730d15e8..b4e0c01a23a 100644 --- a/addons/mail/static/src/js/utils.js +++ b/addons/mail/static/src/js/utils.js @@ -7,7 +7,7 @@ var web_client = require('web.web_client'); function send_notification(title, content) { - if (Notification && Notification.permission === "granted") { + if (window.Notification && Notification.permission === "granted") { if (bus.is_master) { _send_native_notification(title, content); }