Maelstromeous / ps2alerts-api

src/QueryObjects/QueryObject.php

Doc Comments +2 added lines, -2 removed lines

@@ -49,7 +49,7 @@ 
      * in the array and added to.
      * e.g. $queryObject->addSelect('COUNT(ResultID) AS COUNT');
      *
-     * @param string $string
+     * @param string $array
      */
     public function addSelect($array)
     {
@@ -171,7 +171,7 @@ 
     /**
      * Allows setting of workaround flags
      *
-     * @param array|string $flags
+     * @param string $flags
      */
     public function setFlags($flags)
     {

Spacing +5 added lines, -5 removed lines

@@ -118,7 +118,7 @@ 
      */
     public function setOrderBy($string)
     {
-        if (! empty($string)) {
+        if (!empty($string)) {
             $this->orderBy = $string;
         }
     }
@@ -140,7 +140,7 @@ 
      */
     public function setOrderByDirection($string)
     {
-        if (! empty($string)) {
+        if (!empty($string)) {
             $this->orderByDirection = $string;
         }
     }
@@ -162,7 +162,7 @@ 
      */
     public function setDimension($string)
     {
-        if (! empty($string)) {
+        if (!empty($string)) {
             $this->dimension = $string;
         }
     }
@@ -184,7 +184,7 @@ 
      */
     public function setLimit($limit)
     {
-        if (! empty($limit)) {
+        if (!empty($limit)) {
             $this->limit = $limit;
         }
     }
@@ -206,7 +206,7 @@ 
      */
     public function setFlags($flags)
     {
-        if (! empty($flags)) {
+        if (!empty($flags)) {
             $this->flags = $flags;
         }
     }

src/Loader/Statistics/AbstractStatisticsLoader.php

Doc Comments +2 added lines, -2 removed lines

@@ -150,10 +150,10 @@
      * Takes common requests and appends them to the query object. Any other
      * special requirements will be handled after
      *
-     * @param  Ps2alerts\Api\QueryObjects\QueryObject $queryObject
+     * @param  QueryObject $queryObject
      * @param  array                                  $post
      *
-     * @return Ps2alerts\Api\QueryObjects\QueryObject
+     * @return QueryObject
      */
     public function setupQueryObject(QueryObject $queryObject, array $post)
     {

Spacing +17 added lines, -17 removed lines

@@ -75,34 +75,34 @@ 
      */
     public function appendRedisKey($post, $redisKey)
     {
-        if (! empty($post['selects'])) {
+        if (!empty($post['selects'])) {
             $whereMD5 = md5($post['selects']);
             $redisKey .= "/select:{$whereMD5}";
         }
 
-        if (! empty($post['wheres'])) {
+        if (!empty($post['wheres'])) {
             $whereMD5 = md5($post['wheres']);
             $redisKey .= "/where:{$whereMD5}";
         }
 
-        if (! empty($post['whereIns'])) {
+        if (!empty($post['whereIns'])) {
             $whereInMD5 = md5($post['whereIns']);
             $redisKey .= "/whereIn:{$whereInMD5}";
         }
 
-        if (! empty($post['orderBy'])) {
+        if (!empty($post['orderBy'])) {
             $orderMD5 = md5($post['orderBy']);
             $redisKey .= "/order:{$orderMD5}";
         }
 
-        if (! empty($post['limit'])) {
+        if (!empty($post['limit'])) {
             // Enforce a max limit
             if ($post['limit'] > 50) {
                 $post['limit'] = 50;
             }
         }
 
-        if (empty($post['limit']) || ! isset($post['limit'])) {
+        if (empty($post['limit']) || !isset($post['limit'])) {
             $post['limit'] = 50;
         }
 
@@ -120,20 +120,20 @@ 
      */
     public function processPostVars($post)
     {
-        if (! empty($post['wheres'])) {
+        if (!empty($post['wheres'])) {
             $return['wheres'] = json_decode($post['wheres'], true);
             $this->getLogDriver()->addDebug(json_encode($return['wheres']));
         }
 
-        if (! empty($post['whereIns'])) {
+        if (!empty($post['whereIns'])) {
             $return['whereIns'] = json_decode($post['whereIns'], true);
         }
 
-        if (! empty($post['orderBy'])) {
+        if (!empty($post['orderBy'])) {
             $return['orderBy'] = json_decode($post['orderBy'], true);
         }
 
-        if (empty($post['limit']) || ! isset($post['limit'])) {
+        if (empty($post['limit']) || !isset($post['limit'])) {
             $post['limit'] = 10;
         }
 
@@ -157,7 +157,7 @@ 
      */
     public function setupQueryObject(QueryObject $queryObject, array $post)
     {
-        if (! empty($post['wheres'])) {
+        if (!empty($post['wheres'])) {
             foreach ($post['wheres'] as $key => $value) {
                 if ($key !== 'lessthan' && $key !== 'morethan') {
                     $queryObject->addWhere([
@@ -167,7 +167,7 @@ 
                 }
             }
 
-            if (! empty($post['wheres']['lessthan'])) {
+            if (!empty($post['wheres']['lessthan'])) {
                 foreach ($post['wheres']['lessthan'] as $key => $value) {
                     $queryObject->addWhere([
                         'col'   => $key,
@@ -177,7 +177,7 @@ 
                 }
             }
 
-            if (! empty($post['wheres']['morethan'])) {
+            if (!empty($post['wheres']['morethan'])) {
                 foreach ($post['wheres']['morethan'] as $key => $value) {
                     $queryObject->addWhere([
                         'col'   => $key,
@@ -188,7 +188,7 @@ 
             }
         }
 
-        if (! empty($post['whereIns'])) {
+        if (!empty($post['whereIns'])) {
             foreach ($post['whereIns'] as $key => $value) {
                 // Escape strings manually, incase of player IDs etc
                 foreach ($value as $i => $val) {
@@ -204,16 +204,16 @@ 
             }
         }
 
-        if (! empty($post['orderBy'])) {
+        if (!empty($post['orderBy'])) {
             $queryObject->setOrderBy(array_keys($post['orderBy'])[0]);
             $queryObject->setOrderByDirection(array_values($post['orderBy'])[0]);
         }
 
-        if (! empty($post['limit'])) {
+        if (!empty($post['limit'])) {
             $queryObject->setLimit($post['limit']);
         }
 
-        if (! empty($this->getFlags())) {
+        if (!empty($this->getFlags())) {
             // If there are some funky things we have to do, set them.
             $queryObject->setFlags($this->getFlags());
         }

src/Loader/Statistics/AlertStatisticsLoader.php

Doc Comments +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
      * Construct
      *
      * @param \Ps2alerts\Api\Repository\AlertRepository    $repository
-     * @param \Ps2alerts\Api\Helper\DataFormatter          $dataFormatter
+     * @param DataFormatterHelper          $dataFormatter
      */
     public function __construct(
         AlertRepository     $repository,

Unused Use Statements -1 removed lines

@@ -5,7 +5,6 @@
 use Ps2alerts\Api\Loader\Statistics\AbstractStatisticsLoader;
 use Ps2alerts\Api\QueryObjects\QueryObject;
 use Ps2alerts\Api\Repository\AlertRepository;
-use Ps2alerts\Api\Validator\AlertInputValidator;
 use Ps2alerts\Api\Helper\DataFormatterHelper;
 
 class AlertStatisticsLoader extends AbstractStatisticsLoader

Spacing +5 added lines, -5 removed lines

@@ -89,9 +89,9 @@ 
             return $this->getFromRedis($masterRedisKey);
         }
 
-        $servers  = [1,10,13,17,25,1000,2000];
-        $zones    = [2,4,6,8];
-        $factions = ['vs','nc','tr','draw'];
+        $servers  = [1, 10, 13, 17, 25, 1000, 2000];
+        $zones    = [2, 4, 6, 8];
+        $factions = ['vs', 'nc', 'tr', 'draw'];
 
         $results = [];
         $this->setCacheExpireTime(3600); // 1 Hour
@@ -188,7 +188,7 @@ 
         $maxDate = date('Y-m-d'); // Today unless set
 
         // If there is a minimum date set
-        if (! empty($post['wheres']['morethan']['ResultEndTime'])) {
+        if (!empty($post['wheres']['morethan']['ResultEndTime'])) {
             if (is_integer($post['wheres']['morethan']['ResultEndTime'])) {
                 $minDate = date('Y-m-d', $post['wheres']['morethan']['ResultEndTime']);
             } else {
@@ -197,7 +197,7 @@ 
         }
 
         // If there is a maximum date set
-        if (! empty($post['wheres']['lessthan']['ResultEndTime'])) {
+        if (!empty($post['wheres']['lessthan']['ResultEndTime'])) {
             if (is_integer($post['wheres']['lessthan']['ResultEndTime'])) {
                 $maxDate = date('Y-m-d', $post['wheres']['lessthan']['ResultEndTime']);
             } else {

src/Repository/AbstractEndpointRepository.php

Spacing +7 added lines, -7 removed lines

@@ -70,14 +70,14 @@ 
         $query = $this->newQuery();
 
         // Setup select statements
-        if (! empty($queryObject->getSelects())) {
+        if (!empty($queryObject->getSelects())) {
             $query->cols($queryObject->getSelects());
         } else {
             $query->cols(['*']);
         }
 
         // Workarounds :-/
-        if (! empty($queryObject->getFlags())) {
+        if (!empty($queryObject->getFlags())) {
             if ($queryObject->getFlags() === 'outfitIDs') {
                 // Prevent the VS, NC and TR "no outfit" workaround
                 $queryObject->addWhere([
@@ -89,7 +89,7 @@ 
         }
 
         // Setup where statements
-        if (! empty($queryObject->getWheres())) {
+        if (!empty($queryObject->getWheres())) {
             foreach ($queryObject->getWheres() as $where) {
                 $col = $where['col'];
 
@@ -103,7 +103,7 @@ 
                 // Generate a UUID so that any columns that are used multiple
                 // times always have unique values. Tiny overhead for the benefit
                 $uuid = $this->getUuidDriver()->uuid4()->toString();
-                $bind = str_replace('-', '', $col.$uuid);
+                $bind = str_replace('-', '', $col . $uuid);
 
                 $op = (isset($where['op']) ? $where['op'] : '=');
 
@@ -113,7 +113,7 @@ 
         }
 
         // Setup where statements
-        if (! empty($queryObject->getWhereIns())) {
+        if (!empty($queryObject->getWhereIns())) {
             foreach ($queryObject->getWhereIns() as $whereIn) {
                 $col = $whereIn['col'];
 
@@ -129,7 +129,7 @@ 
         }
 
         // Set up order statement
-        if (! empty($queryObject->getOrderBy())) {
+        if (!empty($queryObject->getOrderBy())) {
             $orderBy = $queryObject->getOrderBy();
             if ($orderBy === 'primary') {
                 $orderBy = $this->getPrimaryKey();
@@ -143,7 +143,7 @@ 
             ]);
         }
 
-        if (! empty($queryObject->getLimit())) {
+        if (!empty($queryObject->getLimit())) {
             // Only set a limit if it's required
             if ($queryObject->getLimit() !== 'unlimited') {
                 $query->limit($queryObject->getLimit());

src/ServiceProvider/UuidServiceProvider.php

Spacing +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
      */
     public function register()
     {
-        $this->getContainer()->add('Ramsey\Uuid\Uuid', function () {
+        $this->getContainer()->add('Ramsey\Uuid\Uuid', function() {
             $uuid = Uuid::Uuid4();
             return $uuid;
         });

src/container.php

Indentation +9 added lines, -9 removed lines

@@ -17,17 +17,17 @@ 
 
 // Inflectors
 $container->inflector('Ps2alerts\Api\Contract\ConfigAwareInterface')
-          ->invokeMethod('setConfig', ['config']);
+            ->invokeMethod('setConfig', ['config']);
 $container->inflector('Ps2alerts\Api\Contract\DatabaseAwareInterface')
-          ->invokeMethod('setDatabaseDriver', ['Aura\Sql']);
+            ->invokeMethod('setDatabaseDriver', ['Aura\Sql']);
 $container->inflector('Ps2alerts\Api\Contract\LogAwareInterface')
-          ->invokeMethod('setLogDriver', ['Monolog\Logger']);
+            ->invokeMethod('setLogDriver', ['Monolog\Logger']);
 $container->inflector('Ps2alerts\Api\Contract\TemplateAwareInterface')
-          ->invokeMethod('setTemplateDriver', ['Twig_Environment']);
+            ->invokeMethod('setTemplateDriver', ['Twig_Environment']);
 $container->inflector('Ps2alerts\Api\Contract\RedisAwareInterface')
-          ->invokeMethod('setRedisDriver', ['redis']);
+            ->invokeMethod('setRedisDriver', ['redis']);
 $container->inflector('Ps2alerts\Api\Contract\UuidAwareInterface')
-          ->invokeMethod('setUuidDriver', ['Ramsey\Uuid\Uuid']);
+            ->invokeMethod('setUuidDriver', ['Ramsey\Uuid\Uuid']);
 
 // Repositories
 $container->add('Ps2alerts\Api\Repository\AlertRepository');
@@ -35,11 +35,11 @@ 
 $container->add('Ps2alerts\Api\Helper\DataFormatterHelper');
 
 $container->add('Ps2alerts\Api\Loader\Statistics\AlertStatisticsLoader')
-          ->withArgument('Ps2alerts\Api\Repository\AlertRepository')
-          ->withArgument('Ps2alerts\Api\Helper\DataFormatterHelper');
+            ->withArgument('Ps2alerts\Api\Repository\AlertRepository')
+            ->withArgument('Ps2alerts\Api\Helper\DataFormatterHelper');
 
 // Container Inflector
 $container->inflector('League\Container\ContainerAwareInterface')
-          ->invokeMethod('setContainer', [$container]);
+            ->invokeMethod('setContainer', [$container]);
 
 return $container;