@@ -5,9 +5,7 @@ |
||
| 5 | 5 | |
| 6 | 6 | |
| 7 | 7 | use PhpParser\Node; |
| 8 | -use PhpParser\Node\Stmt\Catch_; |
|
| 9 | 8 | use PHPStan\Analyser\Scope; |
| 10 | -use PHPStan\Broker\Broker; |
|
| 11 | 9 | use PHPStan\Rules\Rule; |
| 12 | 10 | |
| 13 | 11 | /** |
@@ -7,7 +7,6 @@ |
||
| 7 | 7 | use PhpParser\Node; |
| 8 | 8 | use PhpParser\Node\Stmt\Catch_; |
| 9 | 9 | use PHPStan\Analyser\Scope; |
| 10 | -use PHPStan\Broker\Broker; |
|
| 11 | 10 | use PHPStan\Rules\Rule; |
| 12 | 11 | |
| 13 | 12 | class EmptyExceptionRule implements Rule |
@@ -9,7 +9,6 @@ |
||
| 9 | 9 | use PhpParser\NodeTraverser; |
| 10 | 10 | use PhpParser\NodeVisitorAbstract; |
| 11 | 11 | use PHPStan\Analyser\Scope; |
| 12 | -use PHPStan\Broker\Broker; |
|
| 13 | 12 | use PHPStan\Rules\Rule; |
| 14 | 13 | |
| 15 | 14 | /** |
@@ -3,12 +3,9 @@ |
||
| 3 | 3 | |
| 4 | 4 | namespace TheCodingMachine\PHPStan\Rules\TypeHints; |
| 5 | 5 | |
| 6 | -use Roave\BetterReflection\Reflection\ReflectionFunction; |
|
| 7 | 6 | use Roave\BetterReflection\Reflection\ReflectionFunctionAbstract; |
| 8 | -use Roave\BetterReflection\Reflection\ReflectionMethod; |
|
| 9 | 7 | use Roave\BetterReflection\Reflection\ReflectionParameter; |
| 10 | 8 | use PhpParser\Node; |
| 11 | -use PHPStan\Reflection\MethodReflection; |
|
| 12 | 9 | |
| 13 | 10 | class MissingTypeHintInMethodRule extends AbstractMissingTypeHintRule |
| 14 | 11 | { |
@@ -1,5 +1,5 @@ discard block |
||
| 1 | 1 | <?php |
| 2 | -declare(strict_types=1); |
|
| 2 | +declare(strict_types = 1); |
|
| 3 | 3 | |
| 4 | 4 | namespace TheCodingMachine\PHPStan\Rules\TypeHints; |
| 5 | 5 | |
@@ -174,9 +174,9 @@ discard block |
||
| 174 | 174 | foreach ($docblockWithoutNullable as $docblockTypehint) { |
| 175 | 175 | if (!$docblockTypehint instanceof Array_) { |
| 176 | 176 | if ($context instanceof ReflectionParameter) { |
| 177 | - return sprintf('%s, mismatching type-hints for parameter %s. PHP type hint is "array" and docblock type hint is %s.', $this->getContext($context), $context->getName(), (string)$docblockTypehint); |
|
| 177 | + return sprintf('%s, mismatching type-hints for parameter %s. PHP type hint is "array" and docblock type hint is %s.', $this->getContext($context), $context->getName(), (string) $docblockTypehint); |
|
| 178 | 178 | } else { |
| 179 | - return sprintf('%s, mismatching type-hints for return type. PHP type hint is "array" and docblock declared return type is %s.', $this->getContext($context), (string)$docblockTypehint); |
|
| 179 | + return sprintf('%s, mismatching type-hints for return type. PHP type hint is "array" and docblock declared return type is %s.', $this->getContext($context), (string) $docblockTypehint); |
|
| 180 | 180 | } |
| 181 | 181 | } |
| 182 | 182 | |
@@ -244,11 +244,11 @@ discard block |
||
| 244 | 244 | $type = $types[0]; |
| 245 | 245 | |
| 246 | 246 | if ($this->isNativeType($type)) { |
| 247 | - return ($isNullable?'?':'').((string)$type); |
|
| 247 | + return ($isNullable ? '?' : '').((string) $type); |
|
| 248 | 248 | } |
| 249 | 249 | |
| 250 | 250 | if ($type instanceof Array_) { |
| 251 | - return ($isNullable?'?':'').'array'; |
|
| 251 | + return ($isNullable ? '?' : '').'array'; |
|
| 252 | 252 | } |
| 253 | 253 | |
| 254 | 254 | // TODO: more definitions to add here |
@@ -256,7 +256,7 @@ discard block |
||
| 256 | 256 | // Manage array of things => (cast to array) |
| 257 | 257 | |
| 258 | 258 | if ($type instanceof Object_) { |
| 259 | - return ($isNullable?'?':'').((string)$type); |
|
| 259 | + return ($isNullable ? '?' : '').((string) $type); |
|
| 260 | 260 | } |
| 261 | 261 | |
| 262 | 262 | return null; |
@@ -4,7 +4,6 @@ discard block |
||
| 4 | 4 | namespace TheCodingMachine\PHPStan\Rules\TypeHints; |
| 5 | 5 | |
| 6 | 6 | |
| 7 | -use phpDocumentor\Reflection\DocBlockFactory; |
|
| 8 | 7 | use phpDocumentor\Reflection\Type; |
| 9 | 8 | use phpDocumentor\Reflection\Types\Array_; |
| 10 | 9 | use phpDocumentor\Reflection\Types\Boolean; |
@@ -25,8 +24,6 @@ discard block |
||
| 25 | 24 | use Roave\BetterReflection\Reflection\ReflectionFunction; |
| 26 | 25 | use Roave\BetterReflection\Reflection\ReflectionMethod; |
| 27 | 26 | use Roave\BetterReflection\Reflection\ReflectionParameter; |
| 28 | -use Roave\BetterReflection\TypesFinder\PhpDocumentor\NamespaceNodeToReflectionTypeContext; |
|
| 29 | -use Roave\BetterReflection\TypesFinder\ResolveTypes; |
|
| 30 | 27 | |
| 31 | 28 | abstract class AbstractMissingTypeHintRule implements Rule |
| 32 | 29 | { |