Implement Scope->getPhpVersion()

Author: staabm

src/Analyser/MutatingScope.php

@@ -47,6 +47,7 @@
 use PHPStan\Parser\NewAssignedToPropertyVisitor;
 use PHPStan\Parser\Parser;
 use PHPStan\Php\PhpVersion;
+use PHPStan\Php\PhpVersions;
 use PHPStan\PhpDoc\Tag\TemplateTag;
 use PHPStan\Reflection\Assertions;
 use PHPStan\Reflection\Callables\CallableParametersAcceptor;
@@ -144,6 +145,7 @@
 use function implode;
 use function in_array;
 use function is_bool;
+use function is_int;
 use function is_numeric;
 use function is_string;
 use function ltrim;
@@ -5721,4 +5723,32 @@ public function getIterableValueType(Type $iteratee): Type
 		return $iteratee->getIterableValueType();
 	}
 
+	public function getPhpVersions(): PhpVersions
+	{
+		$versionId = $this->getType(new ConstFetch(new Name('PHP_VERSION_ID')));
+		if ($versionId instanceof IntegerRangeType) {
+			if ($versionId->getMin() !== null && $versionId->getMax() !== null) {
+				return new PhpVersions([$versionId->getMin(), $versionId->getMax()]);
+			}
+			if ($versionId->getMin() !== null) {
+				return new PhpVersions([$versionId->getMin()]);
+			}
+			return new PhpVersions([$versionId->getMax()]);
+		}
+
+		$scalars = $versionId->getConstantScalarValues();
+		if ($scalars !== []) {
+			$ints = [];
+			foreach ($scalars as $scalar) {
+				if (!is_int($scalar)) {
+					throw new ShouldNotHappenException();
+				}
+				$ints[] = $scalar;
+			}
+			return new PhpVersions($ints);
+		}
+
+		return new PhpVersions([$this->phpVersion->getVersionId()]);
+	}
+
 }

src/Analyser/Scope.php

@@ -6,6 +6,7 @@
 use PhpParser\Node\Expr;
 use PhpParser\Node\Name;
 use PhpParser\Node\Param;
+use PHPStan\Php\PhpVersions;
 use PHPStan\Reflection\ClassConstantReflection;
 use PHPStan\Reflection\ClassMemberAccessAnswerer;
 use PHPStan\Reflection\ClassReflection;
@@ -136,4 +137,6 @@ public function filterByFalseyValue(Expr $expr): self;
 
 	public function isInFirstLevelStatement(): bool;
 
+	public function getPhpVersions(): PhpVersions;
+
 }

src/Php/PhpVersions.php

@@ -0,0 +1,58 @@
+<?php declare(strict_types = 1);
+
+namespace PHPStan\Php;
+
+use PHPStan\ShouldNotHappenException;
+use PHPStan\TrinaryLogic;
+use function max;
+use function min;
+
+/**
+ * @api
+ */
+final class PhpVersions
+{
+
+	private int $minVersionId;
+
+	private int $maxVersionId;
+
+	/**
+	 * @api
+	 * @param list<int> $phpVersionIds
+	 */
+	public function __construct(
+		array $phpVersionIds,
+	)
+	{
+		if ($phpVersionIds === []) {
+			throw new ShouldNotHappenException();
+		}
+
+		$normalizedPhpVersionIds = [];
+		foreach ($phpVersionIds as $versionId) {
+			// drop patch version part and replace with 00
+			$normalizedPhpVersionIds[] = ((int) ($versionId / 100)) * 100;
+		}
+
+		$this->minVersionId = min($normalizedPhpVersionIds);
+		$this->maxVersionId = max($normalizedPhpVersionIds);
+	}
+
+	public function producesWarningForFinalPrivateMethods(): TrinaryLogic
+	{
+		return $this->minPhpVersion(80000);
+	}
+
+	private function minPhpVersion(int $versionId): TrinaryLogic
+	{
+		if ($this->minVersionId >= $versionId) {
+			return TrinaryLogic::createYes();
+		}
+		if ($this->maxVersionId >= $versionId) {
+			return TrinaryLogic::createMaybe();
+		}
+		return TrinaryLogic::createNo();
+	}
+
+}

src/Rules/Methods/FinalPrivateMethodRule.php

@@ -5,7 +5,6 @@
 use PhpParser\Node;
 use PHPStan\Analyser\Scope;
 use PHPStan\Node\InClassMethodNode;
-use PHPStan\Php\PhpVersion;
 use PHPStan\Rules\Rule;
 use PHPStan\Rules\RuleErrorBuilder;
 use function sprintf;
@@ -14,12 +13,6 @@
 final class FinalPrivateMethodRule implements Rule
 {
 
-	public function __construct(
-		private PhpVersion $phpVersion,
-	)
-	{
-	}
-
 	public function getNodeType(): string
 	{
 		return InClassMethodNode::class;
@@ -28,7 +21,7 @@ public function getNodeType(): string
 	public function processNode(Node $node, Scope $scope): array
 	{
 		$method = $node->getMethodReflection();
-		if (!$this->phpVersion->producesWarningForFinalPrivateMethods()) {
+		if (!$scope->getPhpVersions()->producesWarningForFinalPrivateMethods()->yes()) {
 			return [];
 		}
 

tests/PHPStan/Rules/Methods/FinalPrivateMethodRuleTest.php

@@ -9,14 +9,9 @@
 /** @extends RuleTestCase<FinalPrivateMethodRule> */
 class FinalPrivateMethodRuleTest extends RuleTestCase
 {
-
-	private int $phpVersionId;
-
 	protected function getRule(): Rule
 	{
-		return new FinalPrivateMethodRule(
-			new PhpVersion($this->phpVersionId),
-		);
+		return new FinalPrivateMethodRule();
 	}
 
 	public function dataRule(): array
@@ -44,7 +39,9 @@ public function dataRule(): array
 	 */
 	public function testRule(int $phpVersion, array $errors): void
 	{
-		$this->phpVersionId = $phpVersion;
+		if (PHP_VERSION_ID < $phpVersion) {
+			$this->markTestSkipped('Test requires PHP ' . $phpVersion);
+		}
 		$this->analyse([__DIR__ . '/data/final-private-method.php'], $errors);
 	}