lifeboat-app / php-sdk

src/services/Orders.php

Spacing +2 added lines, -2 removed lines

@@ -51,7 +51,7 @@ 
     public function fetch(int $id): ?Order
     {
         /** @var Order|null $fetch */
-        $fetch = $this->_get('api/orders/order/' . $id);
+        $fetch = $this->_get('api/orders/order/'.$id);
         return $fetch;
     }
 
@@ -78,7 +78,7 @@ 
     public function update(int $id, array $data): ?Order
     {
         /** @var Order|null $post */
-        $post = $this->_post('api/orders/order/' . $id, $data);
+        $post = $this->_post('api/orders/order/'.$id, $data);
         return $post;
     }
 

src/models/Model.php

Spacing +6 added lines, -6 removed lines

@@ -35,19 +35,19 @@ 
                     // Model or Object
                     if (!array_key_exists('field_name', $value)) {
                         $this->$field = ObjectFactory::make($client, $value);
-                    } else {
+                    }else {
                         // Has One Relation - Deprecated
                         $name = $value['field_name'];
                         $this->$name    = $value['field_value'];
                         $this->$field   = ObjectFactory::make($client, $value['object_data']);
                     }
-                } else {
+                }else {
                     // HasMany / ManyMany Relation
                     $list = [];
                     foreach ($value as $item) {
                         if (ArrayLib::is_associative($item)) {
                             $list[] = ObjectFactory::make($client, $item);
-                        } else {
+                        }else {
                             $list[] = $item;
                         }
                     }
@@ -74,7 +74,7 @@ 
                 $data[$key] = [];
                 /** @var self $item */
                 foreach ($value as $item) $data[$key][] = $item->toArray(true);
-            } else $data[$key] = $value;
+            }else $data[$key] = $value;
         }
 
         return $data;
@@ -98,7 +98,7 @@ 
             if ($model === static::class) return new $service($this->getClient());
         }
 
-        throw new ErrorException("Could not determine which service to use for " . static::class);
+        throw new ErrorException("Could not determine which service to use for ".static::class);
     }
 
     /**
@@ -112,7 +112,7 @@ 
         if ($this->exists()) {
             if (!method_exists($service, 'update')) return $this;
             return $service->update($this->ID, $this->toArray());
-        } else {
+        }else {
             if (!method_exists($service, 'create')) return $this;
             return $service->create($this->toArray());
         }

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/services/TaxCodes.php

Spacing +2 added lines, -2 removed lines

@@ -22,7 +22,7 @@ 
     public function fetch(int $id): ?TaxCode
     {
         /** @var TaxCode|null $fetch */
-        $fetch = $this->_get('api/tax_code/' . $id);
+        $fetch = $this->_get('api/tax_code/'.$id);
         return $fetch;
     }
 
@@ -49,7 +49,7 @@ 
     public function update(int $id, array $data): ?TaxCode
     {
         /** @var TaxCode|null $post */
-        $post = $this->_post('api/tax_code/' . $id, $data);
+        $post = $this->_post('api/tax_code/'.$id, $data);
         return $post;
     }
 

src/services/Locations.php

Spacing +2 added lines, -2 removed lines

@@ -22,7 +22,7 @@ 
     public function fetch(int $id): ?Location
     {
         /** @var Location|null $fetch */
-        $fetch = $this->_get('api/locations/location/' . $id);
+        $fetch = $this->_get('api/locations/location/'.$id);
         return $fetch;
     }
 
@@ -49,7 +49,7 @@ 
     public function update(int $id, array $data): ?Location
     {
         /** @var Location|null $post */
-        $post = $this->_post('api/locations/location/' . $id, $data);
+        $post = $this->_post('api/locations/location/'.$id, $data);
         return $post;
     }
 

src/services/DeliveryZones.php

Spacing +2 added lines, -2 removed lines

@@ -22,7 +22,7 @@ 
     public function fetch(int $id): ?DeliveryZone
     {
         /** @var DeliveryZone|null $fetch */
-        $fetch = $this->_get('api/delivery-zones/zone/' . $id);
+        $fetch = $this->_get('api/delivery-zones/zone/'.$id);
         return $fetch;
     }
 
@@ -49,7 +49,7 @@ 
     public function update(int $id, array $data): ?DeliveryZone
     {
         /** @var DeliveryZone|null $post */
-        $post = $this->_post('api/delivery-zones/zone/' . $id, $data);
+        $post = $this->_post('api/delivery-zones/zone/'.$id, $data);
         return $post;
     }
 

src/services/Products.php

Spacing +4 added lines, -4 removed lines

@@ -39,7 +39,7 @@ 
     public function fetch(int $id): ?Product
     {
         /** @var Product|null $fetch */
-        $fetch = $this->_get('api/products/product/' . $id);
+        $fetch = $this->_get('api/products/product/'.$id);
         return $fetch;
     }
 
