[Php74] Skip property hook on RestoreDefaultNullToNullableTypePropert…

…yRector (#6736)
rectorphp · Feb 14, 2025 · 6137332 · 6137332
1 parent aca623e
commit 6137332
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 0 deletions.
diff --git a/...roperty/RestoreDefaultNullToNullableTypePropertyRector/Fixture/skip_property_hook.php.inc b/...roperty/RestoreDefaultNullToNullableTypePropertyRector/Fixture/skip_property_hook.php.inc
@@ -0,0 +1,10 @@
+<?php
+
+namespace Rector\Tests\Php74\Rector\Property\RestoreDefaultNullToNullableTypePropertyRector\Fixture;
+
+final class SkipPropertyHook
+{
+    public ?string $bar {
+        get => 'bar';
+    }
+}
diff --git a/rules/Php74/Rector/Property/RestoreDefaultNullToNullableTypePropertyRector.php b/rules/Php74/Rector/Property/RestoreDefaultNullToNullableTypePropertyRector.php
@@ -115,6 +115,10 @@ private function shouldSkipProperty(Property $property, Class_ $class): bool
             return true;
         }
 
+        if ($property->hooks !== []) {
+            return true;
+        }
+
         // is variable assigned in constructor
         $propertyName = $this->getName($property);
         return $this->constructorAssignDetector->isPropertyAssignedConditionally($class, $propertyName);