Align rector classes with rest of cakephp formatting standards

All mechanical changes from the modification in `phpcs.xml` and running `composer cs-fix`.
cakephp · Sep 6, 2024 · eb188cd · eb188cd
1 parent c2f65ec
commit eb188cd
Show file tree

Hide file tree

Showing 14 changed files with 66 additions and 61 deletions.
diff --git a/phpcs.xml b/phpcs.xml
@@ -4,7 +4,6 @@
 
     <rule ref="CakePHP"/>
 
-    <exclude-pattern>src/Rector</exclude-pattern>
     <exclude-pattern>tests/test_apps</exclude-pattern>
     <exclude-pattern>tests/TestCase/Rector</exclude-pattern>
 </ruleset>
diff --git a/src/Rector/NodeAnalyzer/FluentChainMethodCallNodeAnalyzer.php b/src/Rector/NodeAnalyzer/FluentChainMethodCallNodeAnalyzer.php
@@ -1,5 +1,5 @@
 <?php
-declare (strict_types=1);
+declare(strict_types=1);
 
 namespace Cake\Upgrade\Rector\NodeAnalyzer;
 
@@ -20,7 +20,7 @@ final class FluentChainMethodCallNodeAnalyzer
     /**
      * @api doctrine
      */
-    public function resolveRootMethodCall(MethodCall $methodCall) : ?MethodCall
+    public function resolveRootMethodCall(MethodCall $methodCall): ?MethodCall
     {
         $callerNode = $methodCall->var;
         while ($callerNode instanceof MethodCall && $callerNode->var instanceof MethodCall) {
@@ -29,17 +29,20 @@ public function resolveRootMethodCall(MethodCall $methodCall) : ?MethodCall
         if ($callerNode instanceof MethodCall) {
             return $callerNode;
         }
+
         return null;
     }
+
     /**
      * @return \PhpParser\Node\Expr|\PhpParser\Node\Name
      */
-    public function resolveRootExpr(MethodCall $methodCall)
+    public function resolveRootExpr(MethodCall $methodCall): Expr|Name
     {
         $callerNode = $methodCall->var;
         while ($callerNode instanceof MethodCall || $callerNode instanceof StaticCall) {
             $callerNode = $callerNode instanceof StaticCall ? $callerNode->class : $callerNode->var;
         }
+
         return $callerNode;
     }
 }
diff --git a/src/Rector/Rector/MethodCall/AddMethodCallArgsRector.php b/src/Rector/Rector/MethodCall/AddMethodCallArgsRector.php
@@ -25,7 +25,7 @@ final class AddMethodCallArgsRector extends AbstractRector implements Configurab
     public const ADD_METHOD_CALL_ARGS = 'add_method_call_args';
 
     /**
-     * @var \Cake\Upgrade\Rector\ValueObject\AddMethodCallArgs[]
+     * @var array<\Cake\Upgrade\Rector\ValueObject\AddMethodCallArgs>
      */
     private array $callsWithAddMethodCallArgs = [];
 
