From: moodler <moodler>
Date: Fri, 30 Mar 2007 16:01:44 +0000 (+0000)
Subject: Merged fix for MDL-8093 from 1.8
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=b373a751d24817a1dfa50d51d3faf29e2e8e90d5;p=moodle.git

Merged fix for MDL-8093 from 1.8
---

diff --git a/user/index.php b/user/index.php
index 853a22a49d..eaa014de3c 100644
--- a/user/index.php
+++ b/user/index.php
@@ -49,15 +49,25 @@
     }
 
     $rolenames = array();
+    $avoidroles = array();
 
     if ($roles = get_roles_used_in_context($context, true)) {
-        // We should exclude "admin" users (those with "doanything" at site level) because 
+        // We should ONLY allow roles with moodle/course:view because otherwise we get little niggly issues 
+        // like MDL-8093
+        // We should further exclude "admin" users (those with "doanything" at site level) because 
         // Otherwise they appear in every participant list
 
+        $canviewroles    = get_roles_with_capability('moodle/course:view', CAP_ALLOW, $context);
         $doanythingroles = get_roles_with_capability('moodle/site:doanything', CAP_ALLOW, $sitecontext);
 
         foreach ($roles as $role) {
+            if (!isset($canviewroles[$role->id])) {   // Avoid this role (eg course creator)
+                $avoidroles[] = $role->id;
+                unset($roles[$role->id]);
+                continue;
+            }
             if (isset($doanythingroles[$role->id])) {   // Avoid this role (ie admin)
+                $avoidroles[] = $role->id;
                 unset($roles[$role->id]);
                 continue;
             }
@@ -138,6 +148,7 @@
     }
 
 
+
 /// setting up tags
     if ($course->id == SITEID) {
         $filtertype = 'site';
@@ -327,14 +338,10 @@
     
     $hiddensql = has_capability('moodle/role:viewhiddenassigns', $context)? '':' AND r.hidden = 0 ';   
 
-    // excluse users with these admin role assignments
-    if ($doanythingroles) {
+    // exclude users with roles we are avoiding
+    if ($avoidroles) {
         $adminroles = 'AND r.roleid NOT IN (';
- 
-        foreach ($doanythingroles as $aroleid=>$role) {
-            $adminroles .= "$aroleid,";
-        }
-        $adminroles = rtrim($adminroles,",");
+        $adminroles .= implode(',', $avoidroles);
         $adminroles .= ')';
     } else {
         $adminroles = '';