Fix trailing commas

Author: tvdijen

src/Controller/AuthorizationController.php

@@ -36,7 +36,7 @@ public function __construct(
         private readonly AuthenticationService $authenticationService,
         private readonly AuthorizationServer $authorizationServer,
         private readonly ModuleConfig $moduleConfig,
-        private readonly LoggerService $loggerService
+        private readonly LoggerService $loggerService,
     ) {
     }
 
@@ -139,7 +139,7 @@ protected function validateAcr(AuthorizationRequest $authorizationRequest): void
         $message = sprintf(
             'No ACRs configured for current auth source, whilst specification mandates one. ' .
             'Falling back to generic ACR (%s).',
-            $genericAcr
+            $genericAcr,
         );
         $this->loggerService->warning($message);
         $authorizationRequest->setAcr($genericAcr);

src/Controller/Client/CreateController.php

@@ -39,7 +39,7 @@ public function __construct(
         private readonly TemplateFactory $templateFactory,
         private readonly FormFactory $formFactory,
         private readonly SessionMessagesService $messages,
-        private readonly AuthContextService $authContextService
+        private readonly AuthContextService $authContextService,
     ) {
     }
 
@@ -95,7 +95,7 @@ public function __invoke(): Template|RedirectResponse
                 empty($client['auth_source']) ? null : (string)$client['auth_source'],
                 empty($client['owner']) ? null : (string)$client['owner'],
                 $postLogoutRedirectUris,
-                empty($client['backchannel_logout_uri']) ? null : (string)$client['backchannel_logout_uri']
+                empty($client['backchannel_logout_uri']) ? null : (string)$client['backchannel_logout_uri'],
             ));
 
             // Also persist allowed origins for this client.

src/Controller/Client/DeleteController.php

@@ -40,7 +40,7 @@ public function __construct(
         ClientRepository $clientRepository,
         private readonly TemplateFactory $templateFactory,
         private readonly SessionMessagesService $messages,
-        AuthContextService $authContextService
+        AuthContextService $authContextService,
     ) {
         $this->clientRepository = $clientRepository;
         $this->authContextService = $authContextService;

src/Controller/Client/EditController.php

@@ -44,7 +44,7 @@ public function __construct(
         private readonly TemplateFactory $templateFactory,
         private readonly FormFactory $formFactory,
         private readonly SessionMessagesService $messages,
-        AuthContextService $authContextService
+        AuthContextService $authContextService,
     ) {
         $this->clientRepository = $clientRepository;
         $this->authContextService = $authContextService;
@@ -102,7 +102,7 @@ public function __invoke(ServerRequest $request): Template|RedirectResponse
                 empty($data['auth_source']) ? null : (string)$data['auth_source'],
                 $client->getOwner(),
                 $postLogoutRedirectUris,
-                empty($data['backchannel_logout_uri']) ? null : (string)$data['backchannel_logout_uri']
+                empty($data['backchannel_logout_uri']) ? null : (string)$data['backchannel_logout_uri'],
             ), $authedUser);
 
             // Also persist allowed origins for this client.
@@ -111,15 +111,15 @@ public function __invoke(ServerRequest $request): Template|RedirectResponse
             $this->messages->addMessage('{oidc:client:updated}');
 
             return new RedirectResponse(
-                (new HTTP())->addURLParameters('show.php', ['client_id' => $client->getIdentifier()])
+                (new HTTP())->addURLParameters('show.php', ['client_id' => $client->getIdentifier()]),
             );
         }
 
         return $this->templateFactory->render('oidc:clients/edit.twig', [
             'form' => $form,
             'regexUri' => ClientForm::REGEX_URI,
             'regexAllowedOriginUrl' => ClientForm::REGEX_ALLOWED_ORIGIN_URL,
-            'regexHttpUri' => ClientForm::REGEX_HTTP_URI
+            'regexHttpUri' => ClientForm::REGEX_HTTP_URI,
         ]);
     }
 }

src/Controller/Client/IndexController.php

@@ -28,7 +28,7 @@ class IndexController
     public function __construct(
         private readonly ClientRepository $clientRepository,
         private readonly TemplateFactory $templateFactory,
-        private readonly AuthContextService $authContextService
+        private readonly AuthContextService $authContextService,
     ) {
     }
 

src/Controller/Client/ResetSecretController.php

@@ -35,7 +35,7 @@ class ResetSecretController
     public function __construct(
         ClientRepository $clientRepository,
         private readonly SessionMessagesService $messages,
-        AuthContextService $authContextService
+        AuthContextService $authContextService,
     ) {
         $this->clientRepository = $clientRepository;
         $this->authContextService = $authContextService;
@@ -69,7 +69,7 @@ public function __invoke(ServerRequest $request): RedirectResponse
         }
 
         return new RedirectResponse(
-            (new HTTP())->addURLParameters('show.php', ['client_id' => $client->getIdentifier()])
+            (new HTTP())->addURLParameters('show.php', ['client_id' => $client->getIdentifier()]),
         );
     }
 }

src/Controller/Client/ShowController.php

