From: martinlanghoff <martinlanghoff>
Date: Wed, 19 Sep 2007 07:26:27 +0000 (+0000)
Subject: course/lib: print_course() bugfix when $rusers is empty
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=431cad0d62c3429a294a4588d9b0c79b3ed90521;p=moodle.git

course/lib: print_course() bugfix when $rusers is empty
---

diff --git a/course/lib.php b/course/lib.php
index c2ef2d25b0..b59b5811dc 100644
--- a/course/lib.php
+++ b/course/lib.php
@@ -1814,13 +1814,17 @@ function print_course($course) {
         $coursemanagerroles = split(',', $managerroles);
         $canseehidden = has_capability('moodle/role:viewhiddenassigns', $context);
         $rusers = get_role_users($coursemanagerroles, $context, 
-                                 true, '', 'r.sortorder ASC, u.lastname ASC', $canseehidden) ;
-        foreach ($rusers as $teacher) {
-            $fullname = fullname($teacher, has_capability('moodle/site:viewfullnames', $context)); 
-            $namesarray[] = format_string($teacher->rolename).': <a href="'.$CFG->wwwroot.'/user/view.php?id='.
-                $teacher->id.'&amp;course='.SITEID.'">'.$fullname.'</a>'; 
+                                 true, '', 'r.sortorder ASC, u.lastname ASC', $canseehidden);
+        if (is_array($rusers) && count($rusers)) {
+            $canviewfullnames = has_capability('moodle/site:viewfullnames', $context);
+            foreach ($rusers as $teacher) {
+                $fullname = fullname($teacher, $canviewfullnames); 
+                $namesarray[] = format_string($teacher->rolename) 
+                    . ': <a href="'.$CFG->wwwroot.'/user/view.php?id='.$teacher->id.'&amp;course='.SITEID.'">'
+                    . $fullname . '</a>'; 
+            }
         }
-        
+
         if (!empty($namesarray)) {
             echo "<ul class=\"teachers\">\n<li>";
             echo implode('</li><li>', $namesarray);