Zend\Code\DocBlockScanner doesn't recognize anymore tags without values about zendframework

[zfbot]

(Originally posted by: adamlundrigan on 02/23/12)

I believe this issue has been rectified already. I ran this unit test against Zend\Code\Scanner\DocBlockScanner:

/**
 * @group ZF2-110
 */
public function testDocBlockScannerParsesTagsWithNoValuesProperly()
{
    $docComment = <<<EOB
/**
 * @mytag
 */
EOB;
    $tokenScanner = new DocBlockScanner($docComment);
    $tags = $tokenScanner->getTags();
    $this->assertCount(1, $tags);
    $this->assertArrayHasKey('name', $tags[0]);
    $this->assertEquals('@mytag', $tags[0]['name']);
    $this->assertArrayHasKey('value', $tags[0]);
    $this->assertEquals('', $tags[0]['value']);
}

And it passed. I've issued a PR (#820) to get this test added to the ZF2 test suite.

[zfbot]

(Originally posted by: mtorromeo on 02/23/12)

It seems I was wrong thinking that the problem was simply about tags without values. I still couldn't get my tags recognized with latest master and I found out that the issue was caused by a bug in the docblock cleaning function.

The regular expression used with single quotes failed to recognize tabs (\t) and also the cleaned docblock was not passed to the scanner.

Here is the fix:

diff --git a/library/Zend/Code/Reflection/DocBlockReflection.php b/library/Zend/Code/Reflection/DocBlockReflection.php
index 564ce62..63c35d4 100644
--- a/library/Zend/Code/Reflection/DocBlockReflection.php
+++ b/library/Zend/Code/Reflection/DocBlockReflection.php
@@ -259,10 +259,10 @@ class DocBlockReflection implements Reflection
         $docComment = $this->docComment; // localize variable

         // create a clean docComment
-        $this->cleanDocComment = preg_replace('#[ \t]*(?:\/\*\*|\*\/|\*)?[ ]{0,1}(.*)?#', '$1', $docComment);
+        $this->cleanDocComment = preg_replace("#[ \t]*(?:\/\*\*|\*\/|\*)?[ ]{0,1}(.*)?#", '$1', $docComment);
         $this->cleanDocComment = ltrim($this->cleanDocComment, "\r\n"); // @todo should be changed to remove first and last empty line

-        $scanner = new DocBlockScanner($docComment);
+        $scanner = new DocBlockScanner($this->cleanDocComment);
         $this->shortDescription = ltrim($scanner->getShortDescription());
         $this->longDescription  = ltrim($scanner->getLongDescription());
         foreach ($scanner->getTags() as $tag) {

[zfbot]

(Originally posted by: maks3w on 05/14/12)

Hi Massimiliano,

You can send a Pull Request in Github with your patch. Anyway I tried to proposal but break some tests.

Regards.

[zfbot]

This issue was ported from the ZF2 Jira Issue Tracker at
http://framework.zend.com/issues/browse/ZF2-110

Known GitHub users mentioned in the original message or comment:
@mtorromeo, @Maks3w, @adamlundrigan