From 3b2127c690959e1e43063b09472028a69a62fcb2 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 be5496c55c7..eb70f330076 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