From: jerome mouneyrac <jerome@moodle.com>
Date: Wed, 13 Jan 2010 10:51:15 +0000 (+0000)
Subject: webservice MDL-20805 fix sql request so return only unique ID
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=f692098303f742d31b8ff362048a4c4aff7fb77d;p=moodle.git

webservice MDL-20805 fix sql request so return only unique ID
---

diff --git a/admin/webservice/forms.php b/admin/webservice/forms.php
index 9f347eb5ed..e763b3e07d 100644
--- a/admin/webservice/forms.php
+++ b/admin/webservice/forms.php
@@ -142,17 +142,16 @@ class web_service_token_form extends moodleform {
 
         if (empty($data->nouserselection)) {
             //user searchable selector
-            $sql = "SELECT user.id, user.firstname, user.lastname, rassign.roleid
+            $sql = "SELECT user.id, user.firstname, user.lastname
             FROM {user} user
-            LEFT JOIN {role_assignments} rassign
-            ON user.id = rassign.userid
+            WHERE user.id NOT IN (  SELECT adminuser.id
+                                    FROM {user} adminuser, {role_assignments} rassign
+                                    WHERE rassign.roleid = 1 AND rassign.userid = adminuser.id)
             ORDER BY user.lastname";
             $users = $DB->get_records_sql($sql,array());
             $options = array();
             foreach ($users as $userid => $user) {
-                if ($user->roleid != 1) {
                     $options[$userid] = $user->firstname. " " . $user->lastname;
-                }
             }
             $mform->addElement('searchableselector', 'user', get_string('user'),$options);
             $mform->addRule('user', get_string('required'), 'required', null, 'client');