PhpdocOnlyAllowedAnnotationsFixer - fix for invalid tag (#312)

Author: kubawerlos

README.md

@@ -8,7 +8,7 @@
 
 [![CI Status](https://github.com/kubawerlos/php-cs-fixer-custom-fixers/workflows/CI/badge.svg?branch=master&event=push)](https://github.com/kubawerlos/php-cs-fixer-custom-fixers/actions)
 [![Code coverage](https://img.shields.io/coveralls/github/kubawerlos/php-cs-fixer-custom-fixers/master.svg)](https://coveralls.io/github/kubawerlos/php-cs-fixer-custom-fixers?branch=master)
-![Tests](https://img.shields.io/badge/tests-2176-brightgreen.svg)
+![Tests](https://img.shields.io/badge/tests-2177-brightgreen.svg)
 [![Mutation testing badge](https://badge.stryker-mutator.io/github.com/kubawerlos/php-cs-fixer-custom-fixers/master)](https://stryker-mutator.github.io)
 [![Psalm type coverage](https://shepherd.dev/github/kubawerlos/php-cs-fixer-custom-fixers/coverage.svg)](https://shepherd.dev/github/kubawerlos/php-cs-fixer-custom-fixers)
 

src/Fixer/PhpdocOnlyAllowedAnnotationsFixer.php

@@ -97,11 +97,12 @@ public function fix(\SplFileInfo $file, Tokens $tokens): void
  $docBlock = new DocBlock($token->getContent());
 
  foreach ($docBlock->getAnnotations() as $annotation) {
- Preg::match('/@([a-zA-Z0-9\Q_-\\\E]+)/', $annotation->getContent(), $matches);
-
- if (\in_array($matches[1], $this->elements, true)) {
+ if (Preg::match('/@([a-zA-Z0-9_\\-\\\\]+)/', $annotation->getContent(), $matches) === 1
+ && \in_array($matches[1], $this->elements, true)
+ ) {
  continue;
  }
+
  $annotation->remove();
  }
 

tests/Fixer/PhpdocOnlyAllowedAnnotationsFixerTest.php

@@ -121,5 +121,18 @@ public static function provideFixCases(): iterable
 ',
  ['elements' => ['foo', 'bar']],
  ];
+
+ yield [
+ '<?php
+ /**
+ */
+ ',
+ '<?php
+ /**
+ * @#
+ * @return Foo
+ */
+ ',
+ ];
  }
 }