@@ -36,7 +36,7 @@ public function __construct(
         ClientRepository $clientRepository,
         private readonly AllowedOriginRepository $allowedOriginRepository,
         private readonly TemplateFactory $templateFactory,
-        AuthContextService $authContextService
+        AuthContextService $authContextService,
     ) {
         $this->clientRepository = $clientRepository;
         $this->authContextService = $authContextService;
@@ -52,7 +52,7 @@ public function __invoke(ServerRequest $request): Template
 
         return $this->templateFactory->render('oidc:clients/show.twig', [
             'client' => $client,
-            'allowedOrigins' => $allowedOrigins
+            'allowedOrigins' => $allowedOrigins,
         ]);
     }
 }

src/Controller/InstallerController.php

@@ -34,7 +34,7 @@ public function __construct(
         private readonly TemplateFactory $templateFactory,
         private readonly SessionMessagesService $messages,
         private readonly DatabaseMigration $databaseMigration,
-        private readonly DatabaseLegacyOAuth2Import $databaseLegacyOAuth2Import
+        private readonly DatabaseLegacyOAuth2Import $databaseLegacyOAuth2Import,
     ) {
     }
 

src/Controller/LogoutController.php

@@ -28,7 +28,7 @@ public function __construct(
         protected SessionService $sessionService,
         protected LogoutTicketStoreBuilder $sessionLogoutTicketStoreBuilder,
         protected LoggerService $loggerService,
-        protected TemplateFactory $templateFactory
+        protected TemplateFactory $templateFactory,
     ) {
     }
 
@@ -87,7 +87,7 @@ public function __invoke(ServerRequest $request): Response
                 }
             } catch (Throwable $exception) {
                 $this->loggerService->warning(
-                    sprintf('Logout: could not get session with ID %s, error: %s', $sidClaim, $exception->getMessage())
+                    sprintf('Logout: could not get session with ID %s, error: %s', $sidClaim, $exception->getMessage()),
                 );
             }
         }
@@ -131,8 +131,8 @@ public static function logoutHandler(): void
         $sessionLogoutTicketStore = LogoutTicketStoreBuilder::getStaticInstance();
         $sessionLogoutTickets = $sessionLogoutTicketStore->getAll();
 
-        if (! empty($sessionLogoutTickets)) {
-            //  TODO low mivanci This could brake since interface does not mandate type. Move to strong typing.
+        if (!empty($sessionLogoutTickets)) {
+            // TODO low mivanci This could brake since interface does not mandate type. Move to strong typing.
             /** @var array $sessionLogoutTicket */
             foreach ($sessionLogoutTickets as $sessionLogoutTicket) {
                 $sid = (string)$sessionLogoutTicket['sid'];
@@ -144,7 +144,7 @@ public static function logoutHandler(): void
                     if (($sessionLogoutTicketSession = Session::getSession($sid)) !== null) {
                         $relyingPartyAssociations = array_merge(
                             $relyingPartyAssociations,
-                            SessionService::getRelyingPartyAssociationsForSession($sessionLogoutTicketSession)
+                            SessionService::getRelyingPartyAssociationsForSession($sessionLogoutTicketSession),
                         );
 
                         SessionService::clearRelyingPartyAssociationsForSession($sessionLogoutTicketSession);
@@ -154,14 +154,14 @@ public static function logoutHandler(): void
                         sprintf(
                             'Session Ticket Logout: could not get session with ID %s, error: %s',
                             $sid,
-                            $exception->getMessage()
-                        )
+                            $exception->getMessage(),
+                        ),
                     );
                 }
             }
 
             $sessionLogoutTicketStore->deleteMultiple(
-                array_map(fn(array $slt): string => (string)$slt['sid'], $sessionLogoutTickets)
+                array_map(fn(array $slt): string => (string)$slt['sid'], $sessionLogoutTickets),
             );
         }
 
@@ -183,7 +183,7 @@ protected function resolveResponse(LogoutRequest $logoutRequest, bool $wasLogout
         }
 
         return $this->templateFactory->render('oidc:/logout.twig', [
-            'wasLogoutActionCalled' => $wasLogoutActionCalled
+            'wasLogoutActionCalled' => $wasLogoutActionCalled,
         ]);
     }
 }

src/Controller/UserInfoController.php

@@ -37,7 +37,7 @@ public function __construct(
         private readonly AccessTokenRepository $accessTokenRepository,
         private readonly UserRepository $userRepository,
         private readonly AllowedOriginRepository $allowedOriginRepository,
-        private readonly ClaimTranslatorExtractor $claimTranslatorExtractor
+        private readonly ClaimTranslatorExtractor $claimTranslatorExtractor,
     ) {
     }
 
@@ -70,7 +70,7 @@ public function __invoke(ServerRequest $request): Response
         $requestedClaims =  $accessToken->getRequestedClaims();
         $additionalClaims = $this->claimTranslatorExtractor->extractAdditionalUserInfoClaims(
             $requestedClaims,
-            $user->getClaims()
+            $user->getClaims(),
         );
         $claims = array_merge($additionalClaims, $claims);