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

2008-11-24 04:55:57 +00:00
parent 840b4f0b67
commit abfc0304e1
1 changed files with 2 additions and 2 deletions
@@ -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