From 690c946c150d8c73f84e2ade4681e4d4d535d9a3 Mon Sep 17 00:00:00 2001
From: martinlanghoff <martinlanghoff>
Date: Wed, 19 Sep 2007 07:09:50 +0000
Subject: [PATCH] get_role_users() callers rework to use the new $gethidden

---
 admin/roles/assign.php | 2 +-
 course/info.php        | 3 ++-
 enrol/manual/enrol.php | 3 ++-
 group/assign.php       | 3 ++-
 4 files changed, 7 insertions(+), 4 deletions(-)

diff --git a/admin/roles/assign.php b/admin/roles/assign.php
index 417d878b99..aa62e9e0d7 100755
--- a/admin/roles/assign.php
+++ b/admin/roles/assign.php
@@ -281,7 +281,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, r.hidden')) {
+        if (!$contextusers = get_role_users($roleid, $context, false, 'u.id, u.firstname, u.lastname, u.email, ra.hidden')) {
             $contextusers = array();
         }
 
diff --git a/course/info.php b/course/info.php
index 267d9dc355..1abdda729e 100644
--- a/course/info.php
+++ b/course/info.php
@@ -55,7 +55,8 @@
         $coursemanagerroles = split(',', $managerroles);
         foreach ($coursemanagerroles as $roleid) {
             $role = get_record('role','id',$roleid);
-            if ($users = get_role_users($roleid, $context, true, '', 'u.lastname ASC', true)) {
+            $canseehidden = has_capability('moodle/role:viewhiddenassigns', $context);
+            if ($users = get_role_users($roleid, $context, true, '', 'u.lastname ASC', $canseehidden)) {
                 foreach ($users as $teacher) {
                     $fullname = fullname($teacher, has_capability('moodle/site:viewfullnames', $context)); 
                     $namesarray[] = format_string($role->name).': <a href="'.$CFG->wwwroot.'/user/view.php?id='.
diff --git a/enrol/manual/enrol.php b/enrol/manual/enrol.php
index db8964f888..ea3d09b166 100644
--- a/enrol/manual/enrol.php
+++ b/enrol/manual/enrol.php
@@ -368,8 +368,9 @@ function print_enrolmentkeyfrom($course) {
 
     // if a keyholder role is defined we list teachers in that role (if any exist)
     $contactslisted = false;
+    $canseehidden = has_capability('moodle/role:viewhiddenassigns', $context);
     if (!empty($CFG->enrol_manual_keyholderrole)) {
-        if ($contacts = get_role_users($CFG->enrol_manual_keyholderrole, get_context_instance(CONTEXT_COURSE, $course->id), true )) {
+        if ($contacts = get_role_users($CFG->enrol_manual_keyholderrole, get_context_instance(CONTEXT_COURSE, $course->id),$canseehidden  )) {
             // guest user has a slightly different message
             if ($guest) {
                 print_string('enrolmentkeyfromguest', '', ':<br />' );
diff --git a/group/assign.php b/group/assign.php
index 75b7fe0c6f..9cfff8dacd 100644
--- a/group/assign.php
+++ b/group/assign.php
@@ -78,7 +78,8 @@ if ($currentmembers) {
         $coursemanagerroles = split(',', $managerroles);
         foreach ($coursemanagerroles as $roleid) {
             $role = get_record('role','id',$roleid);
-            $managers = get_role_users($roleid, $context, true, 'u.id', 'u.id ASC', true);
+            $canseehidden = has_capability('moodle/role:viewhiddenassigns', $context);
+            $managers = get_role_users($roleid, $context, true, 'u.id', 'u.id ASC', $canseehidden);
         }
     }
     
-- 
2.39.5