Maelstromeous / ps2alerts-api

src/ServiceProvider/DatabaseServiceProvider.php

Spacing +4 added lines, -4 removed lines

@@ -23,7 +23,7 @@ 
      */
     public function register()
     {
-        $this->getContainer()->share('Database', function () {
+        $this->getContainer()->share('Database', function() {
             $config = $this->getContainer()->get('config')['database'];
 
             $pdo = new ExtendedPdo(
@@ -35,7 +35,7 @@ 
             return $pdo;
         });
 
-        $this->getContainer()->share('Database\Data', function () {
+        $this->getContainer()->share('Database\Data', function() {
             $config = $this->getContainer()->get('config')['database_data'];
 
             $pdo = new ExtendedPdo(
@@ -47,7 +47,7 @@ 
             return $pdo;
         });
 
-        $this->getContainer()->share('Database\Archive', function () {
+        $this->getContainer()->share('Database\Archive', function() {
             $config = $this->getContainer()->get('config')['database_archive'];
 
             $pdo = new ExtendedPdo(
@@ -59,7 +59,7 @@ 
             return $pdo;
         });
 
-        $this->getContainer()->add('Aura\SqlQuery\QueryFactory', function () {
+        $this->getContainer()->add('Aura\SqlQuery\QueryFactory', function() {
             return new QueryFactory('mysql');
         });
     }

src/ServiceProvider/RedisServiceProvider.php

Spacing +2 added lines, -2 removed lines

@@ -23,7 +23,7 @@ 
      */
     public function register()
     {
-        $this->getContainer()->add('redis', function () {
+        $this->getContainer()->add('redis', function() {
             $redisConfig = $this->getContainer()->get('config')['redis'];
 
             $args = [
@@ -33,7 +33,7 @@ 
                 'scheme'   => 'tcp',
             ];
 
-            if (! empty($redisConfig['pass'])) {
+            if (!empty($redisConfig['pass'])) {
                 $args['password'] = $redisConfig['pass'];
             }
 

src/ServiceProvider/FractalServiceProvider.php

Spacing +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
      */
     public function register()
     {
-        $this->getContainer()->share('League\Fractal\Manager', function () {
+        $this->getContainer()->share('League\Fractal\Manager', function() {
             return new Manager;
         });
     }

src/ServiceProvider/LogServiceProvider.php

Spacing +2 added lines, -2 removed lines

@@ -21,7 +21,7 @@ 
      */
     public function register()
     {
-        $this->getContainer()->share('Monolog\Logger', function () {
+        $this->getContainer()->share('Monolog\Logger', function() {
             $log = new Logger('app');
 
             $config = $this->getContainer()->get('config');
@@ -39,7 +39,7 @@ 
             }
 
             $log->pushHandler(
-                new StreamHandler(__DIR__ . '/../../logs/app.log', Logger::DEBUG)
+                new StreamHandler(__DIR__.'/../../logs/app.log', Logger::DEBUG)
             );
 
             return $log;

src/ServiceProvider/HttpClientServiceProvider.php

Spacing +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
      */
     public function register()
     {
-        $this->getContainer()->add('GuzzleHttp\Client', function () {
+        $this->getContainer()->add('GuzzleHttp\Client', function() {
             return new Client([
                 'timeout' => 10.0
             ]);

src/Command/DeleteMissingAlertsCommand.php

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
         $missing = 0;
         $max = $result->ResultID;
 
-        while($count < $max) {
+        while ($count < $max) {
             $count++;
 
             $per = round(($count / $max) * 100, 2);

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 
 use Ps2alerts\Api\Command\BaseCommand;
 use Ps2alerts\Api\Repository\AlertRepository;
-use Symfony\Component\Console\Input\InputArgument;
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Output\OutputInterface;
 

src/Command/CommandsCommon.php

Spacing +2 added lines, -2 removed lines

@@ -1,6 +1,6 @@
 <?php
 
 function convert($size) {
-    $unit=array('B','KB','MB','GB','TB','PB');
-    return @round($size/pow(1024,($i=floor(log($size,1024)))),2).' '.$unit[$i];
+    $unit = array('B', 'KB', 'MB', 'GB', 'TB', 'PB');
+    return @round($size / pow(1024, ($i = floor(log($size, 1024)))), 2).' '.$unit[$i];
 }

src/Command/DeleteAlertCommand.php

Spacing +4 added lines, -4 removed lines

@@ -194,7 +194,7 @@ 
     ) {
         // Check each cols to make sure we handle SUM(BLAH) AS BLAH issues
 
-        foreach($cols as $key => $col) {
+        foreach ($cols as $key => $col) {
             if (strpos($col, 'AS ') !== false) {
                 $pos = strrpos($col, 'AS ') + 3; // Plus 3 for "AS "
                 $len = strlen($col);
@@ -210,7 +210,7 @@ 
         $query->from($table);
         $query->where('resultID = ?', $id);
 
-        if (! empty($groupBy)) {
+        if (!empty($groupBy)) {
             $query->groupBy($groupBy);
         }
 
@@ -226,7 +226,7 @@ 
             $update->table($totalsTable);
             $update->where("{$filter} = ?", $row->$filter);
 
-            foreach($fields as $field) {
+            foreach ($fields as $field) {
                 $update->set($field, "{$field} - {$row->$field}");
             }
 
@@ -239,7 +239,7 @@ 
 
     protected function deleteAllFromTables(array $tables, $id, OutputInterface $output)
     {
-        foreach($tables as $table) {
+        foreach ($tables as $table) {
             $delete = $this->auraFactory->newDelete();
             $delete->from($table);
             $delete->where('resultID = ?', $id);

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

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

Doc Comments +11 added lines, -5 removed lines

@@ -115,7 +115,7 @@ 
     /**
      * Processes players for alert
      * @param  string $id Alert ID
-     * @return void
+     * @return integer
      */
     protected function processPlayers($id)
     {
@@ -149,7 +149,7 @@ 
     /**
      * Processes outfits for alert
      * @param  string $id Alert ID
-     * @return void
+     * @return integer
      */
     protected function processOutfits($id)
     {
@@ -181,7 +181,7 @@ 
     /**
      * Processes XPs for alert
      * @param  string $id Alert ID
-     * @return void
+     * @return integer
      */
     protected function processXP($id)
     {
@@ -208,11 +208,11 @@ 
     /**
      * Executes the process based on inputs
      * @param  string $id          Alert ID
-     * @param  array  $cols        Columns to look for
+     * @param  string[]  $cols        Columns to look for
      * @param  string $table       Table to look for
      * @param  string $totalsTable Table total to update if applicable
      * @param  string $filter      Column to filter on
-     * @param  array  $fields      Fields to summarize
+     * @param  string[]  $fields      Fields to summarize
      * @param  array  $groupBy     Fields to group by
      * @return int
      */
@@ -270,6 +270,9 @@ 
         return $count;
     }
 
+    /**
+     * @param string $id
+     */
     protected function deleteAllFromTables(array $tables, $id, OutputInterface $output)
     {
         foreach($tables as $table) {
@@ -288,6 +291,9 @@ 
         }
     }
 
+    /**
+     * @param string $id
+     */
     protected function deleteAlert($id)
     {
         $delete = $this->auraFactory->newDelete();

src/Controller/Endpoint/Admin/AdminEndpointController.php

Indentation +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
     /**
      * Construct
      *
-    */
+     */
     public function __construct()
     {