Minor: Format code - refs BT#21930

AngelFQC · Sep 13, 2024 · 2b5eeb5 · 2b5eeb5
1 parent 274f9f2
commit 2b5eeb5
Show file tree

Hide file tree

Showing 3 changed files with 40 additions and 42 deletions.
diff --git a/plugin/azure_active_directory/src/AzureActiveDirectory.php b/plugin/azure_active_directory/src/AzureActiveDirectory.php
@@ -306,6 +306,44 @@ public function registerUser(
         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 @@ private function formatUserData(
             $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);
-            },
-        ];
-    }
 }
diff --git a/plugin/azure_active_directory/src/AzureCommand.php b/plugin/azure_active_directory/src/AzureCommand.php
@@ -138,9 +138,9 @@ protected function getAzureGroups(): Generator
     }
 
     /**
-     * @return Generator<int, array<string, string>>
-     *
      * @throws Exception
+     *
+     * @return Generator<int, array<string, string>>
      */
     protected function getAzureGroupMembers(string $groupUid): Generator
     {

diff --git a/plugin/azure_active_directory/src/AzureSyncUsergroupsCommand.php b/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
 {
     /**
-Original file line number
+Diff line change
@@ Expand Up / @@ -2,8 +2,6 @@ @@
     /* For license terms, see /license.txt */
-    use League\OAuth2\Client\Token\AccessTokenInterface;
     class AzureSyncUsergroupsCommand extends AzureCommand
     {
         /**
@@ Expand Down @@