From b1cff118c7626d44156bad7f8435346086b1b79e Mon Sep 17 00:00:00 2001
From: martinlanghoff <martinlanghoff>
Date: Wed, 19 Sep 2007 07:54:23 +0000
Subject: [PATCH] datalib:get_courses_wmanagers() handle empty
$CFG->coursemanager more gracefully
Having no roles set as coursemanager is a valid setting.
get_courses_wmanagers() should not produce invalid SQL on it...
actually, it should not even try to get the course managers.
---
lib/datalib.php | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/lib/datalib.php b/lib/datalib.php
index 4601662ee8..87da06debd 100644
--- a/lib/datalib.php
+++ b/lib/datalib.php
@@ -698,6 +698,11 @@ function get_courses_wmanagers($categoryid=0, $sort="c.sortorder ASC", $fields=a
return array(); // no courses!
}
+ $CFG->coursemanager = trim($CFG->coursemanager);
+ if (empty($CFG->coursemanager)) {
+ return $courses;
+ }
+
$managerroles = split(',', $CFG->coursemanager);
$catctxids = '';
if (count($managerroles)) {
--
2.39.5