diff --git a/classes/webservice.php b/classes/webservice.php index b3ef5639..136b210a 100644 --- a/classes/webservice.php +++ b/classes/webservice.php @@ -473,7 +473,7 @@ private function get_least_recently_active_paid_user_id() { continue; } // Skip the protected group. - if (!in_array($this->protectedgroup,$user->group_ids ?? array(),true)) { + if (!in_array($this->protectedgroup, $user->group_ids ?? array(), true)) { continue; } // We need the login time. @@ -502,14 +502,14 @@ private function get_least_recently_active_paid_user_id() { public function get_groups() { // Classic: group:read:admin. // Granular: group:read:list_groups:admin. - // Not essential scope, execute only if scope has been granted + // Not essential scope, execute only if scope has been granted. if ($this->has_scope(['group:read:list_groups:admin']) || $this->has_scope(['group:read:admin'])) { try { $response = $this->make_call('/groups'); - - } catch (moodle_exception $error) { + } + catch (moodle_exception $error) { $response = ''; - } + } } else { $response = ''; } diff --git a/settings.php b/settings.php index 0be569e4..619899b2 100644 --- a/settings.php +++ b/settings.php @@ -171,8 +171,8 @@ 1 ); $settings->add($recycleonjoin); - - // Only call to the web services and load the setting if the connection is OK + + // Only call to the web services and load the setting if the connection is OK. if (isset($status) && $status === 'connectionok') { $zoomgrps = []; $groupobj = zoom_webservice()->get_groups(); @@ -184,8 +184,8 @@ $protectedgroup = new admin_setting_configselect( 'zoom/protectedgroup', - get_string('protectedgroup','mod_zoom'), - get_string('protectedgroup_desc','mod_zoom'), + get_string('protectedgroup', 'mod_zoom'), + get_string('protectedgroup_desc', 'mod_zoom'), 0, $zoomgrps); $settings->add($protectedgroup);