From c1596896f149fbcebe28718b87c714f8d592dd3e Mon Sep 17 00:00:00 2001
From: Dan Marsden <dan@danmarsden.com>
Date: Mon, 6 Jan 2014 08:36:18 +1300
Subject: [PATCH] correct usage of group check Fixes #26

---
 locallib.php | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/locallib.php b/locallib.php
index 2977b34..0982f44 100644
--- a/locallib.php
+++ b/locallib.php
@@ -1070,7 +1070,7 @@ class attendance {
         global $DB;
 
         if (!array_key_exists($userid, $this->userstatusesstat)) {
-            if (!empty($this->get_group_mode())) {
+            if ($this->get_group_mode()) {
                 $qry = "SELECT al.statusid, count(al.statusid) AS stcnt
                       FROM {attendance_log} al
                       JOIN {attendance_sessions} ats ON al.sessionid = ats.id
@@ -1142,7 +1142,7 @@ class attendance {
         } else {
             $where = "ats.attendanceid = :aid AND ats.sessdate >= :csdate";
         }
-        if (!empty($this->get_group_mode())) {
+        if ($this->get_group_mode()) {
             $sql = "SELECT ats.id, ats.sessdate, ats.groupid, al.statusid, al.remarks
                   FROM {attendance_sessions} ats
                   JOIN {attendance_log} al ON ats.id = al.sessionid AND al.studentid = :uid
@@ -1192,7 +1192,7 @@ class attendance {
         // If the array's index is a number it will not merge entries.
         // It would be better as a UNION query butunfortunatly MS SQL does not seem to support doing a DISTINCT on a the description field.
         $id = $DB->sql_concat(':value', 'ats.id');
-        if (!empty($this->get_group_mode())) {
+        if ($this->get_group_mode()) {
             $sql = "SELECT $id, ats.id, ats.groupid, ats.sessdate, ats.duration, ats.description, al.statusid, al.remarks
                   FROM {attendance_sessions} ats
             RIGHT JOIN {attendance_log} al