[CodingStyle] Skip inside readonly class on AddArrayDefaultToArrayPro…

…pertyRector (#3612)
rectorphp · Apr 13, 2023 · f81f92c · f81f92c
1 parent edf54c8
commit f81f92c
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 0 deletions.
diff --git a/...Rector/Class_/AddArrayDefaultToArrayPropertyRector/Fixture/skip_in_readonly_class.php.inc b/...Rector/Class_/AddArrayDefaultToArrayPropertyRector/Fixture/skip_in_readonly_class.php.inc
@@ -0,0 +1,10 @@
+<?php
+
+namespace Rector\Tests\CodingStyle\Rector\Class_\AddArrayDefaultToArrayPropertyRector\Fixture;
+
+final readonly class SkipInReadonlyClass
+{
+    /** @var string[] */
+    private array $arr;
+}
+
diff --git a/rules/CodingStyle/Rector/Class_/AddArrayDefaultToArrayPropertyRector.php b/rules/CodingStyle/Rector/Class_/AddArrayDefaultToArrayPropertyRector.php
@@ -93,6 +93,10 @@ public function getNodeTypes(): array
      */
     public function refactor(Node $node): ?Node
     {
+        if ($node->isReadonly()) {
+            return null;
+        }
+
         $changedProperties = $this->collectPropertyNamesWithMissingDefaultArray($node);
         if ($changedProperties === []) {
             return null;