summaryrefslogtreecommitdiff
path: root/addons/web/static/src/js/view_form.js
diff options
context:
space:
mode:
authorXavier Morel <xmo@openerp.com>2012-12-06 14:51:14 +0100
committerXavier Morel <xmo@openerp.com>2012-12-06 14:51:14 +0100
commit7ac99cbeafc88f0fe6f6c2ee425c9ef87180a2ce (patch)
tree33be658f2ccacade894ed9c5e9d5d35976d89ad1 /addons/web/static/src/js/view_form.js
parent96408a36f03f7184c7a43e7b68d3eea5e73e924e (diff)
[FIX] form selection field: don't interpret value 0 as false in the template when the code doesn't do so
bzr revid: xmo@openerp.com-20121206135114-nuwc3wdc6zatbl3d
Diffstat (limited to 'addons/web/static/src/js/view_form.js')
-rw-r--r--addons/web/static/src/js/view_form.js11
1 files changed, 4 insertions, 7 deletions
diff --git a/addons/web/static/src/js/view_form.js b/addons/web/static/src/js/view_form.js
index 21c3b3f43d9..d5feb342ebe 100644
--- a/addons/web/static/src/js/view_form.js
+++ b/addons/web/static/src/js/view_form.js
@@ -2680,13 +2680,10 @@ instance.web.form.FieldSelection = instance.web.form.AbstractField.extend(instan
init: function(field_manager, node) {
var self = this;
this._super(field_manager, node);
- this.values = _.clone(this.field.selection);
- _.each(this.values, function(v, i) {
- if (v[0] === false && v[1] === '') {
- self.values.splice(i, 1);
- }
- });
- this.values.unshift([false, '']);
+ this.values = _(this.field.selection).chain()
+ .reject(function (v) { return v[0] === false && v[1] === ''; })
+ .unshift([false, ''])
+ .value();
},
initialize_content: function() {
// Flag indicating whether we're in an event chain containing a change