From bf50512f6f128d5046bb203810b485adeb812c9a Mon Sep 17 00:00:00 2001
From: Simon Podlipsky <simon@podlipsky.net>
Date: Tue, 18 Jan 2022 09:18:44 +0100
Subject: [PATCH] Do not run
 `ChangeReadOnlyPropertyWithDefaultValueToConstantRector` on properties with
 attributes

---
 .../Fixture/skip_attribute.php.inc                    |  9 +++++++++
 ...adOnlyPropertyWithDefaultValueToConstantRector.php | 11 ++++++++++-
 2 files changed, 19 insertions(+), 1 deletion(-)
 create mode 100644 rules-tests/Privatization/Rector/Property/ChangeReadOnlyPropertyWithDefaultValueToConstantRector/Fixture/skip_attribute.php.inc

diff --git a/rules-tests/Privatization/Rector/Property/ChangeReadOnlyPropertyWithDefaultValueToConstantRector/Fixture/skip_attribute.php.inc b/rules-tests/Privatization/Rector/Property/ChangeReadOnlyPropertyWithDefaultValueToConstantRector/Fixture/skip_attribute.php.inc
new file mode 100644
index 00000000000..1a5211275b8
--- /dev/null
+++ b/rules-tests/Privatization/Rector/Property/ChangeReadOnlyPropertyWithDefaultValueToConstantRector/Fixture/skip_attribute.php.inc
@@ -0,0 +1,9 @@
+<?php
+
+namespace Rector\Tests\Privatization\Rector\Property\ChangeReadOnlyPropertyWithDefaultValueToConstantRector\Fixture;
+
+class SkipAttribute
+{
+    #[ORM\Column(type: 'json')]
+    private string $parameters = '{}';
+}
diff --git a/rules/Privatization/Rector/Property/ChangeReadOnlyPropertyWithDefaultValueToConstantRector.php b/rules/Privatization/Rector/Property/ChangeReadOnlyPropertyWithDefaultValueToConstantRector.php
index 21930a6766a..e30b4c01063 100644
--- a/rules/Privatization/Rector/Property/ChangeReadOnlyPropertyWithDefaultValueToConstantRector.php
+++ b/rules/Privatization/Rector/Property/ChangeReadOnlyPropertyWithDefaultValueToConstantRector.php
@@ -13,6 +13,7 @@
 use Rector\Core\Rector\AbstractRector;
 use Rector\Privatization\NodeFactory\ClassConstantFactory;
 use Rector\Privatization\NodeReplacer\PropertyFetchWithConstFetchReplacer;
+use Symplify\PHPStanRules\Reflection\PropertyAnalyzer;
 use Symplify\RuleDocGenerator\ValueObject\CodeSample\CodeSample;
 use Symplify\RuleDocGenerator\ValueObject\RuleDefinition;
 
@@ -130,6 +131,14 @@ private function shouldSkip(Property $property): bool
             return true;
         }
 
-        return $this->isObjectType($classLike, new ObjectType('PHP_CodeSniffer\Sniffs\Sniff'));
+        if ($property->attrGroups !== []) {
+            return true;
+        }
+
+        if ($this->isObjectType($classLike, new ObjectType('PHP_CodeSniffer\Sniffs\Sniff'))) {
+            return true;
+        }
+
+        return false;
     }
 }