From 4f5fc6ee552e5f2edca6401d04761a3c19797dc7 Mon Sep 17 00:00:00 2001 From: Christophe Matthieu Date: Tue, 6 Feb 2018 11:17:06 +0100 Subject: [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. --- addons/web/static/src/js/_deprecated/data.js | 1 + 1 file changed, 1 insertion(+) (limited to 'addons/web/static/src/js/_deprecated/data.js') diff --git a/addons/web/static/src/js/_deprecated/data.js b/addons/web/static/src/js/_deprecated/data.js index 14b3daeb2c6..e13ca14da22 100644 --- a/addons/web/static/src/js/_deprecated/data.js +++ b/addons/web/static/src/js/_deprecated/data.js @@ -314,6 +314,7 @@ var DataSet = Class.extend(mixins.PropertiesMixin, { */ init: function (parent, model, context) { mixins.PropertiesMixin.init.call(this); + this.setParent(parent); this.model = model; this.context = context || {}; this.index = null; -- cgit v1.2.3