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

Add isNumeric constraint on Venmo phone validation

Browse Source
This commit is contained in:
Sahat Yalkabov
2014-02-11 07:50:47 -05:00
parent d14732e0df
commit f0b5ad7bf0
1 changed files with 5 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
controllers/api.js
View File

@ -443,6 +443,7 @@ exports.postVenmo = function(req, res, next) {
}
var token = _.findWhere(req.user.tokens, { kind: 'venmo' });
var formData = {
access_token: token.accessToken,
note: req.body.note,
@ -451,21 +452,21 @@ exports.postVenmo = function(req, res, next) {
if (validator.isEmail(req.body.user)) {
formData.email = req.body.user;
} else if (validator.isLength(req.body.user, 10, 11)) {
} else if (validator.isNumberic(req.body.user) &&
validator.isLength(req.body.user, 10, 11)) {
formData.phone = req.body.user;
} else {
formData.user_id = req.body.user;
}
// Send money
request.post('https://api.venmo.com/v1/payments', { form: formData }, function(err, request, body) {
request.post('https://sandbox-api.venmo.com/v1/payments', { form: formData }, function(err, request, body) {
if (err) return next(err);
if (request.statusCode !== 200) {
req.flash('errors', { msg: JSON.parse(body).error.message });
return res.redirect('/api/venmo');
}
req.flash('success', 'Venmo money transfer complete');
req.flash('success', { msg: 'Venmo money transfer complete' });
res.redirect('/api/venmo');
});
};