OAuth2-Framework / oauth2-framework

src/Bundle/Server/DependencyInjection/OAuth2FrameworkServerExtension.php

Doc Comments +2 added lines, -2 removed lines

@@ -82,7 +82,7 @@ 
 
     /**
      * @param string           $path
-     * @param array            $sources
+     * @param SourceInterface[]            $sources
      * @param array            $mergedConfig
      * @param ContainerBuilder $container
      */
@@ -100,7 +100,7 @@ 
     /**
      * @param array            $bundleConfig
      * @param string           $path
-     * @param array            $sources
+     * @param SourceInterface[]            $sources
      * @param ContainerBuilder $container
      */
     private function prependSources(array $bundleConfig, string $path, array $sources, ContainerBuilder $container)

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 /*
  * The MIT License (MIT)

src/Bundle/Server/Form/FormFactory.php

Doc Comments +1 added lines, -1 removed lines

@@ -59,7 +59,7 @@
 
     /**
      * @param array      $options
-     * @param mixed|null $data
+     * @param Model\AuthorizationModel $data
      *
      * @return FormInterface
      */

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 /*
  * The MIT License (MIT)

src/Bundle/Server/Tests/Context/OIDCContext.php

Unused Use Statements -2 removed lines

@@ -34,8 +34,6 @@
 use Jose\Object\JWSInterface;
 use OAuth2Framework\Component\Server\Model\Client\Client;
 use OAuth2Framework\Component\Server\Model\Client\ClientId;
-use OAuth2Framework\Component\Server\Model\IdToken\IdTokenId;
-use OAuth2Framework\Component\Server\Model\IdToken\IdTokenLoader;
 use OAuth2Framework\Bundle\Server\Model\ClientRepository;
 
 final class OIDCContext implements Context

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 /*
  * The MIT License (MIT)

src/Component/Server/Tests/Application/Application.php

Unused Use Statements -5 removed lines

@@ -22,10 +22,6 @@ 
 use Interop\Http\Factory\ServerRequestFactoryInterface;
 use Interop\Http\Factory\UriFactoryInterface;
 use Jose\Checker\CheckerManager;
-use Jose\Checker\CriticalHeaderChecker;
-use Jose\Checker\ExpirationTimeChecker;
-use Jose\Checker\IssuedAtChecker;
-use Jose\Checker\NotBeforeChecker;
 use Jose\Decrypter;
 use Jose\Encrypter;
 use Jose\Factory\JWKFactory;
@@ -216,7 +212,6 @@ 
 use OAuth2Framework\Component\Server\Tests\Stub\SecurityLayer;
 use OAuth2Framework\Component\Server\Tests\Stub\ServiceLocator;
 use OAuth2Framework\Component\Server\Tests\Stub\SessionStateParameterExtension;
-use OAuth2Framework\Component\Server\Tests\Stub\SubjectChecker;
 use OAuth2Framework\Component\Server\Tests\Stub\TrustedIssuer;
 use OAuth2Framework\Component\Server\Tests\Stub\UriExtension;
 use OAuth2Framework\Component\Server\Tests\Stub\UserAccountManager;

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 /*
  * The MIT License (MIT)

src/Component/Server/Tests/Stub/AuthCodeRepository.php

Doc Comments +3 added lines, -3 removed lines

@@ -202,7 +202,7 @@ 
      * @param string                  $redirectUri
      * @param DataBag                 $parameters
      * @param DataBag                 $metadatas
-     * @param array                   $scopes
+     * @param string[]                   $scopes
      * @param \DateTimeImmutable|null $expiresAt
      * @param bool                    $withRefreshToken
      * @param ResourceServerId|null   $resourceServerId
@@ -239,7 +239,7 @@ 
      * @param string                  $redirectUri
      * @param DataBag                 $parameters
      * @param DataBag                 $metadatas
-     * @param array                   $scopes
+     * @param string[]                   $scopes
      * @param \DateTimeImmutable|null $expiresAt
      * @param bool                    $withRefreshToken
      * @param ResourceServerId|null   $resourceServerId
@@ -277,7 +277,7 @@ 
      * @param string                  $redirectUri
      * @param DataBag                 $parameters
      * @param DataBag                 $metadatas
-     * @param array                   $scopes
+     * @param string[]                   $scopes
      * @param \DateTimeImmutable|null $expiresAt
      * @param bool                    $withRefreshToken
      * @param ResourceServerId|null   $resourceServerId

Spacing +3 added lines, -3 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 /*
  * The MIT License (MIT)
@@ -140,7 +140,7 @@ 
     /**
      * {@inheritdoc}
      */
