From 5613dbca342c6a052a86cd15fce40345298af89d Mon Sep 17 00:00:00 2001
From: toyomoyo <toyomoyo>
Date: Tue, 21 Nov 2006 06:42:38 +0000
Subject: [PATCH] giving some indication a role assignment is a hidden one

---
 admin/roles/assign.html | 7 ++++++-
 admin/roles/assign.php  | 2 +-
 2 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/admin/roles/assign.html b/admin/roles/assign.html
index 620d77bcf7..87e4e03b3a 100755
--- a/admin/roles/assign.html
+++ b/admin/roles/assign.html
@@ -25,7 +25,12 @@
           <?php
               foreach ($contextusers as $contextuser) {
                   $fullname = fullname($contextuser, true);
-                  echo "<option value=\"$contextuser->id\">".$fullname.", ".$contextuser->email."</option>\n";
+                  if ($contextuser->hidden) {
+                      $hidden=' ('.get_string('hidden').') ';  
+                  } else {
+                      $hidden="";  
+                  }
+                  echo "<option value=\"$contextuser->id\">".$fullname.", ".$contextuser->email.$hidden."</option>\n";
               }
           ?>
 
diff --git a/admin/roles/assign.php b/admin/roles/assign.php
index 48ee603c52..48db187816 100755
--- a/admin/roles/assign.php
+++ b/admin/roles/assign.php
@@ -191,7 +191,7 @@
     /// Get all existing participants in this context.
         // Why is this not done with get_users???
 
-        if (!$contextusers = get_role_users($roleid, $context, false, 'u.id, u.firstname, u.lastname, u.email')) {
+        if (!$contextusers = get_role_users($roleid, $context, false, 'u.id, u.firstname, u.lastname, u.email, r.hidden')) {
             $contextusers = array();
         }
 
-- 
2.39.5