Improved annotations parsing

Author: kukulich

SlevomatCodingStandard/Helpers/AnnotationHelper.php

@@ -66,7 +66,7 @@ public static function getAnnotations(\PHP_CodeSniffer\Files\File $codeSnifferFi
 				$annotationName = $tokens[$i]['content'];
 				$annotationParameters = null;
 				$annotationContent = null;
-				if (preg_match('~^(@[a-zA-Z\\\\]+)(?:\(([^)]*)\))?(?:\\s+(.+))?($)~s', $annotationCode, $matches)) {
+				if (preg_match('~^(@[a-zA-Z\\\\]+)(?:\((.*?)\))?(?:\\s+(.+))?($)~s', $annotationCode, $matches)) {
 					$annotationName = $matches[1];
 					$annotationParameters = trim($matches[2]);
 					if ($annotationParameters === '') {

tests/Helpers/AnnotationHelperTest.php

@@ -70,6 +70,14 @@ public function testFunctionWithParametrizedAnnotation(): void
 		$this->assertNull($annotations[0]->getContent());
 	}
 
+	public function testFunctionWithParametrizedAnnotationContainingParenthesis(): void
+	{
+		$annotations = AnnotationHelper::getAnnotationsByName($this->getTestedCodeSnifferFile(), $this->findFunctionPointerByName($this->getTestedCodeSnifferFile(), 'withParametrizedAnnotationContainingParenthesis'), '@Security');
+		$this->assertCount(1, $annotations);
+		$this->assertSame('"is_granted(\'ROLE_ADMIN\')"', $annotations[0]->getParameters());
+		$this->assertNull($annotations[0]->getContent());
+	}
+
 	public function testFunctionWithMultilineParametrizedAnnotation(): void
 	{
 		$annotations = AnnotationHelper::getAnnotationsByName($this->getTestedCodeSnifferFile(), $this->findFunctionPointerByName($this->getTestedCodeSnifferFile(), 'withMultilineParametrizedAnnotation'), '@Route');

tests/Helpers/data/annotation.php

@@ -36,6 +36,11 @@ public function withAnnotation($b, $c);
 	 */
 	public function withParametrizedAnnotation();
 
+	/**
+	 * @Security("is_granted('ROLE_ADMIN')")
+	 */
+	public function withParametrizedAnnotationContainingParenthesis();
+
 	/**
 	 * @Assert\Callback()
 	 */

tests/Sniffs/TypeHints/TypeHintDeclarationSniffTest.php

@@ -22,6 +22,7 @@ public function testNoErrors(): void
 				'@see',
 				'@Assert\Callback',
 				'@Something\\',
+				'@Security',
 			],
 		]));
 	}

tests/Sniffs/TypeHints/data/typeHintDeclarationNoErrors.php

@@ -654,4 +654,12 @@ public function parameterHasDescriptionContainingVariable(bool $foo, int $bar) :
 	{
 	}
 
+	/**
+	 * @Security("is_granted('ROLE_ADMIN')")
+	 */
+	public function usefullSymfonySecurityAnnotation()
+	{
+
+	}
+
 }