[Php81] Skip non private modifier on ReadOnlyPropertyRector (#1367)

* [Php81] Skip non private vmodifier on ReadOnlyPropertyRector

* [ci-review] Rector Rectify

* [ci-review] Rector Rectify

Co-authored-by: GitHub Action <action@github.com>

rules-tests/Php81/Rector/Property/ReadOnlyPropertyRector/Fixture/skip_non_private_property.php.inc

@@ -0,0 +1,18 @@
+<?php
+
+namespace Rector\Tests\Php81\Rector\Property\ReadOnlyPropertyRector\Fixture;
+
+final class SkipNonPrivateProperty
+{
+    public string $name;
+
+    public function __construct(string $name)
+    {
+        $this->name = $name;
+    }
+
+    public function getName()
+    {
+        return $this->name;
+    }
+}

rules-tests/Php81/Rector/Property/ReadOnlyPropertyRector/Fixture/skip_non_private_property2.php.inc

@@ -0,0 +1,16 @@
+<?php
+
+namespace Rector\Tests\Php81\Rector\Property\ReadOnlyPropertyRector\Fixture;
+
+class SkipNonPrivateProperty2
+{
+    public function __construct(
+        public string $name
+    ) {
+    }
+
+    public function getName()
+    {
+        return $this->name;
+    }
+}

rules-tests/Privatization/NodeManipulator/VisibilityManipulatorTest.php

@@ -21,9 +21,9 @@ protected function setUp(): void
 
     public function test(): void
     {
-        $node = new ClassMethod('SomeClass');
-        $node->flags = Visibility::PUBLIC | Visibility::STATIC;
-        $this->visibilityManipulator->changeNodeVisibility($node, Visibility::PROTECTED);
-        $this->assertSame(Visibility::PROTECTED | Visibility::STATIC, $node->flags);
+        $classMethod = new ClassMethod('SomeClass');
+        $classMethod->flags = Visibility::PUBLIC | Visibility::STATIC;
+        $this->visibilityManipulator->changeNodeVisibility($classMethod, Visibility::PROTECTED);
+        $this->assertSame(Visibility::PROTECTED | Visibility::STATIC, $classMethod->flags);
     }
 }

rules/Php81/Rector/Property/ReadOnlyPropertyRector.php

@@ -11,6 +11,7 @@
 use Rector\Core\NodeManipulator\PropertyManipulator;
 use Rector\Core\Rector\AbstractRector;
 use Rector\Core\ValueObject\PhpVersionFeature;
+use Rector\Core\ValueObject\Visibility;
 use Rector\Privatization\NodeManipulator\VisibilityManipulator;
 use Rector\VersionBonding\Contract\MinPhpVersionInterface;
 use Symplify\RuleDocGenerator\ValueObject\CodeSample\CodeSample;
@@ -101,6 +102,10 @@ public function refactor(Node $node): ?Node
             return null;
         }
 
+        if ($node->flags !== Visibility::PRIVATE) {
+            return null;
+        }
+
         $this->visibilityManipulator->makeReadonly($node);
         return $node;
     }
@@ -112,7 +117,7 @@ public function provideMinPhpVersion(): int
 
     private function refactorParam(Param $param): Param | null
     {
-        if ($param->flags === 0) {
+        if ($param->flags !== Visibility::PRIVATE) {
             return null;
         }
 

rules/Privatization/NodeManipulator/VisibilityManipulator.php

@@ -12,6 +12,9 @@
 use Rector\Core\ValueObject\Visibility;
 use Webmozart\Assert\Assert;
 
+/**
+ * @see \Rector\Tests\Privatization\NodeManipulator\VisibilityManipulatorTest
+ */
 final class VisibilityManipulator
 {
     public function hasVisibility(ClassMethod | Property | ClassConst | Param $node, int $visibility): bool