Avoid reprint on DowngradeReadonlyPropertyRector

Author: samsonasik

rules/DowngradePhp81/Rector/Property/DowngradeReadonlyPropertyRector.php

@@ -102,17 +102,13 @@ public function refactor(Node $node): ?Node
             return null;
         }
 
-        $hasChangedDoc = false;
         $hasChanged = false;
         foreach ($node->params as $param) {
             if (! $this->visibilityManipulator->isReadonly($param)) {
                 continue;
             }
 
-            if ($this->addPhpDocTag($param)) {
-                $hasChangedDoc = true;
-            }
-
+            $this->addPhpDocTag($param);
             $this->visibilityManipulator->removeReadonly($param);
             $hasChanged = true;
         }
@@ -121,23 +117,18 @@ public function refactor(Node $node): ?Node
             return null;
         }
 
-        if ($hasChangedDoc) {
-            $node->setAttribute(AttributeKey::ORIGINAL_NODE, null);
-        }
-
         return $node;
     }
 
-    private function addPhpDocTag(Property|Param $node): bool
+    private function addPhpDocTag(Property|Param $node): void
     {
         $phpDocInfo = $this->phpDocInfoFactory->createFromNodeOrEmpty($node);
 
         if ($phpDocInfo->hasByName(self::TAGNAME)) {
-            return false;
+            return;
         }
 
         $phpDocInfo->addPhpDocTagNode(new PhpDocTagNode('@' . self::TAGNAME, new GenericTagValueNode('')));
         $this->docBlockUpdater->updateRefactoredNodeWithPhpDocInfo($node);
-        return true;
     }
 }

rules/DowngradePhp85/Rector/Class_/DowngradeFinalPropertyPromotionRector.php

@@ -110,16 +110,16 @@ public function refactor(Node $node): ?ClassMethod
         return null;
     }
 
-    private function addPhpDocTag(Property|Param $node): bool
+    private function addPhpDocTag(Property|Param $node): void
     {
         $phpDocInfo = $this->phpDocInfoFactory->createFromNodeOrEmpty($node);
 
         if ($phpDocInfo->hasByName(self::TAGNAME)) {
-            return false;
+            return;
         }
 
         $phpDocInfo->addPhpDocTagNode(new PhpDocTagNode('@' . self::TAGNAME, new GenericTagValueNode('')));
         $this->docBlockUpdater->updateRefactoredNodeWithPhpDocInfo($node);
-        return true;
+        return;
     }
 }

tests/Issues/DowngradeReadonlyClassPropertyToPhp80/Fixture/fixture.php.inc

@@ -17,12 +17,10 @@ namespace Rector\Tests\Issues\DowngradeReadonlyClassPropertyToPhp80\Fixture;
 
 final class Fixture
 {
-    public function __construct(
-        /**
-         * @readonly
-         */
-        public array $property
-    )
+    public function __construct(/**
+     * @readonly
+     */
+    public array $property)
     {
     }
 }