From 45d2376797e8eed1cbe46592e04c6f76729856fc Mon Sep 17 00:00:00 2001 From: pardyot-s Date: Wed, 8 Aug 2018 10:39:03 +0530 Subject: [PATCH 1/2] (fix): email validation --- lib/api/account-validation.js | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/lib/api/account-validation.js b/lib/api/account-validation.js index 7c2220cbd1f..f365be1ea33 100644 --- a/lib/api/account-validation.js +++ b/lib/api/account-validation.js @@ -50,13 +50,15 @@ const validationMethods = { check(email, Match.OptionalOrNull(String)); check(optional, Match.OptionalOrNull(Boolean)); - const processedEmail = email.trim(); + if(email) { + const processedEmail = email.trim(); - // Valid - if (optional === true && processedEmail.length === 0) { - return true; - } else if (processedEmail.indexOf("@") !== -1) { - return true; + // Valid + if (optional === true && processedEmail.length === 0) { + return true; + } else if (processedEmail.indexOf("@") !== -1) { + return true; + } } // Invalid From 5cc9c37df2b6e940f91971ea272b3ca7743e586c Mon Sep 17 00:00:00 2001 From: pardyot-s Date: Wed, 8 Aug 2018 11:10:41 +0530 Subject: [PATCH 2/2] lint fixed --- lib/api/account-validation.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/api/account-validation.js b/lib/api/account-validation.js index f365be1ea33..fb3701dc278 100644 --- a/lib/api/account-validation.js +++ b/lib/api/account-validation.js @@ -50,7 +50,7 @@ const validationMethods = { check(email, Match.OptionalOrNull(String)); check(optional, Match.OptionalOrNull(Boolean)); - if(email) { + if (email) { const processedEmail = email.trim(); // Valid