From: skodak <skodak>
Date: Wed, 4 Mar 2009 13:36:06 +0000 (+0000)
Subject: MDL-16852 fixed left join trouble; merged from MOODLE_19_STABLE
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=0a94af2927467e35273ea0ca5cb8a82765c66f1b;p=moodle.git

MDL-16852 fixed left join trouble; merged from MOODLE_19_STABLE
---

diff --git a/course/report/participation/index.php b/course/report/participation/index.php
index bde37aa05a..56ba3d3be7 100644
--- a/course/report/participation/index.php
+++ b/course/report/participation/index.php
@@ -188,11 +188,8 @@
         $sql = "SELECT ra.userid, u.firstname, u.lastname, u.idnumber, COUNT(l.action) AS count
                   FROM {role_assignments} ra
                        JOIN {user} u           ON u.id = ra.userid
-                       LEFT JOIN {log} l ON l.userid = ra.userid
-                 WHERE ra.contextid $relatedcontexts AND ra.roleid = :roleid AND
-                       (l.id IS NULL OR
-                          (l.cmid = :instanceid AND l.time > :timefrom AND $actionsql)
-                       )";
+                       LEFT JOIN {log} l ON (l.userid = ra.userid AND l.cmid = :instanceid AND l.time > :timefrom AND $actionsql)
+                 WHERE ra.contextid $relatedcontexts AND ra.roleid = :roleid";
         $params['roleid'] = $roleid;
         $params['instanceid'] = $instanceid;
         $params['timefrom'] = $timefrom;
@@ -210,11 +207,8 @@
         $countsql = "SELECT COUNT(DISTINCT(ra.userid))
                        FROM {role_assignments} ra
                             JOIN {user} u           ON u.id = ra.userid
-                            LEFT OUTER JOIN {log} l ON l.userid = ra.userid
-                      WHERE ra.contextid $relatedcontexts AND ra.roleid = :roleid AND
-                            (l.id IS NULL OR
-                               (l.cmid = :instanceid AND l.time > :timefrom AND $actionsql)
-                            )";
+                            LEFT OUTER JOIN {log} l ON (l.userid = ra.userid AND l.cmid = :instanceid AND l.time > :timefrom AND $actionsql)
+                      WHERE ra.contextid $relatedcontexts AND ra.roleid = :roleid";
 
         $totalcount = $DB->count_records_sql($countsql, $params);