Minor: Format code - refs BT#21930

plugin/azure_active_directory/src/AzureActiveDirectory.php

@@ -306,6 +306,44 @@ class AzureActiveDirectory extends Plugin
         return $userId;
     }
 
+    /**
+     * @return array<string, string|false>
+     */
+    public function getGroupUidByRole(): array
+    {
+        $groupUidList = [
+            'admin' => $this->get(self::SETTING_GROUP_ID_ADMIN),
+            'sessionAdmin' => $this->get(self::SETTING_GROUP_ID_SESSION_ADMIN),
+            'teacher' => $this->get(self::SETTING_GROUP_ID_TEACHER),
+        ];
+
+        return array_filter($groupUidList);
+    }
+
+    /**
+     * @return array<string, callable>
+     */
+    public function getUpdateActionByRole(): array
+    {
+        return [
+            'admin' => function (User $user) {
+                $user->setStatus(COURSEMANAGER);
+
+                UserManager::addUserAsAdmin($user, false);
+            },
+            'sessionAdmin' => function (User $user) {
+                $user->setStatus(SESSIONADMIN);
+
+                UserManager::removeUserAdmin($user, false);
+            },
+            'teacher' => function (User $user) {
+                $user->setStatus(COURSEMANAGER);
+
+                UserManager::removeUserAdmin($user, false);
+            },
+        ];
+    }
+
     /**
      * @throws Exception
      */
@@ -347,42 +385,4 @@ class AzureActiveDirectory extends Plugin
             $extra,
         ];
     }
-
-    /**
-     * @return array<string, string|false>
-     */
-    public function getGroupUidByRole(): array
-    {
-        $groupUidList = [
-            'admin' => $this->get(self::SETTING_GROUP_ID_ADMIN),
-            'sessionAdmin' => $this->get(self::SETTING_GROUP_ID_SESSION_ADMIN),
-            'teacher' => $this->get(self::SETTING_GROUP_ID_TEACHER),
-        ];
-
-        return array_filter($groupUidList);
-    }
-
-    /**
-     * @return array<string, callable>
-     */
-    public function getUpdateActionByRole(): array
-    {
-        return [
-            'admin' => function (User $user) {
-                $user->setStatus(COURSEMANAGER);
-
-                UserManager::addUserAsAdmin($user, false);
-            },
-            'sessionAdmin' => function (User $user) {
-                $user->setStatus(SESSIONADMIN);
-
-                UserManager::removeUserAdmin($user, false);
-            },
-            'teacher' => function (User $user) {
-                $user->setStatus(COURSEMANAGER);
-
-                UserManager::removeUserAdmin($user, false);
-            },
-        ];
-    }
 }

plugin/azure_active_directory/src/AzureCommand.php

@@ -138,9 +138,9 @@ abstract class AzureCommand
     }
 
     /**
-     * @return Generator<int, array<string, string>>
-     *
      * @throws Exception
+     *
+     * @return Generator<int, array<string, string>>
      */
     protected function getAzureGroupMembers(string $groupUid): Generator
     {

plugin/azure_active_directory/src/AzureSyncUsergroupsCommand.php

@@ -2,8 +2,6 @@
 
 /* For license terms, see /license.txt */
 
-use League\OAuth2\Client\Token\AccessTokenInterface;
-
 class AzureSyncUsergroupsCommand extends AzureCommand
 {
     /**