No need to specify DISTINCT because u.* guarantees unique ids.

Also, DISTINCT clauses cannot be used with TEXT/BINARY columns
on MSSQL and Oracle.

diff --git a/lib/accesslib.php b/lib/accesslib.php
index fb533b37d3f4871f13511e733ee72ddb1b11b68f..1816143a0a2c89b80eeb4caaf950395e39e45d8e 100755 (executable)
--- a/lib/accesslib.php
+++ b/lib/accesslib.php
@@ -2003,7 +2003,7 @@ function get_overridable_roles ($context) {
  * @param $limitfrom - number of records to skip (offset)
  * @param $limitnum - number of records to fetch 
  */
-function get_users_by_capability($context, $capability, $fields='distinct u.*', $sort='', $limitfrom='', $limitnum='') {
+function get_users_by_capability($context, $capability, $fields='u.*', $sort='', $limitfrom='', $limitnum='') {
     
     global $CFG;