fix

Author: samsonasik

rules/Php82/Rector/Class_/ReadOnlyClassRector.php

@@ -83,7 +83,7 @@ public function refactor(Node $node): ?Node
             return null;
         }
 
-        $this->visibilityManipulator->changeNodeVisibility($node, Visibility::READONLY);
+        $this->visibilityManipulator->makeReadonly($node);
 
         $constructClassMethod = $node->getMethod(MethodName::CONSTRUCT);
         if ($constructClassMethod instanceof ClassMethod) {

rules/Privatization/NodeManipulator/VisibilityManipulator.php

@@ -78,16 +78,15 @@ public function removeVisibility(ClassMethod | Property | ClassConst $node): voi
         }
     }
 
-    public function changeNodeVisibility(Class_ | ClassMethod | Property | ClassConst $node, int $visibility): void
+    public function changeNodeVisibility(ClassMethod | Property | ClassConst $node, int $visibility): void
     {
         Assert::oneOf($visibility, [
             Visibility::PUBLIC,
             Visibility::PROTECTED,
             Visibility::PRIVATE,
             Visibility::STATIC,
             Visibility::ABSTRACT,
-            Visibility::FINAL,
-            Visibility::READONLY,
+            Visibility::FINAL
         ]);
 
         $this->replaceVisibilityFlag($node, $visibility);
@@ -118,7 +117,7 @@ public function removeAbstract(ClassMethod $classMethod): void
         $classMethod->flags -= Class_::MODIFIER_ABSTRACT;
     }
 
-    public function makeReadonly(Property | Param $node): void
+    public function makeReadonly(Class_ | Property | Param $node): void
     {
         $this->addVisibilityFlag($node, Visibility::READONLY);
     }
@@ -147,14 +146,14 @@ private function removeVisibilityFlag(
         $node->flags &= ~$visibility;
     }
 
-    private function replaceVisibilityFlag(Class_ | ClassMethod | Property | ClassConst $node, int $visibility): void
+    private function replaceVisibilityFlag(ClassMethod | Property | ClassConst $node, int $visibility): void
     {
         $isStatic = $node instanceof ClassMethod && $node->isStatic();
         if ($isStatic) {
             $this->makeNonStatic($node);
         }
 
-        if ($visibility !== Visibility::STATIC && $visibility !== Visibility::ABSTRACT && $visibility !== Visibility::FINAL && $visibility !== Visibility::READONLY) {
+        if ($visibility !== Visibility::STATIC && $visibility !== Visibility::ABSTRACT && $visibility !== Visibility::FINAL) {
             $this->removeVisibility($node);
         }