From fa9c2829d05746bc98b2af309295b1a212754463 Mon Sep 17 00:00:00 2001 From: Xavier Morel Date: Mon, 4 Jul 2011 14:10:57 +0200 Subject: [PATCH] [FIX] value-content tests on set_value for some widgets note: NaN != NaN, need to use isNaN bzr revid: xmo@openerp.com-20110704121057-k9vrgim1pfc1g8u1 --- addons/base/static/src/js/form.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/addons/base/static/src/js/form.js b/addons/base/static/src/js/form.js index 98248ae37b9..874ff247a6f 100644 --- a/addons/base/static/src/js/form.js +++ b/addons/base/static/src/js/form.js @@ -935,7 +935,7 @@ openerp.base.form.FieldDatetime = openerp.base.form.Field.extend({ }, set_value: function(value) { this._super.apply(this, arguments); - if (value == null || value == false) { + if (!value) { this.$element.find('input').val(''); } else { this.$element.find('input').unbind('change'); @@ -1086,7 +1086,7 @@ openerp.base.form.FieldProgressBar = openerp.base.form.Field.extend({ set_value: function(value) { this._super.apply(this, arguments); var show_value = Number(value); - if (show_value === NaN) { + if (isNaN(show_value)) { show_value = 0; } this.$element.find('div').progressbar('option', 'value', show_value).find('span').html(show_value + '%');