fix code style issues

Author: xabbuh

src/Controller/Connect/AbstractController.php

@@ -57,7 +57,7 @@ public function __construct(
         TokenStorageInterface $tokenStorage,
         UserCheckerInterface $userChecker,
         Environment $twig,
-        ?AccountConnectorInterface $accountConnector
+        ?AccountConnectorInterface $accountConnector,
     ) {
         $this->resourceOwnerMapLocator = $resourceOwnerMapLocator;
         $this->requestStack = $requestStack;
@@ -81,7 +81,7 @@ protected function getResourceOwnerByName(string $name): ResourceOwnerInterface
             }
         }
 
-        throw new NotFoundHttpException(sprintf("No resource owner with name '%s'.", $name));
+        throw new NotFoundHttpException(\sprintf("No resource owner with name '%s'.", $name));
     }
 
     /**

src/Controller/Connect/ConnectController.php

@@ -11,6 +11,7 @@
 
 namespace HWI\Bundle\OAuthBundle\Controller\Connect;
 
+use Exception;
 use HWI\Bundle\OAuthBundle\Connect\AccountConnectorInterface;
 use HWI\Bundle\OAuthBundle\Event\GetResponseUserEvent;
 use HWI\Bundle\OAuthBundle\HWIOAuthEvents;
@@ -62,7 +63,7 @@ public function __construct(
         bool $failedUseReferer,
         string $failedAuthPath,
         bool $enableConnectConfirmation,
-        ?AccountConnectorInterface $accountConnector
+        ?AccountConnectorInterface $accountConnector,
     ) {
         parent::__construct(
             $resourceOwnerMapLocator,
@@ -89,7 +90,7 @@ public function __construct(
      *
      * @param string $service name of the resource owner to connect to
      *
-     * @throws \Exception
+     * @throws Exception
      * @throws NotFoundHttpException if `connect` functionality was not enabled
      * @throws AccessDeniedException if no user is authenticated
      */

src/Controller/Connect/RegisterController.php

@@ -11,6 +11,7 @@
 
 namespace HWI\Bundle\OAuthBundle\Controller\Connect;
 
+use Exception;
 use HWI\Bundle\OAuthBundle\Connect\AccountConnectorInterface;
 use HWI\Bundle\OAuthBundle\Event\FilterUserResponseEvent;
 use HWI\Bundle\OAuthBundle\Event\FormEvent;
@@ -19,6 +20,8 @@
 use HWI\Bundle\OAuthBundle\HWIOAuthEvents;
 use HWI\Bundle\OAuthBundle\Security\Core\Exception\AccountNotLinkedException;
 use HWI\Bundle\OAuthBundle\Security\Http\ResourceOwnerMapLocator;
+use InvalidArgumentException;
+use RuntimeException;
 use Symfony\Component\Form\FormFactoryInterface;
 use Symfony\Component\HttpFoundation\RedirectResponse;
 use Symfony\Component\HttpFoundation\Request;
