Merged MDL-13810 Fixed check for ul.timeaccess

2008-03-06 02:15:00 +00:00
parent 68be1fe1e7
commit fffb5d240a
1 changed files with 3 additions and 3 deletions
@@ -4525,11 +4525,11 @@ function get_users_by_capability($context, $capability, $fields='', $sort='',
    $sortby = $sort ? " ORDER BY $sort " : '';

    // User lastaccess JOIN
-    if ($sort == 'ul.timeaccess') {
+    if (strpos($sort, 'ul.timeaccess') === FALSE) {  // user_lastaccess is not required MDL-13810
+        $uljoin = '';
+    } else {
        $uljoin = "LEFT OUTER JOIN {$CFG->prefix}user_lastaccess ul 
                         ON (ul.userid = u.id AND ul.courseid = {$context->instanceid})";
-    } else {
-        $uljoin = '';
    }

    //