Conver usernames to lowercase for internal use. This prevent duplicate users

when using external authentication and posgresql.

diff --git a/lib/moodlelib.php b/lib/moodlelib.php
index 6e683b18fcdaff848f1b21163e806457110f0f42..bfc626f1cb252e8fed90c135f8519386bd8a0277 100644 (file)
--- a/lib/moodlelib.php
+++ b/lib/moodlelib.php
@@ -489,7 +489,8 @@ function get_moodle_cookie() {
 function create_user_record($username, $password) {
 /// Creates a bare-bones user record 
     global $REMOTE_ADDR, $CFG;
-    
+    //just in case check text case
+    $username = trim(moodle_strtolower($username));
     if (function_exists(auth_get_userinfo)) {
         if ($newinfo = auth_get_userinfo($username)) {
             foreach ($newinfo as $key => $value){

diff --git a/login/index.php b/login/index.php
index c64c53b5fbcf7975b17fba31478b64eddd81af16..1718696a1e3451f76f71d11242298aaa8d7b83b6 100644 (file)
--- a/login/index.php
+++ b/login/index.php
@@ -21,8 +21,8 @@
 
 
     if ($frm = data_submitted()) {
+        $frm->username = trim(moodle_strtolower($frm->username));
         $user = authenticate_user_login($frm->username, $frm->password);
-
         update_login_count();
 
         if ($user) {

diff --git a/login/signup.php b/login/signup.php
index 8bab6c76c59e482ccdef79393bf2d50ccca8cbb8..16ecc068c0358177e769a3fccf4e508b9b0a5f89 100644 (file)
--- a/login/signup.php
+++ b/login/signup.php
@@ -6,7 +6,7 @@
 
     if ($user = data_submitted()) {
         validate_form($user, $err);
-
+        $user->username= trim(moodle_strtolower($user->username));
         if (count((array)$err) == 0) {
             $plainpass = $user->password;
             $user->password = md5($user->password);
@@ -74,16 +74,17 @@
 
 function validate_form($user, &$err) {
     global $CFG;
-    if (empty($user->username))
+    if (empty($user->username)){
         $err->username = get_string("missingusername");
-
-    else if (record_exists("user", "username", $user->username))
-        $err->username = get_string("usernameexists");
-
-    else {
-        $string = eregi_replace("[^([:alnum:])]", "", $user->username);
-        if (strcmp($user->username, $string)) 
+    }else{
+        $user->username = trim(moodle_strtolower($user->username));
+        if (record_exists("user", "username", $user->username)){
+            $err->username = get_string("usernameexists");
+        }else {
+            $string = eregi_replace("[^([:alnum:])]", "", $user->username);
+            if (strcmp($user->username, $string)) 
             $err->username = get_string("alphanumerical");
+        }
     }
 
     if (isset($CFG->auth_user_create) and $CFG->auth_user_create==1 and function_exists('auth_user_exists') ){

diff --git a/user/edit.php b/user/edit.php
index 71371fb7f2719a191eb56b0e5976ee850ccf1063..53efbecf9899a68f2c562bb5644a7ab0cbf945ec 100644 (file)
--- a/user/edit.php
+++ b/user/edit.php
@@ -45,7 +45,7 @@
        if ($usernew = data_submitted()) {
         $usernew->firstname = strip_tags($usernew->firstname);
         $usernew->lastname  = strip_tags($usernew->lastname);
-
+        $usernew->username = trim(moodle_strtolower($usernew->username));
         if (empty($_FILES['imagefile'])) {
             $_FILES['imagefile'] = NULL;    // To avoid using uninitialised variable later
         }