@@ -66,7 +66,7 @@ 
     public function update(int $id, array $data): ?Product
     {
         /** @var Product|null $post */
-        $post = $this->_post('api/products/product/' . $id, $data);
+        $post = $this->_post('api/products/product/'.$id, $data);
         return $post;
     }
 
@@ -78,7 +78,7 @@ 
      */
     public function delete(int $id): bool
     {
-        return $this->_delete('api/products/product/' . $id);
+        return $this->_delete('api/products/product/'.$id);
     }
 
     /**
@@ -91,7 +91,7 @@ 
      */
     public function setStockLevel(int $id, int $quantity, Location $location): bool
     {
-        return (bool) $this->_post('api/products/inventory/'. $id . '/quantity', [
+        return (bool) $this->_post('api/products/inventory/'.$id.'/quantity', [
             'location'  => $location->ID,
             'quantity'  => $quantity
         ]);

src/App.php

Spacing +4 added lines, -4 removed lines

@@ -14,7 +14,7 @@ 
  */
 class App extends Connector {
 
-    const CODE_URL      = '/oauth/code';
+    const CODE_URL = '/oauth/code';
 
     const ACCESS_TOKEN_PARAM    = 'lb_app_access_token';
     const API_CHALLENGE_PARAM   = 'lb_app_api_challenge';
@@ -31,7 +31,7 @@ 
     public function __construct(string $app_id, string $app_secret, string $auth_domain = self::AUTH_DOMAIN)
     {
         if (!$app_id || !$app_secret) {
-            throw new InvalidArgumentException(static::class . "expects an app_id and app_secret");
+            throw new InvalidArgumentException(static::class."expects an app_id and app_secret");
         }
 
         $this->setAppID($app_id);
@@ -164,7 +164,7 @@ 
         if (!$response->isValid() || !$json || !array_key_exists('access_token', $json)) {
             if (array_key_exists('error', $json)) throw new OAuthException($json['error']);
             return '';
-        } else {
+        }else {
             $this->setAccessToken($json['access_token']);
 
             if (array_key_exists('store_data', $json) &&
@@ -213,7 +213,7 @@ 
 
         if (!$response->isValid() || !$json || !array_key_exists('access_token', $json)) {
             throw new OAuthException($response->getError());
-        } else {
+        }else {
             $this->setAccessToken($json['access_token']);
         }
 

Braces +12 added lines, -4 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] ?? '');
@@ -140,7 +142,9 @@ 
         $url    = URL::setGetVar('process_url', urlencode($process_url), $url);
         $url    = URL::setGetVar('error_url', urlencode($error_url), $url);
 
-        if ($this->getSiteKey()) $url = URL::setGetVar('site_key', $this->getSiteKey(), $url);
+        if ($this->getSiteKey()) {
+            $url = URL::setGetVar('site_key', $this->getSiteKey(), $url);
+        }
 
         return URL::setGetVar('challenge', Utils::pack($challenge), $url);
     }
@@ -164,7 +168,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']);
@@ -186,7 +192,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(),

Spacing +3 added lines, -3 removed lines

@@ -2,7 +2,7 @@ 
 
 namespace Lifeboat;
 
-require_once __DIR__ . DIRECTORY_SEPARATOR . 'includes' . DIRECTORY_SEPARATOR . 'functions.php';
+require_once __DIR__.DIRECTORY_SEPARATOR.'includes'.DIRECTORY_SEPARATOR.'functions.php';
 
 use Lifeboat\Exceptions\BadMethodException;
 use Lifeboat\Exceptions\OAuthException;
@@ -142,7 +142,7 @@ 
     public function curl_api(string $url, string $method = 'GET', array $data = [], array $headers = []): CurlResponse
     {
         $uri = URL::is_absolute_url($url) ? $url
-            : 'https://' . rtrim($this->getHost(), '/') . '/' . ltrim($url, '/');
+            : 'https://'.rtrim($this->getHost(), '/').'/'.ltrim($url, '/');
 
         $curl = new Curl($uri, $data, $headers);
 
@@ -178,6 +178,6 @@ 
      */
     protected function auth_url(string $path): string
     {
-        return rtrim($this->_auth_domain, '/') . '/' . ltrim($path, '/');
+        return rtrim($this->_auth_domain, '/').'/'.ltrim($path, '/');
     }
 }

examples/Auth.php

Spacing +1 added lines, -1 removed lines

@@ -90,7 +90,7 @@
         $challenge = self::get_app()->getAPIChallenge();
 
         // Redirect to the auth URL
-        header("Location: " . self::get_app()->getAuthURL($process, $error, $challenge));
+        header("Location: ".self::get_app()->getAuthURL($process, $error, $challenge));
         flush();
         die();
     }