php-guard / oauth-server

src/OAuth2/GrantTypes/AuthorizationCodeGrantType.php

Spacing +1 added lines, -1 removed lines

@@ -73,7 +73,7 @@
 
         $this->authorizationCodeStorage->revoke($authorizationCode->getCode());
 
-        if($authorizationCode->getExpiresAt() < time()) {
+        if ($authorizationCode->getExpiresAt() < time()) {
             throw new OAuthException('invalid_grant', 'Authorization code has expired',
                 'https://tools.ietf.org/html/rfc6749#section-4.1.3');
         }

src/OAuth2/EndpointMessages/Authorization/AuthorizationRequest.php

Spacing +2 added lines, -2 removed lines

@@ -96,11 +96,11 @@
      * @throws \Exception
      */
     public function validate() {
-        if(!$this->responseType) {
+        if (!$this->responseType) {
             throw new \Exception('Missing response_type parameter');
         }
 
-        if(!$this->clientId) {
+        if (!$this->clientId) {
             throw new \Exception('Missing client_id parameter');
         }
         return true;

src/OAuth2/OpenID/Credentials/IdToken.php

Spacing +2 added lines, -2 removed lines

@@ -30,12 +30,12 @@
     {
         $missingClaims = array_diff(self::REQUIRED_CLAIMS, array_keys($claims));
         if (!empty($missingClaims)) {
-            throw new \Exception('Missing claims : ' . implode(', ', $missingClaims));
+            throw new \Exception('Missing claims : '.implode(', ', $missingClaims));
         }
 
         $undefinedClaims = array_diff(array_keys($claims), self::DEFINED_CLAIMS);
         if (!empty($undefinedClaims)) {
-            throw new \Exception('Undefined claims : ' . implode(', ', $undefinedClaims));
+            throw new \Exception('Undefined claims : '.implode(', ', $undefinedClaims));
         }
 
         // todo check nonce required if present in authentication request

src/OAuth2/OpenID/Endpoints/AuthorizationEndpoint.php

Spacing +8 added lines, -8 removed lines

@@ -38,7 +38,7 @@ 
         /**
          * @var RegisteredClient $client
          */
-        if($res = $this->verify($request, $result)) {
+        if ($res = $this->verify($request, $result)) {
             return $res;
         }
 
@@ -56,7 +56,7 @@ 
         $result = [];
 
         try {
-            if(!$resourceOwner->isConsentGivenForClient($client)) {
+            if (!$resourceOwner->isConsentGivenForClient($client)) {
                 throw new OAuthException('access_denied',
                     'The resource owner server denied the request',
                     'https://tools.ietf.org/html/rfc6749#section-4.1.1');
@@ -68,7 +68,7 @@ 
              * @var ResponseTypeInterface $responseType
              */
             foreach ($responseTypes as $responseType) {
-                if($responseType->getExtendedResponseTypes()) {
+                if ($responseType->getExtendedResponseTypes()) {
                     $extendedResponseTypes = array_merge($extendedResponseTypes, $responseType->getExtendedResponseTypes());
                 }
                 $responseTypeNames[] = $responseType->getResponseType();
@@ -78,7 +78,7 @@ 
              * @var ResponseTypeInterface $responseType
              */
             foreach ($responseTypes as $responseType) {
-                if(!in_array($responseType->getResponseType(), $extendedResponseTypes)) {
+                if (!in_array($responseType->getResponseType(), $extendedResponseTypes)) {
                     $extendedResponseTypes = null;
                     if ($responseType->getExtendedResponseTypes()) {
                         $extendedResponseTypeNames = array_intersect($responseType->getExtendedResponseTypes(), array_keys($responseTypes));
@@ -98,7 +98,7 @@ 
                 $data['state'] ?? null);
         }
 
-        if(isset($data['state'])) {
+        if (isset($data['state'])) {
             $result['state'] = $data['state'];
         }
 
@@ -258,7 +258,7 @@ 
         if (!$scopePolicyManager->checkScope($client, $scope)) {
             $supportedScopes = implode(', ', $scopePolicyManager->getSupportedScopes($client));
             throw new OAuthException('invalid_scope',
-                'Some of requested scopes are not supported. Scope supported : ' . $supportedScopes,
+                'Some of requested scopes are not supported. Scope supported : '.$supportedScopes,
                 'https://tools.ietf.org/html/rfc6749#section-4.1');
         }
 
@@ -417,12 +417,12 @@ 
             $responseType = $this->server->getResponseTypeRepository()->getResponseType($responseTypeName);
             if (!$responseType) {
                 throw new OAuthException('invalid_request',
-                    'Unknown response_type : ' . $responseTypeName);
+                    'Unknown response_type : '.$responseTypeName);
             }
 
             if (!$responseType->isMultiValuedResponseTypeSupported()) {
                 throw new OAuthException('invalid_request',
-                    'Multi-valued response_type not supported with response_type : ' . $responseTypeName);
+                    'Multi-valued response_type not supported with response_type : '.$responseTypeName);
             }
 
             $responseTypes[$responseTypeName] = $responseType;

src/OAuth2/OpenID/ResponseTypes/IdTokenResponseType.php

Spacing +6 added lines, -6 removed lines

@@ -73,25 +73,25 @@ 
         ];
 
         // todo, include if auth_time is marked as an essential claim by the client otherwise it is optional (conf ?)
-        if(isset($data['max_age']) && $data['max_age']) {
+        if (isset($data['max_age']) && $data['max_age']) {
             $claims['auth_time'] = $resourceOwner->getTimeWhenAuthenticationOccured();
         }
 
-        if(isset($data['nonce']) && !is_null($data['nonce'])) {
+        if (isset($data['nonce']) && !is_null($data['nonce'])) {
             $claims['nonce'] = $data['nonce'];
         }
 
-        if(empty($extendedResponseTypes)) {
+        if (empty($extendedResponseTypes)) {
             $standardClaims = $this->userInfoClaimsStorage->getClaims($resourceOwner);
 
             foreach ($this->userInfoClaimsStorage->getClaimsByScope($scope) as $claimRequested) {
-                if(isset($standardClaims[$claimRequested]) && $standardClaims[$claimRequested]) {
+                if (isset($standardClaims[$claimRequested]) && $standardClaims[$claimRequested]) {
                     $claims[$claimRequested] = $standardClaims[$claimRequested];
                 }
             }
         }
 
-        if(isset($extendedResponseTypes['code'])) {
+        if (isset($extendedResponseTypes['code'])) {
             //c_hash
             /**
              * @var \OAuth2\ResponseTypes\ResponseTypeInterface $responseType
@@ -102,7 +102,7 @@ 
             $claims['c_hash'] = 'todo'; //todo
         }
 
-        if(isset($extendedResponseTypes['token'])) {
+        if (isset($extendedResponseTypes['token'])) {
             //at_hash
             /**
              * @var \OAuth2\ResponseTypes\ResponseTypeInterface $responseType

src/OAuth2/OpenID/encours/Test.php

Spacing +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
         $certsRepository = $this->sqlOauth->getRepository(Cert::class);
 
         $actualCerts = $certsRepository->findBy([], ['createdAt' => 'DESC'], 2);
-        if(!isset($actualCerts[0])) {
+        if (!isset($actualCerts[0])) {
             list('privKey' => $privKey, 'pubKey' => $pubKey, 'rsa' => $rsa) = $this->generateRSAKeys();
 
             $oldCert = new Cert();

src/OAuth2/OpenID/Models/AddressClaim.php

Spacing +6 added lines, -6 removed lines

@@ -142,22 +142,22 @@
     function jsonSerialize()
     {
         $data = [];
-        if($this->formatted) {
+        if ($this->formatted) {
             $data['formatted'] = $this->formatted;
         }
-        if($this->streetAddress) {
+        if ($this->streetAddress) {
             $data['street_address'] = $this->streetAddress;
         }
-        if($this->locality) {
+        if ($this->locality) {
             $data['locality'] = $this->locality;
         }
-        if($this->region) {
+        if ($this->region) {
             $data['region'] = $this->region;
         }
-        if($this->postalCode) {
+        if ($this->postalCode) {
             $data['postal_code'] = $this->postalCode;
         }
-        if($this->country) {
+        if ($this->country) {
             $data['country'] = $this->country;
         }
         return empty($data) ? '' : json_encode($data);

src/OAuth2/OpenID/Models/StandardClaims.php

Spacing +1 added lines, -1 removed lines

@@ -445,7 +445,7 @@
     {
         $getter = 'get'.ucfirst($this->snakeToCamel($offset));
         $result = $this->{$getter}();
-        if(is_object($result)) {
+        if (is_object($result)) {
             return json_encode($result);
         }
         return $result;

src/OAuth2/Repositories/StorageRepository.php

Spacing +3 added lines, -3 removed lines

@@ -34,10 +34,10 @@ 
     public function __construct(array $storages)
     {
         foreach (static::STORAGES_INTERFACES as $name => $interface) {
-            if(!isset($storages[$name])) {
+            if (!isset($storages[$name])) {
                 throw new \Exception("Missing storage '$name'");
             }
-            if(!is_a($storages[$name], $interface)) {
+            if (!is_a($storages[$name], $interface)) {
                 throw new \Exception("Storage '$name' must implement '$interface'");
             }
             $this->storages[$name] = $storages[$name];
@@ -50,7 +50,7 @@ 
      * @throws \Exception
      */
     public function getStorage(string $name) {
-        if(!isset($this->storages[$name])) {
+        if (!isset($this->storages[$name])) {
             throw new \Exception("Unknown storage '$name'");
         }
         return $this->storages[$name];