Skip to content

Commit

Permalink
[IMP] web: modify notification.js like in enterprise
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
ged-odoo committed Jul 24, 2015
1 parent cf66cfe commit 4f5d498
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion addons/web/static/src/js/widgets/notification.js
Original file line number Diff line number Diff line change
Expand Up @@ -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));
Expand Down

0 comments on commit 4f5d498

Please sign in to comment.