From fffb5d240ae82cec450e3637b24e5e5ef5bc91ad Mon Sep 17 00:00:00 2001
From: moodler <moodler>
Date: Thu, 6 Mar 2008 02:15:00 +0000
Subject: [PATCH] Merged MDL-13810  Fixed check for ul.timeaccess

---
 lib/accesslib.php | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/lib/accesslib.php b/lib/accesslib.php
index 8b290741b952f..04a4123599305 100755
--- a/lib/accesslib.php
+++ b/lib/accesslib.php
@@ -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 = '';
     }
 
     //