fbett / le-acme2-php

src/LE_ACME2Tests/Authorizer/HTTPTest.php

Spacing +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
 
     public function testDirectoryPath() {
 
-        if(!file_exists($this->_directoryPath)) {
+        if (!file_exists($this->_directoryPath)) {
             mkdir($this->_directoryPath);
         }
 

src/LE_ACME2Tests/OrderTest.php

Spacing +3 added lines, -3 removed lines

@@ -10,7 +10,7 @@ 
 
         $account = \LE_ACME2\Account::get($this->_accountEmail);
 
-        if(\LE_ACME2\Order::exists($account, $this->_orderSubjects)) {
+        if (\LE_ACME2\Order::exists($account, $this->_orderSubjects)) {
             $this->markTestSkipped('Skipped: Order does already exist');
         }
 
@@ -24,7 +24,7 @@ 
 
         $account = \LE_ACME2\Account::get($this->_accountEmail);
 
-        if(\LE_ACME2\Order::exists($account, $this->_orderSubjects)) {
+        if (\LE_ACME2\Order::exists($account, $this->_orderSubjects)) {
             // Skipping order modification tests, when the order already exists
             // to reduce the LE api usage while developing
             TestHelper::getInstance()->setSkipOrderModificationTests(true);
@@ -49,7 +49,7 @@ 
 
         $account = \LE_ACME2\Account::get($this->_accountEmail);
 
-        if(\LE_ACME2\Order::exists($account, $this->_umlautsOrderSubjects)) {
+        if (\LE_ACME2\Order::exists($account, $this->_umlautsOrderSubjects)) {
             // Skipping order modification tests, when the order already exists
             // to reduce the LE api usage while developing
             TestHelper::getInstance()->setSkipOrderModificationTests(true);

src/LE_ACME2Tests/TestHelper.php

Spacing +2 added lines, -2 removed lines

@@ -12,9 +12,9 @@
 
     private function __construct() {
 
-        $projectPath = realpath($_SERVER[ 'PWD' ]) . '/';
+        $projectPath = realpath($_SERVER['PWD']) . '/';
         $this->_tempPath = $projectPath . 'temp/';
-        if( !file_exists($this->_tempPath) ) {
+        if (!file_exists($this->_tempPath)) {
             mkdir($this->_tempPath);
         }
 

src/LE_ACME2Tests/AccountTest.php

Spacing +7 added lines, -7 removed lines

@@ -29,7 +29,7 @@ 
 
     public function testCommonKeyDirectoryPath() {
 
-        if(!file_exists($this->_commonKeyDirectoryPath)) {
+        if (!file_exists($this->_commonKeyDirectoryPath)) {
             mkdir($this->_commonKeyDirectoryPath);
         }
 
@@ -42,7 +42,7 @@ 
 
     public function testNonExisting() {
 
-        if(\LE_ACME2\Account::exists($this->_accountEmail)) {
+        if (\LE_ACME2\Account::exists($this->_accountEmail)) {
             $this->markTestSkipped('Skipped: Account does already exist');
         }
 
@@ -54,7 +54,7 @@ 
 
     public function testCreate() {
 
-        if(\LE_ACME2\Account::exists($this->_accountEmail)) {
+        if (\LE_ACME2\Account::exists($this->_accountEmail)) {
             // Skipping account modification tests, when the account already exists
             // to reduce the LE api usage while developing
             TestHelper::getInstance()->setSkipAccountModificationTests(true);
@@ -76,7 +76,7 @@ 
 
     public function testInvalidCreate() {
 
-        if(TestHelper::getInstance()->shouldSkipAccountModificationTests()) {
+        if (TestHelper::getInstance()->shouldSkipAccountModificationTests()) {
             $this->expectNotToPerformAssertions();
             return;
         }
@@ -93,7 +93,7 @@ 
 
     public function testModification() {
 
-        if(TestHelper::getInstance()->shouldSkipAccountModificationTests()) {
+        if (TestHelper::getInstance()->shouldSkipAccountModificationTests()) {
             $this->expectNotToPerformAssertions();
             return;
         }
@@ -123,7 +123,7 @@ 
 
     public function testDeactivation() {
 
-        if(TestHelper::getInstance()->shouldSkipAccountModificationTests()) {
+        if (TestHelper::getInstance()->shouldSkipAccountModificationTests()) {
             $this->expectNotToPerformAssertions();
             return;
         }
@@ -149,7 +149,7 @@ 
 
     public function testCreationAfterDeactivation() {
 
-        if(TestHelper::getInstance()->shouldSkipAccountModificationTests()) {
+        if (TestHelper::getInstance()->shouldSkipAccountModificationTests()) {
             $this->expectNotToPerformAssertions();
             return;
         }

src/LE_ACME2/Authorizer/DNS.php

Indentation +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
      * @throws Exception\RateLimitReached
      */
     protected function _existsNotValidChallenges(Response\Authorization\Struct\Challenge $challenge,
-                                                 Response\Authorization\Get $authorizationResponse
+                                                    Response\Authorization\Get $authorizationResponse
     ) : bool {
 
         if($challenge->status == Response\Authorization\Struct\Challenge::STATUS_PENDING) {

Spacing +4 added lines, -4 removed lines

@@ -38,13 +38,13 @@ 
                                                  Response\Authorization\Get $authorizationResponse
     ) : bool {
 
-        if($challenge->status == Response\Authorization\Struct\Challenge::STATUS_PENDING) {
+        if ($challenge->status == Response\Authorization\Struct\Challenge::STATUS_PENDING) {
 
-            if(self::$_dnsWriter === null) {
+            if (self::$_dnsWriter === null) {
                 throw new \RuntimeException('DNS writer is not set');
             }
 
-            if( self::$_dnsWriter->write(
+            if (self::$_dnsWriter->write(
                     $this->_order,
                     $authorizationResponse->getIdentifier()->value,
                     (new ChallengeAuthorizationKey($this->_account))->getEncoded($challenge->token)
@@ -58,7 +58,7 @@ 
             }
         }
 
-        if($challenge->status == Response\Authorization\Struct\Challenge::STATUS_INVALID) {
+        if ($challenge->status == Response\Authorization\Struct\Challenge::STATUS_INVALID) {
             throw new Exception\DNSAuthorizationInvalid(
                 'Received status "' . Response\Authorization\Struct\Challenge::STATUS_INVALID . '" while challenge should be verified'
             );

src/LE_ACME2/Authorizer/AbstractAuthorizer.php

Indentation +1 added lines, -1 removed lines

@@ -130,7 +130,7 @@
      * @throws Exception\AuthorizationInvalid
      */
     protected function _existsNotValidChallenges(Response\Authorization\Struct\Challenge $challenge,
-                                                 Response\Authorization\Get $authorizationResponse
+                                                    Response\Authorization\Get $authorizationResponse
     ) : bool {
 
         if($challenge->status == Response\Authorization\Struct\Challenge::STATUS_PENDING) {

Spacing +12 added lines, -12 removed lines

@@ -44,7 +44,7 @@ 
      */
     protected function _fetchAuthorizationResponses() {
 
-        if(!file_exists($this->_order->getKeyDirectoryPath() . 'private.pem')) {
+        if (!file_exists($this->_order->getKeyDirectoryPath() . 'private.pem')) {
 
             Utilities\Logger::getInstance()->add(
                 Utilities\Logger::LEVEL_DEBUG,
@@ -56,7 +56,7 @@ 
 
         $orderResponse = Cache\OrderResponse::getInstance()->get($this->_order);
 
-        foreach($orderResponse->getAuthorizations() as $authorization) {
+        foreach ($orderResponse->getAuthorizations() as $authorization) {
 
             $this->_authorizationResponses[] = Cache\OrderAuthorizationResponse::getInstance()
                 ->get($this->_order, $authorization, $this->_getChallengeType());
@@ -70,10 +70,10 @@ 
 
     public function shouldStartAuthorization() : bool {
 
-        foreach($this->_authorizationResponses as $response) {
+        foreach ($this->_authorizationResponses as $response) {
 
             $challenge = $response->getChallenge($this->_getChallengeType());
-            if($challenge && $challenge->status == Response\Authorization\Struct\Challenge::STATUS_PENDING) {
+            if ($challenge && $challenge->status == Response\Authorization\Struct\Challenge::STATUS_PENDING) {
 
                 Utilities\Logger::getInstance()->add(
                     Utilities\Logger::LEVEL_DEBUG,
@@ -99,22 +99,22 @@ 
      */
     public function progress() {
 
-        if($this->_progressed) {
+        if ($this->_progressed) {
             return;
         }
 
         $this->_progressed = true;
 
-        if(!$this->_hasValidAuthorizationResponses())
+        if (!$this->_hasValidAuthorizationResponses())
             return;
 
         $existsNotValidChallenges = false;
 
-        foreach($this->_authorizationResponses as $authorizationResponse) {
+        foreach ($this->_authorizationResponses as $authorizationResponse) {
 
             $challenge = $authorizationResponse->getChallenge($this->_getChallengeType());
 
-            if($challenge && $this->_existsNotValidChallenges($challenge, $authorizationResponse)) {
+            if ($challenge && $this->_existsNotValidChallenges($challenge, $authorizationResponse)) {
                 $existsNotValidChallenges = true;
             }
         }
@@ -133,7 +133,7 @@ 
                                                  Response\Authorization\Get $authorizationResponse
     ) : bool {
 
-        if($challenge->status == Response\Authorization\Struct\Challenge::STATUS_PENDING) {
+        if ($challenge->status == Response\Authorization\Struct\Challenge::STATUS_PENDING) {
 
             Utilities\Logger::getInstance()->add(
                 Utilities\Logger::LEVEL_DEBUG,
@@ -143,15 +143,15 @@ 
 
             return true;
         }
-        else if($challenge->status == Response\Authorization\Struct\Challenge::STATUS_PROGRESSING) {
+        else if ($challenge->status == Response\Authorization\Struct\Challenge::STATUS_PROGRESSING) {
 
             // Should come back later
             return true;
         }
-        else if($challenge->status == Response\Authorization\Struct\Challenge::STATUS_VALID) {
+        else if ($challenge->status == Response\Authorization\Struct\Challenge::STATUS_VALID) {
 
         }
-        else if($challenge->status == Response\Authorization\Struct\Challenge::STATUS_INVALID) {
+        else if ($challenge->status == Response\Authorization\Struct\Challenge::STATUS_INVALID) {
             throw new Exception\AuthorizationInvalid(
                 'Received status "' . Response\Authorization\Struct\Challenge::STATUS_INVALID . '" while challenge should be verified'
             );

Braces +7 added lines, -10 removed lines

@@ -105,8 +105,9 @@ 
 
         $this->_progressed = true;
 
-        if(!$this->_hasValidAuthorizationResponses())
-            return;
+        if(!$this->_hasValidAuthorizationResponses()) {
+                    return;
+        }
 
         $existsNotValidChallenges = false;
 
@@ -142,21 +143,17 @@ 
             );
 
             return true;
-        }
-        else if($challenge->status == Response\Authorization\Struct\Challenge::STATUS_PROGRESSING) {
+        } else if($challenge->status == Response\Authorization\Struct\Challenge::STATUS_PROGRESSING) {
 
             // Should come back later
             return true;
-        }
-        else if($challenge->status == Response\Authorization\Struct\Challenge::STATUS_VALID) {
+        } else if($challenge->status == Response\Authorization\Struct\Challenge::STATUS_VALID) {
 
-        }
-        else if($challenge->status == Response\Authorization\Struct\Challenge::STATUS_INVALID) {
+        } else if($challenge->status == Response\Authorization\Struct\Challenge::STATUS_INVALID) {
             throw new Exception\AuthorizationInvalid(
                 'Received status "' . Response\Authorization\Struct\Challenge::STATUS_INVALID . '" while challenge should be verified'
             );
-        }
-        else {
+        } else {
 
             throw new \RuntimeException('Challenge status "' . $challenge->status . '" is not implemented');
         }

src/LE_ACME2/Authorizer/HTTP.php

Indentation +1 added lines, -1 removed lines

@@ -50,7 +50,7 @@
      * @throws Exception\RateLimitReached
      */
     protected function _existsNotValidChallenges(Response\Authorization\Struct\Challenge $challenge,
-                                                 Response\Authorization\Get $authorizationResponse
+                                                    Response\Authorization\Get $authorizationResponse
     ) : bool {
 
         Utilities\Logger::getInstance()->add(

Spacing +6 added lines, -6 removed lines

@@ -20,13 +20,13 @@ 
 
     public static function setDirectoryPath(string $directoryPath) {
 
-        if(!file_exists($directoryPath)) {
+        if (!file_exists($directoryPath)) {
             throw new \RuntimeException('HTTP authorization directory path does not exist');
         }
 
         self::$_directoryPath = realpath($directoryPath) . DIRECTORY_SEPARATOR;
 
-        if(!file_exists(self::$_directoryPath . 'test-token')) {
+        if (!file_exists(self::$_directoryPath . 'test-token')) {
             file_put_contents(self::$_directoryPath . self::TEST_TOKEN, self::TEST_CHALLENGE);
         }
     }
@@ -58,10 +58,10 @@ 
             'Challenge "' . $challenge->token . '" has status:' . $challenge->status
         );
 
-        if($challenge->status == Response\Authorization\Struct\Challenge::STATUS_PENDING) {
+        if ($challenge->status == Response\Authorization\Struct\Challenge::STATUS_PENDING) {
 
             $this->_writeToFile($challenge);
-            if($this->_validateFile($authorizationResponse->getIdentifier()->value, $challenge)) {
+            if ($this->_validateFile($authorizationResponse->getIdentifier()->value, $challenge)) {
 
                 $request = new Request\Authorization\Start($this->_account, $this->_order, $challenge);
                 /* $response = */ $request->getResponse();
@@ -71,7 +71,7 @@ 
             }
         }
 
-        if($challenge->status == Response\Authorization\Struct\Challenge::STATUS_INVALID) {
+        if ($challenge->status == Response\Authorization\Struct\Challenge::STATUS_INVALID) {
             throw new Exception\HTTPAuthorizationInvalid(
                 'Received status "' . Response\Authorization\Struct\Challenge::STATUS_INVALID . '" while challenge should be verified'
             );
@@ -102,7 +102,7 @@ 
         $expectedResponse = $challengeAuthorizationKey->get($challenge->token);
         $response = Utilities\ChallengeHTTP::fetch($domain, $challenge->token);
 
-        if($response != $expectedResponse) {
+        if ($response != $expectedResponse) {
 
             throw new Exception\HTTPAuthorizationInvalid(
                 'HTTP challenge for "' . $domain . '"": ' .

src/LE_ACME2/Order.php

Spacing +32 added lines, -32 removed lines

@@ -40,7 +40,7 @@ 
 
         array_map(function($subject) {
 
-            if(preg_match_all('~(\*\.)~', $subject) > 1)
+            if (preg_match_all('~(\*\.)~', $subject) > 1)
                 throw new \RuntimeException('Cannot create orders with multiple wildcards in one domain.');
 
         }, $subjects);
@@ -79,7 +79,7 @@ 
 
         Utilities\Logger::getInstance()->add(
             Utilities\Logger::LEVEL_INFO,
-            get_class() . '::' . __FUNCTION__ .  ' "' . implode(':', $subjects) . '"'
+            get_class() . '::' . __FUNCTION__ . ' "' . implode(':', $subjects) . '"'
         );
 
         $order = new self($account, $subjects);
@@ -104,7 +104,7 @@ 
             Cache\OrderResponse::getInstance()->set($this, $response);
             return $this;
 
-        } catch(Exception\AbstractException $e) {
+        } catch (Exception\AbstractException $e) {
             $this->_clearKeyDirectory();
             throw $e;
         }
@@ -120,12 +120,12 @@ 
 
         Utilities\Logger::getInstance()->add(
             Utilities\Logger::LEVEL_INFO,
-            get_class() . '::' . __FUNCTION__ .  ' "' . implode(':', $subjects) . '"'
+            get_class() . '::' . __FUNCTION__ . ' "' . implode(':', $subjects) . '"'
         );
 
         $order = new self($account, $subjects);
 
-        if(!self::exists($account, $subjects))
+        if (!self::exists($account, $subjects))
             throw new \RuntimeException('Order does not exist');
 
         return $order;
@@ -143,11 +143,11 @@ 
      */
     protected function _getAuthorizer(string $type) : Authorizer\AbstractAuthorizer {
 
-        if($this->_authorizer === null) {
+        if ($this->_authorizer === null) {
 
-            if($type == self::CHALLENGE_TYPE_HTTP) {
+            if ($type == self::CHALLENGE_TYPE_HTTP) {
                 $this->_authorizer = new Authorizer\HTTP($this->_account, $this);
-            } else if($type == self::CHALLENGE_TYPE_DNS) {
+            } else if ($type == self::CHALLENGE_TYPE_DNS) {
                 $this->_authorizer = new Authorizer\DNS($this->_account, $this);
             } else {
                 throw new \RuntimeException('Challenge type not implemented');
@@ -184,7 +184,7 @@ 
 
         try {
             return $this->_getAuthorizer($type)->shouldStartAuthorization();
-        } catch(Exception\ExpiredAuthorization $e) {
+        } catch (Exception\ExpiredAuthorization $e) {
 
             $this->_clearAfterExpiredAuthorization();
 
@@ -207,7 +207,7 @@ 
 
             return $authorizer->hasFinished();
 
-        } catch(Exception\ExpiredAuthorization $e) {
+        } catch (Exception\ExpiredAuthorization $e) {
 
             $this->_clearAfterExpiredAuthorization();
 
@@ -222,7 +222,7 @@ 
      */
     public function finalize() {
 
-        if(!is_object($this->_authorizer) || !$this->_authorizer->hasFinished()) {
+        if (!is_object($this->_authorizer) || !$this->_authorizer->hasFinished()) {
 
             throw new \RuntimeException('Not all challenges are valid. Please check result of authorize() first!');
         }
@@ -234,7 +234,7 @@ 
 
         $orderResponse = Cache\OrderResponse::getInstance()->get($this);
 
-        if(
+        if (
             $orderResponse->getStatus() == Response\Order\AbstractOrder::STATUS_PENDING /* DEPRECATED AFTER JULI 5TH 2018 */ ||
             $orderResponse->getStatus() == Response\Order\AbstractOrder::STATUS_READY   // ACME draft-12 Section 7.1.6
         ) {
@@ -245,7 +245,7 @@ 
             Cache\OrderResponse::getInstance()->set($this, $orderResponse);
         }
 
-        if($orderResponse->getStatus() == Response\Order\AbstractOrder::STATUS_VALID) {
+        if ($orderResponse->getStatus() == Response\Order\AbstractOrder::STATUS_VALID) {
 
             $request = new Request\Order\GetCertificate($this, $orderResponse);
             $response = $request->getResponse();
@@ -257,7 +257,7 @@ 
             //$certificateValidToTimeTimestamp = $certificateInfo['validTo_time_t'];
             $intermediateInfo = openssl_x509_parse($intermediate);
 
-            if(self::$_preferredChain !== null) {
+            if (self::$_preferredChain !== null) {
                 Utilities\Logger::getInstance()->add(
                     Utilities\Logger::LEVEL_INFO,
                     'Preferred chain is set: ' . self::$_preferredChain,
@@ -265,14 +265,14 @@ 
             }
 
             $found = false;
-            if(self::$_preferredChain !== null && $intermediateInfo['issuer']['CN'] != self::$_preferredChain) {
+            if (self::$_preferredChain !== null && $intermediateInfo['issuer']['CN'] != self::$_preferredChain) {
 
                 Utilities\Logger::getInstance()->add(
                     Utilities\Logger::LEVEL_INFO,
                     'Default certificate does not satisfy preferred chain, trying to fetch alternative'
                 );
 
-                foreach($response->getAlternativeLinks() as $link) {
+                foreach ($response->getAlternativeLinks() as $link) {
 
                     $request = new Request\Order\GetCertificate($this, $orderResponse, $link);
                     $response = $request->getResponse();
@@ -281,7 +281,7 @@ 
                     $alternativeIntermediate = $response->getIntermediate();
 
                     $intermediateInfo = openssl_x509_parse($intermediate);
-                    if($intermediateInfo['issuer']['CN'] != self::$_preferredChain) {
+                    if ($intermediateInfo['issuer']['CN'] != self::$_preferredChain) {
                         continue;
                     }
 
@@ -293,7 +293,7 @@ 
                     break;
                 }
 
-                if(!$found) {
+                if (!$found) {
                     Utilities\Logger::getInstance()->add(
                         Utilities\Logger::LEVEL_INFO,
                         'Preferred chain could not be satisfied, returning default chain'
@@ -325,8 +325,8 @@ 
     protected function _getLatestCertificateDirectory() : ?string {
 
         $files = scandir($this->getKeyDirectoryPath(), SORT_NUMERIC | SORT_DESC);
-        foreach($files as $file) {
-            if(
+        foreach ($files as $file) {
+            if (
                 substr($file, 0, strlen(self::BUNDLE_DIRECTORY_PREFIX)) == self::BUNDLE_DIRECTORY_PREFIX &&
                 is_dir($this->getKeyDirectoryPath() . $file)
             ) {
@@ -343,7 +343,7 @@ 
 
     public function getCertificateBundle() : Struct\CertificateBundle {
 
-        if(!$this->isCertificateBundleAvailable()) {
+        if (!$this->isCertificateBundleAvailable()) {
             throw new \RuntimeException('There is no certificate available');
         }
 
@@ -365,35 +365,35 @@ 
      */
     public function enableAutoRenewal($keyType = self::KEY_TYPE_RSA, int $renewBefore = null) {
 
-        if($keyType === null) {
+        if ($keyType === null) {
             $keyType = self::KEY_TYPE_RSA;
         }
 
-        if(!$this->isCertificateBundleAvailable()) {
+        if (!$this->isCertificateBundleAvailable()) {
             throw new \RuntimeException('There is no certificate available');
         }
 
         $orderResponse = Cache\OrderResponse::getInstance()->get($this);
-        if(
+        if (
             $orderResponse === null ||
             $orderResponse->getStatus() != Response\Order\AbstractOrder::STATUS_VALID
         ) {
             return;
         }
 
-        Utilities\Logger::getInstance()->add(Utilities\Logger::LEVEL_DEBUG,'Auto renewal triggered');
+        Utilities\Logger::getInstance()->add(Utilities\Logger::LEVEL_DEBUG, 'Auto renewal triggered');
 
         $directory = $this->_getLatestCertificateDirectory();
 
         $expireTime = self::_getExpireTimeFromCertificateDirectoryPath($directory);
 
-        if($renewBefore === null) {
+        if ($renewBefore === null) {
             $renewBefore = strtotime('-30 days', $expireTime);
         }
 
-        if($renewBefore < time()) {
+        if ($renewBefore < time()) {
 
-            Utilities\Logger::getInstance()->add(Utilities\Logger::LEVEL_INFO,'Auto renewal: Will recreate order');
+            Utilities\Logger::getInstance()->add(Utilities\Logger::LEVEL_INFO, 'Auto renewal: Will recreate order');
 
             $this->_create($keyType, true);
         }
@@ -407,7 +407,7 @@ 
      */
     public function revokeCertificate(int $reason = 0) : bool {
 
-        if(!$this->isCertificateBundleAvailable()) {
+        if (!$this->isCertificateBundleAvailable()) {
             throw new \RuntimeException('There is no certificate available to revoke');
         }
 
@@ -422,7 +422,7 @@ 
                 $this->_getKeyDirectoryPath('-revoked-' . microtime(true))
             );
             return true;
-        } catch(Exception\InvalidResponse $e) {
+        } catch (Exception\InvalidResponse $e) {
             return false;
         }
     }
@@ -430,12 +430,12 @@ 
     protected static function _getExpireTimeFromCertificateDirectoryPath(string $path) {
 
         $stringPosition = strrpos($path, self::BUNDLE_DIRECTORY_PREFIX);
-        if($stringPosition === false) {
+        if ($stringPosition === false) {
             throw new \RuntimeException('ExpireTime not found in' . $path);
         }
 
         $expireTime = substr($path, $stringPosition + strlen(self::BUNDLE_DIRECTORY_PREFIX));
-        if(
+        if (
             !is_numeric($expireTime) ||
             $expireTime < strtotime('-10 years') ||
             $expireTime > strtotime('+10 years')

Braces +6 added lines, -4 removed lines

@@ -40,8 +40,9 @@ 
 
         array_map(function($subject) {
 
-            if(preg_match_all('~(\*\.)~', $subject) > 1)
-                throw new \RuntimeException('Cannot create orders with multiple wildcards in one domain.');
+            if(preg_match_all('~(\*\.)~', $subject) > 1) {
+                            throw new \RuntimeException('Cannot create orders with multiple wildcards in one domain.');
+            }
 
         }, $subjects);
 
@@ -125,8 +126,9 @@ 
 
         $order = new self($account, $subjects);
 
-        if(!self::exists($account, $subjects))
-            throw new \RuntimeException('Order does not exist');
+        if(!self::exists($account, $subjects)) {
+                    throw new \RuntimeException('Order does not exist');
+        }
 
         return $order;
     }

src/LE_ACME2/Response/Order/AbstractOrder.php

Spacing +2 added lines, -2 removed lines

@@ -51,11 +51,11 @@
      */
     protected function _isValid(): bool {
 
-        if(!parent::_isValid()) {
+        if (!parent::_isValid()) {
             return false;
         }
 
-        if(
+        if (
             $this->getStatus() == AbstractOrder::STATUS_INVALID
         ) {
             throw new Exception\OrderStatusInvalid(