ComponentReflection::parseAnnotation: fix false positive annotation parsing (#151)

Author: mrtnzlml

src/Application/UI/ComponentReflection.php

@@ -195,7 +195,7 @@ public static function convertType(& $val, $type, $isClass = FALSE)
 	 */
 	public static function parseAnnotation(\Reflector $ref, $name)
 	{
-		if (!preg_match_all('#[\\s*]@' . preg_quote($name, '#') . '(?:\(\\s*([^)]*)\\s*\))?#', $ref->getDocComment(), $m)) {
+		if (!preg_match_all('#[\\s*]@' . preg_quote($name, '#') . '(?:\(\\s*([^)]*)\\s*\)|\\s|$)#', $ref->getDocComment(), $m)) {
 			return FALSE;
 		}
 		static $tokens = ['true' => TRUE, 'false' => FALSE, 'null' => NULL];

tests/UI/ComponentReflection.parseAnnotation.phpt

@@ -15,6 +15,8 @@ require __DIR__ . '/../bootstrap.php';
  * @title(value ="Johno's addendum", mode=True,) , out
  * @title
  * @title()
+ * @publi - 'c' is missing intentionally
+ * @privatex
  * @components(item 1)
  * @persistent(true)
  * @persistent(FALSE)
@@ -30,6 +32,8 @@ class TestClass {}
 $rc = new ReflectionClass('TestClass');
 
 Assert::same(['value ="Johno\'s addendum"', 'mode=True', TRUE, TRUE], Reflection::parseAnnotation($rc, 'title'));
+Assert::same(FALSE, Reflection::parseAnnotation($rc, 'public'));
+Assert::same(FALSE, Reflection::parseAnnotation($rc, 'private'));
 Assert::same(['item 1'], Reflection::parseAnnotation($rc, 'components'));
 Assert::same([TRUE, FALSE, NULL], Reflection::parseAnnotation($rc, 'persistent'));
 Assert::same([TRUE], Reflection::parseAnnotation($rc, 'renderable'));