EntityNotFinalRule - only entities that are final on language level should be considered

Author: DanielBadura

src/Rules/Doctrine/ORM/EntityNotFinalRule.php

@@ -34,7 +34,7 @@ public function processNode(Node $node, Scope $scope): array
 		if ($classReflection === null) {
 			throw new \PHPStan\ShouldNotHappenException();
 		}
-		if (!$classReflection->isFinal()) {
+		if (!$classReflection->isFinalByKeyword()) {
 			return [];
 		}
 

tests/Rules/Doctrine/ORM/EntityNotFinalRuleTest.php

@@ -45,6 +45,11 @@ public function ruleProvider(): Iterator
 			],
 		];
 
+		yield 'final annotated entity' => [
+			__DIR__ . '/data/FinalAnnotatedEntity.php',
+			[],
+		];
+
 		yield 'final non-entity' => [
 			__DIR__ . '/data/FinalNonEntity.php',
 			[],

tests/Rules/Doctrine/ORM/data/FinalAnnotatedEntity.php

@@ -0,0 +1,22 @@
+<?php declare(strict_types = 1);
+
+namespace PHPStan\Rules\Doctrine\ORM;
+
+use Doctrine\ORM\Mapping as ORM;
+
+/**
+ * @final
+ * @ORM\Entity()
+ */
+class FinalAnnotatedEntity
+{
+	/**
+	 * @ORM\Id()
+	 * @ORM\GeneratedValue()
+	 * @ORM\Column(type="integer")
+	 *
+	 * @var int
+	 */
+	private $id;
+
+}