Bump to PHPStan ^2.1.8 and change deprecated ClassReflection::isSubClassOf() to ClassReflection::is()

build/target-repository/composer.json

@@ -8,7 +8,7 @@
     ],
     "require": {
         "php": "^7.4|^8.0",
-        "phpstan/phpstan": "^2.1.7"
+        "phpstan/phpstan": "^2.1.8"
     },
     "autoload": {
         "files": [

composer.json

@@ -24,7 +24,7 @@
         "ocramius/package-versions": "^2.9",
         "ondram/ci-detector": "^4.2",
         "phpstan/phpdoc-parser": "^2.0.2",
-        "phpstan/phpstan": "^2.1.7",
+        "phpstan/phpstan": "^2.1.8",
         "react/event-loop": "^1.5",
         "react/promise": "^3.2",
         "react/socket": "^1.15",

rules-tests/Renaming/Rector/Name/RenameClassRector/Source/EnforceExceptionSuffixCallback.php

@@ -18,7 +18,7 @@ public function __invoke(
     ): ?string {
         $fullyQualifiedClassName = (string) $nodeNameResolver->getName($class);
         $classReflection = $reflectionProvider->getClass($fullyQualifiedClassName);
-        if (! $classReflection->isSubclassOf(Exception::class)) {
+        if (! $classReflection->is(Exception::class)) {
             return null;
         }
 

rules/DeadCode/Rector/ClassMethod/RemoveUnusedPrivateMethodRector.php

@@ -147,7 +147,7 @@ private function collectTestMethodsUsesPrivateDataProvider(
         Class_ $class,
         array $classMethods
     ): array {
-        if (! $classReflection->isSubClassOf('PHPUnit\Framework\TestCase')) {
+        if (! $classReflection->is('PHPUnit\Framework\TestCase')) {
             return [];
         }
 

rules/Php70/Rector/MethodCall/ThisCallOnStaticMethodToStaticCallRector.php

@@ -101,7 +101,7 @@ public function refactor(Node $node): ?Node
         $classReflection = $scope->getClassReflection();
 
         // skip PHPUnit calls, as they accept both self:: and $this-> formats
-        if ($classReflection->isSubclassOf('PHPUnit\Framework\TestCase')) {
+        if ($classReflection->is('PHPUnit\Framework\TestCase')) {
             return null;
         }
 

rules/Php70/Rector/StaticCall/StaticCallOnNonStaticToInstanceCallRector.php

@@ -169,7 +169,7 @@ private function shouldSkip(string $methodName, string $className, StaticCall $s
         }
 
         $reflection = $scope->getClassReflection();
-        if ($reflection instanceof ClassReflection && $reflection->isSubclassOf($className)) {
+        if ($reflection instanceof ClassReflection && $reflection->is($className)) {
             return true;
         }
 

rules/Php80/Rector/ClassMethod/AddParamBasedOnParentClassMethodRector.php

@@ -112,7 +112,7 @@ public function refactor(Node $node): ?Node
         }
 
         $currentClassReflection = $this->reflectionResolver->resolveClassReflection($node);
-        $isPDO = $currentClassReflection instanceof ClassReflection && $currentClassReflection->isSubclassOf('PDO');
+        $isPDO = $currentClassReflection instanceof ClassReflection && $currentClassReflection->is('PDO');
 
         // It relies on phpstorm stubs that define 2 kind of query method for both php 7.4 and php 8.0
         // @see https://github.com/JetBrains/phpstorm-stubs/blob/e2e898a29929d2f520fe95bdb2109d8fa895ba4a/PDO/PDO.php#L1096-L1126

rules/Privatization/Rector/Class_/FinalizeTestCaseClassRector.php

@@ -81,7 +81,7 @@ public function refactor(Node $node): ?Node
         }
 
         $classReflection = $this->reflectionProvider->getClass($className);
-        if (! $classReflection->isSubclassOf('PHPUnit\Framework\TestCase')) {
+        if (! $classReflection->is('PHPUnit\Framework\TestCase')) {
             return null;
         }
 

rules/Transform/Rector/ClassMethod/ReturnTypeWillChangeRector.php

@@ -100,7 +100,7 @@ public function refactor(Node $node): ?Node
             }
 
             foreach ($this->returnTypeChangedClassMethodReferences as $returnTypeChangedClassMethodReference) {
-                if (! $classReflection->isSubclassOf($returnTypeChangedClassMethodReference->getClass())) {
+                if (! $classReflection->is($returnTypeChangedClassMethodReference->getClass())) {
                     continue;
                 }
 

rules/TypeDeclaration/Rector/ClassMethod/ReturnTypeFromMockObjectRector.php

@@ -153,6 +153,6 @@ private function isInsideTestCaseClass(Scope $scope): bool
         }
 
         // is phpunit test case?
-        return $classReflection->isSubclassOf(ClassName::TEST_CASE_CLASS);
+        return $classReflection->is(ClassName::TEST_CASE_CLASS);
     }
 }

rules/TypeDeclaration/Rector/Property/AddPropertyTypeDeclarationRector.php

@@ -122,6 +122,6 @@ private function isClassReflectionType(ClassReflection $classReflection, string
             return true;
         }
 
-        return $classReflection->isSubclassOf($type);
+        return $classReflection->is($type);
     }
 }

src/NodeTypeResolver/NodeTypeResolver.php

@@ -332,7 +332,7 @@ public function isMethodStaticCallOrClassMethodObjectType(Node $node, ObjectType
             return true;
         }
 
-        return $classReflection->isSubclassOf($objectType->getClassName());
+        return $classReflection->is($objectType->getClassName());
     }
 
     /**