Make use of getArgs()

rectorphp · May 13, 2023 · 10913e0 · 10913e0
1 parent 3209c44
commit 10913e0
Show file tree

Hide file tree

Showing 21 changed files with 43 additions and 111 deletions.
diff --git a/rules/DowngradePhp54/Rector/FuncCall/DowngradeIndirectCallByArrayRector.php b/rules/DowngradePhp54/Rector/FuncCall/DowngradeIndirectCallByArrayRector.php
@@ -79,7 +79,7 @@ public function refactor(Node $node): ?FuncCall
             return null;
         }
 
-        $args = array_merge([new Arg($node->name)], $node->args);
+        $args = array_merge([new Arg($node->name)], $node->getArgs());
 
         return new FuncCall(new Name('call_user_func'), $args);
     }

diff --git a/rules/DowngradePhp55/Rector/ClassConstFetch/DowngradeClassConstantToStringRector.php b/rules/DowngradePhp55/Rector/ClassConstFetch/DowngradeClassConstantToStringRector.php
@@ -16,7 +16,7 @@
 /**
  * @changelog https://wiki.php.net/rfc/class_name_scalars
  *
- * @see Rector\Tests\DowngradePhp55\Rector\ClassConstFetch\DowngradeClassConstantToStringRector\DowngradeClassConstantToStringRectorTest
+ * @see \Rector\Tests\DowngradePhp55\Rector\ClassConstFetch\DowngradeClassConstantToStringRector\DowngradeClassConstantToStringRectorTest
  */
 final class DowngradeClassConstantToStringRector extends AbstractRector
 {

diff --git a/rules/DowngradePhp55/Rector/FuncCall/DowngradeBoolvalRector.php b/rules/DowngradePhp55/Rector/FuncCall/DowngradeBoolvalRector.php
@@ -82,27 +82,19 @@ public function refactor(Node $node): ?Node
 
     private function refactorBoolval(FuncCall $funcCall): ?Bool_
     {
-        if (! isset($funcCall->args[0])) {
+        if (! isset($funcCall->getArgs()[0])) {
             return null;
         }
 
-        if (! $funcCall->args[0] instanceof Arg) {
-            return null;
-        }
-
-        return new Bool_($funcCall->args[0]->value);
+        return new Bool_($funcCall->getArgs()[0]->value);
     }
 
     private function refactorAsCallback(FuncCall $funcCall): ?FuncCall
     {
         $functionLikeReflection = null;
         $refactored = false;
 
-        foreach ($funcCall->args as $position => $arg) {
-            if (! $arg instanceof Arg) {
-                continue;
-            }
-
+        foreach ($funcCall->getArgs() as $position => $arg) {
             if (! $this->isBoolvalReference($arg)) {
                 continue;
             }

diff --git a/rules/DowngradePhp70/Rector/Expression/DowngradeDefineArrayConstantRector.php b/rules/DowngradePhp70/Rector/Expression/DowngradeDefineArrayConstantRector.php
@@ -5,15 +5,13 @@
 namespace Rector\DowngradePhp70\Rector\Expression;
 
 use PhpParser\Node;
-use PhpParser\Node\Arg;
 use PhpParser\Node\Const_;
 use PhpParser\Node\Expr\Array_;
 use PhpParser\Node\Expr\FuncCall;
 use PhpParser\Node\Scalar\String_;
 use PhpParser\Node\Stmt\ClassMethod;
 use PhpParser\Node\Stmt\Expression;
 use PhpParser\Node\Stmt\Function_;
-use Rector\Core\NodeAnalyzer\ArgsAnalyzer;
 use Rector\Core\Rector\AbstractRector;
 use Symplify\RuleDocGenerator\ValueObject\CodeSample\CodeSample;
 use Symplify\RuleDocGenerator\ValueObject\RuleDefinition;
@@ -23,11 +21,6 @@
  */
 final class DowngradeDefineArrayConstantRector extends AbstractRector
 {
-    public function __construct(
-        private readonly ArgsAnalyzer $argsAnalyzer
-    ) {
-    }
-
     /**
      * @return array<class-string<Node>>
      */
@@ -72,13 +65,11 @@ public function refactor(Node $node): ?Node
         }
 
         $funcCall = $node->expr;
-
         if ($this->shouldSkip($funcCall)) {
             return null;
         }
 
-        /** @var Arg[] $args */
-        $args = $funcCall->args;
+        $args = $funcCall->getArgs();
 
         /** @var String_ $arg0 */
         $arg0 = $args[0]->value;
@@ -96,13 +87,7 @@ private function shouldSkip(FuncCall $funcCall): bool
             return true;
         }
 
-        $args = $funcCall->args;
-
-        if (! $this->argsAnalyzer->isArgsInstanceInArgsPositions($args, [0, 1])) {
-            return true;
-        }
-
-        /** @var Arg[] $args */
+        $args = $funcCall->getArgs();
         if (! $args[0]->value instanceof String_) {
             return true;
         }

diff --git a/rules/DowngradePhp70/Rector/FuncCall/DowngradeDirnameLevelsRector.php b/rules/DowngradePhp70/Rector/FuncCall/DowngradeDirnameLevelsRector.php
@@ -94,15 +94,7 @@ private function getLevelsArg(FuncCall $funcCall): ?Arg
             return null;
         }
 
-        if (! isset($funcCall->args[1])) {
-            return null;
-        }
-
-        if (! $funcCall->args[1] instanceof Arg) {
-            return null;
-        }
-
-        return $funcCall->args[1];
+        return $funcCall->getArgs()[1] ?? null;
     }
 
     private function getLevelsRealValue(Arg $levelsArg): ?int

diff --git a/rules/DowngradePhp70/Rector/FuncCall/DowngradeSessionStartArrayOptionsRector.php b/rules/DowngradePhp70/Rector/FuncCall/DowngradeSessionStartArrayOptionsRector.php
@@ -65,16 +65,13 @@ public function refactor(Node $node): ?Node
             return null;
         }
 
-        if (! isset($node->args[0])) {
-            return null;
-        }
-
-        if (! $node->args[0] instanceof Arg) {
+        if (! isset($node->getArgs()[0])) {
             return null;
         }
 
         /** @var Array_ $options */
-        $options = $node->args[0]->value;
+        $options = $node->getArgs()[0]
+->value;
 
         foreach ($options->items as $option) {
             if (! $option instanceof ArrayItem) {

diff --git a/rules/DowngradePhp70/Rector/FuncCall/DowngradeUncallableValueCallToCallUserFuncRector.php b/rules/DowngradePhp70/Rector/FuncCall/DowngradeUncallableValueCallToCallUserFuncRector.php
@@ -100,7 +100,7 @@ public function refactor(Node $node): ?FuncCall
             return null;
         }
 
-        $args = array_merge([new Arg($node->name)], $node->args);
+        $args = array_merge([new Arg($node->name)], $node->getArgs());
 
         return new FuncCall(new Name('call_user_func'), $args);
     }

diff --git a/rules/DowngradePhp70/Rector/MethodCall/DowngradeClosureCallRector.php b/rules/DowngradePhp70/Rector/MethodCall/DowngradeClosureCallRector.php
@@ -66,7 +66,7 @@ public function refactor(Node $node): ?FuncCall
         }
 
         $methodCall = $this->createBindToCall($node);
-        $args = [new Arg($methodCall), ...array_slice($node->args, 1)];
+        $args = [new Arg($methodCall), ...array_slice($node->getArgs(), 1)];
 
         return new FuncCall(new Name('call_user_func'), $args);
     }

