minor #3891 PhpdocNoEmptyReturnFixer - account for null[] (dmvdbrugge)

This PR was merged into the 2.12 branch.

Discussion
----------

PhpdocNoEmptyReturnFixer - account for null[]

This fixes #3869

Commits
-------

23084e9 PhpdocNoEmptyReturnFixer - account for null[]

Author: keradus

src/DocBlock/Annotation.php

@@ -226,6 +226,22 @@ public function setTypes(array $types)
         $this->clearCache();
     }
 
+    /**
+     * Get the normalized types associated with this annotation, so they can easily be compared.
+     *
+     * @return string[]
+     */
+    public function getNormalizedTypes()
+    {
+        $normalized = array_map(static function ($type) {
+            return strtolower($type);
+        }, $this->getTypes());
+
+        sort($normalized);
+
+        return $normalized;
+    }
+
     /**
      * Remove this annotation by removing all its lines.
      */

src/Fixer/Phpdoc/PhpdocNoEmptyReturnFixer.php

@@ -17,7 +17,6 @@
 use PhpCsFixer\DocBlock\DocBlock;
 use PhpCsFixer\FixerDefinition\CodeSample;
 use PhpCsFixer\FixerDefinition\FixerDefinition;
-use PhpCsFixer\Preg;
 use PhpCsFixer\Tokenizer\Token;
 use PhpCsFixer\Tokenizer\Tokens;
 
@@ -105,7 +104,9 @@ protected function applyFix(\SplFileInfo $file, Tokens $tokens)
      */
     private function fixAnnotation(DocBlock $doc, Annotation $annotation)
     {
-        if (1 === Preg::match('/@return\s+(void|null)(?!\|)/', $doc->getLine($annotation->getStart())->getContent())) {
+        $types = $annotation->getNormalizedTypes();
+
+        if (1 === count($types) && ('null' === $types[0] || 'void' === $types[0])) {
             $annotation->remove();
         }
     }

tests/DocBlock/AnnotationTest.php

@@ -342,6 +342,29 @@ public function provideTypesCases()
         ];
     }
 
+    /**
+     * @param string[] $expected
+     * @param string   $input
+     *
+     * @dataProvider provideNormalizedTypesCases
+     */
+    public function testNormalizedTypes($expected, $input)
+    {
+        $line = new Line($input);
+        $tag = new Annotation([$line]);
+
+        $this->assertSame($expected, $tag->getNormalizedTypes());
+    }
+
+    public function provideNormalizedTypesCases()
+    {
+        return [
+            [['null', 'string'], '* @param StRiNg|NuLl $foo'],
+            [['void'], '* @return Void'],
+            [['bar', 'baz', 'foo', 'null', 'qux'], '* @return Foo|Bar|Baz|Qux|null'],
+        ];
+    }
+
     public function testGetTypesOnBadTag()
     {
         $this->expectException(\RuntimeException::class);

tests/Fixer/Phpdoc/PhpdocNoEmptyReturnFixerTest.php

@@ -58,6 +58,46 @@ public function testFixNull()
      * @return null
      */
 
+EOF;
+
+        $this->doTest($expected, $input);
+    }
+
+    public function testFixVoidCaseInsensitive()
+    {
+        $expected = <<<'EOF'
+<?php
+    /**
+     */
+
+EOF;
+
+        $input = <<<'EOF'
+<?php
+    /**
+     * @return vOId
+     */
+
+EOF;
+
+        $this->doTest($expected, $input);
+    }
+
+    public function testFixNullCaseInsensitive()
+    {
+        $expected = <<<'EOF'
+<?php
+    /**
+     */
+
+EOF;
+
+        $input = <<<'EOF'
+<?php
+    /**
+     * @return nULl
+     */
+
 EOF;
 
         $this->doTest($expected, $input);
@@ -123,6 +163,19 @@ public function testOtherDoNothing()
      * @return int|null
      */
 
+EOF;
+
+        $this->doTest($expected);
+    }
+
+    public function testYetAnotherDoNothing()
+    {
+        $expected = <<<'EOF'
+<?php
+    /**
+     * @return null[]|string[]
+     */
+
 EOF;
 
         $this->doTest($expected);