Merge pull request #20 from mtrajano/analysis-KZZ4aR

Apply fixes from StyleCI

Author: mtrajano

config/laravel-swagger.php

@@ -62,4 +62,4 @@
     */
 
     'parseDocBlock' => true,
-];
+];

src/FormatterManager.php

@@ -40,4 +40,4 @@ public function format()
     {
         return $this->formatter->format();
     }
-}
+}

src/Formatters/Formatter.php

@@ -12,4 +12,4 @@ public function __construct($docs)
     }
 
     abstract public function format();
-}
+}

src/Formatters/JsonFormatter.php

@@ -14,4 +14,4 @@ public function format()
 
         return json_encode($this->docs, JSON_PRETTY_PRINT);
     }
-}
+}

src/Formatters/YamlFormatter.php

@@ -14,4 +14,4 @@ public function format()
 
         return yaml_emit($this->docs);
     }
-}
+}

src/Generator.php

@@ -2,11 +2,11 @@
 
 namespace Mtrajano\LaravelSwagger;
 
-use ReflectionMethod;
-use Illuminate\Support\Str;
-use Illuminate\Routing\Route;
 use Illuminate\Foundation\Http\FormRequest;
+use Illuminate\Routing\Route;
+use Illuminate\Support\Str;
 use phpDocumentor\Reflection\DocBlockFactory;
+use ReflectionMethod;
 
 class Generator
 {
@@ -53,7 +53,9 @@ public function generate()
             foreach ($methods as $method) {
                 $this->method = strtolower($method);
 
-                if (in_array($this->method, $this->config['ignoredMethods'])) continue;
+                if (in_array($this->method, $this->config['ignoredMethods'])) {
+                    continue;
+                }
 
                 $this->generatePath();
             }
@@ -111,18 +113,18 @@ protected function getRouteUri(Route $route)
     protected function generatePath()
     {
         $actionInstance = is_string($this->action) ? $this->getActionClassInstance($this->action) : null;
-        $docBlock = $actionInstance ? ($actionInstance->getDocComment() ?: "") : "";
+        $docBlock = $actionInstance ? ($actionInstance->getDocComment() ?: '') : '';
 
-        list($isDeprecated, $summary, $description) = $this->parseActionDocBlock($docBlock);
+        [$isDeprecated, $summary, $description] = $this->parseActionDocBlock($docBlock);
 
         $this->docs['paths'][$this->uri][$this->method] = [
             'summary' => $summary,
             'description' => $description,
             'deprecated' => $isDeprecated,
             'responses' => [
                 '200' => [
-                    'description' => 'OK'
-                ]
+                    'description' => 'OK',
+                ],
             ],
         ];
 
@@ -148,7 +150,9 @@ protected function addActionParameters()
 
     protected function getFormRules()
     {
-        if (!is_string($this->action)) return false;
+        if (!is_string($this->action)) {
+            return false;
+        }
 
         $parameters = $this->getActionClassInstance($this->action)->getParameters();
 
@@ -175,15 +179,15 @@ protected function getParameterGenerator($rules)
 
     private function getActionClassInstance(string $action)
     {
-        list($class, $method) = Str::parseCallback($action);
+        [$class, $method] = Str::parseCallback($action);
 
         return new ReflectionMethod($class, $method);
     }
 
     private function parseActionDocBlock(string $docBlock)
     {
         if (empty($docBlock) || !$this->config['parseDocBlock']) {
-            return [false, "", ""];
+            return [false, '', ''];
         }
 
         try {
@@ -195,8 +199,8 @@ private function parseActionDocBlock(string $docBlock)
             $description = (string) $parsedComment->getDescription();
 
             return [$isDeprecated, $summary, $description];
-        } catch(\Exception $e) {
-            return [false, "", ""];
+        } catch (\Exception $e) {
+            return [false, '', ''];
         }
     }
-}
+}

src/LaravelSwaggerException.php

@@ -4,4 +4,4 @@
 
 class LaravelSwaggerException extends \Exception
 {
-}
+}

src/Parameters/BodyParameterGenerator.php

@@ -81,7 +81,7 @@ protected function addToProperties(&$properties, $nameTokens, $rules)
 
         if ($type === 'array') {
             $this->addToProperties($properties[$name]['items'], $nameTokens, $rules);
-        } else if ($type === 'object') {
+        } elseif ($type === 'object') {
             $this->addToProperties($properties[$name]['properties'], $nameTokens, $rules);
         }
     }
@@ -98,7 +98,7 @@ protected function getNestedParamType($nameTokens)
     protected function getNewPropObj($type, $rules)
     {
         $propObj = [
-            'type' => $type
+            'type' => $type,
         ];
 
         if ($enums = $this->getEnumValues($rules)) {
@@ -107,10 +107,10 @@ protected function getNewPropObj($type, $rules)
 
         if ($type === 'array') {
             $propObj['items'] = [];
-        } else if ($type === 'object') {
+        } elseif ($type === 'object') {
             $propObj['properties'] = [];
         }
 
         return $propObj;
     }
-}
+}

src/Parameters/Concerns/GeneratesFromRules.php

@@ -19,11 +19,11 @@ protected function getParamType(array $paramRules)
     {
         if (in_array('integer', $paramRules)) {
             return 'integer';
-        } else if (in_array('numeric', $paramRules)) {
+        } elseif (in_array('numeric', $paramRules)) {
             return 'number';
-        } else if (in_array('boolean', $paramRules)) {
+        } elseif (in_array('boolean', $paramRules)) {
             return 'boolean';
-        } else if (in_array('array', $paramRules)) {
+        } elseif (in_array('array', $paramRules)) {
             return 'array';
         } else {
             //date, ip, email, etc..
@@ -54,15 +54,15 @@ protected function getEnumValues(array $paramRules)
             return [];
         }
 
-        list($param, $vals) = explode(':', $in);
+        [$param, $vals] = explode(':', $in);
 
         return explode(',', $vals);
     }
 
     private function getInParameter(array $paramRules)
     {
         foreach ($paramRules as $rule) {
-            if ((is_string($rule) || method_exists($rule , '__toString')) && Str::startsWith($rule, 'in:')) {
+            if ((is_string($rule) || method_exists($rule, '__toString')) && Str::startsWith($rule, 'in:')) {
                 return $rule;
             }
         }

src/Parameters/ParameterGenerator.php

@@ -7,4 +7,4 @@ interface ParameterGenerator
     public function getParameters();
 
     public function getParamLocation();
-}
+}