Added missing trailing dot in error messages

Author: adaamz

src/Rules/Methods/MissingMethodParameterTypehintRule.php

@@ -57,7 +57,7 @@ private function checkMethodParameter(MethodReflection $methodReflection, Parame
 
 		if ($parameterType instanceof MixedType && !$parameterType->isExplicitMixed()) {
 			return sprintf(
-				'Method %s::%s() has parameter $%s with no typehint specified',
+				'Method %s::%s() has parameter $%s with no typehint specified.',
 				$methodReflection->getDeclaringClass()->getDisplayName(),
 				$methodReflection->getName(),
 				$parameterReflection->getName()

src/Rules/Methods/MissingMethodReturnTypehintRule.php

@@ -49,7 +49,7 @@ private function checkMethodReturnType(MethodReflection $methodReflection): ?str
 
 		if ($returnType instanceof MixedType && !$returnType->isExplicitMixed()) {
 			return sprintf(
-				'Method %s::%s() has no return typehint specified',
+				'Method %s::%s() has no return typehint specified.',
 				$methodReflection->getDeclaringClass()->getDisplayName(),
 				$methodReflection->getName()
 			);

src/Rules/Methods/WrongCaseOfInheritedMethodRule.php

@@ -77,7 +77,7 @@ private function findMethod(
 		}
 
 		return sprintf(
-			'Method %s::%s() does not match %s method name: %s::%s()',
+			'Method %s::%s() does not match %s method name: %s::%s().',
 			$declaringClass->getDisplayName(),
 			$methodName,
 			$classReflection->isInterface() ? 'interface' : 'parent',

src/Rules/Properties/MissingPropertyTypehintRule.php

@@ -34,7 +34,7 @@ public function processNode(Node $node, Scope $scope): array
 		if ($returnType instanceof MixedType && !$returnType->isExplicitMixed()) {
 			return [
 				sprintf(
-					'Property %s::$%s has no typehint specified',
+					'Property %s::$%s has no typehint specified.',
 					$propertyReflection->getDeclaringClass()->getDisplayName(),
 					$node->name->name
 				),

tests/Levels/data/onlyBooleans-0.json

@@ -1,6 +1,6 @@
 [
     {
-        "message": "Method PHPStan\\Levels\\OnlyBooleans\\Foo::doFoo() has parameter $mixed with no typehint specified",
+        "message": "Method PHPStan\\Levels\\OnlyBooleans\\Foo::doFoo() has parameter $mixed with no typehint specified.",
         "line": 14,
         "ignorable": true
     },

tests/Rules/Methods/MissingMethodParameterTypehintRuleTest.php

@@ -14,23 +14,23 @@ public function testRule(): void
 	{
 		$this->analyse([__DIR__ . '/data/missing-method-parameter-typehint.php'], [
 			[
-				'Method MissingMethodParameterTypehint\FooInterface::getFoo() has parameter $p1 with no typehint specified',
+				'Method MissingMethodParameterTypehint\FooInterface::getFoo() has parameter $p1 with no typehint specified.',
 				8,
 			],
 			[
-				'Method MissingMethodParameterTypehint\FooParent::getBar() has parameter $p2 with no typehint specified',
+				'Method MissingMethodParameterTypehint\FooParent::getBar() has parameter $p2 with no typehint specified.',
 				15,
 			],
 			[
-				'Method MissingMethodParameterTypehint\Foo::getFoo() has parameter $p1 with no typehint specified',
+				'Method MissingMethodParameterTypehint\Foo::getFoo() has parameter $p1 with no typehint specified.',
 				25,
 			],
 			[
-				'Method MissingMethodParameterTypehint\Foo::getBar() has parameter $p2 with no typehint specified',
+				'Method MissingMethodParameterTypehint\Foo::getBar() has parameter $p2 with no typehint specified.',
 				33,
 			],
 			[
-				'Method MissingMethodParameterTypehint\Foo::getBaz() has parameter $p3 with no typehint specified',
+				'Method MissingMethodParameterTypehint\Foo::getBaz() has parameter $p3 with no typehint specified.',
 				42,
 			],
 		]);

tests/Rules/Methods/MissingMethodReturnTypehintRuleTest.php

@@ -14,19 +14,19 @@ public function testRule(): void
 	{
 		$this->analyse([__DIR__ . '/data/missing-method-return-typehint.php'], [
 			[
-				'Method MissingMethodReturnTypehint\FooInterface::getFoo() has no return typehint specified',
+				'Method MissingMethodReturnTypehint\FooInterface::getFoo() has no return typehint specified.',
 				8,
 			],
 			[
-				'Method MissingMethodReturnTypehint\FooParent::getBar() has no return typehint specified',
+				'Method MissingMethodReturnTypehint\FooParent::getBar() has no return typehint specified.',
 				15,
 			],
 			[
-				'Method MissingMethodReturnTypehint\Foo::getFoo() has no return typehint specified',
+				'Method MissingMethodReturnTypehint\Foo::getFoo() has no return typehint specified.',
 				25,
 			],
 			[
-				'Method MissingMethodReturnTypehint\Foo::getBar() has no return typehint specified',
+				'Method MissingMethodReturnTypehint\Foo::getBar() has no return typehint specified.',
 				33,
 			],
 		]);

tests/Rules/Methods/WrongCaseOfInheritedMethodRuleTest.php

@@ -14,11 +14,11 @@ public function testRule(): void
 	{
 		$this->analyse([__DIR__ . '/data/wrong-case.php'], [
 			[
-				'Method WrongCase\Foo::GETfoo() does not match interface method name: WrongCase\FooInterface::getFoo()',
+				'Method WrongCase\Foo::GETfoo() does not match interface method name: WrongCase\FooInterface::getFoo().',
 				25,
 			],
 			[
-				'Method WrongCase\Foo::GETbar() does not match parent method name: WrongCase\FooParent::getBar()',
+				'Method WrongCase\Foo::GETbar() does not match parent method name: WrongCase\FooParent::getBar().',
 				30,
 			],
 		]);

tests/Rules/Properties/MissingPropertyTypehintRuleTest.php

@@ -14,15 +14,15 @@ public function testRule(): void
 	{
 		$this->analyse([__DIR__ . '/data/missing-property-typehint.php'], [
 			[
-				'Property MissingPropertyTypehint\MyClass::$prop1 has no typehint specified',
+				'Property MissingPropertyTypehint\MyClass::$prop1 has no typehint specified.',
 				7,
 			],
 			[
-				'Property MissingPropertyTypehint\MyClass::$prop2 has no typehint specified',
+				'Property MissingPropertyTypehint\MyClass::$prop2 has no typehint specified.',
 				9,
 			],
 			[
-				'Property MissingPropertyTypehint\MyClass::$prop3 has no typehint specified',
+				'Property MissingPropertyTypehint\MyClass::$prop3 has no typehint specified.',
 				14,
 			],
 		]);