giving some indication a role assignment is a hidden one

diff --git a/admin/roles/assign.html b/admin/roles/assign.html
index 620d77bcf71f69bf31b4c144de831e73f0391d50..87e4e03b3acdc98f99df3e6a6fc1f232719644dd 100755 (executable)
--- 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 48ee603c521b9a75d9e5dc31e19469b41a88ee32..48db187816628285c5de2e2d4f770e955fb93a7a 100755 (executable)
--- 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();
         }