Fix CS.

Author: dereuromark

src/AuthorizationService.php

@@ -35,7 +35,7 @@ class AuthorizationService implements AuthorizationServiceInterface
     protected ResolverInterface $resolver;
 
     /**
-     * Track whether or not authorization was checked.
+     * Track whether authorization was checked.
      *
      * @var bool
      */
@@ -82,7 +82,7 @@ protected function performCheck(
         ?IdentityInterface $user,
         string $action,
         mixed $resource,
-        mixed ...$optionalArgs
+        mixed ...$optionalArgs,
     ): bool|ResultInterface {
         $this->authorizationChecked = true;
         $policy = $this->resolver->getPolicy($resource);
@@ -102,8 +102,8 @@ protected function performCheck(
             is_bool($result) || $result instanceof ResultInterface,
             new Exception(sprintf(
                 'Authorization check method must return `%s` or `bool`.',
-                ResultInterface::class
-            ))
+                ResultInterface::class,
+            )),
         );
 
         return $result;
@@ -144,7 +144,7 @@ protected function getCanHandler(mixed $policy, string $action): Closure
 
         assert(
             method_exists($policy, $method) || method_exists($policy, '__call'),
-            new MissingMethodException([$method, $action, get_class($policy)])
+            new MissingMethodException([$method, $action, get_class($policy)]),
         );
 
         return [$policy, $method](...);
@@ -164,7 +164,7 @@ protected function getScopeHandler(mixed $policy, string $action): Closure
 
         assert(
             method_exists($policy, $method) || method_exists($policy, '__call'),
-            new MissingMethodException([$method, $action, get_class($policy)])
+            new MissingMethodException([$method, $action, get_class($policy)]),
         );
 
         return [$policy, $method](...);

src/AuthorizationServiceInterface.php

@@ -53,7 +53,7 @@ public function canResult(
         ?IdentityInterface $user,
         string $action,
         mixed $resource,
-        mixed ...$optionalArgs
+        mixed ...$optionalArgs,
     ): ResultInterface;
 
     /**
@@ -74,11 +74,11 @@ public function applyScope(
         ?IdentityInterface $user,
         string $action,
         mixed $resource,
-        mixed ...$optionalArgs
+        mixed ...$optionalArgs,
     ): mixed;
 
     /**
-     * Return a boolean based on whether or not this object
+     * Return a boolean based on whether this object
      * has had an authorization operation performed.
      *
      * @return bool

src/Controller/Component/AuthorizationComponent.php

@@ -123,7 +123,7 @@ public function canResult(mixed $resource, ?string $action = null): ResultInterf
     protected function performCheck(
         mixed $resource,
         ?string $action = null,
-        string $method = 'can'
+        string $method = 'can',
     ): ResultInterface|bool {
         $request = $this->getController()->getRequest();
         if ($action === null) {
@@ -197,7 +197,7 @@ public function mapAction(string $controllerAction, string $policyAction)
     /**
      * Allows to map controller actions to policy actions.
      *
-     * @param array $actions Map of controller action to policy action.
+     * @param array<string, mixed> $actions Map of controller action to policy action.
      * @param bool $overwrite Set to true to override configuration. False will merge with current configuration.
      * @return $this
      */
@@ -238,7 +238,7 @@ protected function getService(ServerRequestInterface $request): AuthorizationSer
                 'Expected that `%s` would be an instance of %s, but got %s',
                 $serviceAttribute,
                 AuthorizationServiceInterface::class,
-                $type
+                $type,
             ));
         }
 
@@ -266,7 +266,7 @@ protected function getIdentity(ServerRequestInterface $request): ?IdentityInterf
                 'Expected that `%s` would be an instance of %s, but got %s',
                 $identityAttribute,
                 IdentityInterface::class,
-                $type
+                $type,
             ));
         }
 

src/Exception/ForbiddenException.php

