diff options
| author | Xavier Morel <xmo@openerp.com> | 2011-11-29 11:13:34 +0100 |
|---|---|---|
| committer | Xavier Morel <xmo@openerp.com> | 2011-11-29 11:13:34 +0100 |
| commit | b379d95f446b3c400b2010832e954adf76fd39b3 (patch) | |
| tree | 9fd009c94e780d6bac1dfa15ce215dfb90e28b83 /addons/web/static/src | |
| parent | 90dfdbaf6a3518d53590388da5cfb695a8c210e6 (diff) | |
[IMP] make float-parsing code more readable by fixing some variable naming
bzr revid: xmo@openerp.com-20111129101334-4uu8v21arkn5h0tu
Diffstat (limited to 'addons/web/static/src')
| -rw-r--r-- | addons/web/static/src/js/formats.js | 10 |
1 files 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) |