Maelstromeous / ps2alerts-api

src/Controller/Endpoint/Leaderboards/LeaderboardPlayerEndpointController.php

Doc Comments +1 added lines, -1 removed lines

@@ -32,7 +32,7 @@
     /**
      * Get Player Leaderboard
      *
-     * @return \League\Fractal\Manager
+     * @return \Psr\Http\Message\ResponseInterface
      */
     public function players()
     {

Spacing +2 added lines, -2 removed lines

@@ -52,7 +52,7 @@ 
             $field = $this->getField($_GET['field']);
         }
 
-        if (! isset($field)) {
+        if (!isset($field)) {
             return $this->respondWithError('Field wasn\'t provided and is required.', self::CODE_WRONG_ARGS);
         }
 
@@ -81,7 +81,7 @@ 
 
         // Gets outfit details
         for ($i = 0; $i < $count; $i++) {
-            if (! empty($players[$i]['playerOutfit'])) {
+            if (!empty($players[$i]['playerOutfit'])) {
                 // Gets outfit details
                 try {
                     $outfit = $this->dataEndpoint->getOutfit($players[$i]['playerOutfit']);

public/index.php

Spacing +5 added lines, -5 removed lines

@@ -1,12 +1,12 @@ 
 <?php
 
-include __DIR__ . '/../vendor/autoload.php';
+include __DIR__.'/../vendor/autoload.php';
 
 use League\Route\Http\Exception\NotFoundException;
 
 // ENV loading
 josegonzalez\Dotenv\Loader::load([
-    'filepath' => __DIR__ . '/../.env',
+    'filepath' => __DIR__.'/../.env',
     'toEnv'    => true
 ]);
 
@@ -21,10 +21,10 @@ 
 }
 
 // Container
-$container = include __DIR__ . '/../src/container.php';
+$container = include __DIR__.'/../src/container.php';
 
 // Routes
-$router = include __DIR__ . '/../src/routes.php';
+$router = include __DIR__.'/../src/routes.php';
 
 // FIRE!!!
 try {
@@ -49,7 +49,7 @@ 
     if ($_ENV['ENV'] === 'development') {
         trigger_error($e->getMessage());
     } else {
-        $logger->addError(":warning: Exception IN API: \n\n" . $e->getMessage());
+        $logger->addError(":warning: Exception IN API: \n\n".$e->getMessage());
         $response = $container->get('Zend\Diactoros\Response');
 
         $response->getBody()->write(

src/Controller/Endpoint/Leaderboards/LeaderboardWeaponEndpointController.php

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
             $field = $this->getField($_GET['field']);
         }
 
-        if (! isset($field)) {
+        if (!isset($field)) {
             return $this->respondWithError('Field wasn\'t provided and is required.', self::CODE_WRONG_ARGS);
         }
 

src/Controller/Endpoint/Leaderboards/AbstractLeaderboardEndpointController.php

Spacing +8 added lines, -8 removed lines

@@ -18,19 +18,19 @@ 
     public function validateRequestVars()
     {
         try {
-            if (! empty($_GET['field'])) {
+            if (!empty($_GET['field'])) {
                 $this->parseField($_GET['field']);
             }
 
-            if (! empty($_GET['server'])) {
+            if (!empty($_GET['server'])) {
                 $this->parseServer($_GET['server']);
             }
 
-            if (! empty($_GET['limit'])) {
+            if (!empty($_GET['limit'])) {
                 $this->parseOffset($_GET['limit']);
             }
 
-            if (! empty($_GET['offset'])) {
+            if (!empty($_GET['offset'])) {
                 $this->parseOffset($_GET['offset']);
             }
         } catch (InvalidArgumentException $e) {
@@ -56,7 +56,7 @@ 
             'captures'
         ];
 
-        if (! empty($field) && in_array($field, $validFields)) {
+        if (!empty($field) && in_array($field, $validFields)) {
             return $field;
         }
 
@@ -77,7 +77,7 @@ 
             unset($validServers[$key]);
         }
 
-        if (! empty($server) && in_array($server, $validServers)) {
+        if (!empty($server) && in_array($server, $validServers)) {
             return $server;
         }
 
@@ -91,7 +91,7 @@ 
      */
     public function parseLimit($limit)
     {
-        if (! isset($limit) && ! is_numeric($limit)) {
+        if (!isset($limit) && !is_numeric($limit)) {
             throw new InvalidArgumentException("Limit needs to be in numerical format.");
         }
 
@@ -103,7 +103,7 @@ 
      */
     public function parseOffset($offset)
     {
-        if (! isset($offset) && ! is_numeric($offset)) {
+        if (!isset($offset) && !is_numeric($offset)) {
             throw new InvalidArgumentException("Offset needs to be in numerical format.");
         }
 

src/Controller/Endpoint/Alerts/AlertEndpointController.php

Spacing +4 added lines, -4 removed lines

@@ -93,16 +93,16 @@ 
         $limit  = (int) $_GET['limit'];
 
         // Set defaults if not supplied
-        if (empty($offset) || ! is_numeric($offset)) {
+        if (empty($offset) || !is_numeric($offset)) {
             $offset = 0;
         }
 
-        if (empty($limit) || ! is_numeric($limit)) {
+        if (empty($limit) || !is_numeric($limit)) {
             $limit = 50;
         }
 
         // Check the date difference between two dates. we don't want to run queries for ALL OF ZE ALERTS NOW do we?!
-        if (! empty($dates)) {
+        if (!empty($dates)) {
             try {
                 $this->getDateValidationUtility()->validateTimeDifference($dates, 180); // Allow half a year
             } catch (InvalidArgumentException $e) {
@@ -123,7 +123,7 @@ 
         $query->where('ResultWinner IN (?)', $this->convertStringToArrayForAuraBinds($factions));
         $query->where('ResultTimeType IN (?)', $brackets);
 
-        if (! empty($dates)) {
+        if (!empty($dates)) {
             $this->addDateRangeWhereClause($dates, $query);
         }
 

src/Controller/Endpoint/Alerts/AlertCountsEndpointController.php

Spacing +4 added lines, -4 removed lines

@@ -72,7 +72,7 @@ 
 
             $query->cols([$sql]);
 
-            if (! empty($dates)) {
+            if (!empty($dates)) {
                 $this->addDateRangeWhereClause($dates, $query, true);
             }
 
@@ -178,7 +178,7 @@ 
         $query->cols([$sql]);
 
         $query->where('ResultDateTime IS NOT NULL');
-        if (! empty($dates)) {
+        if (!empty($dates)) {
             $this->addDateRangeWhereClause($dates, $query, true);
         }
         $query->groupBy(['dateIndex']);
@@ -203,11 +203,11 @@ 
             $factionAbv = strtoupper($faction);
 
             $sql .= "SUM(CASE WHEN `ResultWinner` = '{$factionAbv}' ";
-            if (! empty($server)) {
+            if (!empty($server)) {
                 $sql .= "AND `ResultServer` IN ({$server}) ";
             }
 
-            if (! empty($zones)) {
+            if (!empty($zones)) {
                 $sql .= "AND `ResultAlertCont` IN ({$zones}) ";
             }
 

src/Controller/Endpoint/Search/SearchEndpointController.php

Spacing +3 added lines, -3 removed lines

@@ -47,7 +47,7 @@ 
         $this->parsePlayerName($args['term']);
         $players = $this->searchForPlayer($args['term']);
 
-        if (! empty($players)) {
+        if (!empty($players)) {
             return $this->respond(
                 'collection',
                 $players,
@@ -73,7 +73,7 @@ 
 
         $this->parseOutfitName($name);
 
-        if (! empty($outfits)) {
+        if (!empty($outfits)) {
             return $this->respond(
                 'collection',
                 $outfits,
@@ -186,7 +186,7 @@ 
             return $this->respondWithError('Player ID cannot be longer than 19 characters.', self::CODE_WRONG_ARGS);
         }
 
-        if (! is_numeric($id)) {
+        if (!is_numeric($id)) {
             return $this->respondWithError('Player ID must be numeric.', self::CODE_WRONG_ARGS);
         }
 

src/Controller/Endpoint/Data/DataEndpointController.php

Spacing +4 added lines, -4 removed lines

@@ -75,7 +75,7 @@ 
         }
 
         // Now return the character the outfit injected
-        if (! empty($character['data']['outfit'])) {
+        if (!empty($character['data']['outfit'])) {
             try {
                 $outfit = $this->getOutfit($character['data']['outfit']);
             } catch (CensusErrorException $e) {
@@ -137,7 +137,7 @@ 
         // First, check if we have the character in Redis
         $redisCheck = $this->getRedisUtility()->checkRedis('cache', 'character', $id);
 
-        if (! empty($redisCheck)) {
+        if (!empty($redisCheck)) {
             return $redisCheck;
         }
 
@@ -186,7 +186,7 @@ 
         // First, check if we have the outfit in Redis
         $redisCheck = $this->getRedisUtility()->checkRedis('cache', 'outfit', $id);
 
-        if (! empty($redisCheck)) {
+        if (!empty($redisCheck)) {
             return $redisCheck;
         }
 
@@ -245,7 +245,7 @@ 
         ];
 
         // Loop through each environment and get the first result
-        foreach($environments as $env) {
+        foreach ($environments as $env) {
             $url = "https://census.daybreakgames.com/s:{$config['census_service_id']}/get/{$env}/{$endpoint}";
 
             $req  = $guzzle->request('GET', $url);

src/Command/DeleteAlertCommand.php

Upper-Lower-Casing +2 added lines, -2 removed lines

@@ -61,7 +61,7 @@ 
         $alert = $this->alertRepo->readSingleById($id, 'primary', true);
 
         if (empty($force) && empty($alert)) {
-            $output->writeln("ALERT {$id} DOES NOT EXIST!");
+            $output->writeln("alert {$id} DOES NOT EXIST!");
             return false;
         }
 
@@ -106,7 +106,7 @@ 
         $this->deleteAlert($id);
 
         if ($this->verbose === 1) {
-            $output->writeln("Alert {$id} successfully deleted!");
+            $output->writeln("alert {$id} successfully deleted!");
         }
 
         return true;