From 95cbf1d37b163d86dc68ebdbee2b45d3cb2e2700 Mon Sep 17 00:00:00 2001 From: Aaron Bohy Date: Mon, 27 Feb 2017 15:46:13 +0100 Subject: [FIX] web: list_editable_renderer: don't instantiate widgets twice In an editable list view, if there was a readonly field in the table, positionned before editable fields, clicking on that readonly field resulted in the editable field widgets being re-instantiated and appended to the DOM (so there were several widgets per fields/cells). --- addons/web/static/src/js/views/list/list_editable_renderer.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'addons/web/static/src/js') diff --git a/addons/web/static/src/js/views/list/list_editable_renderer.js b/addons/web/static/src/js/views/list/list_editable_renderer.js index 7be3f3d4e29..38b3066a4d3 100644 --- a/addons/web/static/src/js/views/list/list_editable_renderer.js +++ b/addons/web/static/src/js/views/list/list_editable_renderer.js @@ -366,7 +366,7 @@ ListRenderer.include({ // if we are just changing active cell in the same row, activate the // corresponding widget and return - if (rowIndex === this.currentRow && colIndex !== this.currentCol) { + if (rowIndex === this.currentRow) { var w = _.findWhere(this.widgets, { __rowIndex: rowIndex, __colIndex: colIndex -- cgit v1.2.3