Browse Source

Apply fixes from StyleCI

Marcel Pociot 5 years ago
parent
commit
ffee319db4

+ 1 - 1
src/Strategies/Responses/ResponseCalls.php

@@ -304,7 +304,7 @@ class ResponseCalls extends Strategy
             return false;
         }
 
-        if (!empty($context['responses'])) {
+        if (! empty($context['responses'])) {
             // Don't attempt a response call if there are already responses
             return false;
         }

+ 1 - 2
src/Strategies/Responses/UseResponseFileTag.php

@@ -4,7 +4,6 @@ namespace Mpociot\ApiDoc\Strategies\Responses;
 
 use Illuminate\Routing\Route;
 use Mpociot\Reflection\DocBlock;
-use Illuminate\Http\JsonResponse;
 use Mpociot\Reflection\DocBlock\Tag;
 use Mpociot\ApiDoc\Strategies\Strategy;
 use Mpociot\ApiDoc\Tools\RouteDocBlocker;
@@ -65,6 +64,6 @@ class UseResponseFileTag extends Strategy
         }, $responseFileTags);
 
         // Convert responses to [200 => 'response', 401 => 'response']
-        return collect($responses)->pluck("0", "1")->toArray();
+        return collect($responses)->pluck('0', '1')->toArray();
     }
 }

+ 1 - 2
src/Strategies/Responses/UseResponseTag.php

@@ -4,7 +4,6 @@ namespace Mpociot\ApiDoc\Strategies\Responses;
 
 use Illuminate\Routing\Route;
 use Mpociot\Reflection\DocBlock;
-use Illuminate\Http\JsonResponse;
 use Mpociot\Reflection\DocBlock\Tag;
 use Mpociot\ApiDoc\Strategies\Strategy;
 use Mpociot\ApiDoc\Tools\RouteDocBlocker;
@@ -63,6 +62,6 @@ class UseResponseTag extends Strategy
         }, $responseTags);
 
         // Convert responses to [200 => 'response', 401 => 'response']
-        return collect($responses)->pluck("0", "1")->toArray();
+        return collect($responses)->pluck('0', '1')->toArray();
     }
 }

+ 5 - 4
src/Tools/Generator.php

@@ -6,7 +6,6 @@ use ReflectionClass;
 use ReflectionMethod;
 use Illuminate\Routing\Route;
 use Mpociot\ApiDoc\Tools\Traits\ParamHelpers;
-use Symfony\Component\HttpFoundation\Response;
 
 class Generator
 {
@@ -56,7 +55,7 @@ class Generator
         $method = $controller->getMethod($methodName);
 
         $parsedRoute = [
-            'id' => md5($this->getUri($route) . ':' . implode($this->getMethods($route))),
+            'id' => md5($this->getUri($route).':'.implode($this->getMethods($route))),
             'methods' => $this->getMethods($route),
             'uri' => $this->getUri($route),
             'boundUri' => Utils::getFullUrl($route, $rulesToApply['bindings'] ?? ($rulesToApply['response_calls']['bindings'] ?? [])),
@@ -73,7 +72,7 @@ class Generator
 
         $responses = $this->fetchResponses($controller, $method, $route, $rulesToApply, $parsedRoute);
         $parsedRoute['response'] = $responses;
-        $parsedRoute['showresponse'] = !empty($responses);
+        $parsedRoute['showresponse'] = ! empty($responses);
 
         $parsedRoute['headers'] = $rulesToApply['headers'] ?? [];
 
@@ -93,6 +92,7 @@ class Generator
             'description' => '',
             'authenticated' => false,
         ];
+
         return $this->iterateThroughStrategies('metadata', $context, [$route, $controller, $method, $rulesToApply]);
     }
 
@@ -129,7 +129,7 @@ class Generator
             $strategy = new $strategyClass($this->config);
             $arguments[] = $context;
             $results = $strategy(...$arguments);
-            if (!is_null($results)) {
+            if (! is_null($results)) {
                 foreach ($results as $index => $item) {
                     // Using a for loop rather than array_merge or +=
                     // so it does not renumber numeric keys
@@ -144,6 +144,7 @@ class Generator
                 }
             }
         }
+
         return $context[$key];
     }
 }

+ 1 - 1
tests/Unit/LaravelGeneratorTest.php

@@ -28,7 +28,7 @@ class LaravelGeneratorTest extends GeneratorTestCase
     public function createRouteUsesArray(string $httpMethod, string $path, string $controllerMethod, $register = false, $class = TestController::class)
     {
         if ($register) {
-            return RouteFacade::{$httpMethod}($path, [$class. "$controllerMethod"]);
+            return RouteFacade::{$httpMethod}($path, [$class."$controllerMethod"]);
         } else {
             return new Route([$httpMethod], $path, ['uses' => [$class, $controllerMethod]]);
         }