From 32d3c44e3edbcff4b4f06df1d72bf2ea348c02cc Mon Sep 17 00:00:00 2001 From: Fabien Meghazi Date: Thu, 24 Nov 2011 14:46:27 +0100 Subject: [PATCH] [FIX] Do not alter field.selection in SelectionField. Clone the array instead. Revert #1550: debug information should not be altered. bzr revid: fme@openerp.com-20111124134627-mx3k2s22naht07go --- addons/web/static/src/js/view_form.js | 4 ++-- addons/web/static/src/xml/base.xml | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/addons/web/static/src/js/view_form.js b/addons/web/static/src/js/view_form.js index cb6a7bedc95..8b3f4737658 100644 --- a/addons/web/static/src/js/view_form.js +++ b/addons/web/static/src/js/view_form.js @@ -1605,7 +1605,7 @@ openerp.web.form.FieldSelection = openerp.web.form.Field.extend({ init: function(view, node) { var self = this; this._super(view, node); - this.values = this.field.selection; + this.values = _.clone(this.field.selection); _.each(this.values, function(v, i) { if (v[0] === false && v[1] === '') { self.values.splice(i, 1); @@ -3121,7 +3121,7 @@ openerp.web.form.FieldSelectionReadonly = openerp.web.form.FieldReadonly.extend( // lifted straight from r/w version var self = this; this._super(view, node); - this.values = this.field.selection; + this.values = _.clone(this.field.selection); _.each(this.values, function(v, i) { if (v[0] === false && v[1] === '') { self.values.splice(i, 1); diff --git a/addons/web/static/src/xml/base.xml b/addons/web/static/src/xml/base.xml index 0200798f3d5..1fe4d028db4 100644 --- a/addons/web/static/src/xml/base.xml +++ b/addons/web/static/src/xml/base.xml @@ -833,7 +833,7 @@
  • Selection: