Fix PHPStan 1.7.0 incompatibilities

Author: mglaman

src/Reflection/EntityFieldsViaMagicReflectionExtension.php

@@ -2,6 +2,7 @@
 
 namespace mglaman\PHPStanDrupal\Reflection;
 
+use PHPStan\BetterReflection\Reflection\ReflectionClass;
 use PHPStan\Reflection\ClassReflection;
 use PHPStan\Reflection\PropertiesClassReflectionExtension;
 use PHPStan\Reflection\PropertyReflection;
@@ -28,16 +29,15 @@ public function hasProperty(ClassReflection $classReflection, string $propertyNa
             return false;
         }
 
-        $reflection = $classReflection->getNativeReflection();
         // We need to find a way to parse the entity annotation so that at the minimum the `entity_keys` are
         // supported. The real fix is Drupal developers _really_ need to start writing @property definitions in the
         // class doc if they don't get `get` methods.
-        if ($reflection->implementsInterface('Drupal\Core\Entity\ContentEntityInterface')) {
+        if ($classReflection->implementsInterface('Drupal\Core\Entity\ContentEntityInterface')) {
             // @todo revisit if it's a good idea to be true.
             // Content entities have magical __get... so it is kind of true.
             return true;
         }
-        if (self::classObjectIsSuperOfInterface($reflection, self::getFieldItemListInterfaceObject())->yes()) {
+        if (self::classObjectIsSuperOfInterface($classReflection->getName(), self::getFieldItemListInterfaceObject())->yes()) {
             return FieldItemListPropertyReflection::canHandleProperty($classReflection, $propertyName);
         }
 
@@ -46,21 +46,19 @@ public function hasProperty(ClassReflection $classReflection, string $propertyNa
 
     public function getProperty(ClassReflection $classReflection, string $propertyName): PropertyReflection
     {
-        $reflection = $classReflection->getNativeReflection();
-        if ($reflection->implementsInterface('Drupal\Core\Entity\EntityInterface')) {
+        if ($classReflection->implementsInterface('Drupal\Core\Entity\EntityInterface')) {
             return new EntityFieldReflection($classReflection, $propertyName);
         }
-        if (self::classObjectIsSuperOfInterface($reflection, self::getFieldItemListInterfaceObject())->yes()) {
+        if (self::classObjectIsSuperOfInterface($classReflection->getName(), self::getFieldItemListInterfaceObject())->yes()) {
             return new FieldItemListPropertyReflection($classReflection, $propertyName);
         }
 
         throw new \LogicException($classReflection->getName() . "::$propertyName should be handled earlier.");
     }
 
-    public static function classObjectIsSuperOfInterface(\ReflectionClass $reflection, ObjectType $interfaceObject) : TrinaryLogic
+    public static function classObjectIsSuperOfInterface(string $name, ObjectType $interfaceObject) : TrinaryLogic
     {
-        $classObject = new ObjectType($reflection->getName());
-        return $interfaceObject->isSuperTypeOf($classObject);
+        return $interfaceObject->isSuperTypeOf(new ObjectType($name));
     }
 
     protected static function getFieldItemListInterfaceObject() : ObjectType

src/Rules/Drupal/PluginManager/AbstractPluginManagerRule.php

@@ -2,6 +2,7 @@
 
 namespace mglaman\PHPStanDrupal\Rules\Drupal\PluginManager;
 
+use PHPStan\Reflection\ClassReflection;
 use PHPStan\Rules\Rule;
 
 /**
@@ -10,7 +11,7 @@
 abstract class AbstractPluginManagerRule implements Rule
 {
 
-    protected function isPluginManager(\ReflectionClass $classReflection): bool
+    protected function isPluginManager(ClassReflection $classReflection): bool
     {
         return
             !$classReflection->isInterface() &&

src/Rules/Drupal/PluginManager/PluginManagerSetsCacheBackendRule.php

@@ -42,9 +42,7 @@ public function processNode(Node $node, Scope $scope): array
             throw new ShouldNotHappenException();
         }
 
-        $classReflection = $scopeClassReflection->getNativeReflection();
-
-        if (!$this->isPluginManager($classReflection)) {
+        if (!$this->isPluginManager($scopeClassReflection)) {
             return [];
         }