express-validator - npm Package Compare versions

Comparing version 0.4.1 to 0.5.0

lib/express_validator.js

@@ -28,126 +28,136 @@ /*
 
-function checkParam(req, getter) {
-  return function(param, fail_msg) {
+var expressValidator = function(options) {
+  options = options || {};
 
-    var value;
+  var _options = {};
 
-    // If param is not an array, then split by dot notation
-    // returning an array. It will return an array even if
-    // param doesn't have the dot notation.
-    //      'blogpost' = ['blogpost']
-    //      'login.username' = ['login', 'username']
-    // For regex matches you can access the parameters using numbers.
-    if (!Array.isArray(param)) {
-      param = typeof param === 'number' ?
-              [param] :
-              param.split('.').filter(function(e){
-                return e !== '';
-              });
-    }
+  _options.errorFormatter = options.errorFormatter || function(param, msg, value) {
+    return {
+      param : param,
+      msg   : msg,
+      value : value
+    };
+  };
 
-    // Extract value from params
-    param.map(function(item) {
-        if (value === undefined) {
-          value = getter(item)
-        } else {
-          value = value[item];
-        }
-    });
-    param = param.join('.');
+  function checkParam(req, getter) {
+    return function(param, fail_msg) {
 
-    validator.error = function(msg) {
-      var error = {
-        param: param,
-        msg: msg,
-        value: value
-      };
-      if (req._validationErrors === undefined) {
-        req._validationErrors = [];
+      var value;
+
+      // If param is not an array, then split by dot notation
+      // returning an array. It will return an array even if
+      // param doesn't have the dot notation.
+      //      'blogpost' = ['blogpost']
+      //      'login.username' = ['login', 'username']
+      // For regex matches you can access the parameters using numbers.
+      if (!Array.isArray(param)) {
+        param = typeof param === 'number' ?
+                [param] :
+                param.split('.').filter(function(e){
+                  return e !== '';
+                });
       }
-      req._validationErrors.push(error);
 
-      if (req.onErrorCallback) {
-        req.onErrorCallback(msg);
+      // Extract value from params
+      param.map(function(item) {
+          if (value === undefined) {
+            value = getter(item)
+          } else {
+            value = value[item];
+          }
+      });
+      param = param.join('.');
+
+      validator.error = function(msg) {
+        var error = _options.errorFormatter(param, msg, value);
+
+        if (req._validationErrors === undefined) {
+          req._validationErrors = [];
+        }
+        req._validationErrors.push(error);
+
+        if (req.onErrorCallback) {
+          req.onErrorCallback(msg);
+        }
+        return this;
       }
-      return this;
+      return validator.check(value, fail_msg);
     }
-    return validator.check(value, fail_msg);
   }
-}
+  return function(req, res, next) {
 
-var expressValidator = function(req, res, next) {
+    req.updateParam = function(name, value) {
+      // route params like /user/:id
+      if (this.params && this.params.hasOwnProperty(name) &&
+          undefined !== this.params[name]) {
+        return this.params[name] = value;
+      }
+      // query string params
+      if (undefined !== this.query[name]) {
+        return this.query[name] = value;
+      }
+      // request body params via connect.bodyParser
+      if (this.body && undefined !== this.body[name]) {
+        return this.body[name] = value;
+      }
+      return false;
+    };
 
-  req.updateParam = function(name, value) {
-    // route params like /user/:id
-    if (this.params && this.params.hasOwnProperty(name) &&
-        undefined !== this.params[name]) {
-      return this.params[name] = value;
-    }
-    // query string params
-    if (undefined !== this.query[name]) {
-      return this.query[name] = value;
-    }
-    // request body params via connect.bodyParser
-    if (this.body && undefined !== this.body[name]) {
-      return this.body[name] = value;
-    }
-    return false;
-  };
+    req.check = checkParam(req, function(item) {
+      return req.param(item);
+    });
 
-  req.check = checkParam(req, function(item) {
-    return req.param(item);
-  });
+    req.checkBody = checkParam(req, function(item) {
+      return req.body[item];
+    });
 
-  req.checkBody = checkParam(req, function(item) {
-    return req.body[item];
-  });
+    req.checkHeader = function(header, fail_msg) {
+      var to_check;
+      if (header === 'referrer' || header === 'referer') {
+        to_check = this.headers.referer;
+      } else {
+        to_check = this.headers[header];
+      }
+      return validator.check(to_check || '', fail_msg);
+    };
 
-  req.checkHeader = function(header, fail_msg) {
-    var to_check;
-    if (header === 'referrer' || header === 'referer') {
-      to_check = this.headers.referer;
-    } else {
-      to_check = this.headers[header];
-    }
-    return validator.check(to_check || '', fail_msg);
-  };
+    req.onValidationError = function(errback) {
+      req.onErrorCallback = errback;
+    };
 
-  req.onValidationError = function(errback) {
-    req.onErrorCallback = errback;
-  };
-
-  req.validationErrors = function(mapped) {
-    if (req._validationErrors === undefined) {
-      return null;
+    req.validationErrors = function(mapped) {
+      if (req._validationErrors === undefined) {
+        return null;
+      }
+      if (mapped) {
+        var errors = {};
+        req._validationErrors.forEach(function(err) {
+          errors[err.param] = err;
+        });
+        return errors;
+      }
+      return req._validationErrors;
     }
-    if (mapped) {
-      var errors = {};
-      req._validationErrors.forEach(function(err) {
-        errors[err.param] = err;
-      });
-      return errors;
-    }
-    return req._validationErrors;
-  }
 
-  req.filter = function(param) {
-    var self = this;
-    var filter = new Filter();
-    filter.modify = function(str) {
-      this.str = str;
-      // Replace the param with the filtered version
-      self.updateParam(param, str);
+    req.filter = function(param) {
+      var self = this;
+      var filter = new Filter();
+      filter.modify = function(str) {
+        this.str = str;
+        // Replace the param with the filtered version
+        self.updateParam(param, str);
+      };
+      return filter.sanitize(this.param(param));
     };
-    return filter.sanitize(this.param(param));
-  };
 
-  // Create some aliases - might help with code readability
-  req.sanitize = req.filter;
-  req.assert = req.check;
-  req.validate = req.check;
+    // Create some aliases - might help with code readability
+    req.sanitize = req.filter;
+    req.assert = req.check;
+    req.validate = req.check;
 
-  return next();
-};
+    return next();
+  };
+}
 module.exports = expressValidator;
 module.exports.Validator = Validator;
 module.exports.Filter = Filter;

package.json

@@ -10,3 +10,3 @@ {
   ],
-  "version": "0.4.1",
+  "version": "0.5.0",
   "homepage": "https://github.com/ctavan/express-validator",
@@ -13,0 +13,0 @@ "repository": {

README.md

@@ -26,3 +26,3 @@ # express-validator
 app.use(express.bodyParser());
-app.use(expressValidator);
+app.use(expressValidator([options]));
 
@@ -73,2 +73,28 @@ app.post('/:urlparam', function(req, res) {
 
+### Middleware Options
+####`errorFormatter`
+_function(param,msg,value)_
+
+The `errorFormatter` option can be used to specify a function that can be used to formate the objects that populate the error arrow that is returned in `req.validationErrors()`. It should return an `Object` that has `param`, `msg`, and `value` keys defined.
+
+```javascript
+// In this example, the formParam value is going to get morphed into form body format useful for printing.
+app.use(expressValidator({
+  errorFormatter: function(param, msg, value) {
+      var namespace = param.split('.')
+      , root    = namespace.shift()
+      , formParam = root;
+
+    while(namespace.length) {
+      formParam += '[' + namespace.shift() + ']';
+    }
+    return {
+      param : formParam,
+      msg   : msg,
+      value : value
+    };
+  }
+}));
+```
+
 ### Validation errors
@@ -75,0 +101,0 @@

test/helpers/app.js

@@ -17,3 +17,3 @@ // Sample app
   self.app.use(express.bodyParser());
-  self.app.use(expressValidator);
+  self.app.use(expressValidator());
 
@@ -20,0 +20,0 @@ self.app.get(/\/test(\d+)/, self.validation);

No alert changes

Improved metrics

Total package byte prevSize

22106

increased by6.01%

Lines of code

456

increased by1.56%

Number of lines in readme file

268

increased by10.74%

No dependency changes