-    public function create(ClientId $clientId, UserAccountId $userAccountId, array $queryParameters, string $redirectUri, DataBag $parameters, DataBag $metadatas, array $scopes, bool $withRefreshToken, ?ResourceServerId $resourceServerId, ?\DateTimeImmutable $expiresAt): AuthCode
+    public function create(ClientId $clientId, UserAccountId $userAccountId, array $queryParameters, string $redirectUri, DataBag $parameters, DataBag $metadatas, array $scopes, bool $withRefreshToken, ? ResourceServerId $resourceServerId, ? \DateTimeImmutable $expiresAt) : AuthCode
     {
         if (null === $expiresAt) {
             $expiresAt = new \DateTimeImmutable($this->lifetime);
@@ -180,7 +180,7 @@ 
     /**
      * {@inheritdoc}
      */
-    public function find(AuthCodeId $authCodeId): ?AuthCode
+    public function find(AuthCodeId $authCodeId): ? AuthCode
     {
         $authCode = null;
         $events = $this->eventStore->getEvents($authCodeId);

src/Component/Server/Tests/Stub/PreConfiguredAuthorizationRepository.php

Doc Comments +1 added lines, -1 removed lines

@@ -122,7 +122,7 @@
      * @param PreConfiguredAuthorizationId $preConfiguredAuthorizationId
      * @param UserAccountId                $userAccountId
      * @param ClientId                     $clientId
-     * @param array                        $scopes
+     * @param string[]                        $scopes
      */
     private function createAndSavePreConfiguredAuthorization(PreConfiguredAuthorizationId $preConfiguredAuthorizationId, UserAccountId $userAccountId, ClientId $clientId, array $scopes)
     {

Spacing +4 added lines, -4 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 /*
  * The MIT License (MIT)
@@ -56,7 +56,7 @@ 
     /**
      * {@inheritdoc}
      */
-    public function create(UserAccountId $userAccountId, ClientId $clientId, array $scopes, ?ResourceServerId $resourceServerId): PreConfiguredAuthorization
+    public function create(UserAccountId $userAccountId, ClientId $clientId, array $scopes, ? ResourceServerId $resourceServerId) : PreConfiguredAuthorization
     {
         $hash = $this->calculateHash($userAccountId, $clientId, $scopes, $resourceServerId);
         $preConfiguredAuthorization = PreConfiguredAuthorization::createEmpty();
@@ -81,7 +81,7 @@ 
     /**
      * {@inheritdoc}
      */
-    public function find(UserAccountId $userAccountId, ClientId $clientId, array $scopes, ?ResourceServerId $resourceServerId): ?PreConfiguredAuthorization
+    public function find(UserAccountId $userAccountId, ClientId $clientId, array $scopes, ? ResourceServerId $resourceServerId) : ? PreConfiguredAuthorization
     {
         $hash = $this->calculateHash($userAccountId, $clientId, $scopes, $resourceServerId);
         $preConfiguredAuthorization = null;
@@ -104,7 +104,7 @@ 
      *
      * @return PreConfiguredAuthorizationId
      */
-    private function calculateHash(ResourceOwnerId $resourceOwnerId, ClientId $clientId, array $scope, ?ResourceServerId $resourceServerId): PreConfiguredAuthorizationId
+    private function calculateHash(ResourceOwnerId $resourceOwnerId, ClientId $clientId, array $scope, ? ResourceServerId $resourceServerId) : PreConfiguredAuthorizationId
     {
         return PreConfiguredAuthorizationId::create(hash(
             'sha512',

src/Component/Server/Tests/Stub/ScopeRepository.php

Doc Comments +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
     /**
      * ScopeManager constructor.
      *
-     * @param array $availableScopes
+     * @param string[] $availableScopes
      */
     public function __construct(array $availableScopes = [])
     {

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 /*
  * The MIT License (MIT)

src/Component/Client/Behaviour/Metadata.php

Spacing +1 added lines, -1 removed lines

@@ -92,7 +92,7 @@
     {
         return preg_replace_callback(
             '/(^|[a-z])([A-Z])/',
-            function ($m) { return mb_strtolower(mb_strlen($m[1], '8bit') ? sprintf('%s_%s', $m[1], $m[2]) : $m[2], '8bit'); },
+            function($m) { return mb_strtolower(mb_strlen($m[1], '8bit') ? sprintf('%s_%s', $m[1], $m[2]) : $m[2], '8bit'); },
             $word
         );
     }

src/Component/Client/Metadata/ServerMetadata.php

Spacing +1 added lines, -1 removed lines

@@ -69,7 +69,7 @@
             'Invalid URL.'
         );
         Assertion::false(
-            false === $allow_unsecured_connection && 'https://' !==  mb_substr($url, 0, 8, '8bit'),
+            false === $allow_unsecured_connection && 'https://' !== mb_substr($url, 0, 8, '8bit'),
             'Unsecured connection.'
         );