KriKrixs / PHP-BeatSaver-API

lib/functions/MultiQuery.php

Braces +11 added lines, -8 removed lines

@@ -41,8 +41,9 @@ 
             $l_Handles[] = $l_CURL;
         }
 
-        foreach ($l_Handles as $l_Current)
-            curl_multi_add_handle($l_Multi, $l_Current);
+        foreach ($l_Handles as $l_Current) {
+                    curl_multi_add_handle($l_Multi, $l_Current);
+        }
 
         $l_RunningHandles = null;
 
@@ -57,10 +58,11 @@ 
         {
             $l_Error = curl_error($l_Current);
 
-            if (!empty($l_Error) || curl_getinfo($l_Current, CURLINFO_HTTP_CODE) !== 200)
-                $l_Result[] = false;
-            else
-                $l_Result[] = new BeatMap(json_decode(curl_multi_getcontent($l_Current)));
+            if (!empty($l_Error) || curl_getinfo($l_Current, CURLINFO_HTTP_CODE) !== 200) {
+                            $l_Result[] = false;
+            } else {
+                            $l_Result[] = new BeatMap(json_decode(curl_multi_getcontent($l_Current)));
+            }
 
             curl_multi_remove_handle($l_Multi, $l_Current);
         }
@@ -87,8 +89,9 @@ 
                     mkdir($targetDir, 0777, true);
                 }
 
-                if(substr($targetDir, -1) !== "/")
-                    $targetDir .= "/";
+                if(substr($targetDir, -1) !== "/") {
+                                    $targetDir .= "/";
+                }
 
                 //The path & filename to save to.
                 $saveTo = $targetDir . $hash . $extension;

lib/BeatSaverAPI.php

Braces +60 added lines, -23 removed lines

@@ -26,8 +26,9 @@ 
      */
     public function __construct(string $userAgent, bool $needAutoloader = false)
     {
-        if($needAutoloader)
-            $this->autoload("./");
+        if($needAutoloader) {
+                    $this->autoload("./");
+        }
 
         $this->userAgent = $userAgent;
         $this->multiQuery = new MultiQuery(self::BEATSAVER_URL, $userAgent);
@@ -233,8 +234,9 @@ 
             }
         }
 
-        if(isset($mapsArray["errorStatus"]) && $mapsArray["errorStatus"])
-            $result->setErrorStatus( $mapsArray["errorStatus"])->setErrorMessage( $mapsArray["errorMessage"]);
+        if(isset($mapsArray["errorStatus"]) && $mapsArray["errorStatus"]) {
+                    $result->setErrorStatus( $mapsArray["errorStatus"])->setErrorMessage( $mapsArray["errorMessage"]);
+        }
 
         unset($mapsArray["errorStatus"]);
         unset($mapsArray["errorMessage"]);
