MDL-76661 mod_bigbluebuttonbn: fix meeting attendee role comparison.
This commit is contained in:
@@ -131,9 +131,9 @@ class meeting {
|
||||
/**
|
||||
* Get meeting attendees
|
||||
*
|
||||
* @return mixed
|
||||
* @return array[]
|
||||
*/
|
||||
public function get_attendees() {
|
||||
public function get_attendees(): array {
|
||||
return $this->get_meeting_info()->attendees ?? [];
|
||||
}
|
||||
|
||||
@@ -276,7 +276,7 @@ class meeting {
|
||||
}
|
||||
$meetinginfo->attendees = [];
|
||||
if (!empty($info['attendees'])) {
|
||||
// Make sure attendees is an array of object, not a simpleXML object.
|
||||
// Ensure each returned attendee is cast to an array, rather than a simpleXML object.
|
||||
foreach ($info['attendees'] as $attendee) {
|
||||
$meetinginfo->attendees[] = (array) $attendee;
|
||||
}
|
||||
|
||||
@@ -175,9 +175,10 @@ class index implements renderable {
|
||||
protected function get_room_attendee_list(meeting $meeting, string $role): string {
|
||||
$attendees = [];
|
||||
|
||||
// Iterate attendees, matching by their "role" property.
|
||||
foreach ($meeting->get_attendees() as $attendee) {
|
||||
if ((string) $attendee->role == $role) {
|
||||
$attendees[] = $attendee->fullName;
|
||||
if (strcmp((string) $attendee['role'], $role) === 0) {
|
||||
$attendees[] = $attendee['fullName'];
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
Reference in New Issue
Block a user