@@ -51,7 +51,7 @@ public function __construct(
         ?ResultInterface $result = null,
         string|array $message = '',
         ?int $code = null,
-        ?Throwable $previous = null
+        ?Throwable $previous = null,
     ) {
         $this->result = $result;
 

src/Middleware/AuthorizationMiddleware.php

@@ -85,14 +85,14 @@ class AuthorizationMiddleware implements MiddlewareInterface
      * Constructor.
      *
      * @param \Authorization\AuthorizationServiceInterface|\Authorization\AuthorizationServiceProviderInterface $subject Authorization service or provider instance.
-     * @param array $config Config array.
+     * @param array<string, mixed> $config Config array.
      * @param \Cake\Core\ContainerInterface|null $container The container instance from the application
      * @throws \InvalidArgumentException
      */
     public function __construct(
         AuthorizationServiceInterface|AuthorizationServiceProviderInterface $subject,
         array $config = [],
-        ?ContainerInterface $container = null
+        ?ContainerInterface $container = null,
     ) {
         if ($this->_defaultConfig['identityDecorator'] === null) {
             $this->_defaultConfig['identityDecorator'] = interface_exists(AuthenIdentityInterface::class)
@@ -142,7 +142,7 @@ public function process(ServerRequestInterface $request, RequestHandlerInterface
             $response = $this->handleException(
                 $exception,
                 $request,
-                $this->getConfig('unauthorizedHandler')
+                $this->getConfig('unauthorizedHandler'),
             );
         }
 
@@ -157,7 +157,7 @@ public function process(ServerRequestInterface $request, RequestHandlerInterface
      * @throws \RuntimeException When authorization method has not been defined.
      */
     protected function getAuthorizationService(
-        ServerRequestInterface $request
+        ServerRequestInterface $request,
     ): AuthorizationServiceInterface {
         $service = $this->subject;
         if ($this->subject instanceof AuthorizationServiceProviderInterface) {
@@ -168,7 +168,7 @@ protected function getAuthorizationService(
             throw new RuntimeException(sprintf(
                 'Invalid service returned from the provider. `%s` does not implement `%s`.',
                 get_debug_type($service),
-                AuthorizationServiceInterface::class
+                AuthorizationServiceInterface::class,
             ));
         }
 
@@ -184,7 +184,7 @@ protected function getAuthorizationService(
      */
     protected function buildIdentity(
         AuthorizationServiceInterface $service,
-        ArrayAccess|array $identity
+        ArrayAccess|array $identity,
     ): IdentityInterface {
         $class = $this->getConfig('identityDecorator');
 
@@ -200,7 +200,7 @@ protected function buildIdentity(
             throw new RuntimeException(sprintf(
                 'Invalid identity returned by decorator. `%s` does not implement `%s`.',
                 get_debug_type($identity),
-                IdentityInterface::class
+                IdentityInterface::class,
             ));
         }
 

src/Middleware/RequestAuthorizationMiddleware.php

@@ -57,7 +57,7 @@ class RequestAuthorizationMiddleware implements MiddlewareInterface
     /**
      * Constructor
      *
-     * @param array $config Configuration options
+     * @param array<string, mixed> $config Configuration options
      */
     public function __construct(array $config = [])
     {

src/Middleware/UnauthorizedHandler/CakeRedirectHandler.php

@@ -55,7 +55,7 @@ public function __construct()
                 'Class `%s` does not exist. ' .
                 'Make sure you are using a full CakePHP framework ' .
                 'and have autoloading configured properly.',
-                Router::class
+                Router::class,
             );
             throw new RuntimeException($message);
         }

src/Middleware/UnauthorizedHandler/ExceptionHandler.php

@@ -31,7 +31,7 @@ class ExceptionHandler implements HandlerInterface
     public function handle(
         Exception $exception,
         ServerRequestInterface $request,
-        array $options = []
+        array $options = [],
     ): ResponseInterface {
         throw $exception;
     }

src/Middleware/UnauthorizedHandler/HandlerFactory.php

@@ -44,7 +44,7 @@ public static function create(string $name): HandlerInterface
             $message = sprintf(
                 'Handler should implement `%s`, got `%s`.',
                 HandlerInterface::class,
-                get_class($instance)
+                get_class($instance),
             );
             throw new RuntimeException($message);
         }

src/Middleware/UnauthorizedHandler/HandlerInterface.php

@@ -30,12 +30,12 @@ interface HandlerInterface
      *
      * @param \Authorization\Exception\Exception $exception Authorization exception thrown by the application.
      * @param \Psr\Http\Message\ServerRequestInterface $request Server request.
-     * @param array $options Options array.
+     * @param array<string, mixed> $options Options array.
      * @return \Psr\Http\Message\ResponseInterface
      */
     public function handle(
         Exception $exception,
         ServerRequestInterface $request,
-        array $options = []
+        array $options = [],
     ): ResponseInterface;
 }