diff --git a/rules/DowngradePhp70/Rector/New_/DowngradeAnonymousClassRector.php b/rules/DowngradePhp70/Rector/New_/DowngradeAnonymousClassRector.php
@@ -128,7 +128,7 @@ public function refactor(Node $node): ?Node
 
         $this->namespacedNameDecorator->decorate($class);
 
-        return new New_(new Name($className), $node->args);
+        return new New_(new Name($className), $node->getArgs());
     }
 
     private function createAnonymousClassName(): string

diff --git a/rules/DowngradePhp71/Rector/FuncCall/DowngradeIsIterableRector.php b/rules/DowngradePhp71/Rector/FuncCall/DowngradeIsIterableRector.php
@@ -5,7 +5,6 @@
 namespace Rector\DowngradePhp71\Rector\FuncCall;
 
 use PhpParser\Node;
-use PhpParser\Node\Arg;
 use PhpParser\Node\Expr\BinaryOp\BooleanOr;
 use PhpParser\Node\Expr\FuncCall;
 use PhpParser\Node\Expr\Instanceof_;
@@ -68,18 +67,15 @@ public function refactor(Node $node): ?Node
             return null;
         }
 
-        if (! isset($node->args[0])) {
+        if (! isset($node->getArgs()[0])) {
             return null;
         }
 
-        if (! $node->args[0] instanceof Arg) {
-            return null;
-        }
+        $expr = $node->getArgs()[0]
+->value;
 
-        /** @var mixed $arg */
-        $arg = $node->args[0]->value;
-        $funcCall = $this->nodeFactory->createFuncCall('is_array', [$arg]);
-        $instanceof = new Instanceof_($arg, new FullyQualified('Traversable'));
+        $funcCall = $this->nodeFactory->createFuncCall('is_array', [$expr]);
+        $instanceof = new Instanceof_($expr, new FullyQualified('Traversable'));
 
         return new BooleanOr($funcCall, $instanceof);
     }

