@@ -20,6 +20,10 @@ |
||
| 20 | 20 | } |
| 21 | 21 | } |
| 22 | 22 | |
| 23 | + /** |
|
| 24 | + * @param integer $elseCount |
|
| 25 | + * @param integer $ifBody |
|
| 26 | + */ |
|
| 23 | 27 | private static function shouldBeFlipped($elseCount, $ifBody) |
| 24 | 28 | { |
| 25 | 29 | $ifIsLonger = ($elseCount + 10) < $ifBody; |
@@ -58,6 +58,9 @@ |
||
| 58 | 58 | return $stringOutput; |
| 59 | 59 | } |
| 60 | 60 | |
| 61 | + /** |
|
| 62 | + * @param \Closure $refactor |
|
| 63 | + */ |
|
| 61 | 64 | private static function recursiveRefactor($tokens, $refactor) |
| 62 | 65 | { |
| 63 | 66 | $i = $changes = 0; |
@@ -71,6 +71,9 @@ |
||
| 71 | 71 | return [$tokens, $changes]; |
| 72 | 72 | } |
| 73 | 73 | |
| 74 | + /** |
|
| 75 | + * @return string |
|
| 76 | + */ |
|
| 74 | 77 | private static function getKeyword($token) |
| 75 | 78 | { |
| 76 | 79 | return self::scopeKeywords[$token]; |
@@ -60,6 +60,9 @@ |
||
| 60 | 60 | return 'Error on route '.$msg; |
| 61 | 61 | } |
| 62 | 62 | |
| 63 | + /** |
|
| 64 | + * @param ErrorPrinter $errorPrinter |
|
| 65 | + */ |
|
| 63 | 66 | private function checkRouteDefinitions($errorPrinter, $routes) |
| 64 | 67 | { |
| 65 | 68 | foreach ($routes as $route) { |
@@ -56,6 +56,9 @@ discard block |
||
| 56 | 56 | return app(ErrorPrinter::class)->hasErrors() ? 1 : 0; |
| 57 | 57 | } |
| 58 | 58 | |
| 59 | + /** |
|
| 60 | + * @param integer $tries |
|
| 61 | + */ |
|
| 59 | 62 | private function fix($filePath, $tokens, $tries) |
| 60 | 63 | { |
| 61 | 64 | Refactor::saveTokens($filePath, $tokens, $this->option('test')); |
@@ -73,6 +76,9 @@ discard block |
||
| 73 | 76 | return [$fixes, $tokens]; |
| 74 | 77 | } |
| 75 | 78 | |
| 79 | + /** |
|
| 80 | + * @param integer $fixed |
|
| 81 | + */ |
|
| 76 | 82 | private function printFinalMsg($fixed) |
| 77 | 83 | { |
| 78 | 84 | if ($fixed > 0) { |
@@ -59,6 +59,9 @@ discard block |
||
| 59 | 59 | return class_exists($eventName, false) ? $this->addOriginInterfaceListeners($eventName, $listeners) : $listeners; |
| 60 | 60 | } |
| 61 | 61 | |
| 62 | + /** |
|
| 63 | + * @param string $string |
|
| 64 | + */ |
|
| 62 | 65 | private function error($string) |
| 63 | 66 | { |
| 64 | 67 | $len = strlen($string); |
@@ -95,6 +98,9 @@ discard block |
||
| 95 | 98 | } |
| 96 | 99 | } |
| 97 | 100 | |
| 101 | + /** |
|
| 102 | + * @return string |
|
| 103 | + */ |
|
| 98 | 104 | private function stringify($event) |
| 99 | 105 | { |
| 100 | 106 | return is_object($event) ? get_class($event) : $event; |
@@ -116,6 +122,9 @@ discard block |
||
| 116 | 122 | return 'The method of '.$at.' is not callable as an event listener for "'.$e.'" event'; |
| 117 | 123 | } |
| 118 | 124 | |
| 125 | + /** |
|
| 126 | + * @return string |
|
| 127 | + */ |
|
| 119 | 128 | protected function getTypeHintedClass($listenerObj, $methodName) |
| 120 | 129 | { |
| 121 | 130 | try { |
@@ -141,6 +150,9 @@ discard block |
||
| 141 | 150 | return $listeners; |
| 142 | 151 | } |
| 143 | 152 | |
| 153 | + /** |
|
| 154 | + * @param \Closure $listener |
|
| 155 | + */ |
|
| 144 | 156 | private function stringifyClosure($listener) |
| 145 | 157 | { |
| 146 | 158 | try { |
@@ -35,6 +35,9 @@ discard block |
||
| 35 | 35 | } |
| 36 | 36 | } |
| 37 | 37 | |
| 38 | + /** |
|
| 39 | + * @param \Illuminate\Routing\Route $route |
|
| 40 | + */ |
|
| 38 | 41 | private function writeIt($route, $filename) |
| 39 | 42 | { |
| 40 | 43 | try { |
@@ -57,6 +60,9 @@ discard block |
||
| 57 | 60 | } |
| 58 | 61 | } |
| 59 | 62 | |
| 63 | + /** |
|
| 64 | + * @param string $value |
|
| 65 | + */ |
|
| 60 | 66 | private function deduceRoute($value) |
| 61 | 67 | { |
| 62 | 68 | if (Str::containsAll($value, ['@', '\\'])) { |
@@ -68,6 +74,9 @@ discard block |
||
| 68 | 74 | return $route; |
| 69 | 75 | } |
| 70 | 76 | |
| 77 | + /** |
|
| 78 | + * @param \Illuminate\Routing\Route $route |
|
| 79 | + */ |
|
| 71 | 80 | private function printIt($route) |
| 72 | 81 | { |
| 73 | 82 | try { |
@@ -80,6 +89,10 @@ discard block |
||
| 80 | 89 | } |
| 81 | 90 | } |
| 82 | 91 | |
| 92 | + /** |
|
| 93 | + * @param string $action |
|
| 94 | + * @param string $middlewareSection |
|
| 95 | + */ |
|
| 83 | 96 | private function getMovableRoute($route, $methods, $action, $middlewareSection) |
| 84 | 97 | { |
| 85 | 98 | if ($action == '\Illuminate\Routing\ViewController::class') { |
@@ -117,6 +130,9 @@ discard block |
||
| 117 | 130 | return ($middlewares ? '->middleware(['.$middlewares."])" : ''); |
| 118 | 131 | } |
| 119 | 132 | |
| 133 | + /** |
|
| 134 | + * @param Exception $e |
|
| 135 | + */ |
|
| 120 | 136 | private function handleRouteProblem($e) |
| 121 | 137 | { |
| 122 | 138 | $this->info('The route has some problem.'); |
@@ -124,6 +140,9 @@ discard block |
||
| 124 | 140 | $this->info($e->getFile()); |
| 125 | 141 | } |
| 126 | 142 | |
| 143 | + /** |
|
| 144 | + * @param string $middlewares |
|
| 145 | + */ |
|
| 127 | 146 | private function prettyPrintInConsole($route, $middlewares) |
| 128 | 147 | { |
| 129 | 148 | $this->getOutput()->writeln('---------------------------------------------------'); |
@@ -17,10 +17,10 @@ discard block |
||
| 17 | 17 | /** |
| 18 | 18 | * Get all of the listeners and their corresponding events. |
| 19 | 19 | * |
| 20 | - * @param iterable $paths |
|
| 20 | + * @param \Symfony\Component\Finder\Finder $paths |
|
| 21 | 21 | * @param $composerPath |
| 22 | 22 | * @param $composerNamespace |
| 23 | - * @param $command |
|
| 23 | + * @param Commands\CheckPsr4 $command |
|
| 24 | 24 | * |
| 25 | 25 | * @return void |
| 26 | 26 | */ |
@@ -83,6 +83,9 @@ discard block |
||
| 83 | 83 | app(ErrorPrinter::class)->errorsList['total'] = 0; |
| 84 | 84 | } |
| 85 | 85 | |
| 86 | + /** |
|
| 87 | + * @param string $relativePath |
|
| 88 | + */ |
|
| 86 | 89 | private static function warn($currentNamespace, $relativePath) |
| 87 | 90 | { |
| 88 | 91 | /** |
@@ -110,6 +113,9 @@ discard block |
||
| 110 | 113 | return Str::startsWith($buffer, '<?php'); |
| 111 | 114 | } |
| 112 | 115 | |
| 116 | + /** |
|
| 117 | + * @param string $correctNamespace |
|
| 118 | + */ |
|
| 113 | 119 | protected static function doNamespaceCorrection($absFilePath, $currentNamespace, $correctNamespace) |
| 114 | 120 | { |
| 115 | 121 | event('laravel_microscope.namespace_fixing', get_defined_vars()); |
@@ -117,6 +123,9 @@ discard block |
||
| 117 | 123 | event('laravel_microscope.namespace_fixed', get_defined_vars()); |
| 118 | 124 | } |
| 119 | 125 | |
| 126 | + /** |
|
| 127 | + * @param string $correctNamespace |
|
| 128 | + */ |
|
| 120 | 129 | private static function ask($command, $correctNamespace) |
| 121 | 130 | { |
| 122 | 131 | return $command->getOutput()->confirm('Do you want to change it to: '.$correctNamespace, true); |
@@ -28,6 +28,9 @@ discard block |
||
| 28 | 28 | } |
| 29 | 29 | } |
| 30 | 30 | |
| 31 | + /** |
|
| 32 | + * @param string $fullNamespace |
|
| 33 | + */ |
|
| 31 | 34 | protected static function checkActions($tokens, $fullNamespace, $path) |
| 32 | 35 | { |
| 33 | 36 | $class = ClassMethods::read($tokens); |
@@ -73,6 +76,9 @@ discard block |
||
| 73 | 76 | return $routelessActions; |
| 74 | 77 | } |
| 75 | 78 | |
| 79 | + /** |
|
| 80 | + * @param \Illuminate\Routing\Route $route |
|
| 81 | + */ |
|
| 76 | 82 | protected static function getMsg($methods, $route) |
| 77 | 83 | { |
| 78 | 84 | $msg = '/**'."\n"; |