@@ -97,7 +97,7 @@ public function refactor(Node $node): ?Node
     }
 
     /**
-     * @param mixed[] $configuration
+     * @param array<mixed> $configuration
      */
     public function configure(array $configuration): void
     {

diff --git a/src/Rector/Rector/MethodCall/ArrayToFluentCallRector.php b/src/Rector/Rector/MethodCall/ArrayToFluentCallRector.php
@@ -3,13 +3,13 @@
 
 namespace Cake\Upgrade\Rector\Rector\MethodCall;
 
+use Cake\Upgrade\Rector\ValueObject\ArrayItemsAndFluentClass;
+use Cake\Upgrade\Rector\ValueObject\ArrayToFluentCall;
+use Cake\Upgrade\Rector\ValueObject\FactoryMethod;
 use PhpParser\Node;
 use PhpParser\Node\Expr\Array_;
 use PhpParser\Node\Expr\MethodCall;
 use PhpParser\Node\Scalar\String_;
-use Cake\Upgrade\Rector\ValueObject\ArrayItemsAndFluentClass;
-use Cake\Upgrade\Rector\ValueObject\ArrayToFluentCall;
-use Cake\Upgrade\Rector\ValueObject\FactoryMethod;
 use Rector\Contract\Rector\ConfigurableRectorInterface;
 use Rector\Rector\AbstractRector;
 use Symplify\RuleDocGenerator\ValueObject\CodeSample\ConfiguredCodeSample;
@@ -31,12 +31,12 @@ final class ArrayToFluentCallRector extends AbstractRector implements Configurab
     public const FACTORY_METHODS = 'factory_methods';
 
     /**
-     * @var \Cake\Upgrade\Rector\ValueObject\ArrayToFluentCall[]
+     * @var array<\Cake\Upgrade\Rector\ValueObject\ArrayToFluentCall>
      */
     private array $arraysToFluentCalls = [];
 
     /**
-     * @var \Cake\Upgrade\Rector\ValueObject\FactoryMethod[]
+     * @var array<\Cake\Upgrade\Rector\ValueObject\FactoryMethod>
      */
     private array $factoryMethods = [];
 
@@ -115,7 +115,7 @@ public function refactor(Node $node): ?Node
     }
 
     /**
-     * @param mixed[] $configuration
+     * @param array<mixed> $configuration
      */
     public function configure(array $configuration): void
     {

diff --git a/src/Rector/Rector/MethodCall/ModalToGetSetRector.php b/src/Rector/Rector/MethodCall/ModalToGetSetRector.php
@@ -3,11 +3,11 @@
 
 namespace Cake\Upgrade\Rector\Rector\MethodCall;
 
+use Cake\Upgrade\Rector\ValueObject\ModalToGetSet;
 use PhpParser\Node;
 use PhpParser\Node\Expr\Array_;
 use PhpParser\Node\Expr\MethodCall;
 use PhpParser\Node\Identifier;
-use Cake\Upgrade\Rector\ValueObject\ModalToGetSet;
 use Rector\Contract\Rector\ConfigurableRectorInterface;
 use Rector\Rector\AbstractRector;
 use Symplify\RuleDocGenerator\ValueObject\CodeSample\ConfiguredCodeSample;
@@ -27,7 +27,7 @@ final class ModalToGetSetRector extends AbstractRector implements ConfigurableRe
     public const UNPREFIXED_METHODS_TO_GET_SET = 'unprefixed_methods_to_get_set';
 
     /**
-     * @var \Cake\Upgrade\Rector\ValueObject\ModalToGetSet[]
+     * @var array<\Cake\Upgrade\Rector\ValueObject\ModalToGetSet>
      */
     private array $unprefixedMethodsToGetSet = [];
 
@@ -92,7 +92,7 @@ public function refactor(Node $node): ?Node
     }
 
     /**
-     * @param mixed[] $configuration
+     * @param array<mixed> $configuration
      */
     public function configure(array $configuration): void
     {

diff --git a/src/Rector/Rector/MethodCall/OptionsArrayToNamedParametersRector.php b/src/Rector/Rector/MethodCall/OptionsArrayToNamedParametersRector.php
@@ -3,28 +3,25 @@
 
 namespace Cake\Upgrade\Rector\Rector\MethodCall;
 
+use Cake\Upgrade\Rector\ValueObject\OptionsArrayToNamedParameters;
 use PhpParser\Node;
+use PhpParser\Node\Arg;
 use PhpParser\Node\Expr\Array_;
 use PhpParser\Node\Expr\MethodCall;
-use PhpParser\Node\Scalar\String_;
-use Cake\Upgrade\Rector\ValueObject\OptionsArrayToNamedParameters;
-use PhpParser\Node\Arg;
 use PhpParser\Node\Identifier;
 use Rector\Contract\Rector\ConfigurableRectorInterface;
 use Rector\Rector\AbstractRector;
 use Symplify\RuleDocGenerator\ValueObject\CodeSample\ConfiguredCodeSample;
 use Symplify\RuleDocGenerator\ValueObject\RuleDefinition;
 
-use function RectorPrefix202304\dump_node;
-
 final class OptionsArrayToNamedParametersRector extends AbstractRector implements ConfigurableRectorInterface
 {
     public const OPTIONS_TO_NAMED_PARAMETERS = 'options_to_named_parameters';
 
     /**
      * @var \Cake\Upgrade\Rector\ValueObject\OptionsArrayToNamedParameters
      */
-    private $optionsToNamed = [];
+    private OptionsArrayToNamedParameters $optionsToNamed = [];
 
     public function getRuleDefinition(): RuleDefinition
     {
@@ -82,9 +79,10 @@ public function refactor(Node $node): ?Node
             if (!$this->matchTypeAndMethodName($optionsToNamed, $node)) {
                 continue;
             }
-            return $this->replaceMethodCall($optionsToNamed, $node);
 
+            return $this->replaceMethodCall($optionsToNamed, $node);
         }
+
         return null;
     }
 

diff --git a/src/Rector/Rector/MethodCall/RemoveIntermediaryMethodRector.php b/src/Rector/Rector/MethodCall/RemoveIntermediaryMethodRector.php
@@ -3,12 +3,12 @@
 
 namespace Cake\Upgrade\Rector\Rector\MethodCall;
 
+use Cake\Upgrade\Rector\NodeAnalyzer\FluentChainMethodCallNodeAnalyzer;
+use Cake\Upgrade\Rector\ValueObject\RemoveIntermediaryMethod;
 use PhpParser\Node;
 use PhpParser\Node\Expr\MethodCall;
 use PhpParser\Node\Expr\Variable;
 use PhpParser\Node\Identifier;
-use Cake\Upgrade\Rector\NodeAnalyzer\FluentChainMethodCallNodeAnalyzer;
-use Cake\Upgrade\Rector\ValueObject\RemoveIntermediaryMethod;
 use Rector\Contract\Rector\ConfigurableRectorInterface;
 use Rector\Rector\AbstractRector;
 use Symplify\RuleDocGenerator\ValueObject\CodeSample\ConfiguredCodeSample;
@@ -28,7 +28,7 @@ final class RemoveIntermediaryMethodRector extends AbstractRector implements Con
     public const REMOVE_INTERMEDIARY_METHOD = 'remove_intermediary_method';
 
     /**
-     * @var \Cake\Upgrade\Rector\ValueObject\RemoveIntermediaryMethod[]
+     * @var array<\Cake\Upgrade\Rector\ValueObject\RemoveIntermediaryMethod>
      */
     private array $removeIntermediaryMethod = [];
 
@@ -86,7 +86,7 @@ public function refactor(Node $node): ?Node
     }
 
     /**
-     * @param mixed[] $configuration
+     * @param array<mixed> $configuration
      */
     public function configure(array $configuration): void
     {

diff --git a/src/Rector/Rector/MethodCall/RemoveMethodCallRector.php b/src/Rector/Rector/MethodCall/RemoveMethodCallRector.php
@@ -1,6 +1,5 @@
 <?php
-
-declare (strict_types = 1);
+declare(strict_types=1);
 
 namespace Cake\Upgrade\Rector\Rector\MethodCall;
 
@@ -24,9 +23,9 @@ final class RemoveMethodCallRector extends AbstractRector implements Configurabl
     public const REMOVE_METHOD_CALL_ARGS = 'remove_method_call_args';
 
     /**
-     * @var \Cake\Upgrade\Rector\ValueObject\RemoveMethodCall[]
+     * @var array<\Cake\Upgrade\Rector\ValueObject\RemoveMethodCall>
      */
-    private $callsWithRemoveMethodCallArgs = [];
+    private array $callsWithRemoveMethodCallArgs = [];
 
     public function getRuleDefinition(): RuleDefinition
     {
@@ -36,24 +35,26 @@ public function getRuleDefinition(): RuleDefinition
 $obj = new SomeClass();
 $obj->methodCall1();
 $obj->methodCall2();
-CODE_SAMPLE, <<<'CODE_SAMPLE'
+CODE_SAMPLE,
+                <<<'CODE_SAMPLE'
 $obj = new SomeClass();
 $obj->methodCall2();
-CODE_SAMPLE, ['SomeClass', 'methodCall1']
-            )
+CODE_SAMPLE,
+                ['SomeClass', 'methodCall1']
+            ),
         ]);
     }
 
     /**
-     * @return array<class-string<Node>>
+     * @return array<class-string<\PhpParser\Node>>
      */
     public function getNodeTypes(): array
     {
         return [Expression::class];
     }
 
     /**
-     * @param Expression $node
+     * @param \PhpParser\Node\Stmt\Expression $node
      */
     public function refactor(Node $node): ?int
     {
@@ -77,7 +78,7 @@ public function refactor(Node $node): ?int
     }
 
     /**
-     * @param mixed[] $configuration
+     * @param array<mixed> $configuration
      */
     public function configure(array $configuration): void
     {

diff --git a/src/Rector/Rector/MethodCall/RenameMethodCallBasedOnParameterRector.php b/src/Rector/Rector/MethodCall/RenameMethodCallBasedOnParameterRector.php
@@ -3,10 +3,10 @@
 
 namespace Cake\Upgrade\Rector\Rector\MethodCall;
 
+use Cake\Upgrade\Rector\ValueObject\RenameMethodCallBasedOnParameter;
 use PhpParser\Node;
 use PhpParser\Node\Expr\MethodCall;
 use PhpParser\Node\Identifier;
-use Cake\Upgrade\Rector\ValueObject\RenameMethodCallBasedOnParameter;
 use Rector\Contract\Rector\ConfigurableRectorInterface;
 use Rector\PhpParser\Node\Value\ValueResolver;
 use Rector\Rector\AbstractRector;
@@ -27,7 +27,7 @@ final class RenameMethodCallBasedOnParameterRector extends AbstractRector implem
     public const CALLS_WITH_PARAM_RENAMES = 'calls_with_param_renames';
 
     /**
-     * @var \Cake\Upgrade\Rector\ValueObject\RenameMethodCallBasedOnParameter[]
+     * @var array<\Cake\Upgrade\Rector\ValueObject\RenameMethodCallBasedOnParameter>
      */
     private array $callsWithParamRenames = [];
 
@@ -92,7 +92,7 @@ public function refactor(Node $node): ?Node
     }
 
     /**
-     * @param mixed[] $configuration
+     * @param array<mixed> $configuration
      */
     public function configure(array $configuration): void
     {

diff --git a/src/Rector/Rector/MethodCall/TableRegistryLocatorRector.php b/src/Rector/Rector/MethodCall/TableRegistryLocatorRector.php
@@ -1,5 +1,4 @@
 <?php
-
 declare(strict_types=1);
 
 namespace Cake\Upgrade\Rector\Rector\MethodCall;
@@ -23,7 +22,7 @@ public function getRuleDefinition(): RuleDefinition
                 <<<'CODE_SAMPLE'
 TableRegistry::getTableLocator()->get('something');
 CODE_SAMPLE
-            )
+            ),
         ]);
     }
 
@@ -34,7 +33,7 @@ public function getNodeTypes(): array
 
     public function refactor(Node $node): ?Node
     {
-        if(! $node instanceof StaticCall) {
+        if (! $node instanceof StaticCall) {
             return null;
         }