get_role_users() callers rework to use the new $gethidden

andelacruz · Sep 19, 2007 · 690c946 · 690c946
1 parent 47b18c1
commit 690c946
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 4 deletions.
diff --git 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
@@ -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
@@ -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
@@ -78,7 +78,8 @@
         $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);
         }
     }