From f0b5ad7bf0ea59141ce9152a5187bd791d11f980 Mon Sep 17 00:00:00 2001
From: Sahat Yalkabov <sakhat@gmail.com>
Date: Tue, 11 Feb 2014 07:50:47 -0500
Subject: [PATCH] Add isNumeric constraint on Venmo phone validation

---
 controllers/api.js | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/controllers/api.js b/controllers/api.js
index fadd4c1f32..591a884e89 100644
--- a/controllers/api.js
+++ b/controllers/api.js
@@ -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');
   });
 };