lifeboat-app / php-sdk

src/utils/Curl.php

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

examples/ExampleController.php

Braces +3 added lines, -1 removed lines

@@ -115,7 +115,9 @@
             $sites = $this->app->getSites();
 
             // If the user doesn't have access to any Lifeboat site force a re-auth
-            if (count($sites) < 1) return $this->reloadAuth();
+            if (count($sites) < 1) {
+                return $this->reloadAuth();
+            }
 
             // If the user only has access to one Lifeboat site then automatically use that
             if (count($sites) === 1) {

src/App.php

Braces +9 added lines, -3 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] ?? '');
@@ -162,7 +164,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']);
@@ -184,7 +188,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(),