From a1f5b4ae033ca2f0eab1ccf6c248b9a79f1f94cf Mon Sep 17 00:00:00 2001
From: toyomoyo <toyomoyo>
Date: Mon, 23 Apr 2007 03:06:01 +0000
Subject: [PATCH] merged fix for MLD-7544, hidden users show up on participants
 block

---
 blocks/online_users/block_online_users.php | 24 ++++++++++++++++++++--
 1 file changed, 22 insertions(+), 2 deletions(-)

diff --git a/blocks/online_users/block_online_users.php b/blocks/online_users/block_online_users.php
index 67c74e8e580..aaaf81f0f5e 100644
--- a/blocks/online_users/block_online_users.php
+++ b/blocks/online_users/block_online_users.php
@@ -1,4 +1,4 @@
-<?PHP //$Id$
+<?php //$Id$
 
 /**
  * This block needs to be reworked.
@@ -74,9 +74,29 @@ class block_online_users extends block_base {
                       $timeselect
                       $groupselect
                 ORDER BY ul.timeaccess DESC";
-
+        
+        
+        
+        $pcontext = get_related_contexts_string($context);
+    
         if ($pusers = get_records_sql($SQL, 0, 50)) {   // We'll just take the most recent 50 maximum
             foreach ($pusers as $puser) {
+                
+                // if current user can't view hidden role assignment in this context and 
+                // user has a hidden role assigned at this context or any parent contexts,
+                // ignore this user
+                
+                $SQL = "SELECT id,id FROM {$CFG->prefix}role_assignments
+                        WHERE userid = $puser->id
+                        AND contextid $pcontext
+                        AND hidden = 1";
+                
+                if (!has_capability('moodle/role:viewhiddenassigns', $context) && record_exists_sql($SQL)) {
+                    // can't see this user as the current user has no capability
+                    // and this user has a hidden assignment at this context or higher
+                    continue;  
+                }
+              
                 $puser->fullname = fullname($puser);
                 $users[$puser->id] = $puser;  
             }