From 4f5d498eb38b2ffbaaa65f6eae4bb72c55ab7dee Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?G=C3=A9ry=20Debongnie?= Date: Fri, 24 Jul 2015 14:27:44 +0200 Subject: [PATCH] [IMP] web: modify notification.js like in enterprise The enterprise web client had to be modified to fix a crash. This crash does not occur in the community web client, but this commit makes sure that the semantics of the notification manager is the same as the enterprise version, to prevent future problems. --- addons/web/static/src/js/widgets/notification.js | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/addons/web/static/src/js/widgets/notification.js b/addons/web/static/src/js/widgets/notification.js index 314034ef748..e6d26265161 100644 --- a/addons/web/static/src/js/widgets/notification.js +++ b/addons/web/static/src/js/widgets/notification.js @@ -54,7 +54,8 @@ var NotificationManager = Widget.extend({ className: 'o_notification_manager', display: function(notification) { - return notification.appendTo(this.$el); + notification.appendTo(this.$el); + return notification; }, notify: function(title, text, sticky) { return this.display(new Notification(this, title, text, sticky));