Changeset 10526 for main/waeup.cas/trunk/waeup/cas/authenticators.py

Timestamp:

22 Aug 2013, 21:21:52 (11 years ago)

Author:

Henrik Bettermann

Message:

Call core_user_get_users first and bring order into tests.

File:

• main/waeup.cas/trunk/waeup/cas/authenticators.py (modified) (3 diffs)

main/waeup.cas/trunk/waeup/cas/authenticators.py

@@ -190 +190 @@
     name = 'kofa_moodle1'
 
-    def _create_user(self, username, userdata, moodle):
-        try:
-            # Usernames in Moodle must not contain uppercase
-            # letters even if extendedusernamechars is set True.
-            # Usernames in Moodle are case insensitive.
-            # In other words, Moodle changes usernames from
-            # uppercase to lowercase when you login.
-            result = moodle.core_user_create_users([
-                {'username':username.lower(),
-                 'password':'dummy',
-                 'firstname':userdata['firstname'],
-                 'lastname':userdata['lastname'],
-                 'email':userdata['email']}])
-        except xmlrpclib.Fault:
-            faultstring = sys.exc_info()[1].faultString
-            if 'Email address already exists' in faultstring:
-                return (False,
-                    'Another Moodle user is using the same email address.'
-                    ' Email addresses can\'t be used twice in Moodle.')
-            # If user exists just pass. Unfortunately the error message
-            # given by Moodle is not always the same.
-            if not ('Username already exists' in faultstring or
-                    'Unknown error' in faultstring):
-                return (False, faultstring)
+    def _update_user(self, username, userdata, moodle):
         try:
             result = moodle.core_user_get_users([
@@ -220 +197 @@
             faultstring = sys.exc_info()[1].faultString
             return (False, faultstring)
-        user_id = result['users'][0]['id']
+        if result['users']:
+            user_id = result['users'][0]['id']
+        else:
+            try:
+                # Usernames in Moodle must not contain uppercase
+                # letters even if extendedusernamechars is set True.
+                # Usernames in Moodle are case insensitive.
+                # In other words, Moodle changes the username from
+                # uppercase to lowercase when you login.
+                result = moodle.core_user_create_users([
+                    {'username':username.lower(),
+                     'password':'dummy',
+                     'firstname':userdata['firstname'],
+                     'lastname':userdata['lastname'],
+                     'email':userdata['email']}])
+            except xmlrpclib.Fault:
+                faultstring = sys.exc_info()[1].faultString
+                if 'Email address already exists' in faultstring:
+                    return (False,
+                        'Another Moodle user is using the same email address.'
+                        ' Email addresses can\'t be used twice in Moodle.')
+                return (False, faultstring)
+            user_id = result[0]['id']
         # Due to a lack of Moodle (Moodle requires an LDAP
         # connection) the authentication method can't
@@ -255 +254 @@
             if principal['type'] == 'student':
                 userdata = proxy.get_student_moodle_data(kofa_username)
-                return self._create_user(username, userdata, moodle)
+                return self._update_user(username, userdata, moodle)
             if principal['type'] == 'applicant':
                 userdata = proxy.get_applicant_moodle_data(kofa_username)
-                return self._create_user(username, userdata, moodle)
+                return self._update_user(username, userdata, moodle)
             return (False, 'User not eligible')
         return (False, 'Invalid username or password')