Maelstromeous / ps2alerts-api

src/Controller/Endpoint/Search/SearchEndpointController.php

Doc Comments +11 added lines, -11 removed lines

@@ -17,11 +17,11 @@ 
     /**
      * Construct
      *
-     * @param League\Fractal\Manager                                   $fractal
-     * @param Ps2alerts\Api\Transformer\Search\OutfitSearchTransformer $outfitSearchTransformer
-     * @param Ps2alerts\Api\Repository\Metrics\OutfitTotalRepository   $outfitTotalRepo
-     * @param Ps2alerts\Api\Transformer\Search\PlayerSearchTransformer $playerSearchTransformer
-     * @param Ps2alerts\Api\Repository\Metrics\PlayerTotalRepository   $playerTotalRepo
+     * @param Manager                                   $fractal
+     * @param OutfitSearchTransformer $outfitSearchTransformer
+     * @param OutfitTotalRepository   $outfitTotalRepo
+     * @param PlayerSearchTransformer $playerSearchTransformer
+     * @param PlayerTotalRepository   $playerTotalRepo
      */
     public function __construct(
         Manager                 $fractal,
@@ -40,11 +40,11 @@ 
     /**
      * Endpoint to return potential players based on search term
      *
-     * @param  Psr\Http\Message\ServerRequestInterface  $request
-     * @param  Psr\Http\Message\ResponseInterface $response
+     * @param  ServerRequestInterface  $request
+     * @param  ResponseInterface $response
      * @param  array                                     $args
      *
-     * @return \League\Fractal\TransformerAbstract
+     * @return ResponseInterface|null
      */
     public function getPlayersByTerm(ServerRequestInterface $request, ResponseInterface $response, array $args)
     {
@@ -67,11 +67,11 @@ 
     /**
      * Endpoint to return potential players based on search term
      *
-     * @param  Psr\Http\Message\ServerRequestInterface  $request
-     * @param  Psr\Http\Message\ResponseInterface $response
+     * @param  ServerRequestInterface  $request
+     * @param  ResponseInterface $response
      * @param  array                                     $args
      *
-     * @return \League\Fractal\TransformerAbstract
+     * @return ResponseInterface|null
      */
     public function getOutfitsByTerm(ServerRequestInterface $request, ResponseInterface $response, array $args)
     {

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 
 use League\Fractal\Manager;
 use Ps2alerts\Api\Controller\Endpoint\AbstractEndpointController;
-use Ps2alerts\Api\Exception\InvalidArgumentException;
 use Ps2alerts\Api\Repository\Metrics\OutfitTotalRepository;
 use Ps2alerts\Api\Repository\Metrics\PlayerTotalRepository;
 use Ps2alerts\Api\Transformer\Search\OutfitSearchTransformer;

Spacing +3 added lines, -3 removed lines

@@ -52,7 +52,7 @@ 
         if ($this->parsePlayerName($args['term'], $response)) {
             $players = $this->searchForPlayer($args['term']);
 
-            if (! empty($players)) {
+            if (!empty($players)) {
                 return $this->respond(
                     'collection',
                     $players,
@@ -81,7 +81,7 @@ 
         if ($this->parseOutfitName($name, $response)) {
             $outfits = $this->searchForOutfit($name);
 
-            if (! empty($outfits)) {
+            if (!empty($outfits)) {
                 return $this->respond(
                     'collection',
                     $outfits,
@@ -199,7 +199,7 @@ 
             return $this->errorWrongArgs('Player ID cannot be longer than 19 characters.');
         }
 
-        if (! is_numeric($id)) {
+        if (!is_numeric($id)) {
             return $this->errorWrongArgs('Player ID must be numeric.');
         }
 

src/Transformer/AlertTransformer.php

Unused Use Statements -1 removed lines

@@ -14,7 +14,6 @@
 use Ps2alerts\Api\Repository\Metrics\VehicleTotalRepository;
 use Ps2alerts\Api\Repository\Metrics\WeaponTotalRepository;
 use Ps2alerts\Api\Repository\Metrics\XpRepository;
-
 use Ps2alerts\Api\Transformer\Metrics\ClassTransformer;
 use Ps2alerts\Api\Transformer\Metrics\CombatHistoryTransformer;
 use Ps2alerts\Api\Transformer\Metrics\CombatTransformer;

src/Transformer/DataTransformer.php

Doc Comments -1 removed lines

@@ -57,7 +57,6 @@
     /**
      * The tranform method required by Fractal to parse the data and return proper typing and fields.
      *
-     * @param  array $data Data to transform
      *
      * @return array
      */

src/Transformer/Profiles/OutfitProfileTransformer.php

Doc Comments +3 added lines

@@ -219,6 +219,9 @@
         return $players;
     }
 
+    /**
+     * @param integer $offset
+     */
     public function getOutfitMembers($data, $offset)
     {
         $client = $this->getHttpClientDriver();

Spacing +2 added lines, -2 removed lines

@@ -191,7 +191,7 @@ 
         $key = "ps2alerts:api:profiles:outfit:players:{$data['outfitID']}";
 
         // If we have this cached already
-        if (! empty($redis->exists($key))) {
+        if (!empty($redis->exists($key))) {
             return json_decode($redis->get($key), true);
         }
 
@@ -227,7 +227,7 @@ 
         $key = "ps2alerts:api:profiles:outfit:census:members:{$data['outfitID']}:{$offset}";
 
         // If we have this cached already
-        if (! empty($redis->exists($key))) {
+        if (!empty($redis->exists($key))) {
             return json_decode($redis->get($key));
         }
 

src/Transformer/Profiles/PlayerProfileTransformer.php

Doc Comments +3 added lines, -4 removed lines

@@ -72,7 +72,7 @@ 
      *
      * @param  array $data
      *
-     * @return \League\Fractal\Resource\Collection
+     * @return \League\Fractal\Resource\Item
      */
     public function includeCensus($data)
     {
@@ -144,9 +144,8 @@ 
     /**
      * Gets Metrics for a player
      *
-     * @param  array $player
      *
-     * @return array
+     * @return \League\Fractal\Resource\Item
      */
     public function includeMetrics($data)
     {
@@ -179,7 +178,7 @@ 
      *
      * @param  array $data
      *
-     * @return \League\Fractal\Resource\Collection
+     * @return \League\Fractal\Resource\Item
      */
     public function includeOutfit($data)
     {

src/Repository/AbstractEndpointRepository.php

Indentation +2 added lines, -2 removed lines

@@ -184,7 +184,7 @@ 
         $key = $this->returnKeyType($keyType);
 
         $query->cols(['*'])
-              ->where("{$key} = ?", $id);
+                ->where("{$key} = ?", $id);
 
         return $this->fireStatementAndReturn($query, true, $object, $archive);
     }
@@ -221,7 +221,7 @@ 
         $key = $this->returnKeyType($keyType);
 
         $query->cols(['*'])
-              ->where("{$key} = ?", $id);
+                ->where("{$key} = ?", $id);
 
         return $this->fireStatementAndReturn($query, false, false, $archive);
     }

src/Transformer/Data/OutfitTransformer.php

Spacing +1 added lines, -1 removed lines

@@ -18,7 +18,7 @@
         $obj = [
             'id'          => (string) $data->outfit_id, // Bigint
             'name'        => (string) $data->name,
-            'tag'         => (! empty($data->alias)) ? (string) $data->alias : null,
+            'tag'         => (!empty($data->alias)) ? (string) $data->alias : null,
             'faction'     => (int) $data->leader->faction_id,
             'count'       => (int) $data->member_count,
             'leader'      => (string) $data->leader_character_id,

src/Transformer/Leaderboards/PlayerLeaderboardTransformer.php

Spacing +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
      */
     public function transform($data)
     {
-        if (! isset($data['playerOutfit'])) {
+        if (!isset($data['playerOutfit'])) {
             $outfit = null;
         } else {
             $outfit = $data['playerOutfit'];

src/container.php

Indentation +21 added lines, -21 removed lines

@@ -19,43 +19,43 @@
 
 // Inflectors
 $container->inflector('Ps2alerts\Api\Contract\ConfigAwareInterface')
-          ->invokeMethod('setConfig', ['config']);
+            ->invokeMethod('setConfig', ['config']);
 $container->inflector('Ps2alerts\Api\Contract\DatabaseAwareInterface')
-          ->invokeMethod('setDatabaseDriver', ['Database'])
-          ->invokeMethod('setDatabaseDataDriver', ['Database\Data'])
-          ->invokeMethod('setDatabaseArchiveDriver', ['Database\Archive']);
+            ->invokeMethod('setDatabaseDriver', ['Database'])
+            ->invokeMethod('setDatabaseDataDriver', ['Database\Data'])
+            ->invokeMethod('setDatabaseArchiveDriver', ['Database\Archive']);
 $container->inflector('Ps2alerts\Api\Contract\LogAwareInterface')
-          ->invokeMethod('setLogDriver', ['Monolog\Logger']);
+            ->invokeMethod('setLogDriver', ['Monolog\Logger']);
 $container->inflector('Ps2alerts\Api\Contract\HttpClientAwareInterface')
-          ->invokeMethod('setHttpClientDriver', ['GuzzleHttp\Client']);
+            ->invokeMethod('setHttpClientDriver', ['GuzzleHttp\Client']);
 $container->inflector('Ps2alerts\Api\Contract\HttpMessageAwareInterface')
-          ->invokeMethods([
-              'setResponse' => ['Zend\Diactoros\Response'],
-              'setRequest'  => ['Zend\Diactoros\ServerRequest']
-           ]);
+            ->invokeMethods([
+                'setResponse' => ['Zend\Diactoros\Response'],
+                'setRequest'  => ['Zend\Diactoros\ServerRequest']
+            ]);
 $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']);
 
 // Container Inflector
 $container->inflector('League\Container\ContainerAwareInterface')
-          ->invokeMethod('setContainer', [$container]);
+            ->invokeMethod('setContainer', [$container]);
 
 // Processing deps
 $container->add('Ps2alerts\Api\Factory\AuraFactory')
-          ->withArgument('Aura\SqlQuery\QueryFactory');
+            ->withArgument('Aura\SqlQuery\QueryFactory');
 
 $container->add('Ps2alerts\Api\Transformer\DataTransformer')
-          ->withArgument('Ps2alerts\Api\Repository\Data\FacilityDataRepository')
-          ->withArgument('Ps2alerts\Api\Repository\Data\VehicleDataRepository')
-          ->withArgument('Ps2alerts\Api\Repository\Data\WeaponDataRepository')
-          ->withArgument('Ps2alerts\Api\Repository\Data\XpDataRepository');
+            ->withArgument('Ps2alerts\Api\Repository\Data\FacilityDataRepository')
+            ->withArgument('Ps2alerts\Api\Repository\Data\VehicleDataRepository')
+            ->withArgument('Ps2alerts\Api\Repository\Data\WeaponDataRepository')
+            ->withArgument('Ps2alerts\Api\Repository\Data\XpDataRepository');
 
 $container->add('Ps2alerts\Api\Controller\Endpoint\Data\DataEndpointController')
-          ->withArgument('League\Fractal\Manager')
-          ->withArgument('Ps2alerts\Api\Transformer\DataTransformer');
+            ->withArgument('League\Fractal\Manager')
+            ->withArgument('Ps2alerts\Api\Transformer\DataTransformer');
 
 return $container;