From e4ac74450f346f47d397a510d8efe65bd5fa0f1f Mon Sep 17 00:00:00 2001 From: Martin Geubelle Date: Wed, 26 Apr 2017 12:26:44 +0200 Subject: [FIX] web, calendar: reintroduce many2manyattendee fields widget This widget behaviour hadn't been correctly implemented with the new views. This commit also introduces some tests. --- addons/web/static/src/js/fields/relational_fields.js | 20 +++++++++++++++----- 1 file changed, 15 insertions(+), 5 deletions(-) (limited to 'addons/web/static/src/js/fields/relational_fields.js') diff --git a/addons/web/static/src/js/fields/relational_fields.js b/addons/web/static/src/js/fields/relational_fields.js index f1eaad118ab..459b529c748 100644 --- a/addons/web/static/src/js/fields/relational_fields.js +++ b/addons/web/static/src/js/fields/relational_fields.js @@ -1201,6 +1201,20 @@ var FieldMany2ManyTags = AbstractField.extend({ }); } }, + /** + * Get the QWeb rendering context used by the tag template; this computation + * is placed in a separate function for other tags to override it. + * + * @private + * @returns {Object} + */ + _getRenderTagsContext: function () { + var elements = this.value ? _.pluck(this.value.data, 'data') : []; + return { + elements: elements, + readonly: this.mode === "readonly", + }; + }, /** * @private * @param {any} id @@ -1246,11 +1260,7 @@ var FieldMany2ManyTags = AbstractField.extend({ * @private */ _renderTags: function () { - var elements = this.value ? _.pluck(this.value.data, 'data') : []; - this.$el.html(qweb.render(this.tag_template, { - elements: elements, - readonly: this.mode === "readonly", - })); + this.$el.html(qweb.render(this.tag_template, this._getRenderTagsContext())); }, //-------------------------------------------------------------------------- -- cgit v1.2.3