Ask for email address again to help avoid typos

diff --git a/lang/en/moodle.php b/lang/en/moodle.php
index d9ee3849e030d35d047e3323ecfcfb7fc9c4164c..65472bb9b4a54541fe2fa6244e8041135174c07d 100644 (file)
--- a/lang/en/moodle.php
+++ b/lang/en/moodle.php
@@ -238,6 +238,7 @@ $string['editthisactivity'] = 'Edit this activity';
 $string['editthiscategory'] = 'Edit this category';
 $string['edituser'] = 'Edit user accounts';
 $string['email'] = 'Email address';
+$string['emailagain'] = 'Email (again)';
 $string['emailconfirm'] = 'Confirm your account';
 $string['emailconfirmation'] = 'Hi $a->firstname,
 
@@ -620,6 +621,7 @@ $string['potentialteachers'] = 'Potential teachers';
 $string['preferredlanguage'] = 'Preferred language';
 $string['preview'] = 'Preview';
 $string['previeworchoose'] = 'Preview or choose a theme';
+$string['previous'] = 'Previous';
 $string['publicdirectory'] = 'Public directory';
 $string['publicdirectory0'] = 'Please do not publish this site';
 $string['publicdirectory1'] = 'Publish the site name only';

diff --git a/login/signup.php b/login/signup.php
index 3721c8b3ddcbfb60a4351200ea896afcf67cd43c..7058404284ebb74c48507d37a1e83208b7271886 100644 (file)
--- a/login/signup.php
+++ b/login/signup.php
@@ -94,31 +94,45 @@ function validate_form($user, &$err) {
     }         
 
 
-    if (empty($user->password)) 
+    if (empty($user->password)) {
         $err->password = get_string("missingpassword");
+    }
 
-    if (empty($user->firstname))
+    if (empty($user->firstname)) {
         $err->firstname = get_string("missingfirstname");
+    }
         
-    if (empty($user->lastname))
+    if (empty($user->lastname)) {
         $err->lastname = get_string("missinglastname");
+    }
         
 
-    if (empty($user->email))
+    if (empty($user->email)) {
         $err->email = get_string("missingemail");
         
-    else if (! validate_email($user->email))
+    } else if (! validate_email($user->email)) {
         $err->email = get_string("invalidemail");
     
-    else if (record_exists("user", "email", $user->email)) 
+    } else if (record_exists("user", "email", $user->email)) {
         $err->email = get_string("emailexists")." <A HREF=forgot_password.php>".get_string("newpassword")."?</A>";
+    }
+    
 
+    if (empty($user->email2)) {
+        $err->email2 = get_string("missingemail");
 
-    if (empty($user->city)) 
+    } else if ($user->email2 != $user->email) {
+        $err->email2 = get_string("invalidemail");
+    }
+
+
+    if (empty($user->city)) {
         $err->city = get_string("missingcity");
+    }
 
-    if (empty($user->country)) 
+    if (empty($user->country)) {
         $err->country = get_string("missingcountry");
+    }
 
     return;
 }

diff --git a/login/signup_form.html b/login/signup_form.html
index 53e7f446ef51be1b3d6c74ca1a77c5550fdddb80..d2d446f6e25286e1e23d61563d58457546e1b2f7 100644 (file)
--- a/login/signup_form.html
+++ b/login/signup_form.html
@@ -30,13 +30,13 @@
        <td colspan="2"><p><b><?php print_string("createuserandpass") ?>:</b></p></td>\r
 </tr>\r
 <tr valign="top">\r
-       <td><p><?php print_string("username") ?>:</p></td>\r
+       <td align=right><p><?php print_string("username") ?>:</p></td>\r
        <td><input type="text" name="username" size="12" value="<?php p($user->username) ?>" />\r
        <?php if (!empty($err->username)) { formerr($err->username); } ?>\r
        </td>\r
 </tr>\r
 <tr valign="top">\r
-       <td><p><?php print_string("password") ?>:</p></td>\r
+       <td align=right><p><?php print_string("password") ?>:</p></td>\r
        <td><input type="password" name="password" size="12" value="<?php p($user->password) ?>" />\r
        <?php if (!empty($err->password)) { formerr($err->password); } ?>\r
        </td>\r
@@ -48,31 +48,37 @@
               </td>\r
             </tr>\r
 <tr valign="top">\r
-       <td><p><?php print_string("email") ?>:</p></td>\r
+       <td align=right><p><?php print_string("email") ?>:</p></td>\r
        <td><input type="text" name="email" size="25" value="<?php p($user->email) ?>" />\r
        <?php if (!empty($err->email)) { formerr($err->email); } ?>\r
        </td>\r
 </tr>\r
 <tr valign="top">\r
-       <td><p><?php print_string("firstname") ?>:</p></td>\r
+       <td align=right><p><?php print_string("emailagain") ?>:</p></td>\r
+       <td><input type="text" name="email2" size="25" value="<?php p($user->email2) ?>" />\r
+       <?php if (!empty($err->email2)) { formerr($err->email2); } ?>\r
+       </td>\r
+</tr>\r
+<tr valign="top">\r
+       <td align=right><p><?php print_string("firstname") ?>:</p></td>\r
        <td><input type="text" name="firstname" size="25" value="<?php p($user->firstname) ?>" />\r
        <?php if (!empty($err->firstname)) { formerr($err->firstname);} ?>\r
        </td>\r
 </tr>\r
 <tr valign="top">\r
-       <td><p><?php print_string("lastname") ?>:</p></td>\r
+       <td align=right><p><?php print_string("lastname") ?>:</p></td>\r
        <td><input type="text" name="lastname" size="25" value="<?php p($user->lastname) ?>" />\r
        <?php if (!empty($err->lastname)) { formerr($err->lastname);} ?>\r
        </td>\r
 </tr>\r
 <tr valign="top">\r
-       <td><p><?php print_string("city") ?>:</p></td>\r
+       <td align=right><p><?php print_string("city") ?>:</p></td>\r
        <td><input type="text" name="city" size="25" value="<?php p($user->city) ?>" />\r
        <?php if (!empty($err->city)) { formerr($err->city);} ?>\r
        </td>\r
 </tr>\r
 <tr valign="top">\r
-       <td><p><?php print_string("country") ?>:</p></td>\r
+       <td align=right><p><?php print_string("country") ?>:</p></td>\r
        <td><?php choose_from_menu (get_list_of_countries(), "country", $user->country, get_string("selectacountry"), "", "") ?>\r
        <?php if (!empty($err->country)) { formerr($err->country);} ?>\r
        </td>\r