[IMP] mail, mail_bot: improved push notifications permission response

This commit makes small improvements on the messaging menu with
the new module `mail_bot`:

[IMP] When OdooBot has a request, it now adds "+1" to the counter of
      the messaging menu. The related preview also has a bold title,
      and displays "(1)" next to the title.

[REF] Static previews are no longer in mail_service. This was only
      used for "OdooBot has a request" preview. This logic has now
      been moved to the module `mail_bot` (was previously in `mail`).

[FIX] Title of response of push notifications permission clearly
      states whether permission has been granted or denied
      (was displaying "[Object object]" before this commit).

Task-ID 1877502

3 files changed, 263 insertions, 1 deletions

diff --git a/addons/mail_bot/static/src/js/mailbot_service.js b/addons/mail_bot/static/src/js/mailbot_service.js
index 300e95d06db..a75780ab090 100644
--- a/addons/mail_bot/static/src/js/mailbot_service.js
+++ b/addons/mail_bot/static/src/js/mailbot_service.js
@@ -5,8 +5,9 @@ var AbstractService = require('web.AbstractService');
 var core = require('web.core');
 var session = require('web.session');
 
+var _t = core._t;
+
 var MailBotService =  AbstractService.extend({
-    name: 'mailbot_service',
     /**
      * @override
      */
@@ -22,7 +23,40 @@ var MailBotService =  AbstractService.extend({
             }, 2*60*1000);
         }
     },
+    /**
+     * Get the previews related to the OdooBot (conversation not included).
+     * For instance, when there is no conversation with OdooBot and OdooBot has
+     * a request, it should display a preview in the systray messaging menu.
+     *
+     * @param {string|undefined} [filter]
+     * @returns {Object[]} list of objects that are compatible with the
+     *   'mail.Preview' template.
+     */
+    getPreviews: function (filter) {
+        var previews = [];
+        if (this.hasRequest() && (filter === 'mailbox_inbox' || !filter)) {
+            previews.push({
+                title: _t("OdooBot has a request"),
+                imageSRC: "/mail/static/src/img/odoobot.png",
+                status: 'bot',
+                body:  _t("Enable desktop notifications to chat"),
+                id: 'request_notification',
+                unreadCounter: 1,
+            });
+        }
+        return previews;
+    },
+    /**
+     * Tell whether OdooBot has a request or not.
+     *
+     * @returns {boolean}
+     */
+    hasRequest: function () {
+        return window.Notification && window.Notification.permission === "default";
+    },
 });
+
 core.serviceRegistry.add('mailbot_service', MailBotService);
 return MailBotService;
+
 });
