Fix parseInt() radix issue, closes #80

validatorjs · Jun 5, 2012 · a3ccb72 · a3ccb72
1 parent 336681d
commit a3ccb72
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/lib/filter.js b/lib/filter.js
@@ -74,8 +74,7 @@ Filter.prototype.toFloat = function() {
 }
 
 Filter.prototype.toInt = function(radix) {
-    radix = radix || 10;
-    this.modify(parseInt(this.str*1, radix));
+    this.modify(parseInt(this.str, radix || 10));
     return this.wrap(this.str);
 }
 

diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 { "name"          : "validator",
   "description"   : "Data validation, filtering and sanitization for node.js",
-  "version"       : "0.4.6-2",
+  "version"       : "0.4.6-3",
   "homepage"      : "http://github.com/chriso/node-validator",
   "keywords"      : ["validator", "validation", "assert", "params", "sanitization", "xss", "entities", "sanitize", "sanitisation", "input"],
   "author"        : "Chris O'Hara <cohara87@gmail.com>",

diff --git a/test/filter.test.js b/test/filter.test.js
@@ -100,6 +100,7 @@ module.exports = {
 
     'test #toInt()': function () {
         assert.ok(3 === Filter.sanitize('3').toInt());
+        assert.ok(255 === Filter.sanitize('ff').toInt(16));
         assert.ok(3 === Filter.sanitize('   3   ').toInt());
     },