minor #5439 VisibilityRequiredFixer - support type alternation for pr…

…operties (keradus)

This PR was merged into the 2.17 branch.

Discussion
----------

VisibilityRequiredFixer - support type alternation for properties

Commits
-------

6081f93 VisibilityRequiredFixer - support type alternation for properties

src/Fixer/ClassNotation/VisibilityRequiredFixer.php

@@ -97,7 +97,7 @@ protected function createConfigurationDefinition()
     protected function applyFix(\SplFileInfo $file, Tokens $tokens)
     {
         $tokensAnalyzer = new TokensAnalyzer($tokens);
-        $propertyTypeDeclarationKinds = [T_STRING, T_NS_SEPARATOR, CT::T_NULLABLE_TYPE, CT::T_ARRAY_TYPEHINT];
+        $propertyTypeDeclarationKinds = [T_STRING, T_NS_SEPARATOR, CT::T_NULLABLE_TYPE, CT::T_ARRAY_TYPEHINT, CT::T_TYPE_ALTERNATION];
 
         foreach (array_reverse($tokensAnalyzer->getClassyElements(), true) as $index => $element) {
             if (!\in_array($element['type'], $this->configuration['elements'], true)) {

tests/Fixer/ClassNotation/VisibilityRequiredFixerTest.php

@@ -827,4 +827,27 @@ public function provideFix74Cases()
             '<?php class Foo { static public ?array $foo; }',
         ];
     }
+
+    /**
+     * @param string      $expected
+     * @param null|string $input
+     *
+     * @requires PHP 8.0
+     * @dataProvider provideFix80Cases
+     */
+    public function testFix80($expected, $input = null)
+    {
+        $this->doTest($expected, $input);
+    }
+
+    public function provideFix80Cases()
+    {
+        yield [
+            '<?php class Foo { private int|float|null $foo; }',
+        ];
+
+        yield [
+            '<?php class Foo { private int | /* or empty */ null $foo; }',
+        ];
+    }
 }