bug #8 fix coding standards (azjezz)

This PR was squashed before being merged into the 0.1-dev branch.

Discussion
----------

fix coding standards

Commits
-------

22148bf fix coding standards

Author: chalasr

.github/workflows/unit-tests.yml

@@ -21,9 +21,6 @@ jobs:
                 dependencies:
                     - "lowest"
                     - "highest"
-                include:
-                    - { symfony-version: "4.4.*", php-version: "7.1", dependencies: "lowest" }
-                    - { symfony-version: "4.4.*", php-version: "7.1", dependencies: "highest" }
 
         steps:
             -   name: "checkout"

.php_cs.dist

@@ -29,7 +29,7 @@ return PhpCsFixer\Config::create()
         ],
         'list_syntax' => ['syntax' => 'short'],
         'no_null_property_initialization' => true,
-        'no_superfluous_phpdoc_tags' => true,
+        'phpdoc_to_comment' => false,
         'ordered_imports' => [
             'imports_order' => [
                 OrderedImportsFixer::IMPORT_TYPE_CONST,

composer.json

@@ -16,12 +16,12 @@
         }
     ],
     "require": {
-        "php": ">=7.1",
+        "php": ">=7.2",
         "doctrine/doctrine-bundle": "^2.0",
         "doctrine/orm": "^2.7",
         "league/oauth2-server": "^8.0",
         "psr/http-factory": "^1.0",
-        "sensio/framework-extra-bundle": "^5.5",
+        "sensio/framework-extra-bundle": "^5.6",
         "symfony/framework-bundle": "^4.4|^5.0",
         "symfony/psr-http-message-bridge": "^2.0",
         "symfony/security-bundle": "^4.4|^5.0"

src/Command/UpdateClientCommand.php

@@ -102,7 +102,7 @@ private function updateClientFromInput(Client $client, InputInterface $input): C
         return $client
             ->setRedirectUris(...array_map(static function (string $redirectUri): RedirectUri {
                 return new RedirectUri($redirectUri);
-            },  $redirectUriStrings))
+            }, $redirectUriStrings))
             ->setGrants(...array_map(static function (string $grant): Grant {
                 return new Grant($grant);
             }, $grantStrings))

src/DBAL/Type/Grant.php

@@ -31,7 +31,7 @@ public function getName(): string
      */
     protected function convertDatabaseValues(array $values): array
     {
-        return array_map(static function(string $value): GrantModel {
+        return array_map(static function (string $value): GrantModel {
             return new GrantModel($value);
         }, $values);
     }

src/DBAL/Type/ImplodedArray.php

@@ -20,7 +20,7 @@ abstract class ImplodedArray extends TextType
     /**
      * {@inheritdoc}
      */
-    public function convertToDatabaseValue($value, AbstractPlatform $platform)
+    public function convertToDatabaseValue($value, AbstractPlatform $platform): ?string
     {
         if (!\is_array($value)) {
             throw new \LogicException('This type can only be used in combination with arrays.');

src/DBAL/Type/RedirectUri.php

@@ -31,7 +31,7 @@ public function getName(): string
      */
     protected function convertDatabaseValues(array $values): array
     {
-        return array_map(static function(string $value): RedirectUriModel {
+        return array_map(static function (string $value): RedirectUriModel {
             return new RedirectUriModel($value);
         }, $values);
     }

src/DBAL/Type/Scope.php

@@ -31,7 +31,7 @@ public function getName(): string
      */
     protected function convertDatabaseValues(array $values): array
     {
-        return array_map(static function(string $value): ScopeModel {
+        return array_map(static function (string $value): ScopeModel {
             return new ScopeModel($value);
         }, $values);
     }

src/Event/AuthorizationRequestResolveEvent.php

@@ -75,6 +75,9 @@ public function hasResponse(): bool
         return $this->response instanceof ResponseInterface;
     }
 
+    /**
+     * @psalm-mutation-free
+     */
     public function getResponse(): ?ResponseInterface
     {
         return $this->response;
@@ -93,11 +96,17 @@ public function getGrantTypeId(): string
         return $this->authorizationRequest->getGrantTypeId();
     }
 
+    /**
+     * @psalm-mutation-free
+     */
     public function getClient(): Client
     {
         return $this->client;
     }
 
+    /**
+     * @psalm-mutation-free
+     */
     public function getUser(): ?UserInterface
     {
         return $this->user;

src/Event/AuthorizationRequestResolveEventFactory.php

@@ -28,7 +28,7 @@ public function __construct(ScopeConverterInterface $scopeConverter, ClientManag
 
     public function fromAuthorizationRequest(AuthorizationRequest $authorizationRequest): AuthorizationRequestResolveEvent
     {
-        $scopes = $this->scopeConverter->toDomainArray(\array_values($authorizationRequest->getScopes()));
+        $scopes = $this->scopeConverter->toDomainArray(array_values($authorizationRequest->getScopes()));
 
         $client = $this->clientManager->find($authorizationRequest->getClient()->getIdentifier());