lifeboat-app / php-sdk

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

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

src/App.php

Spacing +2 added lines, -2 removed lines

@@ -22,7 +22,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);
@@ -128,7 +128,7 @@ 
 
         if (!$response->isValid() || !$json || !array_key_exists('access_token', $json)) {
             return $json['access_token'];
-        } else {
+        }else {
             return '';
         }
     }

src/includes/functions.php

Spacing +1 added lines, -1 removed lines

@@ -9,7 +9,7 @@
     if (!$date) return null;
 
     try {
-        $date = new \DateTime($date . ' CET');
+        $date = new \DateTime($date.' CET');
         return $date;
     } catch (Exception $e) {
         return null;

tests/factory/ObjectFactoryTest.php

Spacing +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
      */
     public function testFactory()
     {
-        $mock_data  = ['ID' => 0];
+        $mock_data = ['ID' => 0];
 
         foreach (ClassMap::MODELS as $name => $class) {
             $this->assertInstanceOf($class, ObjectFactory::create($this->getMockClient(), $name, $mock_data));

tests/UtilsTest.php

Spacing +6 added lines, -6 removed lines

@@ -21,7 +21,7 @@ 
     public function test_is_associative_array()
     {
         $this->assertEquals(true, ArrayLib::is_associative(['a' => 1]));
-        $this->assertEquals(false, ArrayLib::is_associative([1,2,3]));
+        $this->assertEquals(false, ArrayLib::is_associative([1, 2, 3]));
         $this->assertEquals(false, ArrayLib::is_associative([]));
 
         try {
@@ -42,16 +42,16 @@ 
     public function test_set_get_var()
     {
         $rel_path = '/path';
-        $this->assertEquals($rel_path . '?var=1', URL::setGetVar('var', 1, $rel_path));
+        $this->assertEquals($rel_path.'?var=1', URL::setGetVar('var', 1, $rel_path));
 
         $rel_path_t = '/path/';
-        $this->assertEquals($rel_path_t . '?var=1', URL::setGetVar('var', 1, $rel_path_t));
+        $this->assertEquals($rel_path_t.'?var=1', URL::setGetVar('var', 1, $rel_path_t));
 
         $absolute = 'https://test.com';
-        $this->assertEquals($absolute . '?var=1', URL::setGetVar('var', 1, $absolute));
+        $this->assertEquals($absolute.'?var=1', URL::setGetVar('var', 1, $absolute));
 
         $absolute_t = 'https://test.com/';
-        $this->assertEquals($absolute_t . '?var=1', URL::setGetVar('var', 1, $absolute_t));
+        $this->assertEquals($absolute_t.'?var=1', URL::setGetVar('var', 1, $absolute_t));
     }
 
     /**
@@ -71,7 +71,7 @@ 
         while (count($generated) < 100000) $generated[] = Utils::create_random_string();
         $count  = count($generated);
         $unique = count(array_unique($generated));
-        $random = 100 - ((100 / $count) * $unique);
+        $random = 100-((100 / $count) * $unique);
 
         // Less than 2% repeat
         $this->assertTrue($random < 2);

src/utils/URL.php

Spacing +8 added lines, -8 removed lines

@@ -20,13 +20,13 @@ 
     public static function setGetVar(string $key, string $value, string $url): string
     {
         if (!self::is_absolute_url($url)) {
-            $url = 'http://dummy.com/' . ltrim($url, '/');
+            $url = 'http://dummy.com/'.ltrim($url, '/');
         }
 
         // try to parse uri
         $parts = parse_url($url);
         if (empty($parts)) {
-            throw new InvalidArgumentException("Can't parse URL: " . $url);
+            throw new InvalidArgumentException("Can't parse URL: ".$url);
         }
 
         // Parse params and add new variable
@@ -40,18 +40,18 @@ 
         // Generate URI segments and formatting
         $scheme = (array_key_exists('scheme', $parts)) ? $parts['scheme'] : 'http';
         $user = (array_key_exists('user', $parts)) ? $parts['user'] : '';
-        $port = (array_key_exists('port', $parts) && $parts['port']) ? ':' . $parts['port'] : '';
+        $port = (array_key_exists('port', $parts) && $parts['port']) ? ':'.$parts['port'] : '';
 
         if ($user != '') {
             // format in either user:pass@host.com or user@host.com
-            $user .= (array_key_exists('pass', $parts) && $parts['pass']) ? ':' . $parts['pass'] . '@' : '';
+            $user .= (array_key_exists('pass', $parts) && $parts['pass']) ? ':'.$parts['pass'].'@' : '';
         }
 
         // handle URL params which are existing / new
-        $params = ($params) ? '?' . http_build_query($params) : '';
+        $params = ($params) ? '?'.http_build_query($params) : '';
 
         // Recompile URI segments
-        $newUri = $scheme . '://' . $user . $parts['host'] . $port . ($parts['path'] ?? '') . $params;
+        $newUri = $scheme.'://'.$user.$parts['host'].$port.($parts['path'] ?? '').$params;
 
         return str_replace('http://dummy.com', '', $newUri);
     }
@@ -64,10 +64,10 @@ 
     {
         // Strip off the query and fragment parts of the URL before checking
         if (($queryPosition = strpos($url, '?')) !== false) {
-            $url = substr($url, 0, $queryPosition - 1);
+            $url = substr($url, 0, $queryPosition-1);
         }
         if (($hashPosition = strpos($url, '#')) !== false) {
-            $url = substr($url, 0, $hashPosition - 1);
+            $url = substr($url, 0, $hashPosition-1);
         }
         $colonPosition = strpos($url, ':');
         $slashPosition = strpos($url, '/');

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

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