Invalid @param annotation should be ignored

Author: kukulich

SlevomatCodingStandard/Sniffs/TypeHints/TypeHintDeclarationSniff.php

@@ -781,12 +781,14 @@ private function getFunctionParameterTypeHintsDefinitions(\PHP_CodeSniffer_File
 				continue;
 			}
 
-			$parameterAnnotationParts = preg_split('~\\s+~', $parameterAnnotation->getContent(), 2);
-			$parameterTypeHintDefinition = $parameterAnnotationParts[0];
-			if (isset($parameterAnnotationParts[1]) && preg_match('~^(?:\.{3}\\s*)?(\$\\S+)~', $parameterAnnotationParts[1], $matches)) {
-				$parametersTypeHintsDefinitions[$matches[1]] = $parameterTypeHintDefinition;
+			if (!preg_match('~^([^$]\\S*)(?:\\s+(?:\.{3}\\s*)?(\$\\S+))?~', $parameterAnnotation->getContent(), $matches)) {
+				continue;
+			}
+
+			if (isset($matches[2])) {
+				$parametersTypeHintsDefinitions[$matches[2]] = $matches[1];
 			} elseif (isset($parametersNames[$parameterAnnotationNo])) {
-				$parametersTypeHintsDefinitions[$parametersNames[$parameterAnnotationNo]] = $parameterTypeHintDefinition;
+				$parametersTypeHintsDefinitions[$parametersNames[$parameterAnnotationNo]] = $matches[1];
 			}
 		}
 

tests/Sniffs/TypeHints/data/fixableParameterTypeHintsWithDisabledNullableTypeHints.fixed.php

@@ -64,6 +64,14 @@ function optionalWithoutTypeHint(string $a = null)
 
 }
 
+/**
+ * @param $a string
+ */
+function invalidAnnotation($a)
+{
+
+}
+
 class Foo
 {
 
@@ -127,4 +135,12 @@ public function optionalWithoutTypeHint(string $a = null)
 
 	}
 
+	/**
+	 * @param $a string
+	 */
+	private function invalidAnnotation($a)
+	{
+
+	}
+
 }

tests/Sniffs/TypeHints/data/fixableParameterTypeHintsWithDisabledNullableTypeHints.php

@@ -64,6 +64,14 @@ function optionalWithoutTypeHint($a = null)
 
 }
 
+/**
+ * @param $a string
+ */
+function invalidAnnotation($a)
+{
+
+}
+
 class Foo
 {
 
@@ -127,4 +135,12 @@ public function optionalWithoutTypeHint($a = null)
 
 	}
 
+	/**
+	 * @param $a string
+	 */
+	private function invalidAnnotation($a)
+	{
+
+	}
+
 }

tests/Sniffs/TypeHints/data/fixableParameterTypeHintsWithEnabledNullableTypeHints.fixed.php

@@ -174,6 +174,14 @@ function traversableDateTimeImmutable($a)
 
 }
 
+/**
+ * @param $a string
+ */
+function invalidAnnotation($a)
+{
+
+}
+
 abstract class Foo
 {
 
@@ -356,4 +364,12 @@ protected function traversableDateTimeImmutable($a)
 
 	}
 
+	/**
+	 * @param $a string
+	 */
+	private function invalidAnnotation($a)
+	{
+
+	}
+
 }

tests/Sniffs/TypeHints/data/fixableParameterTypeHintsWithEnabledNullableTypeHints.php

@@ -174,6 +174,14 @@ function traversableDateTimeImmutable($a)
 
 }
 
+/**
+ * @param $a string
+ */
+function invalidAnnotation($a)
+{
+
+}
+
 abstract class Foo
 {
 
@@ -356,4 +364,12 @@ protected function traversableDateTimeImmutable($a)
 
 	}
 
+	/**
+	 * @param $a string
+	 */
+	private function invalidAnnotation($a)
+	{
+
+	}
+
 }