Change deprecated ClassReflection::isSubClassOf() to ClassReflection::is() for PHPStan ^2.1.8 (#186)

Author: samsonasik

build/target-repository/composer.json

@@ -5,7 +5,7 @@
     "license": "MIT",
     "require": {
         "php": "^7.4|^8.0",
-        "phpstan/phpstan": "^2.0",
+        "phpstan/phpstan": "^2.1.8",
         "nette/utils": "^3.2|^4.0",
         "webmozart/assert": "^1.11"
     },

composer.json

@@ -6,7 +6,7 @@
     "require": {
         "php": ">=8.2",
         "webmozart/assert": "^1.11",
-        "phpstan/phpstan": "^2.0",
+        "phpstan/phpstan": "^2.1.8",
         "nette/utils": "^3.2|^4.0",
         "phpstan/phpdoc-parser": "^2.0"
     },

src/NodeAnalyzer/EnumAnalyzer.php

@@ -34,7 +34,7 @@ public function detect(Scope $scope, ClassLike $classLike): bool
             return true;
         }
 
-        if ($classReflection->isSubclassOf('MyCLabs\Enum\Enum')) {
+        if ($classReflection->is('MyCLabs\Enum\Enum')) {
             return true;
         }
 

src/Rules/ClassNameRespectsParentSuffixRule.php

@@ -93,7 +93,7 @@ public function processNode(Node $node, Scope $scope): array
     private function processClassNameAndShort(ClassReflection $classReflection): array
     {
         foreach ($this->parentClasses as $parentClass) {
-            if (! $classReflection->isSubclassOf($parentClass)) {
+            if (! $classReflection->is($parentClass)) {
                 continue;
             }
 

src/Rules/Complexity/NoJustPropertyAssignRule.php

@@ -141,7 +141,7 @@ private function shouldSkipCurrentClass(Scope $scope): bool
             $classReflection = $scope->getClassReflection();
 
             // skip Symfony form types as rather static
-            if ($classReflection->isSubclassOf(AbstractType::class)) {
+            if ($classReflection->is(AbstractType::class)) {
                 return true;
             }
         }

src/Rules/Doctrine/NoGetRepositoryOnServiceRepositoryEntityRule.php

@@ -90,11 +90,11 @@ private function shouldSkipTestClass(Scope $scope): bool
             return true;
         }
 
-        if ($classReflection->isSubclassOf(TestClassName::PHPUNIT_TEST_CASE)) {
+        if ($classReflection->is(TestClassName::PHPUNIT_TEST_CASE)) {
             return true;
         }
 
-        return $classReflection->isSubclassOf(TestClassName::BEHAT_CONTEXT);
+        return $classReflection->is(TestClassName::BEHAT_CONTEXT);
     }
 
     private function resolveRepositoryClassFromGetRepositoryEntity(MethodCall $methodCall, Scope $scope): ?string
@@ -130,10 +130,10 @@ private function isServiceRepositoryClassReflection(string $repositoryClassName)
         }
 
         $repositoryClassReflection = $this->reflectionProvider->getClass($repositoryClassName);
-        if ($repositoryClassReflection->isSubclassOf(DoctrineClass::ODM_SERVICE_REPOSITORY)) {
+        if ($repositoryClassReflection->is(DoctrineClass::ODM_SERVICE_REPOSITORY)) {
             return true;
         }
 
-        return $repositoryClassReflection->isSubclassOf(DoctrineClass::ORM_SERVICE_REPOSITORY);
+        return $repositoryClassReflection->is(DoctrineClass::ORM_SERVICE_REPOSITORY);
     }
 }

src/Rules/Doctrine/NoRepositoryCallInDataFixtureRule.php

@@ -69,6 +69,6 @@ private function isDataFixtureClass(Scope $scope): bool
         }
 
         $classReflection = $scope->getClassReflection();
-        return $classReflection->isSubclassOf(DoctrineClass::FIXTURE_INTERFACE);
+        return $classReflection->is(DoctrineClass::FIXTURE_INTERFACE);
     }
 }

src/Rules/Domain/RequireExceptionNamespaceRule.php

@@ -37,7 +37,7 @@ public function processNode(Node $node, Scope $scope): array
             return [];
         }
 
-        if (! $classReflection->isSubclassOf('Exception')) {
+        if (! $classReflection->is('Exception')) {
             return [];
         }
 

src/Rules/Rector/PhpUpgradeDowngradeRegisteredInSetRule.php

@@ -110,12 +110,12 @@ private function matchRectorClassName(Scope $scope): ?string
             return null;
         }
 
-        if (! $classReflection->isSubclassOf(ClassName::RECTOR)) {
+        if (! $classReflection->is(ClassName::RECTOR)) {
             return null;
         }
 
         // configurable Rector can be registered optionally
-        if ($classReflection->isSubclassOf(ClassName::CONFIGURABLE_RECTOR)) {
+        if ($classReflection->is(ClassName::CONFIGURABLE_RECTOR)) {
             return null;
         }
 

src/Rules/SeeAnnotationToTestRule.php

@@ -95,7 +95,7 @@ private function shouldSkipClassReflection(ClassReflection $classReflection): bo
         }
 
         foreach ($this->requiredSeeTypes as $requiredSeeType) {
-            if ($classReflection->isSubclassOf($requiredSeeType)) {
+            if ($classReflection->is($requiredSeeType)) {
                 return false;
             }
         }