From b379d95f446b3c400b2010832e954adf76fd39b3 Mon Sep 17 00:00:00 2001 From: Xavier Morel Date: Tue, 29 Nov 2011 11:13:34 +0100 Subject: [PATCH] [IMP] make float-parsing code more readable by fixing some variable naming bzr revid: xmo@openerp.com-20111129101334-4uu8v21arkn5h0tu --- addons/web/static/src/js/formats.js | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/addons/web/static/src/js/formats.js b/addons/web/static/src/js/formats.js index 2fa36f3d221..2df651158eb 100644 --- a/addons/web/static/src/js/formats.js +++ b/addons/web/static/src/js/formats.js @@ -159,16 +159,16 @@ openerp.web.parse_value = function (value, descriptor, value_if_empty) { if (!isNaN(tmp)) return tmp; - var tmp2 = tmp = value; + var tmp2 = value; do { tmp = tmp2; tmp2 = tmp.replace(openerp.web._t.database.parameters.thousands_sep, ""); } while(tmp !== tmp2); - tmp = tmp.replace(openerp.web._t.database.parameters.decimal_point, "."); - tmp = Number(tmp); - if (isNaN(tmp)) + var reformatted_value = tmp.replace(openerp.web._t.database.parameters.decimal_point, "."); + var parsed = Number(reformatted_value); + if (isNaN(parsed)) throw new Error(value + " is not a correct float"); - return tmp; + return parsed; case 'float_time': var float_time_pair = value.split(":"); if (float_time_pair.length != 2)