[transform] tidy up array dim fetch to method call rector (#7694)

* [transform] tidy up array dim fetch to method call rector

* fix fixture autoload by using unique function names

Author: TomasVotruba

rules-tests/CodeQuality/Rector/FuncCall/SortNamedParamRector/Fixture/skip_first_class_callable.php.inc

@@ -2,8 +2,8 @@
 
 namespace Rector\Tests\CodeQuality\Rector\FuncCall\SortNamedParamRector\Fixture;
 
-function run($foo = null, $bar = null, $baz = null) {}
+function skip_first_class_callable($foo = null, $bar = null, $baz = null) {}
 
-run(...);
+skip_first_class_callable(...);
 
 ?>

rules-tests/CodeQuality/Rector/FuncCall/SortNamedParamRector/Fixture/skip_no_arg.php.inc

@@ -2,8 +2,7 @@
 
 namespace Rector\Tests\CodeQuality\Rector\FuncCall\SortNamedParamRector\Fixture;
 
-function run(?string $foo = null) {}
+function skip_single_arg(?string $foo = null) {}
 
-run(foo: 'test');
-
-?>
+skip_single_arg(foo: 'test');
+skip_single_arg();

rules-tests/CodeQuality/Rector/FuncCall/SortNamedParamRector/Fixture/skip_single_arg.php.inc

@@ -2,10 +2,8 @@
 
 namespace Rector\Tests\CodeQuality\Rector\FuncCall\SortNamedParamRector\Fixture;
 
-function run($foo = null, $bar = null, $baz = null) {}
+function skip_sorted($foo = null, $bar = null, $baz = null) {}
 
-run(foo: $foo, bar: $bar);
+skip_sorted(foo: $foo, bar: $bar);
 
-run($foo, bar: $bar, baz: $baz);
-
-?>
+skip_sorted($foo, bar: $bar, baz: $baz);

rules-tests/CodeQuality/Rector/FuncCall/SortNamedParamRector/Fixture/skip_sorted.php.inc

@@ -4,9 +4,6 @@ namespace Rector\Tests\Transform\Rector\ArrayDimFetch\ArrayDimFetchToMethodCallR
 
 /** @var \Container $object */
 $object['key'];
-$object['key'] = 'value';
-isset($object['key']);
-unset($object['key']);
 
 ?>
 -----
@@ -16,8 +13,5 @@ namespace Rector\Tests\Transform\Rector\ArrayDimFetch\ArrayDimFetchToMethodCallR
 
 /** @var \Container $object */
 $object->get('key');
-$object['key'] = 'value';
-isset($object['key']);
-unset($object['key']);
 
 ?>

…r/Fixture/partial_transformation.php.inc …tor/Fixture/handle_container_get.php.incrules-tests/Transform/Rector/ArrayDimFetch/ArrayDimFetchToMethodCallRector/Fixture/partial_transformation.php.inc renamed to rules-tests/Transform/Rector/ArrayDimFetch/ArrayDimFetchToMethodCallRector/Fixture/handle_container_get.php.inc rules-tests/Transform/Rector/ArrayDimFetch/ArrayDimFetchToMethodCallRector/Fixture/partial_transformation.php.inc renamed to rules-tests/Transform/Rector/ArrayDimFetch/ArrayDimFetchToMethodCallRector/Fixture/handle_container_get.php.inc

@@ -0,0 +1,8 @@
+<?php
+
+namespace Rector\Tests\Transform\Rector\ArrayDimFetch\ArrayDimFetchToMethodCallRector\Fixture;
+
+/** @var \Container $object */
+$object['key'] = 'value';
+isset($object['key']);
+unset($object['key']);

rules-tests/Transform/Rector/ArrayDimFetch/ArrayDimFetchToMethodCallRector/Fixture/skip_container_set_isset_unset.php.inc

@@ -19,6 +19,7 @@
 use PhpParser\NodeVisitor;
 use PHPStan\Type\ObjectType;
 use Rector\Contract\Rector\ConfigurableRectorInterface;
+use Rector\NodeTypeResolver\Node\AttributeKey;
 use Rector\Rector\AbstractRector;
 use Rector\Transform\ValueObject\ArrayDimFetchToMethodCall;
 use Symplify\RuleDocGenerator\ValueObject\CodeSample\ConfiguredCodeSample;
@@ -86,12 +87,15 @@ public function refactor(Node $node): array|Expr|null|int
             if (! $node->var instanceof ArrayDimFetch) {
                 return null;
             }
+            return $this->createExplicitMethodCall($node->var, 'set', $node->expr);
+        }
 
-            return $this->getMethodCall($node->var, 'set', $node->expr)
-                ?? NodeVisitor::DONT_TRAVERSE_CHILDREN;
+        // is part of assign, skip
+        if ($node->getAttribute(AttributeKey::IS_BEING_ASSIGNED)) {
+            return null;
         }
 
-        return $this->getMethodCall($node, 'get');
+        return $this->createExplicitMethodCall($node, 'get');
     }
 
     public function configure(array $configuration): void
@@ -108,7 +112,7 @@ private function handleIsset(Isset_ $isset): Expr|int|null
 
         foreach ($isset->vars as $var) {
             if ($var instanceof ArrayDimFetch) {
-                $methodCall = $this->getMethodCall($var, 'exists');
+                $methodCall = $this->createExplicitMethodCall($var, 'exists');
 
                 if ($methodCall instanceof MethodCall) {
                     $exprs[] = $methodCall;
@@ -147,7 +151,7 @@ private function handleUnset(Unset_ $unset): array|int
 
         foreach ($unset->vars as $var) {
             if ($var instanceof ArrayDimFetch) {
-                $methodCall = $this->getMethodCall($var, 'unset');
+                $methodCall = $this->createExplicitMethodCall($var, 'unset');
 
                 if ($methodCall instanceof MethodCall) {
                     $stmts[] = new Expression($methodCall);
@@ -173,8 +177,11 @@ private function handleUnset(Unset_ $unset): array|int
     /**
      * @param 'get'|'set'|'exists'|'unset' $action
      */
-    private function getMethodCall(ArrayDimFetch $arrayDimFetch, string $action, ?Expr $expr = null): ?MethodCall
-    {
+    private function createExplicitMethodCall(
+        ArrayDimFetch $arrayDimFetch,
+        string $action,
+        ?Expr $expr = null
+    ): ?MethodCall {
         if (! $arrayDimFetch->dim instanceof Node) {
             return null;
         }

rules/Transform/Rector/ArrayDimFetch/ArrayDimFetchToMethodCallRector.php

@@ -4,12 +4,12 @@
 
 namespace Rector\Reporting;
 
-use Rector\PhpParser\Enum\NodeGroup;
 use Rector\Configuration\Deprecation\Contract\DeprecatedInterface;
 use Rector\Configuration\Option;
 use Rector\Configuration\Parameter\SimpleParameterProvider;
 use Rector\Contract\PhpParser\Node\StmtsAwareInterface;
 use Rector\Contract\Rector\RectorInterface;
+use Rector\PhpParser\Enum\NodeGroup;
 use Symfony\Component\Console\Style\SymfonyStyle;
 
 final readonly class DeprecatedRulesReporter