role assign: Don't get duplicate records in potential_assignees_below_course user...

diff --git a/admin/roles/lib.php b/admin/roles/lib.php
index 647c508d25ac5381dc129395a879f46f7cbf1a8e..517de9886af070554e3ea6964389ab4bd48c72dc 100644 (file)
--- a/admin/roles/lib.php
+++ b/admin/roles/lib.php
@@ -1006,8 +1006,8 @@ class potential_assignees_below_course extends role_assign_user_selector_base {
         }
         $roleids =  '('.implode(',', $validroleids).')';
 
-        $fields      = 'SELECT ' . $this->required_fields_sql('u');
-        $countfields = 'SELECT COUNT(1)';
+        $fields      = 'SELECT DISTINCT ' . $this->required_fields_sql('u');
+        $countfields = 'SELECT COUNT(DISTINCT u.id)';
 
         $sql   = " FROM {user} u
                    JOIN {role_assignments} ra ON ra.userid = u.id