[ci-review] Rector Rectify

Author: actions-user

rules/DeadCode/Rector/ClassMethod/RemoveEmptyClassMethodRector.php

@@ -5,7 +5,6 @@
 namespace Rector\DeadCode\Rector\ClassMethod;
 
 use PhpParser\Node;
-use PhpParser\Node\Name\FullyQualified;
 use PhpParser\Node\Stmt\Class_;
 use PhpParser\Node\Stmt\ClassMethod;
 use PHPStan\PhpDocParser\Ast\PhpDoc\DeprecatedTagValueNode;

rules/Php81/Rector/FuncCall/NullToStrictIntPregSlitFuncCallLimitArgRector.php

@@ -5,9 +5,7 @@
 namespace Rector\Php81\Rector\FuncCall;
 
 use PhpParser\Node;
-use PhpParser\Node\Arg;
 use PhpParser\Node\Expr\FuncCall;
-use PhpParser\Node\Identifier;
 use PHPStan\Analyser\Scope;
 use PHPStan\Reflection\ClassReflection;
 use PHPStan\Reflection\FunctionReflection;
@@ -129,26 +127,6 @@ public function provideMinPhpVersion(): int
         return PhpVersionFeature::DEPRECATE_NULL_ARG_IN_STRING_FUNCTION;
     }
 
-    /**
-     * @param Arg[] $args
-     */
-    private function resolveNamedPosition(array $args): ?int
-    {
-        foreach ($args as $position => $arg) {
-            if (! $arg->name instanceof Identifier) {
-                continue;
-            }
-
-            if (! $this->isName($arg->name, 'limit')) {
-                continue;
-            }
-
-            return $position;
-        }
-
-        return null;
-    }
-
     private function shouldSkip(FuncCall $funcCall): bool
     {
         if (! $this->isName($funcCall, 'preg_split')) {

rules/Php85/Rector/Class_/SleepToSerializeRector.php

@@ -4,6 +4,8 @@
 
 namespace Rector\Php85\Rector\Class_;
 
+use PhpParser\Node\Scalar\String_;
+use PhpParser\Node\Expr\Variable;
 use PhpParser\Node;
 use PhpParser\Node\Expr\Array_;
 use PhpParser\Node\Expr\ArrayItem;
@@ -28,8 +30,8 @@
 final class SleepToSerializeRector extends AbstractRector implements MinPhpVersionInterface
 {
     public function __construct(
-        private BetterNodeFinder $betterNodeFinder,
-        private ReturnAnalyzer $returnAnalyzer
+        private readonly BetterNodeFinder $betterNodeFinder,
+        private readonly ReturnAnalyzer $returnAnalyzer
     ) {
     }
 
@@ -110,18 +112,19 @@ public function refactor(Node $node): ?Node
                 return null;
             }
 
-            if (count($return->expr->items) > 0) {
+            if ($return->expr->items !== []) {
                 $newItems = [];
                 foreach ($return->expr->items as $item) {
-                    if ($item !== null && $item->value instanceof Node\Scalar\String_) {
+                    if ($item !== null && $item->value instanceof String_) {
                         $propName = $item->value->value;
                         $newItems[] = new ArrayItem(
-                            new PropertyFetch(new Node\Expr\Variable('this'), $propName),
+                            new PropertyFetch(new Variable('this'), $propName),
                             $item->value
                         );
                     }
                 }
-                if (count($newItems) > 0) {
+
+                if ($newItems !== []) {
                     $hasChanged = true;
                     $return->expr->items = $newItems;
                 }

src/NodeAnalyzer/ArgsAnalyzer.php

@@ -8,10 +8,10 @@
 use PhpParser\Node\Identifier;
 use Rector\NodeNameResolver\NodeNameResolver;
 
-final class ArgsAnalyzer
+final readonly class ArgsAnalyzer
 {
     public function __construct(
-        private readonly NodeNameResolver $nodeNameResolver
+        private NodeNameResolver $nodeNameResolver
     ) {
     }