lifeboat-app / php-sdk

tests/services/ServicesTest.php

Spacing +1 added lines, -1 removed lines

@@ -148,7 +148,7 @@
         string $url,
         string $list_class = ListResource::class,
         array $params = []
-    ){
+    ) {
         $this->assertInstanceOf($list_class, $all);
         $this->assertEquals($url, $all->getURL());
         $this->assertEquals($this->getMockClient(), $all->getClient());

src/App.php

Spacing +2 added lines, -2 removed lines

@@ -25,7 +25,7 @@ 
     public function __construct(string $app_id, string $app_secret, $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);
@@ -157,7 +157,7 @@ 
 
         if (!$response->isValid() || !$json || !array_key_exists('access_token', $json)) {
             return '';
-        } else {
+        }else {
             $this->setAccessToken($json['access_token']);
             return $this->getAccessToken(false);
         }

Braces +6 added lines, -2 removed lines

@@ -109,7 +109,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] ?? '');
@@ -169,7 +171,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/utils/Curl.php

Spacing +3 added lines, -3 removed lines

@@ -21,7 +21,7 @@ 
     private $_method = 'GET';
     private $_url    = '';
     private $_data    = [];
-    private $_isfile   = false;
+    private $_isfile = false;
     private $_headers = [
         'Content-Type'      => 'application/x-www-form-urlencoded',
         'X-Requested-By'    => self::USER_AGENT
@@ -191,7 +191,7 @@ 
         }
 
         if ($this->_enable_cache && $this->getMethod() === 'GET') {
-            $cache_key = urlencode($request_uri) . implode(',', $send_headers);
+            $cache_key = urlencode($request_uri).implode(',', $send_headers);
             if (array_key_exists($cache_key, self::$_cache)) {
                 return self::$_cache[$cache_key];
             }
@@ -211,7 +211,7 @@ 
         curl_setopt($ch, CURLOPT_AUTOREFERER, true);
 
         if (!empty($send_headers))  curl_setopt($ch, CURLOPT_HTTPHEADER, $send_headers);
-        if (!is_null($post_data))   {
+        if (!is_null($post_data)) {
             curl_setopt($ch, CURLOPT_POST, 1);
             curl_setopt($ch, CURLOPT_POSTFIELDS, $post_data);
         }

Braces +27 added lines, -9 removed lines

@@ -39,8 +39,12 @@ 
     public function __construct(string $url, array $data = [], array $headers = [])
     {
         $this->setURL($url);
-        foreach ($data as $name => $value)      $this->addDataParam($name, $value);
-        foreach ($headers as $name => $value)   $this->addHeader($name, $value);
+        foreach ($data as $name => $value) {
+            $this->addDataParam($name, $value);
+        }
+        foreach ($headers as $name => $value) {
+            $this->addHeader($name, $value);
+        }
     }
 
     /**
@@ -99,7 +103,9 @@ 
 
     public function removeDataParam(string $name): Curl
     {
-        if (array_key_exists($name, $this->_data)) unset($this->_data[$name]);
+        if (array_key_exists($name, $this->_data)) {
+            unset($this->_data[$name]);
+        }
         return $this;
     }
 
@@ -128,7 +134,9 @@ 
      */
     public function removeHeader(string $name): Curl
     {
-        if (array_key_exists($name, $this->_headers)) unset($this->_headers[$name]);
+        if (array_key_exists($name, $this->_headers)) {
+            unset($this->_headers[$name]);
+        }
         return $this;
     }
 
@@ -173,13 +181,17 @@ 
         $request_uri    = $this->getURL();
 
         //  Headers
-        foreach ($this->getHeaders() as $k => $v) $send_headers[] = "{$k}: {$v}";
+        foreach ($this->getHeaders() as $k => $v) {
+            $send_headers[] = "{$k}: {$v}";
+        }
 
         // Request Data
         switch ($this->getMethod()) {
             case 'GET':
             case 'DELETE':
-                foreach ($this->getDataParams() as $name => $value) $request_uri = URL::setGetVar($name, $value, $request_uri);
+                foreach ($this->getDataParams() as $name => $value) {
+                    $request_uri = URL::setGetVar($name, $value, $request_uri);
+                }
                 break;
 
             case 'POST':
@@ -210,7 +222,9 @@ 
         curl_setopt($ch, CURLINFO_HEADER_OUT, true);
         curl_setopt($ch, CURLOPT_AUTOREFERER, true);
 
-        if (!empty($send_headers))  curl_setopt($ch, CURLOPT_HTTPHEADER, $send_headers);
+        if (!empty($send_headers)) {
+            curl_setopt($ch, CURLOPT_HTTPHEADER, $send_headers);
+        }
         if (!is_null($post_data))   {
             curl_setopt($ch, CURLOPT_POST, 1);
             curl_setopt($ch, CURLOPT_POSTFIELDS, $post_data);
@@ -219,10 +233,14 @@ 
         $result     = curl_exec($ch);
         $http_code  = curl_getinfo($ch, CURLINFO_HTTP_CODE);
 
-        if ($http_code === 401 && $retry_on_401) return $this->curl(false);
+        if ($http_code === 401 && $retry_on_401) {
+            return $this->curl(false);
+        }
 
         $response = new CurlResponse((int) $http_code, (string) $result);
-        if ($this->_enable_cache && isset($cache_key)) self::$_cache[$cache_key] = $response;
+        if ($this->_enable_cache && isset($cache_key)) {
+            self::$_cache[$cache_key] = $response;
+        }
 
         return $response;
     }

src/models/Order.php

Spacing +1 added lines, -1 removed lines

@@ -93,6 +93,6 @@
             default: return '';
         }
 
-        return ($value !== 0) ?  number_format($value, 2) . $this->Currency : '-';
+        return ($value !== 0) ?  number_format($value, 2).$this->Currency : '-';
     }
 }

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());
         }
     }