diff --git a/addons/mail_bot/static/src/js/systray_messaging_menu.js b/addons/mail_bot/static/src/js/systray_messaging_menu.js
new file mode 100644
index 00000000000..05c0ee54f24
--- /dev/null
+++ b/addons/mail_bot/static/src/js/systray_messaging_menu.js
@@ -0,0 +1,121 @@
+odoo.define('mail_bot.systray.MessagingMenu', function (require) {
+"use strict";
+
+var MessagingMenu = require('mail.systray.MessagingMenu');
+var core = require('web.core');
+
+var _t = core._t;
+
+return MessagingMenu.include({
+
+    //--------------------------------------------------------------------------
+    // Private
+    //--------------------------------------------------------------------------
+
+    /**
+     * Override so that 'mailbot has a request' is included in the computation
+     * of of the counter.
+     *
+     * @override
+     * @private
+     * @returns {integer}
+     */
+    _computeCounter: function () {
+        var counter = this._super.apply(this, arguments);
+        if (this.call('mailbot_service', 'hasRequest')) {
+            counter++;
+        }
+        return counter;
+    },
+    /**
+     * Override so that the mailbot previews are included in the systray
+     * messaging menu (e.g. 'OdooBot has a request')
+     *
+     * @override
+     * @private
+     * @returns {$.Promise<Object[]>} resolved with list of previews that are
+     *   compatible with the 'mail.Preview' template.
+     */
+    _getPreviews: function () {
+        var mailbotPreviews = this.call('mailbot_service', 'getPreviews', this._filter);
+        return this._super.apply(this, arguments).then(function (previews) {
+            return _.union(mailbotPreviews, previews);
+        });
+    },
+    /**
+     * Handle the response of the user when prompted whether push notifications
+     * are granted or denied.
+     *
+     * Also refreshes the counter after a response from a push notification
+     * request. This is useful because the counter contains a part for the
+     * OdooBot, and the OdooBot influences the counter by 1 when it requests
+     * for notifications. This should no longer be the case when push
+     * notifications are either granted or denied.
+     *
+     * @private
+     * @param {string} value
+     */
+    _handleResponseNotificationPermission: function (value) {
+        if (value !== 'granted') {
+            this.call('bus_service', 'sendNotification', _t('Permission denied'),
+                _t('Odoo will not have the permission to send native notifications on this device.'));
+        } else {
+            this.call('bus_service', 'sendNotification', _t('Yay, push notifications are enabled!'),
+                _t('Odoo has now the permission to send you native notifications on this device.'));
+        }
+        this._updateCounter();
+    },
+    /**
+     * Display the browser notification request dialog when the user clicks on
+     * systray's corresponding notification
+     *
+     * @private
+     */
+    _requestNotificationPermission: function () {
+        var def = window.Notification && window.Notification.requestPermission();
+        if (def) {
+            def.then(this._handleResponseNotificationPermission.bind(this));
+        }
+        this.$('.o_mail_navbar_request_permission').slideUp();
+    },
+
+    //--------------------------------------------------------------------------
+    // Handlers
+    //--------------------------------------------------------------------------
+
+    /**
+     * Override so that it handles preview related to OdooBot
+     *
+     * @override
+     * @private
+     * @param {MouseEvent} ev
+     */
+    _onClickPreview: function (ev) {
+        var previewID = $(ev.currentTarget).data('preview-id');
+        if (previewID === 'request_notification') {
+            this._requestNotificationPermission();
+        } else {
+            this._super.apply(this, arguments);
+        }
+    },
+    /**
+     * Override so that it handles clicking on 'mark as read' similarly to
+     * requesting push notification permission.
+     *
+     * @override
+     * @private
+     * @param {MouseEvent} ev
+     */
+    _onClickPreviewMarkAsRead: function (ev) {
+        ev.stopPropagation();
+        var $preview = $(ev.currentTarget).closest('.o_mail_preview');
+        var previewID = $preview.data('preview-id');
+        if (previewID === 'request_notification') {
+            this._requestNotificationPermission();
+        } else {
+            this._super.apply(this, arguments);
+        }
+    }
+});
+
+});
diff --git a/addons/mail_bot/static/tests/systray_messaging_menu_tests.js b/addons/mail_bot/static/tests/systray_messaging_menu_tests.js
new file mode 100644
index 00000000000..a04aba7056a
--- /dev/null
+++ b/addons/mail_bot/static/tests/systray_messaging_menu_tests.js
@@ -0,0 +1,107 @@
+odoo.define('mail_bot.systray.MessagingMenuTests', function (require) {
+"use strict";
+
+var MessagingMenu = require('mail.systray.MessagingMenu');
+var mailTestUtils = require('mail.testUtils');
+
+var MailBotService = require('mail_bot.MailBotService');
+
+var testUtils = require('web.test_utils');
+
+QUnit.module('mail_bot', {}, function () {
+QUnit.module('MessagingMenu', {
+    beforeEach: function () {
+        var self = this;
+
+        this.data = {
+            'mail.channel': {
+                fields: {},
+                records: [],
+            },
+            'mail.message': {
+                fields: {},
+                records: [],
+            },
+        };
+
+        // Patch mailbot_service so that it does do not do any RPC, and
+        // the detection of push notifications permissions can be simulated in
+        // the test cases. By default, shows 'OdooBot has a request'
+        this.hasMailbotRequest = true;
+        testUtils.patch(MailBotService, {
+            /**
+             * @override
+             */
+            start: function () {},
+            /**
+             * @override
+             * @returns {boolean}
+             */
+            hasRequest: function () {
+                return self.hasMailbotRequest;
+            },
+        });
+
+        this.services = _.extend({}, mailTestUtils.getMailServices(), {
+            mailbot_service: MailBotService
+        });
+    },
+    afterEach: function () {
+        // unpatch MailBotService
+        testUtils.unpatch(MailBotService);
+    }
+});
+
+QUnit.test('messaging menu widget: rendering with OdooBot has a request', function (assert) {
+    assert.expect(5);
+
+    var messagingMenu = new MessagingMenu();
+    testUtils.addMockEnvironment(messagingMenu, {
+        data: this.data,
+        services: this.services,
+    });
+    messagingMenu.appendTo($('#qunit-fixture'));
+
+    assert.strictEqual(messagingMenu.$('.o_notification_counter').length, 1,
+        "should display a notification counter next to the messaging menu");
+    assert.strictEqual(messagingMenu.$('.o_notification_counter').text(), '1',
+        "should display a counter of '1' next to the messaging menu");
+
+    messagingMenu.$('.dropdown-toggle').click();
+    assert.strictEqual(messagingMenu.$('.o_preview_info').length, 1,
+        "should display a preview in the messaging menu");
+    assert.strictEqual(messagingMenu.$('.o_preview_name').text().trim(),
+        'OdooBot has a request',
+        "preview should display that OdooBot has a request");
+    assert.strictEqual(messagingMenu.$('.o_preview_counter').text().replace(/\s/g, ''),
+        '(1)', "should display an counter of '1' next to the preview");
+
+    messagingMenu.destroy();
+});
+
+QUnit.test('messaging menu widget: rendering without OdooBot has a request', function (assert) {
+    assert.expect(3);
+
+    this.hasMailbotRequest = false;
+
+    var messagingMenu = new MessagingMenu();
+    testUtils.addMockEnvironment(messagingMenu, {
+        data: this.data,
+        services: this.services,
+    });
+    messagingMenu.appendTo($('#qunit-fixture'));
+
+    assert.strictEqual(messagingMenu.$('.o_notification_counter').length, 1,
+        "should display a notification counter next to the messaging menu");
+    assert.strictEqual(messagingMenu.$('.o_notification_counter').text(), '0',
+        "should display a counter of '0' next to the messaging menu");
+    messagingMenu.$('.dropdown-toggle').click();
+    assert.strictEqual(messagingMenu.$('.o_preview_info').length, 0,
+        "should display no preview in the messaging menu");
+
+    messagingMenu.destroy();
+});
+
+
+});
+});