Merge pull request #390 from ergebnis/fix/wrapping

Fix: Wrapping
ergebnis · Nov 8, 2021 · 1687e7f · 1687e7f
2 parents 5433f83 + f50f947
commit 1687e7f
Show file tree

Hide file tree

Showing 23 changed files with 100 additions and 50 deletions.
diff --git a/src/Classes/FinalRule.php b/src/Classes/FinalRule.php
@@ -48,8 +48,10 @@ final class FinalRule implements Rule
     /**
      * @param string[] $classesNotRequiredToBeAbstractOrFinal
      */
-    public function __construct(bool $allowAbstractClasses, array $classesNotRequiredToBeAbstractOrFinal)
-    {
+    public function __construct(
+        bool $allowAbstractClasses,
+        array $classesNotRequiredToBeAbstractOrFinal
+    ) {
         $this->allowAbstractClasses = $allowAbstractClasses;
         $this->classesNotRequiredToBeAbstractOrFinal = \array_map(static function (string $classNotRequiredToBeAbstractOrFinal): string {
             return $classNotRequiredToBeAbstractOrFinal;
@@ -65,8 +67,10 @@ public function getNodeType(): string
         return Node\Stmt\Class_::class;
     }
 
-    public function processNode(Node $node, Scope $scope): array
-    {
+    public function processNode(
+        Node $node,
+        Scope $scope
+    ): array {
         if (!$node instanceof Node\Stmt\Class_) {
             throw new ShouldNotHappenException(\sprintf(
                 'Expected node to be instance of "%s", but got instance of "%s" instead.',

diff --git a/src/Classes/NoExtendsRule.php b/src/Classes/NoExtendsRule.php
@@ -51,8 +51,10 @@ public function getNodeType(): string
         return Node\Stmt\Class_::class;
     }
 
-    public function processNode(Node $node, Scope $scope): array
-    {
+    public function processNode(
+        Node $node,
+        Scope $scope
+    ): array {
         if (!$node instanceof Node\Stmt\Class_) {
             throw new ShouldNotHappenException(\sprintf(
                 'Expected node to be instance of "%s", but got instance of "%s" instead.',

diff --git a/src/Classes/PHPUnit/Framework/TestCaseWithSuffixRule.php b/src/Classes/PHPUnit/Framework/TestCaseWithSuffixRule.php
@@ -43,8 +43,10 @@ public function getNodeType(): string
         return Node\Stmt\Class_::class;
     }
 
-    public function processNode(Node $node, Analyser\Scope $scope): array
-    {
+    public function processNode(
+        Node $node,
+        Analyser\Scope $scope
+    ): array {
         if (!$node instanceof Node\Stmt\Class_) {
             throw new ShouldNotHappenException(\sprintf(
                 'Expected node to be instance of "%s", but got instance of "%s" instead.',

diff --git a/src/Closures/NoNullableReturnTypeDeclarationRule.php b/src/Closures/NoNullableReturnTypeDeclarationRule.php
@@ -25,8 +25,10 @@ public function getNodeType(): string
         return Node\Expr\Closure::class;
     }
 
-    public function processNode(Node $node, Scope $scope): array
-    {
+    public function processNode(
+        Node $node,
+        Scope $scope
+    ): array {
         if (!$node instanceof Node\Expr\Closure) {
             throw new ShouldNotHappenException(\sprintf(
                 'Expected node to be instance of "%s", but got instance of "%s" instead.',

diff --git a/src/Closures/NoParameterWithNullDefaultValueRule.php b/src/Closures/NoParameterWithNullDefaultValueRule.php
@@ -25,8 +25,10 @@ public function getNodeType(): string
         return Node\Expr\Closure::class;
     }
 
-    public function processNode(Node $node, Scope $scope): array
-    {
+    public function processNode(
+        Node $node,
+        Scope $scope
+    ): array {
         if (!$node instanceof Node\Expr\Closure) {
             throw new ShouldNotHappenException(\sprintf(
                 'Expected node to be instance of "%s", but got instance of "%s" instead.',

diff --git a/src/Closures/NoParameterWithNullableTypeDeclarationRule.php b/src/Closures/NoParameterWithNullableTypeDeclarationRule.php
@@ -25,8 +25,10 @@ public function getNodeType(): string
         return Node\Expr\Closure::class;
     }
 
-    public function processNode(Node $node, Scope $scope): array
-    {
+    public function processNode(
+        Node $node,
+        Scope $scope
+    ): array {
         if (!$node instanceof Node\Expr\Closure) {
             throw new ShouldNotHappenException(\sprintf(
                 'Expected node to be instance of "%s", but got instance of "%s" instead.',

diff --git a/src/Expressions/NoCompactRule.php b/src/Expressions/NoCompactRule.php
@@ -25,8 +25,10 @@ public function getNodeType(): string
         return Node\Expr\FuncCall::class;
     }
 
-    public function processNode(Node $node, Scope $scope): array
-    {
+    public function processNode(
+        Node $node,
+        Scope $scope
+    ): array {
         if (!$node instanceof Node\Expr\FuncCall) {
             throw new ShouldNotHappenException(\sprintf(
                 'Expected node to be instance of "%s", but got instance of "%s" instead.',

diff --git a/src/Expressions/NoEmptyRule.php b/src/Expressions/NoEmptyRule.php
@@ -24,8 +24,10 @@ public function getNodeType(): string
         return Node\Expr\Empty_::class;
     }
 
-    public function processNode(Node $node, Scope $scope): array
-    {
+    public function processNode(
+        Node $node,
+        Scope $scope
+    ): array {
         return [
             'Language construct empty() should not be used.',
         ];

diff --git a/src/Expressions/NoErrorSuppressionRule.php b/src/Expressions/NoErrorSuppressionRule.php
@@ -24,8 +24,10 @@ public function getNodeType(): string
         return Node\Expr\ErrorSuppress::class;
     }
 
-    public function processNode(Node $node, Scope $scope): array
-    {
+    public function processNode(
+        Node $node,
+        Scope $scope
+    ): array {
         return [
             'Error suppression via "@" should not be used.',
         ];

diff --git a/src/Expressions/NoEvalRule.php b/src/Expressions/NoEvalRule.php
@@ -24,8 +24,10 @@ public function getNodeType(): string
         return Node\Expr\Eval_::class;
     }
 
-    public function processNode(Node $node, Scope $scope): array
-    {
+    public function processNode(
+        Node $node,
+        Scope $scope
+    ): array {
         return [
             'Language construct eval() should not be used.',
         ];

diff --git a/src/Expressions/NoIssetRule.php b/src/Expressions/NoIssetRule.php
@@ -24,8 +24,10 @@ public function getNodeType(): string
         return Node\Expr\Isset_::class;
     }
 
-    public function processNode(Node $node, Scope $scope): array
-    {
+    public function processNode(
+        Node $node,
+        Scope $scope
+    ): array {
         return [
             'Language construct isset() should not be used.',
         ];

diff --git a/src/Files/DeclareStrictTypesRule.php b/src/Files/DeclareStrictTypesRule.php
@@ -26,8 +26,10 @@ public function getNodeType(): string
         return FileNode::class;
     }
 
-    public function processNode(Node $node, Scope $scope): array
-    {
+    public function processNode(
+        Node $node,
+        Scope $scope
+    ): array {
         if (!$node instanceof FileNode) {
             throw new ShouldNotHappenException(\sprintf(
                 'Expected node to be instance of "%s", but got instance of "%s" instead.',

diff --git a/src/Functions/NoNullableReturnTypeDeclarationRule.php b/src/Functions/NoNullableReturnTypeDeclarationRule.php
@@ -25,8 +25,10 @@ public function getNodeType(): string
         return Node\Stmt\Function_::class;
     }
 
-    public function processNode(Node $node, Scope $scope): array
-    {
+    public function processNode(
+        Node $node,
+        Scope $scope
+    ): array {
         if (!$node instanceof Node\Stmt\Function_) {
             throw new ShouldNotHappenException(\sprintf(
                 'Expected node to be instance of "%s", but got instance of "%s" instead.',

diff --git a/src/Functions/NoParameterWithNullDefaultValueRule.php b/src/Functions/NoParameterWithNullDefaultValueRule.php
@@ -25,8 +25,10 @@ public function getNodeType(): string
         return Node\Stmt\Function_::class;
     }
 
-    public function processNode(Node $node, Scope $scope): array
-    {
+    public function processNode(
+        Node $node,
+        Scope $scope
+    ): array {
         if (!$node instanceof Node\Stmt\Function_) {
             throw new ShouldNotHappenException(\sprintf(
                 'Expected node to be instance of "%s", but got instance of "%s" instead.',

diff --git a/src/Functions/NoParameterWithNullableTypeDeclarationRule.php b/src/Functions/NoParameterWithNullableTypeDeclarationRule.php
@@ -25,8 +25,10 @@ public function getNodeType(): string
         return Node\Stmt\Function_::class;
     }
 
-    public function processNode(Node $node, Scope $scope): array
-    {
+    public function processNode(
+        Node $node,
+        Scope $scope
+    ): array {
         if (!$node instanceof Node\Stmt\Function_) {
             throw new ShouldNotHappenException(\sprintf(
                 'Expected node to be instance of "%s", but got instance of "%s" instead.',

diff --git a/src/Methods/FinalInAbstractClassRule.php b/src/Methods/FinalInAbstractClassRule.php
@@ -26,8 +26,10 @@ public function getNodeType(): string
         return Node\Stmt\ClassMethod::class;
     }
 
-    public function processNode(Node $node, Scope $scope): array
-    {
+    public function processNode(
+        Node $node,
+        Scope $scope
+    ): array {
         if (!$node instanceof Node\Stmt\ClassMethod) {
             throw new ShouldNotHappenException(\sprintf(
                 'Expected node to be instance of "%s", but got instance of "%s" instead.',

diff --git a/src/Methods/NoConstructorParameterWithDefaultValueRule.php b/src/Methods/NoConstructorParameterWithDefaultValueRule.php
@@ -26,8 +26,10 @@ public function getNodeType(): string
         return Node\Stmt\ClassMethod::class;
     }
 
-    public function processNode(Node $node, Scope $scope): array
-    {
+    public function processNode(
+        Node $node,
+        Scope $scope
+    ): array {
         if (!$node instanceof Node\Stmt\ClassMethod) {
             throw new ShouldNotHappenException(\sprintf(
                 'Expected node to be instance of "%s", but got instance of "%s" instead.',

diff --git a/src/Methods/NoNullableReturnTypeDeclarationRule.php b/src/Methods/NoNullableReturnTypeDeclarationRule.php
@@ -26,8 +26,10 @@ public function getNodeType(): string
         return Node\Stmt\ClassMethod::class;
     }
 
-    public function processNode(Node $node, Scope $scope): array
-    {
+    public function processNode(
+        Node $node,
+        Scope $scope
+    ): array {
         if (!$node instanceof Node\Stmt\ClassMethod) {
             throw new ShouldNotHappenException(\sprintf(
                 'Expected node to be instance of "%s", but got instance of "%s" instead.',

diff --git a/src/Methods/NoParameterWithContainerTypeDeclarationRule.php b/src/Methods/NoParameterWithContainerTypeDeclarationRule.php
@@ -34,8 +34,10 @@ final class NoParameterWithContainerTypeDeclarationRule implements Rule
     /**
      * @param array<int, string> $interfacesImplementedByContainers
      */
-    public function __construct(Reflection\ReflectionProvider $reflectionProvider, array $interfacesImplementedByContainers)
-    {
+    public function __construct(
+        Reflection\ReflectionProvider $reflectionProvider,
+        array $interfacesImplementedByContainers
+    ) {
         $this->reflectionProvider = $reflectionProvider;
         $this->interfacesImplementedByContainers = \array_filter(
             \array_map(static function (string $interfaceImplementedByContainers): string {
@@ -52,8 +54,10 @@ public function getNodeType(): string
         return Node\Stmt\ClassMethod::class;
     }
 
-    public function processNode(Node $node, Scope $scope): array
-    {
+    public function processNode(
+        Node $node,
+        Scope $scope
+    ): array {
         if (!$node instanceof Node\Stmt\ClassMethod) {
             throw new ShouldNotHappenException(\sprintf(
                 'Expected node to be instance of "%s", but got instance of "%s" instead.',

diff --git a/src/Methods/NoParameterWithNullDefaultValueRule.php b/src/Methods/NoParameterWithNullDefaultValueRule.php
@@ -26,8 +26,10 @@ public function getNodeType(): string
         return Node\Stmt\ClassMethod::class;
     }
 
-    public function processNode(Node $node, Scope $scope): array
-    {
+    public function processNode(
+        Node $node,
+        Scope $scope
+    ): array {
         if (!$node instanceof Node\Stmt\ClassMethod) {
             throw new ShouldNotHappenException(\sprintf(
                 'Expected node to be instance of "%s", but got instance of "%s" instead.',

diff --git a/src/Methods/NoParameterWithNullableTypeDeclarationRule.php b/src/Methods/NoParameterWithNullableTypeDeclarationRule.php
@@ -26,8 +26,10 @@ public function getNodeType(): string
         return Node\Stmt\ClassMethod::class;
     }
 
-    public function processNode(Node $node, Scope $scope): array
-    {
+    public function processNode(
+        Node $node,
+        Scope $scope
+    ): array {
         if (!$node instanceof Node\Stmt\ClassMethod) {
             throw new ShouldNotHappenException(\sprintf(
                 'Expected node to be instance of "%s", but got instance of "%s" instead.',

diff --git a/src/Methods/PrivateInFinalClassRule.php b/src/Methods/PrivateInFinalClassRule.php
@@ -26,8 +26,10 @@ public function getNodeType(): string
         return Node\Stmt\ClassMethod::class;
     }
 
-    public function processNode(Node $node, Scope $scope): array
-    {
+    public function processNode(
+        Node $node,
+        Scope $scope
+    ): array {
         if (!$node instanceof Node\Stmt\ClassMethod) {
             throw new ShouldNotHappenException(\sprintf(
                 'Expected node to be instance of "%s", but got instance of "%s" instead.',

diff --git a/src/Statements/NoSwitchRule.php b/src/Statements/NoSwitchRule.php
@@ -24,8 +24,10 @@ public function getNodeType(): string
         return Node\Stmt\Switch_::class;
     }
 
-    public function processNode(Node $node, Scope $scope): array
-    {
+    public function processNode(
+        Node $node,
+        Scope $scope
+    ): array {
         return [
             'Control structures using switch should not be used.',
         ];