diff options
| author | Christophe Matthieu <chm@odoo.com> | 2018-02-06 11:17:06 +0100 |
|---|---|---|
| committer | Christophe Matthieu <chm@odoo.com> | 2018-02-08 15:45:30 +0100 |
| commit | 4f5fc6ee552e5f2edca6401d04761a3c19797dc7 (patch) | |
| tree | 849e0deef856f5b72ad30ba12126ae056c97cfac /addons/mail/static/src | |
| parent | 83795d37f3b512797dd78de027e444f305ab9ef1 (diff) | |
[FIX] web: properly set parent when using mixins
The parentedMixin doesn't automatically set the parent, so when this mixin
is used, the parent must be set manually. This wasn't done at several places,
and thus those elements weren't destroyed with their actual parent.
In master, we should probably make the ParentedMixin set the parent itself.
Diffstat (limited to 'addons/mail/static/src')
| -rw-r--r-- | addons/mail/static/src/js/chat_manager.js | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/addons/mail/static/src/js/chat_manager.js b/addons/mail/static/src/js/chat_manager.js index f7aaf73637e..2a64dcda828 100644 --- a/addons/mail/static/src/js/chat_manager.js +++ b/addons/mail/static/src/js/chat_manager.js @@ -611,7 +611,7 @@ var ChatManager = Class.extend(Mixins.EventDispatcherMixin, ServicesMixin, { Mixins.EventDispatcherMixin.init.call(this); this.setParent(parent); - this.bus = new Bus(); + this.bus = new Bus(this); this.bus.on('client_action_open', null, function (open) { client_action_open = open; }); |