From 5a701d1fbe0ec4f1c2ceb6cb38ccc26fa709f324 Mon Sep 17 00:00:00 2001 From: Michael Aherne Date: Fri, 11 Jan 2013 11:12:33 +0000 Subject: [PATCH] MDL-27164 groups Do not show multiple instances of the same role as multiple --- group/lib.php | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/group/lib.php b/group/lib.php index 024e81b0ed8..4b6a49bbe14 100644 --- a/group/lib.php +++ b/group/lib.php @@ -767,7 +767,7 @@ function groups_calculate_role_people($rs, $context) { $roles[$roledata->id] = $roledata; } // Record that user has role - $users[$rec->userid]->roles[] = $roles[$rec->roleid]; + $users[$rec->userid]->roles[$rec->roleid] = $roles[$rec->roleid]; } } $rs->close(); @@ -797,7 +797,8 @@ function groups_calculate_role_people($rs, $context) { } else if($rolecount > 1) { $roleid = '*'; } else { - $roleid = $userdata->roles[0]->id; + $userrole = reset($userdata->roles); + $roleid = $userrole->id; } $roles[$roleid]->users[$userid] = $userdata; }