[ARVADOS] updated: cd5e5afa6a37320a2d8f3f01e725bc7b18f800a0

git at public.curoverse.com git at public.curoverse.com
Fri Apr 4 15:50:18 EDT 2014


Summary of changes:
 .../app/views/users/_setup_popup.html.erb          |   13 +++++++++++--
 1 files changed, 11 insertions(+), 2 deletions(-)

       via  cd5e5afa6a37320a2d8f3f01e725bc7b18f800a0 (commit)
      from  ac5cd5913a208c463206d2fa6cf62e42148682a4 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit cd5e5afa6a37320a2d8f3f01e725bc7b18f800a0
Author: radhika chippada <radhika at radhika.curoverse>
Date:   Fri Apr 4 15:45:53 2014 -0400

    Add email format validation to the input text field in the setup modal.

diff --git a/apps/workbench/app/views/users/_setup_popup.html.erb b/apps/workbench/app/views/users/_setup_popup.html.erb
index aed57da..90cab2c 100644
--- a/apps/workbench/app/views/users/_setup_popup.html.erb
+++ b/apps/workbench/app/views/users/_setup_popup.html.erb
@@ -75,7 +75,8 @@
   var email_disabled = document.forms["setup_form"]["email"].disabled;
   var email_value = document.forms["setup_form"]["email"].value;
   var prefix_value = document.forms["setup_form"]["openid_prefix"].value;
-  if ((email_disabled == false) && (email_value == null || email_value == "" || prefix_value == null || prefix_value == "")) {
+  if ((email_disabled == false) && (email_value == null || email_value == "" || 
+        prefix_value == null || prefix_value == "")) {
     $register.attr('disabled', true);
   }
 
@@ -85,7 +86,15 @@
     var email_disabled = document.forms["setup_form"]["email"].disabled;
     var email_value = document.forms["setup_form"]["email"].value;
     var prefix_value = document.forms["setup_form"]["openid_prefix"].value;
-    if ((email_disabled == false) && (email_value == null || email_value == "" || prefix_value == null || prefix_value == "")) {
+
+    var emailRegExp = /^([\w-\.]+@([\w-]+\.)+[\w-]{2,4})?$/;
+    var validEmail = false;
+    if (emailRegExp.test(email_value )) {
+      validEmail = true;
+    }
+
+    if ((email_disabled == false) && (!validEmail || email_value == null || 
+            email_value == "" || prefix_value == null || prefix_value == "")){
       trigger = true;
     }
 

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list