fix cs

rectorphp · May 18, 2021 · 4c2eca2 · 4c2eca2
1 parent 35d72f9
commit 4c2eca2
Show file tree

Hide file tree

Showing 6 changed files with 19 additions and 8 deletions.
diff --git a/config/services.php b/config/services.php
@@ -24,7 +24,6 @@
 use PHPStan\PhpDocParser\Parser\PhpDocParser;
 use PHPStan\PhpDocParser\Parser\TypeParser;
 use PHPStan\Reflection\ReflectionProvider;
-use PrettyXml\Formatter;
 use Rector\BetterPhpDocParser\PhpDocParser\BetterPhpDocParser;
 use Rector\BetterPhpDocParser\PhpDocParser\BetterTypeParser;
 use Rector\Caching\Cache\NetteCacheFactory;

diff --git a/rules/DeadCode/Rector/Assign/RemoveUnusedVariableAssignRector.php b/rules/DeadCode/Rector/Assign/RemoveUnusedVariableAssignRector.php
@@ -184,9 +184,9 @@ private function isUsedInAssignExpr(Expr $expr, Assign $assign): bool
             $previousAssign = $this->betterNodeFinder->findFirstPreviousOfNode(
                 $assign,
                 fn (Node $node): bool => $node instanceof Assign && $this->usedVariableNameAnalyzer->isVariableNamed(
-                $node->var,
-                $variable
-            )
+                    $node->var,
+                    $variable
+                )
             );
 
             if ($previousAssign instanceof Assign) {

diff --git a/rules/DeadCode/Rector/FunctionLike/RemoveOverriddenValuesRector.php b/rules/DeadCode/Rector/FunctionLike/RemoveOverriddenValuesRector.php
@@ -231,7 +231,10 @@ private function shouldRemoveAssignNode(
 
         $isVariableAssigned = (bool) $this->betterNodeFinder->findFirst(
             $assignNode->expr,
-            fn (Node $node): bool => $this->nodeComparator->areNodesEqual($node, $nodeByTypeAndPosition->getVariableNode())
+            fn (Node $node): bool => $this->nodeComparator->areNodesEqual(
+                $node,
+                $nodeByTypeAndPosition->getVariableNode()
+            )
         );
 
         return ! $isVariableAssigned;

diff --git a/rules/Naming/Naming/PropertyNaming.php b/rules/Naming/Naming/PropertyNaming.php
@@ -293,7 +293,10 @@ private function filterClassMethodsWithPropertyFetchReturnOnly(
 
         return array_filter(
             $prefixedClassMethods,
-            fn (ClassMethod $classMethod): bool => $this->doesClassMethodMatchReturnPropertyFetch($classMethod, $classMethodName)
+            fn (ClassMethod $classMethod): bool => $this->doesClassMethodMatchReturnPropertyFetch(
+                $classMethod,
+                $classMethodName
+            )
         );
     }
 

diff --git a/rules/Php80/Rector/Switch_/ChangeSwitchToMatchRector.php b/rules/Php80/Rector/Switch_/ChangeSwitchToMatchRector.php
@@ -140,7 +140,10 @@ private function changeToAssign(Switch_ $switch, Match_ $match, Expr $assignExpr
     {
         $prevInitializedAssign = $this->betterNodeFinder->findFirstPreviousOfNode(
             $switch,
-            fn (Node $node): bool => $node instanceof Assign && $this->nodeComparator->areNodesEqual($node->var, $assignExpr)
+            fn (Node $node): bool => $node instanceof Assign && $this->nodeComparator->areNodesEqual(
+                $node->var,
+                $assignExpr
+            )
         );
 
         $assign = new Assign($assignExpr, $match);

diff --git a/src/NodeManipulator/ClassMethodAssignManipulator.php b/src/NodeManipulator/ClassMethodAssignManipulator.php
@@ -125,7 +125,10 @@ private function filterOutArrayDestructedVariables(array $variableAssigns, Class
 
         return array_filter(
             $variableAssigns,
-            fn (Assign $assign): bool => ! $this->nodeNameResolver->isNames($assign->var, $arrayDestructionCreatedVariables)
+            fn (Assign $assign): bool => ! $this->nodeNameResolver->isNames(
+                $assign->var,
+                $arrayDestructionCreatedVariables
+            )
         );
     }