lifeboat-app / php-sdk

src/services/Orders.php

Spacing +2 added lines, -2 removed lines

@@ -43,7 +43,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;
     }
 
@@ -70,7 +70,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/Address.php

Braces +3 added lines, -1 removed lines

@@ -59,7 +59,9 @@
      */
     public function Customer(): ?Customer
     {
-        if (!$this->CustomerID) return null;
+        if (!$this->CustomerID) {
+            return null;
+        }
 
         /** @var Customer|null $customer */
         $customer = $this->getClient()->customers->fetch($this->CustomerID);

src/resource/SimpleList.php

Spacing +1 added lines, -1 removed lines

@@ -62,7 +62,7 @@
         $t = $this->getItems();
         $c = $this->count();
 
-        return (function () use ($t, $c) {
+        return (function() use ($t, $c) {
             $i = 0;
 
             while ($i < $c) {

Braces +6 added lines, -2 removed lines

@@ -42,7 +42,9 @@ 
 
             foreach ($list as $item) {
                 $obj = ObjectFactory::make($this->getClient(), $item);
-                if (!$obj) continue;
+                if (!$obj) {
+                    continue;
+                }
 
                 $this->_items[] = $obj;
             }
@@ -76,6 +78,8 @@ 
      */
     public function first(): ?Model
     {
-        foreach ($this as $obj) return $obj;
+        foreach ($this as $obj) {
+            return $obj;
+        }
     }
 }

src/services/CustomPages.php

Spacing +2 added lines, -2 removed lines

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

src/services/Collections.php

Spacing +3 added lines, -3 removed lines

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

src/factory/ObjectFactory.php

Braces +6 added lines, -2 removed lines

@@ -21,7 +21,9 @@ 
     public static function create(Connector $connector, string $model, array $data = []): Model
     {
         $model = strtolower($model);
-        if (!array_key_exists($model, ClassMap::MODELS)) return new LifeboatModel($connector, $data);
+        if (!array_key_exists($model, ClassMap::MODELS)) {
+            return new LifeboatModel($connector, $data);
+        }
 
         $cls = ClassMap::MODELS[$model];
         return new $cls($connector, $data);
@@ -35,7 +37,9 @@ 
     public static function make(Connector $connector, array $data): Model
     {
         $model = $data['model'] ?? '';
-        if (!$model) return new LifeboatModel($connector, $data);
+        if (!$model) {
+            return new LifeboatModel($connector, $data);
+        }
         return self::create($connector, $model, $data);
     }
 }

src/Client.php

Spacing +2 added lines, -2 removed lines

@@ -21,11 +21,11 @@
     public function __construct(string $_api_key, string $_api_secret, $_auth_domain = self::AUTH_DOMAIN)
     {
         if (!$_api_key) {
-            throw new InvalidArgumentException(static::class . ' parameter 1 should be your api key');
+            throw new InvalidArgumentException(static::class.' parameter 1 should be your api key');
         }
 
         if (!$_api_secret) {
-            throw new InvalidArgumentException(static::class . ' parameter 2 should be your api secret');
+            throw new InvalidArgumentException(static::class.' parameter 2 should be your api secret');
         }
 
         $this->_api_key = $_api_key;

src/models/Model.php

Spacing +5 added lines, -5 removed lines

@@ -34,19 +34,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;
                         }
                     }
@@ -83,7 +83,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);
     }
 
     /**
@@ -97,7 +97,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 +9 added lines, -3 removed lines

@@ -80,7 +80,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);
@@ -95,10 +97,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());
         }
     }

tests/AppTest.php

Spacing +4 added lines, -4 removed lines

@@ -54,10 +54,10 @@
     {
         $app    = new App('id', 'secret', 'http://test.domain');
         $url    = $app->getAuthURL('process.php', 'error.php', 'challenge');
-        $expect = 'http://test.domain/oauth/code?app_id=id&process_url=' .
-            urlencode('process.php') .
-            '&error_url=' . urlencode('error.php') .
-            '&challenge=' . Utils::pack('challenge');
+        $expect = 'http://test.domain/oauth/code?app_id=id&process_url='.
+            urlencode('process.php').
+            '&error_url='.urlencode('error.php').
+            '&challenge='.Utils::pack('challenge');
 
         $this->assertEquals($expect, $url);
     }