limoncello-php-dist / passport

src/Traits/BasicClientAuthenticationTrait.php

Doc Comments +1 added lines, -1 removed lines

@@ -32,7 +32,7 @@
      * @param array                              $parameters
      * @param string                             $realm
      *
-     * @return ClientInterface|null
+     * @return \Limoncello\Passport\Contracts\Entities\ClientInterface|null
      *
      * @SuppressWarnings(PHPMD.ElseExpression)
      * @SuppressWarnings(PHPMD.NPathComplexity)

Spacing +2 added lines, -2 removed lines

@@ -43,7 +43,7 @@ 
         ServerRequestInterface $request,
         array $parameters,
         $realm = 'OAuth'
-    ): ?ClientInterface {
+    ): ? ClientInterface {
         // A client may use Basic authentication.
         //
         // Or
@@ -57,7 +57,7 @@ 
         // try to parse `Authorization` header for client ID and credentials
         $clientId          = null;
         $clientCredentials = null;
-        $errorHeaders      = ['WWW-Authenticate' => 'Basic realm="' . $realm . '"'];
+        $errorHeaders      = ['WWW-Authenticate' => 'Basic realm="'.$realm.'"'];
         if (empty($headerArray = $authorizationHeader) === false) {
             $errorCode = OAuthTokenBodyException::ERROR_INVALID_CLIENT;
             if (empty($authHeader = $headerArray[0]) === true ||

src/Entities/Token.php

Spacing +13 added lines, -13 removed lines

@@ -154,9 +154,9 @@ 
     {
         if ($this->hasDynamicProperty(static::FIELD_ID) === true) {
             $this
-                ->setIdentifier((int)$this->{static::FIELD_ID})
+                ->setIdentifier((int) $this->{static::FIELD_ID})
                 ->setClientIdentifier($this->{static::FIELD_ID_CLIENT})
-                ->setUserIdentifier((int)$this->{static::FIELD_ID_USER})
+                ->setUserIdentifier((int) $this->{static::FIELD_ID_USER})
                 ->setRedirectUriString($this->{static::FIELD_REDIRECT_URI})
                 ->setCode($this->{static::FIELD_CODE})
                 ->setType($this->{static::FIELD_TYPE})
@@ -171,7 +171,7 @@ 
     /**
      * @inheritdoc
      */
-    public function getIdentifier(): ?int
+    public function getIdentifier(): ? int
     {
         return $this->identifierField;
     }
@@ -259,7 +259,7 @@ 
     /**
      * @inheritdoc
      */
-    public function getRedirectUriString(): ?string
+    public function getRedirectUriString(): ? string
     {
         return $this->redirectUriString;
     }
@@ -267,7 +267,7 @@ 
     /**
      * @inheritdoc
      */
-    public function setRedirectUriString(?string $uri): TokenInterface
+    public function setRedirectUriString(? string $uri) : TokenInterface
     {
         $this->redirectUriString = $uri;
 
@@ -341,7 +341,7 @@ 
     /**
      * @inheritdoc
      */
-    public function setCode(?string $code): TokenInterface
+    public function setCode(? string $code) : TokenInterface
     {
         $this->codeField = $code;
 
@@ -359,7 +359,7 @@ 
     /**
      * @inheritdoc
      */
-    public function setValue(?string $value): TokenInterface
+    public function setValue(? string $value) : TokenInterface
     {
         $this->valueField = $value;
 
@@ -369,7 +369,7 @@ 
     /**
      * @inheritdoc
      */
-    public function getType(): ?string
+    public function getType(): ? string
     {
         return $this->typeField;
     }
@@ -377,7 +377,7 @@ 
     /**
      * @inheritdoc
      */
-    public function setType(?string $type): TokenInterface
+    public function setType(? string $type) : TokenInterface
     {
         $this->typeField = $type;
 
@@ -395,7 +395,7 @@ 
     /**
      * @inheritdoc
      */
-    public function setRefreshValue(?string $refreshValue): TokenInterface
+    public function setRefreshValue(? string $refreshValue) : TokenInterface
     {
         $this->refreshValueField = $refreshValue;
 
@@ -405,7 +405,7 @@ 
     /**
      * @inheritdoc
      */
-    public function getCodeCreatedAt(): ?DateTimeInterface
+    public function getCodeCreatedAt(): ? DateTimeInterface
     {
         if ($this->codeCreatedAtField === null && ($codeCreatedAt = $this->{static::FIELD_CODE_CREATED_AT}) !== null) {
             $this->codeCreatedAtField = $this->parseDateTime($codeCreatedAt);
@@ -427,7 +427,7 @@ 
     /**
      * @inheritdoc
      */
-    public function getValueCreatedAt(): ?DateTimeInterface
+    public function getValueCreatedAt(): ? DateTimeInterface
     {
         if ($this->valueCreatedAtField === null &&
             ($tokenCreatedAt = $this->{static::FIELD_VALUE_CREATED_AT}) !== null
@@ -451,7 +451,7 @@ 
     /**
      * @inheritdoc
      */
-    public function getRefreshCreatedAt(): ?DateTimeInterface
+    public function getRefreshCreatedAt(): ? DateTimeInterface
     {
         if ($this->refreshCreatedAtField === null &&
             ($tokenCreatedAt = $this->{static::FIELD_VALUE_CREATED_AT}) !== null

src/Entities/RedirectUri.php

Spacing +4 added lines, -4 removed lines

@@ -63,7 +63,7 @@ 
     {
         if ($this->hasDynamicProperty(static::FIELD_ID) === true) {
             $this
-                ->setIdentifier((int)$this->{static::FIELD_ID})
+                ->setIdentifier((int) $this->{static::FIELD_ID})
                 ->setClientIdentifier($this->{static::FIELD_ID_CLIENT})
                 ->setValue($this->{static::FIELD_VALUE});
         }
@@ -72,7 +72,7 @@ 
     /**
      * @inheritdoc
      */
-    public function getIdentifier(): ?int
+    public function getIdentifier(): ? int
     {
         return $this->identifierField;
     }
@@ -90,7 +90,7 @@ 
     /**
      * @inheritdoc
      */
-    public function getClientIdentifier(): ?string
+    public function getClientIdentifier(): ? string
     {
         return $this->clientIdentifierField;
     }
@@ -108,7 +108,7 @@ 
     /**
      * @inheritdoc
      */
-    public function getValue(): ?string
+    public function getValue(): ? string
     {
         return $this->valueField;
     }

src/Entities/DatabaseItem.php

Spacing +2 added lines, -2 removed lines

@@ -48,7 +48,7 @@ 
     /**
      * @inheritdoc
      */
-    public function getCreatedAt(): ?DateTimeInterface
+    public function getCreatedAt(): ? DateTimeInterface
     {
         if ($this->createdAtField === null &&
             $this->hasDynamicProperty(static::FIELD_CREATED_AT) === true &&
@@ -63,7 +63,7 @@ 
     /**
      * @inheritdoc
      */
-    public function getUpdatedAt(): ?DateTimeInterface
+    public function getUpdatedAt(): ? DateTimeInterface
     {
         if ($this->updatedAtField === null &&
             $this->hasDynamicProperty(static::FIELD_UPDATED_AT) === true &&

src/Entities/Scope.php

Spacing +2 added lines, -2 removed lines

@@ -55,7 +55,7 @@ 
     /**
      * @inheritdoc
      */
-    public function getIdentifier(): ?string
+    public function getIdentifier(): ? string
     {
         return $this->identifierField;
     }
@@ -73,7 +73,7 @@ 
     /**
      * @inheritdoc
      */
-    public function getDescription(): ?string
+    public function getDescription(): ? string
     {
         return $this->descriptionField;
     }

src/Entities/DatabaseScheme.php

Spacing +4 added lines, -4 removed lines

@@ -508,7 +508,7 @@ 
     /**
      * @inheritdoc
      */
-    public function getUsersView(): ?string
+    public function getUsersView(): ? string
     {
         return static::VIEW_USERS;
     }
@@ -516,7 +516,7 @@ 
     /**
      * @inheritdoc
      */
-    public function getUsersTable(): ?string
+    public function getUsersTable(): ? string
     {
         return $this->usersTableName;
     }
@@ -524,7 +524,7 @@ 
     /**
      * @inheritdoc
      */
-    public function getUsersIdentityColumn(): ?string
+    public function getUsersIdentityColumn(): ? string
     {
         return $this->usersIdColumn;
     }
@@ -532,7 +532,7 @@ 
     /**
      * @inheritdoc
      */
-    public function getPassportView(): ?string
+    public function getPassportView(): ? string
     {
         return static::VIEW_PASSPORT;
     }

src/Entities/Client.php

Spacing +3 added lines, -3 removed lines

@@ -189,7 +189,7 @@ 
     /**
      * @inheritdoc
      */
-    public function getName(): ?string
+    public function getName(): ? string
     {
         return $this->nameField;
     }
@@ -207,7 +207,7 @@ 
     /**
      * @inheritdoc
      */
-    public function getDescription(): ?string
+    public function getDescription(): ? string
     {
         return $this->descriptionField;
     }
@@ -225,7 +225,7 @@ 
     /**
      * @inheritdoc
      */
-    public function getCredentials(): ?string
+    public function getCredentials(): ? string
     {
         return $this->credentialsField;
     }

src/BasePassportServer.php

Spacing +7 added lines, -7 removed lines

@@ -63,7 +63,7 @@ 
         ServerRequestInterface $request,
         array $parameters,
         $realm = 'OAuth'
-    ): ?ClientInterface;
+    ): ? ClientInterface;
 
     /**
      * @var PassportServerIntegrationInterface
@@ -249,7 +249,7 @@ 
     /**
      * @inheritdoc
      */
-    public function codeReadAuthenticationCode(string $code): ?AuthorizationCodeInterface
+    public function codeReadAuthenticationCode(string $code): ? AuthorizationCodeInterface
     {
         return $this->getIntegration()->getTokenRepository()
             ->readByCode($code, $this->getIntegration()->getCodeExpirationPeriod());
@@ -367,7 +367,7 @@ 
 
         assert(is_string($defaultClientId) === true && empty($defaultClientId) === false);
 
-        $defaultClient   = $this->getIntegration()->getClientRepository()->read($defaultClientId);
+        $defaultClient = $this->getIntegration()->getClientRepository()->read($defaultClientId);
 
         assert($defaultClient !== null);
 
@@ -440,7 +440,7 @@ 
             $tokenRepo->updateValues($updatedToken);
         } else {
             assert(is_array($scope));
-            $tokenRepo->inTransaction(function () use ($tokenRepo, $updatedToken, $scope) {
+            $tokenRepo->inTransaction(function() use ($tokenRepo, $updatedToken, $scope) {
                 $tokenRepo->updateValues($updatedToken);
                 $tokenRepo->unbindScopes($updatedToken->getIdentifier());
                 $tokenRepo->bindScopeIdentifiers($updatedToken->getIdentifier(), $scope);
@@ -492,7 +492,7 @@ 
     {
         $this->logDebug('Sending token as JSON response.');
 
-        $scopeList  = $token->isScopeModified() === false || empty($token->getScopeIdentifiers()) === true ?
+        $scopeList = $token->isScopeModified() === false || empty($token->getScopeIdentifiers()) === true ?
             null : $token->getScopeList();
 
         // for access token format @link https://tools.ietf.org/html/rfc6749#section-5.1
@@ -556,7 +556,7 @@ 
     ): ResponseInterface {
         $this->logDebug('Sending token as redirect response.');
 
-        $scopeList  = $token->isScopeModified() === false || empty($token->getScopeIdentifiers()) === true ?
+        $scopeList = $token->isScopeModified() === false || empty($token->getScopeIdentifiers()) === true ?
             null : $token->getScopeList();
 
         // for access token format @link https://tools.ietf.org/html/rfc6749#section-5.1
@@ -699,7 +699,7 @@ 
      */
     private function filterNulls(array $array): array
     {
-        return array_filter($array, function ($value) {
+        return array_filter($array, function($value) {
             return $value !== null;
         });
     }

src/Integration/BasePassportServerIntegration.php

Spacing +6 added lines, -6 removed lines

@@ -127,7 +127,7 @@ 
         /** @var callable|null $customPropProvider */
         $customPropProvider = $this->settings[C::KEY_TOKEN_CUSTOM_PROPERTIES_PROVIDER] ?? null;
         $wrapper            = $customPropProvider !== null ?
-            function (TokenInterface $token) use ($container, $customPropProvider): array {
+            function(TokenInterface $token) use ($container, $customPropProvider) : array {
                 return call_user_func($customPropProvider, $container, $token);
             } : null;
 
@@ -155,7 +155,7 @@ 
     /**
      * @inheritdoc
      */
-    public function verifyAllowedUserScope(int $userIdentity, array $scope = null): ?array
+    public function verifyAllowedUserScope(int $userIdentity, array $scope = null): ? array
     {
         $validator   = $this->settings[C::KEY_USER_SCOPE_VALIDATOR];
         $nullOrScope = call_user_func($validator, $this->container, $userIdentity, $scope);
@@ -176,7 +176,7 @@ 
      */
     public function generateCodeValue(TokenInterface $token): string
     {
-        $codeValue = bin2hex(random_bytes(16)) . uniqid();
+        $codeValue = bin2hex(random_bytes(16)).uniqid();
 
         assert(is_string($codeValue) === true && empty($codeValue) === false);
 
@@ -188,10 +188,10 @@ 
      */
     public function generateTokenValues(TokenInterface $token): array
     {
-        $tokenValue     = bin2hex(random_bytes(16)) . uniqid();
+        $tokenValue     = bin2hex(random_bytes(16)).uniqid();
         $tokenType      = 'bearer';
         $tokenExpiresIn = $this->getTokenExpirationPeriod();
-        $refreshValue   = bin2hex(random_bytes(16)) . uniqid();
+        $refreshValue   = bin2hex(random_bytes(16)).uniqid();
 
         assert(is_string($tokenValue) === true && empty($tokenValue) === false);
         assert(is_string($tokenType) === true && empty($tokenType) === false);
@@ -260,7 +260,7 @@ 
             self::SCOPE_APPROVAL_IS_SCOPE_MODIFIED => $isScopeModified,
             self::SCOPE_APPROVAL_SCOPE             => $scopeList,
             self::SCOPE_APPROVAL_STATE             => $state,
-        ], function ($value) {
+        ], function($value) {
             return $value !== null;
         });