From 46b3d8983aecd4908bb0b00ca9f1ccf73c4d5ff8 Mon Sep 17 00:00:00 2001
From: Simon Rettberg
Date: Thu, 23 Jun 2016 17:05:06 +0200
Subject: [exams] Add baseconfig API hook

---
 modules-available/exams/inc/exams.inc.php | 31 +++++++++++++++++++------------
 1 file changed, 19 insertions(+), 12 deletions(-)

(limited to 'modules-available/exams/inc')

diff --git a/modules-available/exams/inc/exams.inc.php b/modules-available/exams/inc/exams.inc.php
index 9e5833ba..c01d4bad 100644
--- a/modules-available/exams/inc/exams.inc.php
+++ b/modules-available/exams/inc/exams.inc.php
@@ -1,17 +1,24 @@
 <?php
 
-class Exams {
+class Exams
+{
 
+	/**
+	 * @param int[] of location ids. must bot be an associative array.
+	 * @return: bool true iff for any of the given location ids an exam is scheduled.
+	 **/
+	public static function isInExamMode($locationIds)
+	{
+		if (!is_array($locationIds)) {
+			$locationIds = array($locationIds);
+		} elseif (empty($locationIds)) {
+			return false;
+		}
+		$l = str_repeat(',?', count($locationIds) - 1);
+		$res = Database::queryFirst("SELECT examid FROM exams"
+			. " INNER JOIN exams_x_location USING (examid)"
+			. " WHERE UNIX_TIMESTAMP() BETWEEN starttime AND endtime AND locationid IN (?$l) LIMIT 1", $locationIds);
+		return $res !== false;
+	}
 
-    /**
-     * @param: array of location ids
-     * @return: true iff for any of the given location ids an exam is scheduled
-     **/
-    public static function isInExamMode($locationIds) {
-        // TODO: Better use prepared statement
-        $l = '(' . implode(', ', $locationIds) . ')';
-        $res = Database::queryFirst("SELECT (COUNT(examid) > 0) as examMode FROM exams WHERE starttime < NOW() AND endtime > NOW() AND locationid IN $l", []);
-
-        return $res['examMode'];
-    }
 }
-- 
cgit v1.2.3-55-g7522