@@ -58,7 +61,7 @@ public function __construct(
         string $grantRule,
         ?string $registrationForm,
         ?AccountConnectorInterface $accountConnector,
-        ?RegistrationFormHandlerInterface $formHandler
+        ?RegistrationFormHandlerInterface $formHandler,
     ) {
         parent::__construct(
             $resourceOwnerMapLocator,
@@ -85,7 +88,7 @@ public function __construct(
      *
      * @throws NotFoundHttpException if `connect` functionality was not enabled
      * @throws AccessDeniedException if any user is authenticated
-     * @throws \RuntimeException
+     * @throws RuntimeException
      */
     public function registrationAction(Request $request, string $key): Response
     {
@@ -109,11 +112,11 @@ public function registrationAction(Request $request, string $key): Response
         }
 
         if (!$error instanceof AccountNotLinkedException) {
-            throw new \RuntimeException('Cannot register an account.', 0, $error instanceof \Exception ? $error : null);
+            throw new RuntimeException('Cannot register an account.', 0, $error instanceof Exception ? $error : null);
         }
 
         if (!$this->registrationForm) {
-            throw new \InvalidArgumentException('Registration form class must be set.');
+            throw new InvalidArgumentException('Registration form class must be set.');
         }
 
         $userInformation = $this

src/Controller/LoginController.php

@@ -12,6 +12,7 @@
 namespace HWI\Bundle\OAuthBundle\Controller;
 
 use HWI\Bundle\OAuthBundle\Security\Core\Exception\AccountNotLinkedException;
+use LogicException;
 use Symfony\Component\HttpFoundation\RedirectResponse;
 use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpFoundation\RequestStack;
@@ -46,7 +47,7 @@ public function __construct(
         RequestStack $requestStack,
         Environment $twig,
         bool $connect,
-        string $grantRule
+        string $grantRule,
     ) {
         $this->authenticationUtils = $authenticationUtils;
         $this->router = $router;
@@ -61,7 +62,7 @@ public function __construct(
      * Action that handles the login 'form'. If connecting is enabled the
      * user will be redirected to the appropriate login urls or registration forms.
      *
-     * @throws \LogicException
+     * @throws LogicException
      */
     public function connectAction(Request $request): Response
     {

src/Controller/RedirectToServiceController.php

@@ -14,6 +14,7 @@
 use HWI\Bundle\OAuthBundle\Security\Http\ResourceOwnerMapLocator;
 use HWI\Bundle\OAuthBundle\Security\OAuthUtils;
 use HWI\Bundle\OAuthBundle\Util\DomainWhitelist;
+use RuntimeException;
 use Symfony\Component\HttpFoundation\Exception\SessionNotFoundException;
 use Symfony\Component\HttpFoundation\RedirectResponse;
 use Symfony\Component\HttpFoundation\Request;
@@ -33,7 +34,7 @@ public function __construct(
         private readonly ResourceOwnerMapLocator $resourceOwnerMapLocator,
         private readonly ?string $targetPathParameter,
         private readonly bool $failedUseReferer,
-        private readonly bool $useReferer
+        private readonly bool $useReferer,
     ) {
     }
 
@@ -44,7 +45,7 @@ public function redirectToServiceAction(Request $request, string $service): Redi
     {
         try {
             $authorizationUrl = $this->oauthUtils->getAuthorizationUrl($request, $service);
-        } catch (\RuntimeException $e) {
+        } catch (RuntimeException $e) {
             throw new NotFoundHttpException($e->getMessage(), $e);
         }
 

src/DependencyInjection/CompilerPass/ResourceOwnerCompilerPass.php

@@ -13,6 +13,7 @@
 
 use HWI\Bundle\OAuthBundle\DependencyInjection\Configuration;
 use HWI\Bundle\OAuthBundle\DependencyInjection\HWIOAuthExtension;
+use InvalidArgumentException;
 use Symfony\Component\Config\Definition\Exception\InvalidConfigurationException;
 use Symfony\Component\DependencyInjection\Compiler\CompilerPassInterface;
 use Symfony\Component\DependencyInjection\ContainerBuilder;
@@ -75,9 +76,9 @@ private function registerResourceOwnerTypeClassParameters(ContainerBuilder $cont
             }
 
             if (!Configuration::isResourceOwnerSupported($match['type'])) {
-                $e = new \InvalidArgumentException(sprintf('Unknown resource owner type "%s"', $match['type']));
+                $e = new InvalidArgumentException(\sprintf('Unknown resource owner type "%s"', $match['type']));
 
-                throw new InvalidConfigurationException(sprintf('Invalid configuration for path "hwi_oauth.resource_owners.%s.type": %s', $resourceOwnerName, $e->getMessage()), $e->getCode(), $e);
+                throw new InvalidConfigurationException(\sprintf('Invalid configuration for path "hwi_oauth.resource_owners.%s.type": %s', $resourceOwnerName, $e->getMessage()), $e->getCode(), $e);
             }
         }
     }

src/DependencyInjection/Configuration.php

@@ -14,6 +14,8 @@
 use HWI\Bundle\OAuthBundle\OAuth\ResourceOwner\GenericOAuth1ResourceOwner;
 use HWI\Bundle\OAuthBundle\OAuth\ResourceOwner\GenericOAuth2ResourceOwner;
 use HWI\Bundle\OAuthBundle\OAuth\ResourceOwnerInterface;
+use LogicException;
+use ReflectionClass;
 use Symfony\Component\Config\Definition\BaseNode;
 use Symfony\Component\Config\Definition\Builder\ArrayNodeDefinition;
 use Symfony\Component\Config\Definition\Builder\TreeBuilder;
@@ -73,17 +75,17 @@ public static function isResourceOwnerSupported(string $resourceOwner): bool
 
     public static function registerResourceOwner(string $resourceOwnerClass): void
     {
-        $reflection = new \ReflectionClass($resourceOwnerClass);
+        $reflection = new ReflectionClass($resourceOwnerClass);
         if (!$reflection->implementsInterface(ResourceOwnerInterface::class)) {
-            throw new \LogicException('Resource owner class should implement "ResourceOwnerInterface", or extended class "GenericOAuth1ResourceOwner"/"GenericOAuth2ResourceOwner".');
+            throw new LogicException('Resource owner class should implement "ResourceOwnerInterface", or extended class "GenericOAuth1ResourceOwner"/"GenericOAuth2ResourceOwner".');
         }
 
         $type = \defined("$resourceOwnerClass::TYPE") ? $resourceOwnerClass::TYPE : null;
         if (null === $type) {
             if (preg_match('~(?P<resource_owner>[^\\\\]+)ResourceOwner$~', $resourceOwnerClass, $match)) {
                 $type = strtolower(preg_replace('/([a-z])([A-Z])/', '$1_$2', $match['resource_owner']));
             } else {
-                throw new \LogicException(sprintf('Resource owner class either should have "TYPE" const defined or end with "ResourceOwner" so that type can be calculated by converting its class name without suffix to "snake_case". Given class name is "%s"', $resourceOwnerClass));
+                throw new LogicException(\sprintf('Resource owner class either should have "TYPE" const defined or end with "ResourceOwner" so that type can be calculated by converting its class name without suffix to "snake_case". Given class name is "%s"', $resourceOwnerClass));
             }
         }
 

src/DependencyInjection/HWIOAuthExtension.php

@@ -11,7 +11,10 @@
 
 namespace HWI\Bundle\OAuthBundle\DependencyInjection;
 
+use ArrayIterator;
+use Exception;
 use HWI\Bundle\OAuthBundle\OAuth\ResourceOwnerInterface;
+use RuntimeException;
 use Symfony\Component\Config\Definition\Exception\InvalidConfigurationException;
 use Symfony\Component\Config\Definition\Processor;
 use Symfony\Component\Config\FileLocator;
@@ -35,15 +38,15 @@
 final class HWIOAuthExtension extends Extension
 {
     /**
-     * @var \ArrayIterator<string, true>
+     * @var ArrayIterator<string, true>
      */
-    private \ArrayIterator $firewallNames;
+    private ArrayIterator $firewallNames;
 
     private bool $refreshTokenListenerEnabled = false;
 
     public function __construct()
     {
-        $this->firewallNames = new \ArrayIterator();
+        $this->firewallNames = new ArrayIterator();
     }
 
     public function getConfiguration(array $config, ContainerBuilder $container): Configuration
@@ -54,8 +57,8 @@ public function getConfiguration(array $config, ContainerBuilder $container): Co
     /**
      * {@inheritdoc}
      *
-     * @throws \Exception
-     * @throws \RuntimeException
+     * @throws Exception
+     * @throws RuntimeException
      * @throws InvalidConfigurationException
      * @throws BadMethodCallException
      * @throws InvalidArgumentException
@@ -138,7 +141,7 @@ public function createResourceOwnerService(ContainerBuilder $container, string $
         // handle external resource owners with given class
         if (isset($options['class'])) {
             if (!is_subclass_of($options['class'], ResourceOwnerInterface::class, true)) {
-                throw new InvalidConfigurationException(sprintf('Class "%s" must implement interface "HWI\Bundle\OAuthBundle\OAuth\ResourceOwnerInterface".', $options['class']));
+                throw new InvalidConfigurationException(\sprintf('Class "%s" must implement interface "HWI\Bundle\OAuthBundle\OAuth\ResourceOwnerInterface".', $options['class']));
             }
 
             $definition = new Definition($options['class']);
@@ -166,7 +169,7 @@ public function getAlias(): string
         return 'hwi_oauth';
     }
 
-    public function getFirewallNames(): \ArrayIterator
+    public function getFirewallNames(): ArrayIterator
     {
         return $this->firewallNames;
     }

src/DependencyInjection/Security/Factory/OAuthAuthenticatorFactory.php

@@ -11,6 +11,7 @@
 
 namespace HWI\Bundle\OAuthBundle\DependencyInjection\Security\Factory;
 
+use ArrayIterator;
 use HWI\Bundle\OAuthBundle\Security\Http\Authenticator\OAuthAuthenticator;
 use HWI\Bundle\OAuthBundle\Security\Http\Firewall\RefreshAccessTokenListener;
 use HWI\Bundle\OAuthBundle\Security\Http\Firewall\RefreshAccessTokenListenerOld;
@@ -31,7 +32,7 @@
  */
 final class OAuthAuthenticatorFactory extends AbstractFactory implements AuthenticatorFactoryInterface, FirewallListenerFactoryInterface
 {
-    public function __construct(private \ArrayIterator $firewallNames)
+    public function __construct(private ArrayIterator $firewallNames)
     {
     }
 
@@ -58,7 +59,7 @@ public function createAuthenticator(
         ContainerBuilder $container,
         string $firewallName,
         array $config,
-        string $userProviderId
+        string $userProviderId,
     ): string {
         $authenticatorId = 'security.authenticator.oauth.'.$firewallName;
 
@@ -130,7 +131,7 @@ public function getPosition(): string
         return 'http';
     }
 
-    public function getFirewallNames(): \ArrayIterator
+    public function getFirewallNames(): ArrayIterator
     {
         return $this->firewallNames;
     }

src/HWIOAuthBundle.php

@@ -14,6 +14,7 @@
 use HWI\Bundle\OAuthBundle\DependencyInjection\CompilerPass\EnableRefreshOAuthTokenListenerCompilerPass;
 use HWI\Bundle\OAuthBundle\DependencyInjection\CompilerPass\ResourceOwnerCompilerPass;
 use HWI\Bundle\OAuthBundle\DependencyInjection\Security\Factory\OAuthAuthenticatorFactory;
+use RuntimeException;
 use Symfony\Bundle\SecurityBundle\DependencyInjection\SecurityExtension;
 use Symfony\Component\DependencyInjection\ContainerBuilder;
 use Symfony\Component\DependencyInjection\Extension\ExtensionInterface;
@@ -43,7 +44,7 @@ public function build(ContainerBuilder $container): void
             // Symfony < 5.4 BC layer
             $extension->addSecurityListenerFactory(new OAuthAuthenticatorFactory($firewallNames));
         } else {
-            throw new \RuntimeException('Unsupported Symfony Security component version');
+            throw new RuntimeException('Unsupported Symfony Security component version');
         }
 
         $container->addCompilerPass(new ResourceOwnerCompilerPass());