Maelstromeous / ps2alerts-api

src/container.php

Spacing +1 added lines, -1 removed lines

@@ -3,7 +3,7 @@
 $container = new League\Container\Container;
 
 // Register the request object singleton to be used later in the request cyncle
-$container->singleton('Symfony\Component\HttpFoundation\Request', function () {
+$container->singleton('Symfony\Component\HttpFoundation\Request', function() {
     return Symfony\Component\HttpFoundation\Request::createFromGlobals();
 });
 

Indentation +8 added lines, -8 removed lines

@@ -18,22 +18,22 @@
 
 // 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('setDatabaseDriver', ['Database'])
+            ->invokeMethod('setDatabaseDataDriver', ['Database\Data']);
 $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']);
 
 
 // Container Inflector
 $container->inflector('League\Container\ContainerAwareInterface')
-          ->invokeMethod('setContainer', [$container]);
+            ->invokeMethod('setContainer', [$container]);
 
 return $container;

src/ServiceProvider/ConfigServiceProvider.php

Spacing +1 added lines, -1 removed lines

@@ -18,7 +18,7 @@
      */
     public function register()
     {
-        $this->getContainer()->singleton('config', function () {
+        $this->getContainer()->singleton('config', function() {
             return include __DIR__ . '/../../config/config.php';
         });
     }

src/ServiceProvider/RedisServiceProvider.php

Spacing +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
      */
     public function register()
     {
-        $this->getContainer()->singleton('redis', function () {
+        $this->getContainer()->singleton('redis', function() {
             $redisConfig = $this->getContainer()->get('config')['redis'];
 
             $client = new Client([

src/ServiceProvider/LogServiceProvider.php

Spacing +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
      */
     public function register()
     {
-        $this->getContainer()->singleton('Monolog\Logger', function () {
+        $this->getContainer()->singleton('Monolog\Logger', function() {
             $log = new Logger('app');
             $log->pushHandler(
                 new StreamHandler(__DIR__ . '/../../logs/app.log', Logger::DEBUG)

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;
         });

tests/Endpoints/AlertControllerTest.php

Unused Use Statements -2 removed lines

@@ -1,8 +1,6 @@
 <?php
 
 use GuzzleHttp\Client;
-use GuzzleHttp\Psr7\Request;
-use GuzzleHttp\Psr7\Response;
 
 class AlertControllerTest extends \PHPUnit_Framework_TestCase
 {

src/Controller/Endpoint/AbstractEndpointController.php

Spacing +1 added lines, -1 removed lines

@@ -272,7 +272,7 @@
     {
         $queryString = $request->query->get('embed');
 
-        if (! empty($queryString)) {
+        if (!empty($queryString)) {
             $this->fractal->parseIncludes($request->query->get('embed'));
         }
     }

Doc Comments +10 added lines, -10 removed lines

@@ -104,7 +104,7 @@ 
      * @param  \League\Fractal\TransformerAbstract        $transformer The Transformer to pass through to Fractal
      * @param  \Symfony\Component\HttpFoundation\Response $response The client's response
      *
-     * @return array
+     * @return Response
      */
     protected function respondWithItem($item, $transformer, Response $response)
     {
@@ -134,7 +134,7 @@ 
      * @param  \League\Fractal\TransformerAbstract        $transformer The Transformer to pass through to Fractal
      * @param  \Symfony\Component\HttpFoundation\Response $response    The client's response
      *
-     * @return array
+     * @return Response
      */
     protected function respondWithCollection($collection, $transformer, Response $response)
     {
@@ -145,7 +145,7 @@ 
      * Creates a collection array and sends it back to hence it came.
      *
      * @param  array                               $collection
-     * @param  /League/Fractal/TransformerAbstract $transformer
+     * @param  \League\Fractal\TransformerAbstract $transformer
      *
      * @return array
      */
@@ -182,7 +182,7 @@ 
      * @param  string                                     $message   Response message to put in the error
      * @param  int                                        $errorCode Error code to set
      *
-     * @return array
+     * @return Response
      */
     protected function respondWithError(Response $response, $message, $errorCode)
     {
@@ -209,7 +209,7 @@ 
      * @param  \Symfony\Component\HttpFoundation\Response $response
      * @param  string                                     $message
      *
-     * @return void
+     * @return Response
      */
     public function errorEmpty(Response $response, $message = 'No data / Empty')
     {
@@ -223,7 +223,7 @@ 
      * @param  \Symfony\Component\HttpFoundation\Response $response
      * @param  string                                     $message
      *
-     * @return void
+     * @return Response
      */
     public function errorForbidden(Response $response, $message = 'Forbidden')
     {
@@ -237,7 +237,7 @@ 
      * @param  \Symfony\Component\HttpFoundation\Response $response
      * @param  string                                     $message
      *
-     * @return void
+     * @return Response
      */
     public function errorInternalError(Response $response, $message = 'Internal Error')
     {
@@ -251,7 +251,7 @@ 
      * @param  \Symfony\Component\HttpFoundation\Response $response
      * @param  string                                     $message
      *
-     * @return void
+     * @return Response
      */
     public function errorNotFound(Response $response, $message = 'Resource Not Found')
     {
@@ -265,7 +265,7 @@ 
      * @param  \Symfony\Component\HttpFoundation\Response $response
      * @param  string                                     $message
      *
-     * @return void
+     * @return Response
      */
     public function errorUnauthorized(Response $response, $message = 'Unauthorized')
     {
@@ -290,7 +290,7 @@ 
     /**
      * Reads any requested includes and adds them to the item / collection
      *
-     * @param  Symfony\Component\HttpFoundation\Request $request
+     * @param  Request $request
      *
      * @return void
      */

src/Repository/AbstractEndpointRepository.php

Indentation +2 added lines, -2 removed lines

@@ -89,7 +89,7 @@ 
         $key = $this->returnKeyType($keyType);
 
         $query->cols(['*'])
-              ->where("`{$key}` = '{$id}'");
+                ->where("`{$key}` = '{$id}'");
 
         return $this->fireStatementAndReturn($query, true);
     }
@@ -107,7 +107,7 @@ 
         $key = $this->returnKeyType($keyType);
 
         $query->cols(['*'])
-              ->where("`{$key}` = '{$id}'");
+                ->where("`{$key}` = '{$id}'");
 
         return $this->fireStatementAndReturn($query);
     }

Doc Comments +1 added lines, -1 removed lines

@@ -78,8 +78,8 @@
     /**
      * Allows for Raw SQL firing without the query builder
      *
-     * @param  \Aura\SqlQuery\AbstractQuery $query
      * @param  boolean                      $single
+     * @param string $sql
      *
      * @return array
      */

src/Controller/Endpoint/Alerts/AlertCountsEndpointController.php

Doc Comments +7 added lines, -7 removed lines

@@ -20,9 +20,9 @@ 
     /**
      * Construct
      *
-     * @param Ps2alerts\Api\Repository\AlertRepository   $repository
-     * @param Ps2alerts\Api\Transformer\AlertTransformer $transformer
-     * @param League\Fractal\Manager                     $fractal
+     * @param AlertRepository   $repository
+     * @param AlertTransformer $transformer
+     * @param Manager                     $fractal
      */
     public function __construct(
         AlertRepository  $repository,
@@ -37,10 +37,10 @@ 
     /**
      * Returns the victories of each faction and the totals
      *
-     * @param  Symfony\Component\HttpFoundation\Request  $request
-     * @param  Symfony\Component\HttpFoundation\Response $response
+     * @param  Request  $request
+     * @param  Response $response
      *
-     * @return array
+     * @return Response
      */
     public function getVictories(Request $request, Response $response)
     {
@@ -53,7 +53,7 @@ 
      * @param  \Symfony\Component\HttpFoundation\Request  $request
      * @param  \Symfony\Component\HttpFoundation\Response $response
      *
-     * @return array
+     * @return Response
      */
     public function getDominations(Request $request, Response $response)
     {

Spacing +6 added lines, -6 removed lines

@@ -76,12 +76,12 @@ 
         $servers      = $this->getConfigItem('servers');
         $zones        = $this->getConfigItem('zones');
 
-        if (! empty($serversQuery)) {
+        if (!empty($serversQuery)) {
             $check = explode(',', $serversQuery);
 
             // Run a check on the IDs provided to make sure they're valid and no naughty things are being passed
-            foreach($check as $id) {
-                if (! in_array($id, $servers)) {
+            foreach ($check as $id) {
+                if (!in_array($id, $servers)) {
                     return $this->errorWrongArgs($response, 'Invalid Server Arguments passed');
                 }
             }
@@ -91,12 +91,12 @@ 
             $servers = implode(',', $servers);
         }
 
-        if (! empty($zonesQuery)) {
+        if (!empty($zonesQuery)) {
             $check = explode(',', $zonesQuery);
 
             // Run a check on the IDs provided to make sure they're valid and no naughty things are being passed
-            foreach($check as $id) {
-                if (! in_array($id, $zones)) {
+            foreach ($check as $id) {
+                if (!in_array($id, $zones)) {
                     return $this->errorWrongArgs($response, 'Invalid Zone Arguments passed');
                 }
             }