From b997309cf694cbd5e8484c9516d7055fc5b1180c Mon Sep 17 00:00:00 2001 From: Aaron Bohy Date: Mon, 27 Feb 2017 16:57:20 +0100 Subject: [PATCH] [FIX] web: abstract_view: don't override shared config variable The method 'setController' allows to change the default Controller Widget used by the view. However, before this fix, calling it changed the default Controller of all other Controller instances. --- addons/web/static/src/js/views/abstract_view.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/addons/web/static/src/js/views/abstract_view.js b/addons/web/static/src/js/views/abstract_view.js index c95e4f084fe..22514f3ba75 100644 --- a/addons/web/static/src/js/views/abstract_view.js +++ b/addons/web/static/src/js/views/abstract_view.js @@ -119,7 +119,7 @@ var AbstractView = Class.extend({ return $.when(model.load(this.loadParams), this._loadLibs()).then(function () { var state = model.get(arguments[0]); var renderer = self.createRenderer(parent, state); - var Controller = self.config.Controller; + var Controller = self.Controller || self.config.Controller; var controllerParams = _.extend({ initialState: state, }, self.controllerParams); @@ -149,7 +149,7 @@ var AbstractView = Class.extend({ * @param {Controller} Controller */ setController: function (Controller) { - this.config.Controller = Controller; + this.Controller = Controller; }, //--------------------------------------------------------------------------