Fixes esamattis#329; Allow negative decimal precision

megawac · Nov 30, 2014 · d78c62b · d78c62b
1 parent ee72fe2
commit d78c62b
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 9 deletions.
diff --git a/lib/underscore.string.js b/lib/underscore.string.js
@@ -457,11 +457,10 @@
       return sprintf.apply(null, argv);
     },
 
-    toNumber: function(str, decimals) {
-      if (!str) return 0;
-      str = _s.trim(str);
-      if (!str.match(/^-?\d+(?:\.\d+)?$/)) return NaN;
-      return parseNumber(parseNumber(str).toFixed(~~decimals));
+    toNumber: function(num, precision) {
+      if (num == null) return 0;
+      var factor = Math.pow(10, isFinite(precision) ? precision : 0);
+      return Math.round(num * factor) / factor;
     },
 
     numberFormat : function(number, dec, dsep, tsep) {

diff --git a/test/strings.js b/test/strings.js
@@ -491,10 +491,14 @@ $(document).ready(function() {
   });
 
   test('String: toNumber', function() {
-    deepEqual(_('not a number').toNumber(), NaN);
+    _.each(['not a number', NaN, {}, [/a/], 'alpha6'], function(val) {
+      deepEqual(_('not a number').toNumber(), NaN);
+      equal(_(Math.PI).toNumber(val), 3);
+    });
     equal(_(0).toNumber(), 0);
     equal(_('0').toNumber(), 0);
     equal(_('0.0').toNumber(), 0);
+    equal(_('        0.0    ').toNumber(), 0);
     equal(_('0.1').toNumber(), 0);
     equal(_('0.1').toNumber(1), 0.1);
     equal(_('  0.1 ').toNumber(1), 0.1);
@@ -507,9 +511,22 @@ $(document).ready(function() {
     equal(_(2).toNumber(2), 2.00);
     equal(_(-2).toNumber(), -2);
     equal(_('-2').toNumber(), -2);
-    equal(_('').toNumber(), 0);
-    equal(_(null).toNumber(), 0);
-    equal(_(undefined).toNumber(), 0);
+    equal(_(-2.5123).toNumber(3), -2.512);
+
+    // Negative precisions
+    equal(_(-234).toNumber(-1), -230);
+    equal(_(234).toNumber(-2), 200);
+    equal(_('234').toNumber(-2), 200);
+
+    _.each(['', null, undefined], function(val) {
+      equal(_(val).toNumber(), 0);
+    });
+
+    _.each([Infinity, -Infinity], function(val) {
+      equal(_(val).toNumber(), val);
+      equal(_(val).toNumber(val), val);
+      equal(_(1).toNumber(val), 1);
+    });
   });
 
   test('String: numberFormat', function() {