Skip to content

Commit 4a9b3d8

Browse files
committed
visibility is one of
1 parent 1a281e3 commit 4a9b3d8

2 files changed

Lines changed: 20 additions & 0 deletions

File tree

‎rules/Privatization/NodeManipulator/VisibilityManipulator.php‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -82,6 +82,9 @@ public function makeNonFinal(Class_|ClassMethod|Param $node): void
8282
$node->flags -= Modifiers::FINAL;
8383
}
8484

85+
/**
86+
* @param Visibility::* $visibility
87+
*/
8588
public function changeNodeVisibility(ClassMethod|Property|ClassConst $node, int $visibility): void
8689
{
8790
Assert::oneOf($visibility, [

‎rules/Visibility/ValueObject/ChangeMethodVisibility.php‎

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,15 +5,29 @@
55
namespace Rector\Visibility\ValueObject;
66

77
use Rector\Validation\RectorAssert;
8+
use Rector\ValueObject\Visibility;
9+
use Webmozart\Assert\Assert;
810

911
final readonly class ChangeMethodVisibility
1012
{
13+
/**
14+
* @param Visibility::* $visibility
15+
*/
1116
public function __construct(
1217
private string $class,
1318
private string $method,
1419
private int $visibility
1520
) {
1621
RectorAssert::className($class);
22+
23+
Assert::oneOf($visibility, [
24+
Visibility::PUBLIC,
25+
Visibility::PROTECTED,
26+
Visibility::PRIVATE,
27+
Visibility::STATIC,
28+
Visibility::ABSTRACT,
29+
Visibility::FINAL,
30+
]);
1731
}
1832

1933
public function getClass(): string
@@ -26,6 +40,9 @@ public function getMethod(): string
2640
return $this->method;
2741
}
2842

43+
/**
44+
* @return Visibility::*
45+
*/
2946
public function getVisibility(): int
3047
{
3148
return $this->visibility;

0 commit comments

Comments
 (0)