lifeboat-app / php-sdk

src/models/Model.php

Braces +21 added lines, -8 removed lines

@@ -64,17 +64,24 @@ 
      */
     public function toArray(bool $flatten = false): array
     {
-        if (!$flatten) return iterator_to_array($this->getIterator());
+        if (!$flatten) {
+            return iterator_to_array($this->getIterator());
+        }
 
         $data = [];
 
         foreach ($this->getIterator() as $key => $value) {
-            if ($value instanceof self)         $data[$key] = $value->toArray(true);
-            else if ($value instanceof ListResource) {
+            if ($value instanceof self) {
+                $data[$key] = $value->toArray(true);
+            } else if ($value instanceof ListResource) {
                 $data[$key] = [];
                 /** @var self $item */
-                foreach ($value as $item) $data[$key][] = $item->toArray(true);
-            } else $data[$key] = $value;
+                foreach ($value as $item) {
+                    $data[$key][] = $item->toArray(true);
+                }
+            } else {
+                $data[$key] = $value;
+            }
         }
 
         return $data;
@@ -95,7 +102,9 @@ 
     public function getService(): ApiService
     {
         foreach (ClassMap::SERVICE_MODEL as $service => $model) {
-            if ($model === static::class) return new $service($this->getClient());
+            if ($model === static::class) {
+                return new $service($this->getClient());
+            }
         }
 
         throw new ErrorException("Could not determine which service to use for " . static::class);
@@ -110,10 +119,14 @@ 
         $service = $this->getService();
 
         if ($this->exists()) {
-            if (!method_exists($service, 'update')) return $this;
+            if (!method_exists($service, 'update')) {
+                return $this;
+            }
             return $service->update($this->ID, $this->toArray());
         } else {
-            if (!method_exists($service, 'create')) return $this;
+            if (!method_exists($service, 'create')) {
+                return $this;
+            }
             return $service->create($this->toArray());
         }
     }

src/App.php

Braces +9 added lines, -3 removed lines

@@ -115,7 +115,9 @@ 
      */
     public function getAPIChallenge(bool $check_session = true): string
     {
-        if ($this->_api_challenge) return $this->_api_challenge;
+        if ($this->_api_challenge) {
+            return $this->_api_challenge;
+        }
 
         if ($check_session && session_status() === PHP_SESSION_ACTIVE) {
             $this->setAPIChallenge($_SESSION[self::API_CHALLENGE_PARAM] ?? '');
@@ -162,7 +164,9 @@ 
         $json = $response->getJSON();
         
         if (!$response->isValid() || !$json || !array_key_exists('access_token', $json)) {
-            if (array_key_exists('error', $json)) throw new OAuthException($json['error']);
+            if (array_key_exists('error', $json)) {
+                throw new OAuthException($json['error']);
+            }
             return '';
         } else {
             $this->setAccessToken($json['access_token']);
@@ -184,7 +188,9 @@ 
      */
     public function getAccessToken(bool $check_session = true): string
     {
-        if ($this->_access_token) return $this->_access_token;
+        if ($this->_access_token) {
+            return $this->_access_token;
+        }
 
         if ($check_session && session_status() === PHP_SESSION_ACTIVE) {
             $this->setAccessToken($_SESSION[self::ACCESS_TOKEN_PARAM] ?? '');

src/Connector.php

Braces +18 added lines, -6 removed lines

@@ -60,7 +60,9 @@ 
     public function __get(string $service): ?ApiService
     {
         $obj = ServiceFactory::inst($this, $service);
-        if (!$obj) throw new BadMethodException("Service for `{$service}` does not exist");
+        if (!$obj) {
+            throw new BadMethodException("Service for `{$service}` does not exist");
+        }
 
         return $obj;
     }
@@ -118,7 +120,9 @@ 
      */
     public function getHost(): ?string
     {
-        if (!$this->getActiveSite(true)) return null;
+        if (!$this->getActiveSite(true)) {
+            return null;
+        }
         return $this->_host;
     }
 
@@ -127,7 +131,9 @@ 
      */
     public function getSiteKey(): ?string
     {
-        if (!$this->getActiveSite(true)) return null;
+        if (!$this->getActiveSite(true)) {
+            return null;
+        }
         return $this->_site_key;
     }
 
@@ -151,7 +157,9 @@ 
         $curl->addHeader('Host', $this->getHost());
 
         foreach ($this->getAuthHeaders() as $header => $value) {
-            if ($value) $curl->addHeader($header, $value);
+            if ($value) {
+                $curl->addHeader($header, $value);
+            }
         }
 
         return $curl->curl_json();
@@ -163,8 +171,12 @@ 
      */
     public function getAuthHeaders(): array
     {
-        if (!$this->getAccessToken()) $this->fetchAccessToken();
-        if (!$this->getAccessToken()) throw new OAuthException("Access token has not been retreived");
+        if (!$this->getAccessToken()) {
+            $this->fetchAccessToken();
+        }
+        if (!$this->getAccessToken()) {
+            throw new OAuthException("Access token has not been retreived");
+        }
 
         return [
             'access-token'  => $this->getAccessToken(),