Maelstromeous / ps2alerts-api

src/Controller/Statistics/OutfitTotalsStatisticsEndpoint.php

Doc Comments +1 added lines, -2 removed lines

@@ -12,7 +12,7 @@ 
     /**
      * Construct
      *
-     * @param \Ps2alerts\Api\Loader\Metrics\OutfitTotalsStatisticsLoader $loader
+     * @param OutfitTotalsStatisticsLoader $loader
      */
     public function __construct(OutfitTotalsStatisticsLoader $loader)
     {
@@ -23,7 +23,6 @@ 
      * Returns top X entries
      *
      * @param  \Symfony\Component\HttpFoundation\Request $request
-     * @param  array                                     $args
      *
      * @return \League\Route\Http\JsonResponse
      */

src/Controller/Statistics/PlayerStatisticsEndpoint.php

Doc Comments +1 added lines, -1 removed lines

@@ -12,7 +12,7 @@
     /**
      * Construct
      *
-     * @param \Ps2alerts\Api\Loader\Metrics\PlayerStatisticsLoader $loader
+     * @param PlayerStatisticsLoader $loader
      */
     public function __construct(PlayerStatisticsLoader $loader)
     {

src/Loader/AbstractLoader.php

Doc Comments +2 added lines, -3 removed lines

@@ -77,7 +77,6 @@ 
     /**
      * Gets the Cache namespace key for Redis
      *
-     * @param string $string
      */
     public function getCacheNamespace()
     {
@@ -128,7 +127,7 @@ 
      * Checks for a key within Redis and returns it's existance
      *
      * @param  string  $key
-     * @return boolean
+     * @return false|null
      */
     public function checkRedis($key)
     {
@@ -155,7 +154,7 @@ 
      * Sets JSON encoded value within Redis with an expiry in seconds
      *
      * @param string  $key
-     * @param mixed   $value   Data to encode into JSON and store
+     * @param string   $value   Data to encode into JSON and store
      * @param integer $expires Cache expiry time in seconds
      */
     public function setExpireKey($key, $value, $expires)

src/Loader/Statistics/PlayerStatisticsLoader.php

Doc Comments +3 added lines

@@ -91,6 +91,9 @@
         );
     }
 
+    /**
+     * @param string $field
+     */
     public function validatePostVars($field, $value)
     {
         if ($field === 'metric' && ! empty($value)) {

Spacing +8 added lines, -8 removed lines

@@ -32,10 +32,10 @@ 
         $queryObject = new QueryObject;
 
         // Build based on metric alone (order by playerKills for example)
-        if (! empty($post['metric'] && empty($post['value']))
+        if (!empty($post['metric'] && empty($post['value']))
             && $this->validatePostVars('metric', $post['metric']) === true
         ) {
-            $direction = (! empty($post['direction']) ? $post['direction'] : 'desc');
+            $direction = (!empty($post['direction']) ? $post['direction'] : 'desc');
 
             $queryObject->setOrderBy($post['metric']);
             $queryObject->setOrderByDirection($direction);
@@ -44,11 +44,11 @@ 
         }
 
         // Build based on metric and a value (such as order by playerKills by server)
-        if (! empty($post['value'])
-            && ! empty($post['metric'])
+        if (!empty($post['value'])
+            && !empty($post['metric'])
             && $this->validatePostVars('metric', $post['metric']) === true
         ) {
-            $op = (! empty($post['operator']) ? $post['operator'] : '=');
+            $op = (!empty($post['operator']) ? $post['operator'] : '=');
 
             $queryObject->addWhere([
                 'col' => $post['metric'],
@@ -56,7 +56,7 @@ 
                 'value' => $post['value']
             ]);
 
-            $direction = (! empty($post['direction']) ? $post['direction'] : 'desc');
+            $direction = (!empty($post['direction']) ? $post['direction'] : 'desc');
 
             $queryObject->setOrderBy($post['metric']);
             $queryObject->setOrderByDirection($direction);
@@ -64,7 +64,7 @@ 
             $redisKey .= ":{$post['metric']}{$op}{$post['value']}-{$direction}";
         }
 
-        if (! empty($post['limit'])) {
+        if (!empty($post['limit'])) {
             $post['limit'] = intval($post['limit']);
 
             // Hard encode a 50 limit
@@ -93,7 +93,7 @@ 
 
     public function validatePostVars($field, $value)
     {
-        if ($field === 'metric' && ! empty($value)) {
+        if ($field === 'metric' && !empty($value)) {
             switch ($value) {
                 case 'playerKills':
                 case 'playerDeaths':

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)
     {

src/Repository/AbstractEndpointRepository.php

Spacing +5 added lines, -5 removed lines

@@ -66,14 +66,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([
@@ -85,7 +85,7 @@ 
         }
 
         // Setup where statements
-        if (! empty($queryObject->getWheres())) {
+        if (!empty($queryObject->getWheres())) {
             foreach ($queryObject->getWheres() as $where) {
                 $col = $where['col'];
 
@@ -104,7 +104,7 @@ 
         }
 
         // Set up order statement
-        if (! empty($queryObject->getOrderBy())) {
+        if (!empty($queryObject->getOrderBy())) {
             $orderBy = $queryObject->getOrderBy();
             if ($orderBy === 'primary') {
                 $orderBy = $this->getPrimaryKey();
@@ -118,7 +118,7 @@ 
             ]);
         }
 
-        if (! empty($queryObject->getLimit())) {
+        if (!empty($queryObject->getLimit())) {
             $query->limit($queryObject->getLimit());
         }
 

src/container.php

Indentation +7 added lines, -7 removed lines

@@ -15,24 +15,24 @@
 
 // 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\TemplateAwareInterface')
-          ->invokeMethod('setTemplateDriver', ['Twig_Environment']);
+            ->invokeMethod('setTemplateDriver', ['Twig_Environment']);
 $container->inflector('Ps2alerts\Api\Contract\RedisAwareInterface')
-          ->invokeMethod('setRedisDriver', ['redis']);
+            ->invokeMethod('setRedisDriver', ['redis']);
 
 $container->add('Ps2alerts\Api\Validator\AlertInputValidator');
 
 $container->add('Ps2alerts\Api\Repository\AlertRepository');
 
 $container->add('Ps2alerts\Api\Loader\Statistics\AlertStatisticsLoader')
-          ->withArgument('Ps2alerts\Api\Repository\AlertRepository')
-          ->withArgument('Ps2alerts\Api\Validator\AlertInputValidator');
+            ->withArgument('Ps2alerts\Api\Repository\AlertRepository')
+            ->withArgument('Ps2alerts\Api\Validator\AlertInputValidator');
 
 // Container Inflector
 $container->inflector('League\Container\ContainerAwareInterface')
-          ->invokeMethod('setContainer', [$container]);
+            ->invokeMethod('setContainer', [$container]);
 
 return $container;

src/Loader/Statistics/AlertStatisticsLoader.php

Spacing +4 added lines, -4 removed lines

@@ -54,7 +54,7 @@ 
             'value' => '1'
         ]);
 
-        if (! empty($post['ResultServer'])) {
+        if (!empty($post['ResultServer'])) {
             if ($this->inputValidator->validatePostVars(
                 'ResultServer',
                 $post['ResultServer']
@@ -67,7 +67,7 @@ 
             }
         }
 
-        if (! empty($post['ResultWinner'])) {
+        if (!empty($post['ResultWinner'])) {
             if ($this->inputValidator->validatePostVars(
                 'ResultWinner',
                 $post['ResultWinner']
@@ -80,7 +80,7 @@ 
             }
         }
 
-        if (! empty($post['ResultAlertCont'])) {
+        if (!empty($post['ResultAlertCont'])) {
             if ($this->inputValidator->validatePostVars(
                 'ResultAlertCont',
                 $post['ResultAlertCont']
@@ -93,7 +93,7 @@ 
             }
         }
 
-        if (! empty($post['ResultDomination'])) {
+        if (!empty($post['ResultDomination'])) {
             if ($this->inputValidator->validatePostVars(
                 'ResultDomination',
                 $post['ResultDomination']

src/Loader/Statistics/AbstractStatisticsLoader.php

Spacing +10 added lines, -10 removed lines

@@ -49,26 +49,26 @@ 
      */
     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['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'] = 10;
         }
 
@@ -86,15 +86,15 @@ 
      */
     public function processPostVars($post)
     {
-        if (! empty($post['wheres'])) {
+        if (!empty($post['wheres'])) {
             $return['wheres'] = json_decode($post['wheres'], 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;
         }
 
@@ -133,14 +133,14 @@ 
             ]);
         }
 
-        if (! empty($post['orderBy'])) {
+        if (!empty($post['orderBy'])) {
             $queryObject->setOrderBy(array_keys($post['orderBy'])[0]);
             $queryObject->setOrderByDirection(array_values($post['orderBy'])[0]);
         }
 
         $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());
         }