From 246b51a793770f7dd3f7b0625e12c7a188f38ddd Mon Sep 17 00:00:00 2001 From: Sahat Yalkabov Date: Sat, 1 Feb 2014 03:53:50 -0500 Subject: [PATCH] Removed redundant express-validator validations, notEmpty() --- controllers/api.js | 3 +++ controllers/user.js | 4 ---- 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a/controllers/api.js b/controllers/api.js index 71999dc18f6..a9db5a60453 100644 --- a/controllers/api.js +++ b/controllers/api.js @@ -260,6 +260,7 @@ exports.getTwitter = function(req, res, next) { * GET /api/paypal * PayPal SDK example */ + exports.getPayPal = function(req, res, next) { paypal.configure(secrets.paypal); var payment_details = { @@ -300,6 +301,7 @@ exports.getPayPal = function(req, res, next) { * GET /api/paypal/success * PayPal SDK example */ + exports.getPayPalSuccess = function(req, res, next) { var payment_id = req.session.payment_id; var payment_details = { 'payer_id': req.query.PayerID }; @@ -322,6 +324,7 @@ exports.getPayPalSuccess = function(req, res, next) { * GET /api/paypal/cancel * PayPal SDK example */ + exports.getPayPalCancel = function(req, res, next) { req.session.payment_id = null; res.render('api/paypal', { diff --git a/controllers/user.js b/controllers/user.js index 5dde93a8f06..525712e4b94 100644 --- a/controllers/user.js +++ b/controllers/user.js @@ -23,7 +23,6 @@ exports.getLogin = function(req, res) { */ exports.postLogin = function(req, res, next) { - req.assert('email', 'Email cannot be blank').notEmpty(); req.assert('email', 'Email is not valid').isEmail(); req.assert('password', 'Password cannot be blank').notEmpty(); @@ -69,9 +68,7 @@ exports.getSignup = function(req, res) { */ exports.postSignup = function(req, res, next) { - req.assert('email', 'Email cannot be blank').notEmpty(); req.assert('email', 'Email is not valid').isEmail(); - req.assert('password', 'Password cannot be blank').notEmpty(); req.assert('password', 'Password must be at least 4 characters long').len(4); req.assert('confirmPassword', 'Passwords do not match').equals(req.body.password); @@ -141,7 +138,6 @@ exports.postUpdateProfile = function(req, res, next) { */ exports.postUpdatePassword = function(req, res, next) { - req.assert('password', 'Password cannot be blank').notEmpty(); req.assert('password', 'Password must be at least 4 characters long').len(4); req.assert('confirmPassword', 'Passwords do not match').equals(req.body.password);