Browse Source

Applied fixes from StyleCI

Marcel Pociot 9 years ago
parent
commit
3ba920c92c

+ 6 - 1
src/Mpociot/ApiDoc/Commands/GenerateDocumentation.php

@@ -120,6 +120,7 @@ class GenerateDocumentation extends Command
             list($name, $id) = explode(',', $binding);
             $resultBindings[$name] = $id;
         }
+
         return $resultBindings;
     }
 
@@ -128,7 +129,7 @@ class GenerateDocumentation extends Command
      */
     private function setUserToBeImpersonated($actAs)
     {
-        if (!empty($actAs)) {
+        if (! empty($actAs)) {
             if (version_compare($this->laravel->version(), '5.2.0', '<')) {
                 $userModel = config('auth.model');
                 $user = $userModel::find($actAs);
@@ -157,6 +158,7 @@ class GenerateDocumentation extends Command
      * @param AbstractGenerator  $generator
      * @param $allowedRoutes
      * @param $routePrefix
+     *
      * @return array
      */
     private function processLaravelRoutes(AbstractGenerator $generator, $allowedRoutes, $routePrefix)
@@ -170,6 +172,7 @@ class GenerateDocumentation extends Command
                 $this->info('Processed route: '.$route->getUri());
             }
         }
+
         return $parsedRoutes;
     }
 
@@ -177,6 +180,7 @@ class GenerateDocumentation extends Command
      * @param AbstractGenerator $generator
      * @param $allowedRoutes
      * @param $routePrefix
+     *
      * @return array
      */
     private function processDingoRoutes(AbstractGenerator $generator, $allowedRoutes, $routePrefix)
@@ -190,6 +194,7 @@ class GenerateDocumentation extends Command
                 $this->info('Processed route: '.$route->uri());
             }
         }
+
         return $parsedRoutes;
     }
 }

+ 7 - 4
src/Mpociot/ApiDoc/Generators/AbstractGenerator.php

@@ -12,9 +12,9 @@ use Illuminate\Foundation\Http\FormRequest;
 
 abstract class AbstractGenerator
 {
-
     /**
      * @param Route $route
+     *
      * @return mixed
      */
     abstract protected function getUri(Route $route);
@@ -30,9 +30,11 @@ abstract class AbstractGenerator
     /**
      * @param array $routeData
      * @param array $routeAction
+     *
      * @return mixed
      */
-    protected function getParameters($routeData, $routeAction) {
+    protected function getParameters($routeData, $routeAction)
+    {
         $validator = Validator::make([], $this->getRouteRules($routeAction['uses']));
         foreach ($validator->getRules() as $attribute => $rules) {
             $attributeData = [
@@ -65,10 +67,10 @@ abstract class AbstractGenerator
         return $this->callRoute(array_shift($methods), $uri);
     }
 
-
     /**
      * @param Route $route
      * @param array $bindings
+     *
      * @return mixed
      */
     protected function addRouteModelBindings(Route $route, $bindings)
@@ -77,6 +79,7 @@ abstract class AbstractGenerator
         foreach ($bindings as $model => $id) {
             $uri = str_replace('{'.$model.'}', $id, $uri);
         }
+
         return $uri;
     }
 
@@ -100,7 +103,6 @@ abstract class AbstractGenerator
         ];
     }
 
-
     /**
      * @param  string  $route
      *
@@ -117,6 +119,7 @@ abstract class AbstractGenerator
                 return $tag->getContent();
             }
         }
+
         return 'general';
     }
 

+ 3 - 7
src/Mpociot/ApiDoc/Generators/DingoGenerator.php

@@ -4,11 +4,9 @@ namespace Mpociot\ApiDoc\Generators;
 
 use Illuminate\Routing\Route;
 use Illuminate\Support\Facades\App;
-use Illuminate\Support\Facades\Request;
 
 class DingoGenerator extends AbstractGenerator
 {
-
     /**
      * @param  \Illuminate\Routing\Route $route
      * @param array $bindings
@@ -22,7 +20,7 @@ class DingoGenerator extends AbstractGenerator
         $routeAction = $route->getAction();
         $routeGroup = $this->getRouteGroup($routeAction['uses']);
         $routeDescription = $this->getRouteDescription($routeAction['uses']);
-        
+
         if ($response->headers->get('Content-Type') === 'application/json') {
             $content = json_encode(json_decode($response->getContent()), JSON_PRETTY_PRINT);
         } else {
@@ -30,7 +28,7 @@ class DingoGenerator extends AbstractGenerator
         }
 
         return $this->getParameters([
-            'resource'    => $routeGroup,
+            'resource' => $routeGroup,
             'title' => $routeDescription['short'],
             'description' => $routeDescription['long'],
             'methods' => $route->getMethods(),
@@ -40,7 +38,6 @@ class DingoGenerator extends AbstractGenerator
         ], $routeAction);
     }
 
-
     /**
      * {@inheritdoc}
      */
@@ -56,5 +53,4 @@ class DingoGenerator extends AbstractGenerator
     {
         return $route->uri();
     }
-    
-}
+}

+ 5 - 6
src/Mpociot/ApiDoc/Generators/LaravelGenerator.php

@@ -8,9 +8,9 @@ use Illuminate\Support\Facades\Request;
 
 class LaravelGenerator extends AbstractGenerator
 {
-
     /**
      * @param Route $route
+     *
      * @return mixed
      */
     protected function getUri(Route $route)
@@ -27,11 +27,11 @@ class LaravelGenerator extends AbstractGenerator
     public function processRoute(Route $route, $bindings = [])
     {
         $response = $this->getRouteResponse($route, $bindings);
-        
+
         $routeAction = $route->getAction();
         $routeGroup = $this->getRouteGroup($routeAction['uses']);
         $routeDescription = $this->getRouteDescription($routeAction['uses']);
-        
+
         if ($response->headers->get('Content-Type') === 'application/json') {
             $content = json_encode(json_decode($response->getContent()), JSON_PRETTY_PRINT);
         } else {
@@ -39,7 +39,7 @@ class LaravelGenerator extends AbstractGenerator
         }
 
         return $this->getParameters([
-            'resource'    => $routeGroup,
+            'resource' => $routeGroup,
             'title' => $routeDescription['short'],
             'description' => $routeDescription['long'],
             'methods' => $route->getMethods(),
@@ -83,5 +83,4 @@ class LaravelGenerator extends AbstractGenerator
 
         return $response;
     }
-    
-}
+}

+ 0 - 1
tests/ApiDocGeneratorTest.php

@@ -5,7 +5,6 @@ namespace Mpociot\ApiDoc\Tests;
 use Illuminate\Routing\Route;
 use Mpociot\ApiDoc\Generators\LaravelGenerator;
 use Orchestra\Testbench\TestCase;
-use Mpociot\ApiDoc\AbstractGenerator;
 use Mpociot\ApiDoc\Tests\Fixtures\TestRequest;
 use Mpociot\ApiDoc\Tests\Fixtures\TestController;
 use Illuminate\Support\Facades\Route as RouteFacade;