Add a configuration to skip DocBlock Custom Tag detection (#336)

Author: LuigiCardamone

src/Analyzer/FileParserFactory.php

@@ -9,12 +9,12 @@
 
 class FileParserFactory
 {
-    public static function createFileParser(TargetPhpVersion $targetPhpVersion): FileParser
+    public static function createFileParser(TargetPhpVersion $targetPhpVersion, $parseDocBlockCustomTags = true): FileParser
     {
         return new FileParser(
             new NodeTraverser(),
             new FileVisitor(ClassDescriptionBuilder::create()),
-            new NameResolver(),
+            new NameResolver(null, ['parseDocBlockCustomTags' => $parseDocBlockCustomTags]),
             $targetPhpVersion
         );
     }

src/Analyzer/NameResolver.php

@@ -31,6 +31,9 @@ class NameResolver extends NodeVisitorAbstract
     /** @var bool Whether to replace resolved nodes in place, or to add resolvedNode attributes */
     protected $replaceNodes;
 
+    /** @var bool Whether to parse DocBlock Custom Tags */
+    protected $parseDocBlockCustomTags;
+
     /**
      * Constructs a name resolution visitor.
      *
@@ -40,6 +43,7 @@ class NameResolver extends NodeVisitorAbstract
      *  * replaceNodes (default true): Resolved names are replaced in-place. Otherwise, a
      *    resolvedName attribute is added. (Names that cannot be statically resolved receive a
      *    namespacedName attribute, as usual.)
+     *  * parseDocBlockCustomTags (default true): Whether to parse DocBlock Custom Tags.
      *
      * @param ErrorHandler|null $errorHandler Error handler
      * @param array             $options      Options
@@ -49,6 +53,7 @@ public function __construct(ErrorHandler $errorHandler = null, array $options =
         $this->nameContext = new NameContext($errorHandler ?? new ErrorHandler\Throwing());
         $this->preserveOriginalNames = $options['preserveOriginalNames'] ?? false;
         $this->replaceNodes = $options['replaceNodes'] ?? true;
+        $this->parseDocBlockCustomTags = $options['parseDocBlockCustomTags'] ?? true;
     }
 
     /**
@@ -148,14 +153,16 @@ public function enterNode(Node $node)
                 break;
             }
 
-            if (!($node->type instanceof FullyQualified)) {
-                foreach ($phpDocNode->getTags() as $tagValue) {
-                    if ('@' === $tagValue->name[0] && false === strpos($tagValue->name, '@var')) {
-                        $customTag = str_replace('@', '', $tagValue->name);
-                        $type = $this->resolveName(new Node\Name($customTag), Use_::TYPE_NORMAL);
-                        $node->type = $type;
+            if ($this->parseDocBlockCustomTags) {
+                if (!($node->type instanceof FullyQualified)) {
+                    foreach ($phpDocNode->getTags() as $tagValue) {
+                        if ('@' === $tagValue->name[0] && false === strpos($tagValue->name, '@var')) {
+                            $customTag = str_replace('@', '', $tagValue->name);
+                            $type = $this->resolveName(new Node\Name($customTag), Use_::TYPE_NORMAL);
+                            $node->type = $type;
 
-                        break;
+                            break;
+                        }
                     }
                 }
             }

src/CLI/Config.php

@@ -13,11 +13,14 @@ class Config
     private $classSetRules;
     /** @var bool */
     private $runOnlyARule;
+    /** @var bool */
+    private $parseDocBlockCustomTags;
 
     public function __construct()
     {
         $this->classSetRules = [];
         $this->runOnlyARule = false;
+        $this->parseDocBlockCustomTags = true;
     }
 
     public function add(ClassSet $classSet, ArchRule ...$rules): self
@@ -46,4 +49,16 @@ public function getClassSetRules(): array
     {
         return $this->classSetRules;
     }
+
+    public function setParseDocBlockCustomTags($parseDocBlockCustomTags): self
+    {
+        $this->parseDocBlockCustomTags = $parseDocBlockCustomTags;
+
+        return $this;
+    }
+
+    public function getParseDocBlockCustomTags(): bool
+    {
+        return $this->parseDocBlockCustomTags;
+    }
 }

src/CLI/Runner.php

@@ -31,7 +31,7 @@ public function __construct(bool $stopOnFailure = false)
     public function run(Config $config, Progress $progress, TargetPhpVersion $targetPhpVersion): void
     {
         /** @var FileParser $fileParser */
-        $fileParser = FileParserFactory::createFileParser($targetPhpVersion);
+        $fileParser = FileParserFactory::createFileParser($targetPhpVersion, $config->getParseDocBlockCustomTags());
 
         /** @var ClassSetRules $classSetRule */
         foreach ($config->getClassSetRules() as $classSetRule) {

tests/Unit/Analyzer/FileVisitorTest.php

@@ -704,4 +704,34 @@ class ApplicationLevelDto
 
         $this->assertCount(1, $violations);
     }
+
+    public function test_it_skip_custom_tags_in_docblocks_the_option_parse_doc_block_custom_tags_is_false(): void
+    {
+        $code = <<< 'EOF'
+<?php
+namespace MyProject\AppBundle\Application;
+use Symfony\Component\Validator\Constraints as Assert;
+class ApplicationLevelDto
+{
+/**
+* @Assert\NotBlank
+*/
+     public $foo;
+
+}
+EOF;
+
+        /** @var FileParser $fp */
+        $fp = FileParserFactory::createFileParser(TargetPhpVersion::create('8.1'), false);
+        $fp->parse($code, 'relativePathName');
+
+        $cd = $fp->getClassDescriptions();
+
+        $violations = new Violations();
+
+        $notHaveDependencyOutsideNamespace = new DependsOnlyOnTheseNamespaces('MyProject\AppBundle\Application');
+        $notHaveDependencyOutsideNamespace->evaluate($cd[0], $violations, 'we want to add this rule for our software');
+
+        $this->assertCount(0, $violations);
+    }
 }