lifeboat-app / php-sdk

src/CurlResponse.php

Braces +3 added lines, -1 removed lines

@@ -40,7 +40,9 @@
     public function getJSON(): ?array
     {
         $data = json_decode($this->result, true);
-        if (!is_array($data)) return null;
+        if (!is_array($data)) {
+            return null;
+        }
         
         return $data;
     }

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/App.php

Braces +3 added lines, -1 removed lines

@@ -44,7 +44,9 @@
      */
     public function getAPIChallenge(): string
     {
-        if (!$this->_api_challenge) $this->_api_challenge = Utils::create_random_string(128);
+        if (!$this->_api_challenge) {
+            $this->_api_challenge = Utils::create_random_string(128);
+        }
         return $this->_api_challenge;
     }
 

Spacing +1 added lines, -1 removed lines

@@ -83,7 +83,7 @@
 
         if (!$response->isValid() || !$json || !array_key_exists('access_token', $json)) {
             return $json['access_token'];
-        } else {
+        }else {
             return '';
         }
     }

src/utils/Utils.php

Spacing +2 added lines, -2 removed lines

@@ -17,7 +17,7 @@ 
         $randomString = '';
 
         for ($i = 0; $i < $length; $i++) {
-            $randomString .= $characters[rand(0, $charactersLength - 1)];
+            $randomString .= $characters[rand(0, $charactersLength-1)];
         }
 
         return $randomString;
@@ -29,6 +29,6 @@ 
      */
     public static function pack(string $str): string
     {
-        return rtrim(strtr(base64_encode(pack('H*',  hash('sha256', $str))), '+/', '-_'), '=');
+        return rtrim(strtr(base64_encode(pack('H*', hash('sha256', $str))), '+/', '-_'), '=');
     }
 }

src/utils/Curl.php

Spacing +2 added lines, -2 removed lines

@@ -188,7 +188,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];
             }
@@ -208,7 +208,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,9 +39,15 @@ 
      */
     public function __construct(string $url = null, array $data = [], array $headers = [])
     {
-        if (!is_null($url)) $this->setURL($url);
-        foreach ($data as $name => $value)      $this->addDataParam($name, $value);
-        foreach ($headers as $name => $value)   $this->addHeader($name, $value);
+        if (!is_null($url)) {
+            $this->setURL($url);
+        }
+        foreach ($data as $name => $value) {
+            $this->addDataParam($name, $value);
+        }
+        foreach ($headers as $name => $value) {
+            $this->addHeader($name, $value);
+        }
     }
 
     /**
@@ -98,7 +104,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;
     }
 
@@ -127,7 +135,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;
     }
 
@@ -171,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':
@@ -208,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);
@@ -218,7 +234,9 @@ 
         $http_code  = curl_getinfo($ch, CURLINFO_HTTP_CODE);
 
         $response = new CurlResponse($http_code, $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/resource/ListResource.php

Braces +15 added lines, -5 removed lines

@@ -107,11 +107,15 @@ 
 
             $this->_max_items = (int) $data['available_items'];
 
-            if (empty($data['items'])) return $this->_items[$page] = [];
+            if (empty($data['items'])) {
+                return $this->_items[$page] = [];
+            }
 
             foreach ($data['items'] as $item) {
                 $obj = ObjectFactory::make($this->getClient(), $item);
-                if (!$obj) continue;
+                if (!$obj) {
+                    continue;
+                }
 
                 $this->_items[$page][] = $obj;
             }
@@ -126,7 +130,9 @@ 
      */
     public function offsetGet($offset): ?ObjectResource
     {
-        foreach ($this as $i => $obj) if ($i === $offset) return $obj;
+        foreach ($this as $i => $obj) {
+            if ($i === $offset) return $obj;
+        }
         return null;
     }
 
@@ -157,7 +163,9 @@ 
      */
     public function offsetExists($offset): bool
     {
-        foreach ($this as $i => $obj) if ($offset === $i) return true;
+        foreach ($this as $i => $obj) {
+            if ($offset === $i) return true;
+        }
         return false;
     }
 
@@ -213,6 +221,8 @@ 
      */
     public function first(): ?Model
     {
-        foreach ($this as $obj) return $obj;
+        foreach ($this as $obj) {
+            return $obj;
+        }
     }
 }

Spacing +4 added lines, -4 removed lines

@@ -212,20 +212,20 @@
         $t = $this->getItems(1);
         $c = $this->count();
 
-        return (function () use ($t, $c) {
+        return (function() use ($t, $c) {
             $i = 0;
             $x = 0;
 
             while ($i < $c) {
                 if ($x === 0) {
                     $x = $this->_page_length;
-                    $t = $this->getItems((int) floor($i / $this->_page_length) + 1);
+                    $t = $this->getItems((int) floor($i / $this->_page_length)+1);
                 }
 
-                yield $i => $t[$this->_page_length - $x];
+                yield $i => $t[$this->_page_length-$x];
 
                 $x -= 1;
-                $i ++;
+                $i++;
             }
         })();
     }

src/Connector.php

Spacing +2 added lines, -2 removed lines

@@ -107,7 +107,7 @@ 
     public function curl_api(string $url, string $method = 'GET', array $data = [], array $headers = []): CurlResponse
     {
         $url = URL::is_absolute_url($url) ? $url
-            : 'https://' . rtrim($this->_host, '/') . '/' . ltrim($url, '/');
+            : 'https://'.rtrim($this->_host, '/').'/'.ltrim($url, '/');
 
         $curl = new Curl($url, $data, $headers);
 
@@ -126,6 +126,6 @@ 
      */
     protected function auth_url(string $path): string
     {
-        return $this->_auth_domain . '/' . ltrim($path, '/');
+        return $this->_auth_domain.'/'.ltrim($path, '/');
     }
 }

Braces +3 added lines, -1 removed lines

@@ -45,7 +45,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;
     }

src/factory/ServiceFactory.php

Braces +3 added lines, -1 removed lines

@@ -19,7 +19,9 @@
     public static function inst(Connector $connector, string $service): ?ApiService
     {
         $service = strtolower($service);
-        if (!array_key_exists($service, ClassMap::SERVICES)) return null;
+        if (!array_key_exists($service, ClassMap::SERVICES)) {
+            return null;
+        }
 
         $cls = ClassMap::SERVICES[$service];
         return new $cls($connector);

src/services/ApiService.php

Braces +3 added lines, -1 removed lines

@@ -98,7 +98,9 @@
     {
         $curl = $this->getClient()->curl_api($url, 'DELETE', $params);
 
-        if ($curl->isValid()) return true;
+        if ($curl->isValid()) {
+            return true;
+        }
 
         throw new ApiException($curl->getError());
     }