diff --git a/rules/DowngradePhp71/Rector/StaticCall/DowngradeClosureFromCallableRector.php b/rules/DowngradePhp71/Rector/StaticCall/DowngradeClosureFromCallableRector.php
@@ -70,16 +70,12 @@ public function refactor(Node $node): ?Node
             return null;
         }
 
-        if (! isset($node->args[0])) {
-            return null;
-        }
-
-        if (! $node->args[0] instanceof Arg) {
+        if (! isset($node->getArgs()[0])) {
             return null;
         }
 
         $tempVariable = $this->namedVariableFactory->createVariable($node, 'callable');
-        $expression = new Expression(new Assign($tempVariable, $node->args[0]->value));
+        $expression = new Expression(new Assign($tempVariable, $node->getArgs()[0]->value));
 
         $this->nodesToAddCollector->addNodeBeforeNode($expression, $node);
 

diff --git a/rules/DowngradePhp72/Rector/FuncCall/DowngradeStreamIsattyRector.php b/rules/DowngradePhp72/Rector/FuncCall/DowngradeStreamIsattyRector.php
@@ -112,7 +112,7 @@ public function refactorWithScope(Node $node, Scope $scope): ?Node
 
         $this->nodesToAddCollector->addNodeBeforeNode($assign, $node);
 
-        return new FuncCall($variable, $node->args);
+        return new FuncCall($variable, $node->getArgs());
     }
 
     private function createClosure(): Closure

diff --git a/rules/DowngradePhp73/Rector/FuncCall/DowngradeIsCountableRector.php b/rules/DowngradePhp73/Rector/FuncCall/DowngradeIsCountableRector.php
@@ -5,7 +5,6 @@
 namespace Rector\DowngradePhp73\Rector\FuncCall;
 
 use PhpParser\Node;
-use PhpParser\Node\Arg;
 use PhpParser\Node\Expr\BinaryOp\BooleanOr;
 use PhpParser\Node\Expr\FuncCall;
 use PhpParser\Node\Expr\Instanceof_;
