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

---
 .../exams/baseconfig/getconfig.inc.php             |  8 ++++++
 modules-available/exams/inc/exams.inc.php          | 31 +++++++++++++---------
 2 files changed, 27 insertions(+), 12 deletions(-)
 create mode 100644 modules-available/exams/baseconfig/getconfig.inc.php

(limited to 'modules-available/exams')

diff --git a/modules-available/exams/baseconfig/getconfig.inc.php b/modules-available/exams/baseconfig/getconfig.inc.php
new file mode 100644
index 00000000..d26a20a9
--- /dev/null
+++ b/modules-available/exams/baseconfig/getconfig.inc.php
@@ -0,0 +1,8 @@
+<?php
+
+if (isset($configVars["SLX_LOCATIONS"])) {
+	$locationIds = explode(' ', $configVars["SLX_LOCATIONS"]);
+	if (Exams::isInExamMode($locationIds)) {
+		$configVars['SLX_EXAM'] = 'yes';
+	}
+}
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