@@ -280,8 +282,9 @@ 
                 if($apiResult === false || $apiResult == "Not Found") {
                     $response->setErrorStatus(true)->setErrorMessage("[getMaps] Something went wrong with the API call while calling page number " . $i . ".");
 
-                    if($apiResult == "Not Found")
-                        return $response;
+                    if($apiResult == "Not Found") {
+                                            return $response;
+                    }
                 }
 
                 foreach ($apiResult->docs as $beatmap) {
@@ -362,23 +365,57 @@ 
 
         $endpoint = "/search/text/page?sortOrder=" . $sort[$sortOrder];
 
-        if($mapName)                $endpoint .= "&q=" . urlencode($mapName);
-        if($startDate)              $endpoint .= "&from=" . $startDate->format("Y-m-d");
-        if($endDate)                $endpoint .= "&to=" . $endDate->format("Y-m-d");
-        if($ranked)                 $endpoint .= "&ranked=" . /** @scrutinizer ignore-type */ var_export($ranked, true);
-        if($automapper)             $endpoint .= "&automapper=" . /** @scrutinizer ignore-type */ var_export($automapper, true);
-        if($chroma)                 $endpoint .= "&chroma=" . /** @scrutinizer ignore-type */ var_export($chroma, true);
-        if($noodle)                 $endpoint .= "&noodle=" . /** @scrutinizer ignore-type */ var_export($noodle, true);
-        if($cinema)                 $endpoint .= "&cinema=" . /** @scrutinizer ignore-type */ var_export($cinema, true);
-        if($fullSpread)             $endpoint .= "&fullSpread=" . /** @scrutinizer ignore-type */ var_export($fullSpread, true);
-        if($minBpm)                 $endpoint .= "&minBpm=" . /** @scrutinizer ignore-type */ $minBpm;
-        if($maxBpm)                 $endpoint .= "&maxBpm=" . /** @scrutinizer ignore-type */ $maxBpm;
-        if($minNps)                 $endpoint .= "&minNps=" . /** @scrutinizer ignore-type */ $minNps;
-        if($maxNps)                 $endpoint .= "&maxNps=" . /** @scrutinizer ignore-type */ $maxNps;
-        if($minRating)              $endpoint .= "&minRating=" . /** @scrutinizer ignore-type */ $minRating;
-        if($maxRating)              $endpoint .= "&maxRating=" . /** @scrutinizer ignore-type */ $maxRating;
-        if($minDuration !== null)   $endpoint .= "&minDuration=" . /** @scrutinizer ignore-type */ $minDuration;
-        if($maxDuration !== null)   $endpoint .= "&maxDuration=" . /** @scrutinizer ignore-type */ $maxDuration;
+        if($mapName) {
+            $endpoint .= "&q=" . urlencode($mapName);
+        }
+        if($startDate) {
+            $endpoint .= "&from=" . $startDate->format("Y-m-d");
+        }
+        if($endDate) {
+            $endpoint .= "&to=" . $endDate->format("Y-m-d");
+        }
+        if($ranked) {
+            $endpoint .= "&ranked=" . /** @scrutinizer ignore-type */ var_export($ranked, true);
+        }
+        if($automapper) {
+            $endpoint .= "&automapper=" . /** @scrutinizer ignore-type */ var_export($automapper, true);
+        }
+        if($chroma) {
+            $endpoint .= "&chroma=" . /** @scrutinizer ignore-type */ var_export($chroma, true);
+        }
+        if($noodle) {
+            $endpoint .= "&noodle=" . /** @scrutinizer ignore-type */ var_export($noodle, true);
+        }
+        if($cinema) {
+            $endpoint .= "&cinema=" . /** @scrutinizer ignore-type */ var_export($cinema, true);
+        }
+        if($fullSpread) {
+            $endpoint .= "&fullSpread=" . /** @scrutinizer ignore-type */ var_export($fullSpread, true);
+        }
+        if($minBpm) {
+            $endpoint .= "&minBpm=" . /** @scrutinizer ignore-type */ $minBpm;
+        }
+        if($maxBpm) {
+            $endpoint .= "&maxBpm=" . /** @scrutinizer ignore-type */ $maxBpm;
+        }
+        if($minNps) {
+            $endpoint .= "&minNps=" . /** @scrutinizer ignore-type */ $minNps;
+        }
+        if($maxNps) {
+            $endpoint .= "&maxNps=" . /** @scrutinizer ignore-type */ $maxNps;
+        }
+        if($minRating) {
+            $endpoint .= "&minRating=" . /** @scrutinizer ignore-type */ $minRating;
+        }
+        if($maxRating) {
+            $endpoint .= "&maxRating=" . /** @scrutinizer ignore-type */ $maxRating;
+        }
+        if($minDuration !== null) {
+            $endpoint .= "&minDuration=" . /** @scrutinizer ignore-type */ $minDuration;
+        }
+        if($maxDuration !== null) {
+            $endpoint .= "&maxDuration=" . /** @scrutinizer ignore-type */ $maxDuration;
+        }
 
         return $this->getMapsByEndpoint($endpoint, $numberOfPage, $startPage);
     }