@@ -56,16 +55,12 @@ public function refactor(Node $node): ?Node
             return null;
         }
 
-        if (! isset($node->args[0])) {
+        if (! isset($node->getArgs()[0])) {
             return null;
         }
 
-        if (! $node->args[0] instanceof Arg) {
-            return null;
-        }
-
-        $isArrayFuncCall = $this->nodeFactory->createFuncCall('is_array', $node->args);
-        $instanceof = new Instanceof_($node->args[0]->value, new FullyQualified('Countable'));
+        $isArrayFuncCall = $this->nodeFactory->createFuncCall('is_array', $node->getArgs());
+        $instanceof = new Instanceof_($node->getArgs()[0]->value, new FullyQualified('Countable'));
 
         return new BooleanOr($isArrayFuncCall, $instanceof);
     }

diff --git a/rules/DowngradePhp73/Rector/FuncCall/DowngradeTrailingCommasInFunctionCallsRector.php b/rules/DowngradePhp73/Rector/FuncCall/DowngradeTrailingCommasInFunctionCallsRector.php
@@ -75,9 +75,9 @@ public function getNodeTypes(): array
     public function refactor(Node $node): ?Node
     {
         if ($node->getArgs() !== []) {
-            $lastArgumentPosition = array_key_last($node->args);
+            $lastArgumentPosition = array_key_last($node->getArgs());
 
-            $last = $node->args[$lastArgumentPosition];
+            $last = $node->getArgs()[$lastArgumentPosition];
             if (! $this->followedByCommaAnalyzer->isFollowed($this->file, $last)) {
                 return null;
             }

diff --git a/rules/DowngradePhp73/Rector/FuncCall/SetCookieOptionsArrayToArgumentsRector.php b/rules/DowngradePhp73/Rector/FuncCall/SetCookieOptionsArrayToArgumentsRector.php
@@ -12,7 +12,6 @@
 use PhpParser\Node\Expr\FuncCall;
 use PhpParser\Node\Scalar\String_;
 use Rector\Core\Exception\ShouldNotHappenException;
-use Rector\Core\NodeAnalyzer\ArgsAnalyzer;
 use Rector\Core\Rector\AbstractRector;
 use Symplify\RuleDocGenerator\ValueObject\CodeSample\CodeSample;
 use Symplify\RuleDocGenerator\ValueObject\RuleDefinition;
@@ -48,11 +47,6 @@ final class SetCookieOptionsArrayToArgumentsRector extends AbstractRector
 
     private int $highestIndex = 1;
 
-    public function __construct(
-        private readonly ArgsAnalyzer $argsAnalyzer
-    ) {
-    }
-
     public function getRuleDefinition(): RuleDefinition
     {
         return new RuleDefinition(
@@ -118,24 +112,20 @@ private function composeNewArgs(FuncCall $funcCall): array
     {
         $this->highestIndex = 1;
 
-        if (! $this->argsAnalyzer->isArgsInstanceInArgsPositions($funcCall->args, [0, 1, 2])) {
+        if (count($funcCall->getArgs()) < 3) {
             return [];
         }
 
-        /** @var Arg $firstArg */
-        $firstArg = $funcCall->args[0];
-
-        /** @var Arg $secondArg */
-        $secondArg = $funcCall->args[1];
+        $firstArg = $funcCall->getArgs()[0];
+        $secondArg = $funcCall->getArgs()[1];
 
         $newArgs = [$firstArg, $secondArg];
 
-        /** @var Arg $thirdArg */
-        $thirdArg = $funcCall->args[2];
+        $thirdArg = $funcCall->getArgs()[2];
 
         /** @var Array_ $optionsArray */
         $optionsArray = $thirdArg->value;
-        /** @var ArrayItem|null $arrayItem */
+
         foreach ($optionsArray->items as $arrayItem) {
             if (! $arrayItem instanceof ArrayItem) {
                 continue;

diff --git a/rules/DowngradePhp74/Rector/FuncCall/DowngradeStripTagsCallWithArrayRector.php b/rules/DowngradePhp74/Rector/FuncCall/DowngradeStripTagsCallWithArrayRector.php
@@ -102,8 +102,7 @@ public function refactor(Node $node): ?Node
             return null;
         }
 
-        /** @var Arg $secondArg */
-        $secondArg = $node->args[1];
+        $secondArg = $node->getArgs()[1];
         $allowableTagsParam = $secondArg->value;
 
         if ($allowableTagsParam instanceof Array_) {

diff --git a/rules/DowngradePhp80/Rector/FuncCall/DowngradeNumberFormatNoFourthArgRector.php b/rules/DowngradePhp80/Rector/FuncCall/DowngradeNumberFormatNoFourthArgRector.php
@@ -17,7 +17,7 @@
 /**
  * @changelog https://www.php.net/manual/en/function.number-format.php#refsect1-function.number-format-changelog
  *
- * @see Rector\Tests\DowngradePhp80\Rector\FuncCall\DowngradeNumberFormatNoFourthArgRector\DowngradeNumberFormatNoFourthArgRectorTest
+ * @see \Rector\Tests\DowngradePhp80\Rector\FuncCall\DowngradeNumberFormatNoFourthArgRector\DowngradeNumberFormatNoFourthArgRectorTest
  */
 final class DowngradeNumberFormatNoFourthArgRector extends AbstractRector
 {

diff --git a/rules/DowngradePhp80/Rector/FuncCall/DowngradeStrContainsRector.php b/rules/DowngradePhp80/Rector/FuncCall/DowngradeStrContainsRector.php
@@ -5,28 +5,21 @@
 namespace Rector\DowngradePhp80\Rector\FuncCall;
 
 use PhpParser\Node;
-use PhpParser\Node\Arg;
 use PhpParser\Node\Expr\BinaryOp\Identical;
 use PhpParser\Node\Expr\BinaryOp\NotIdentical;
 use PhpParser\Node\Expr\BooleanNot;
 use PhpParser\Node\Expr\FuncCall;
-use Rector\Core\NodeAnalyzer\ArgsAnalyzer;
 use Rector\Core\Rector\AbstractRector;
 use Symplify\RuleDocGenerator\ValueObject\CodeSample\CodeSample;
 use Symplify\RuleDocGenerator\ValueObject\RuleDefinition;
 
 /**
  * @changelog https://wiki.php.net/rfc/str_contains
  *
- * @see Rector\Tests\DowngradePhp80\Rector\FuncCall\DowngradeStrContainsRector\DowngradeStrContainsRectorTest
+ * @see \Rector\Tests\DowngradePhp80\Rector\FuncCall\DowngradeStrContainsRector\DowngradeStrContainsRectorTest
  */
 final class DowngradeStrContainsRector extends AbstractRector
 {
-    public function __construct(
-        private readonly ArgsAnalyzer $argsAnalyzer
-    ) {
-    }
-
     public function getRuleDefinition(): RuleDefinition
     {
         return new RuleDefinition(
@@ -77,17 +70,14 @@ public function refactor(Node $node): Identical | NotIdentical | null
             return null;
         }
 
-        if (! $this->argsAnalyzer->isArgsInstanceInArgsPositions($funcCall->args, [0, 1])) {
+        if (count($funcCall->getArgs()) < 2) {
             return null;
         }
 
-        /** @var Arg $firstArg */
-        $firstArg = $funcCall->args[0];
-        $haystack = $firstArg->value;
-
-        /** @var Arg $secondArg */
-        $secondArg = $funcCall->args[1];
-        $needle = $secondArg->value;
+        $haystack = $funcCall->getArgs()[0]
+->value;
+        $needle = $funcCall->getArgs()[1]
+->value;
 
         $funcCall = $this->nodeFactory->createFuncCall('strpos', [$haystack, $needle]);