gaspersic/freeCodeCamp
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Removed redundant req.assert for empty email

Browse Source 
Swapped code order of a few things in app.js to make it more consistent in terms of coding style
This commit is contained in:
Sahat Yalkabov
2014-02-02 05:38:38 -05:00
parent 9d77b3030d
commit 1db2daa01a
2 changed files with 7 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
app.js
View File

@ -11,12 +11,6 @@ var mongoose = require('mongoose');
var passport = require('passport'); var passport = require('passport');
var expressValidator = require('express-validator'); var expressValidator = require('express-validator');
/**
* Create Express server.
*/
var app = express();
/** /**
* Load controllers. * Load controllers.
*/ */
@ -33,6 +27,13 @@ var contactController = require('./controllers/contact');
var secrets = require('./config/secrets'); var secrets = require('./config/secrets');
var passportConf = require('./config/passport'); var passportConf = require('./config/passport');
/**
* Create Express server.
*/
var app = express();
/** /**
* Mongoose configuration. * Mongoose configuration.
*/ */

1
controllers/contact.js
View File

@ -22,7 +22,6 @@ exports.getContact = function(req, res) {
exports.postContact = function(req, res) { exports.postContact = function(req, res) {
req.assert('name', 'Name cannot be blank').notEmpty(); req.assert('name', 'Name cannot be blank').notEmpty();
req.assert('email', 'Email cannot be blank').notEmpty();
req.assert('email', 'Email is not valid').isEmail(); req.assert('email', 'Email is not valid').isEmail();
req.assert('message', 'Message cannot be blank').notEmpty(); req.assert('message', 'Message cannot be blank').notEmpty();