smarcet / JOSE4PHP

src/jws/JWSFactory.php

Braces +6 added lines, -4 removed lines

@@ -41,8 +41,8 @@ 
 
         if($spec instanceof IJWS_ParamsSpecification)
         {
-            if($spec->getKey()->getKeyUse()->getString() !== JSONWebKeyPublicKeyUseValues::Signature)
-                throw new InvalidJWKType
+            if($spec->getKey()->getKeyUse()->getString() !== JSONWebKeyPublicKeyUseValues::Signature) {
+                            throw new InvalidJWKType
                 (
                     sprintf
                     (
@@ -50,9 +50,10 @@ 
                         $spec->getKey()->getKeyUse()->getString()
                     )
                 );
+            }
 
-            if($spec->getAlg()->getString() !== $spec->getKey()->getAlgorithm()->getString())
-                throw new InvalidJWKAlgorithm
+            if($spec->getAlg()->getString() !== $spec->getKey()->getAlgorithm()->getString()) {
+                            throw new InvalidJWKAlgorithm
                 (
                     sprintf
                     (
@@ -61,6 +62,7 @@ 
                         $spec->getKey()->getAlgorithm()->getString()
                     )
                 );
+            }
 
             $header = new JOSEHeader
             (

src/jws/impl/specs/JWS_ParamsSpecification.php

Braces +6 added lines, -4 removed lines

@@ -56,11 +56,13 @@
      */
     public function __construct(IJWK $key, StringOrURI $alg, $payload, $signature = ''){
 
-        if(is_null($key))
-            throw new InvalidJWKType();
+        if(is_null($key)) {
+                    throw new InvalidJWKType();
+        }
 
-        if(is_null($payload))
-            throw new JWSInvalidPayloadException('missing payload');
+        if(is_null($payload)) {
+                    throw new JWSInvalidPayloadException('missing payload');
+        }
 
         $this->key = $key;
         $this->alg = $alg;

src/jws/impl/JWS.php

Braces +42 added lines, -32 removed lines

@@ -79,8 +79,9 @@ 
 
         parent::__construct($header, $claim_set);
 
-        if(!is_null($payload))
-            $this->setPayload($payload);
+        if(!is_null($payload)) {
+                    $this->setPayload($payload);
+        }
 
         $this->signature = $signature;
     }
@@ -100,8 +101,9 @@ 
      */
     public function toCompactSerialization()
     {
-        if(!is_null($this->jwk->getId()))
-            $this->header->addHeader(new JOSEHeaderParam(RegisteredJOSEHeaderNames::KeyID, $this->jwk->getId()));
+        if(!is_null($this->jwk->getId())) {
+                    $this->header->addHeader(new JOSEHeaderParam(RegisteredJOSEHeaderNames::KeyID, $this->jwk->getId()));
+        }
 
         if($this->jwk instanceof IAsymmetricJWK)
         {
@@ -134,16 +136,19 @@ 
     public function sign()
     {
 
-        if(is_null($this->jwk))
-            throw new JWSInvalidJWKException;
+        if(is_null($this->jwk)) {
+                    throw new JWSInvalidJWKException;
+        }
 
-        if($this->jwk->getKeyUse()->getString() !== JSONWebKeyPublicKeyUseValues::Signature)
-            throw new JWSInvalidJWKException(sprintf('use %s not supported.', $this->jwk->getKeyUse()->getString()));
+        if($this->jwk->getKeyUse()->getString() !== JSONWebKeyPublicKeyUseValues::Signature) {
+                    throw new JWSInvalidJWKException(sprintf('use %s not supported.', $this->jwk->getKeyUse()->getString()));
+        }
 
         $alg = DigitalSignatures_MACs_Registry::getInstance()->get($this->header->getAlgorithm()->getString());
 
-        if(is_null($alg))
-            throw new JWSNotSupportedAlgorithm(sprintf('alg %s.',$this->header->getAlgorithm()->getString()));
+        if(is_null($alg)) {
+                    throw new JWSNotSupportedAlgorithm(sprintf('alg %s.',$this->header->getAlgorithm()->getString()));
+        }
 
         $secured_input_bytes = JOSEHeaderSerializer::serialize($this->header) . IBasicJWT::SegmentSeparator .$this->getEncodedPayload();
 
@@ -152,12 +157,10 @@ 
         if($alg instanceof DigitalSignatureAlgorithm)
         {
             $this->signature = $alg->sign($key, $secured_input_bytes);
-        }
-        else if($alg instanceof MAC_Algorithm )
+        } else if($alg instanceof MAC_Algorithm )
         {
             $this->signature = $alg->digest($key, $secured_input_bytes);
-        }
-        else
+        } else
         {
             throw new JWSNotSupportedAlgorithm(sprintf('alg %s.',$this->header->getAlgorithm()->getString()));
         }
@@ -171,15 +174,15 @@ 
      */
     public function getEncodedPayload()
     {
-        if(is_null($this->payload))
-            throw new JWSInvalidPayloadException('payload is not set!');
+        if(is_null($this->payload)) {
+                    throw new JWSInvalidPayloadException('payload is not set!');
+        }
 
         $enc_payload = '';
         if($this->payload->isClaimSet() && $this->payload instanceof IJWSPayloadClaimSetSpec)
         {
             $enc_payload = JWTClaimSetSerializer::serialize($this->payload->getClaimSet());
-        }
-        else
+        } else
         {
             $enc_payload = JWTRawSerializer::serialize($this->payload->getRaw());
         }
@@ -233,11 +236,12 @@ 
      */
     public function verify($original_alg)
     {
-        if(is_null($this->jwk))
-            throw new JWSInvalidJWKException;
+        if(is_null($this->jwk)) {
+                    throw new JWSInvalidJWKException;
+        }
 
-        if($this->jwk->getKeyUse()->getString() !== JSONWebKeyPublicKeyUseValues::Signature)
-            throw new JWSInvalidJWKException
+        if($this->jwk->getKeyUse()->getString() !== JSONWebKeyPublicKeyUseValues::Signature) {
+                    throw new JWSInvalidJWKException
             (
                 sprintf
                 (
@@ -245,12 +249,14 @@ 
                     $this->jwk->getKeyUse()->getString()
                 )
             );
+        }
 
-        if(is_null($this->jwk->getAlgorithm()))
-            throw new InvalidJWKAlgorithm('algorithm intended for use with the key is not set! ');
+        if(is_null($this->jwk->getAlgorithm())) {
+                    throw new InvalidJWKAlgorithm('algorithm intended for use with the key is not set! ');
+        }
 
-        if(!is_null($this->jwk->getId()) && !is_null($this->header->getKeyID()) && $this->header->getKeyID()->getValue() != $this->jwk->getId()->getValue())
-            throw new JWSInvalidJWKException
+        if(!is_null($this->jwk->getId()) && !is_null($this->header->getKeyID()) && $this->header->getKeyID()->getValue() != $this->jwk->getId()->getValue()) {
+                    throw new JWSInvalidJWKException
             (
                 sprintf
                 (
@@ -259,16 +265,18 @@ 
                     $this->jwk->getId()->getValue()
                 )
             );
+        }
 
         $alg = DigitalSignatures_MACs_Registry::getInstance()->get($original_alg);
 
-        if(is_null($alg))
-            throw new JWSNotSupportedAlgorithm(sprintf('algo %s', $original_alg));
+        if(is_null($alg)) {
+                    throw new JWSNotSupportedAlgorithm(sprintf('algo %s', $original_alg));
+        }
 
         $former_alg = $this->header->getAlgorithm()->getString();
 
-        if($former_alg != $original_alg)
-            throw new JWSNotSupportedAlgorithm
+        if($former_alg != $original_alg) {
+                    throw new JWSNotSupportedAlgorithm
             (
                 sprintf
                 (
@@ -277,9 +285,10 @@ 
                     $original_alg
                 )
             );
+        }
 
-        if($this->jwk->getAlgorithm()->getValue() !==  $original_alg)
-            throw new InvalidJWKAlgorithm
+        if($this->jwk->getAlgorithm()->getValue() !==  $original_alg) {
+                    throw new InvalidJWKAlgorithm
             (
                 sprintf
                 (
@@ -288,6 +297,7 @@ 
                     $original_alg
                 )
             );
+        }
 
         $secured_input_bytes = JOSEHeaderSerializer::serialize($this->header) . IBasicJWT::SegmentSeparator .$this->getEncodedPayload();
 

src/jws/payloads/JWSPayloadFactory .php

Braces +1 added lines, -2 removed lines

@@ -33,8 +33,7 @@
 
         if($content instanceof IJWTClaimSet){
             return new _JWSPayloadClaimSetSpec($content);
-        }
-        else{
+        } else{
             return new _JWSPayloadRawSpec($content);
         }
     }

src/utils/services/impl/RandomNumberGeneratorService.php

Braces +3 added lines, -1 removed lines

@@ -29,7 +29,9 @@
      */
     public function invoke()
     {
-        if(func_num_args() <= 0) throw new \RuntimeException('you must pass len as arg!');
+        if(func_num_args() <= 0) {
+         throw new \RuntimeException('you must pass len as arg!');
+        }
         $byte_len = func_get_arg(0);
         return ByteUtil::randomBytes($byte_len);
     }

src/utils/services/Utils_Registry.php

Braces +3 added lines, -2 removed lines

@@ -54,8 +54,9 @@
      * @return null|IService
      */
     public function get($service_name){
-        if(!array_key_exists($service_name,  $this->services))
-            throw new \InvalidArgumentException('unknown service!');
+        if(!array_key_exists($service_name,  $this->services)) {
+                    throw new \InvalidArgumentException('unknown service!');
+        }
 
         return $this->services[$service_name];
     }

src/utils/factories/BasicJWTFactory.php

Braces +3 added lines, -2 removed lines

@@ -47,8 +47,9 @@
             case 3:
                 // JWS or unsecured one
                 $header = JOSEHeaderSerializer::deserialize($segments[0]);
-                if($header->getAlgorithm()->getString() === 'none' && empty($segments[2]))
-                    return UnsecuredJWT::fromCompactSerialization($compact_serialization);
+                if($header->getAlgorithm()->getString() === 'none' && empty($segments[2])) {
+                                    return UnsecuredJWT::fromCompactSerialization($compact_serialization);
+                }
                 return JWSFactory::build( new JWS_CompactFormatSpecification($compact_serialization) );
             break;
             case 5:

src/utils/json_types/StringOrURI.php

Braces +6 added lines, -4 removed lines

@@ -35,10 +35,12 @@
      */
     public function getUri(){
 
-        if($this->isString())
-            throw new \RuntimeException('current value is not an uri!');
-        if(filter_var($this->value, FILTER_VALIDATE_URL) === false)
-            throw new \RuntimeException('current value is not an uri!');
+        if($this->isString()) {
+                    throw new \RuntimeException('current value is not an uri!');
+        }
+        if(filter_var($this->value, FILTER_VALIDATE_URL) === false) {
+                    throw new \RuntimeException('current value is not an uri!');
+        }
 
         return (string)$this->value;
     }

src/utils/json_types/JsonArray.php

Braces +3 added lines, -2 removed lines

@@ -41,8 +41,9 @@
 
     public function offsetUnset($offset)
     {
-        if ($this->offsetExists($offset))
-            unset($this->value[$offset]);
+        if ($this->offsetExists($offset)) {
+                    unset($this->value[$offset]);
+        }
     }
 
     public function append($value){