Fix generic type implementations

phansys · phansys · commit 61c77d1fda14 · 2023-09-16T19:10:19.000-03:00
diff --git a/phpstan.neon.dist b/phpstan.neon.dist
@@ -13,8 +13,6 @@ parameters:
     treatPhpDocTypesAsCertain: false
     checkMissingVarTagTypehint: true
     checkMissingTypehints: true
-    # @todo: Remove the "checkGenericClassInNonGenericObjectType" definition.
-    checkGenericClassInNonGenericObjectType: false
     ignoreErrors:
         - '#^Property Gedmo\\Tests\\.+\\Fixture\\[^:]+::\$\w+ is never written, only read\.$#'
         - '#^Property Gedmo\\Tests\\.+\\Fixture\\[^:]+::\$\w+ is never read, only written\.$#'
diff --git a/src/AbstractTrackingListener.php b/src/AbstractTrackingListener.php
@@ -47,6 +47,7 @@ public function getSubscribedEvents()
      * Maps additional metadata for the object.
      *
      * @param LoadClassMetadataEventArgs $eventArgs
+     * @phpstan-param LoadClassMetadataEventArgs<ClassMetadata<object>, ObjectManager> $eventArgs
      *
      * @return void
      */
diff --git a/src/Loggable/LoggableListener.php b/src/Loggable/LoggableListener.php
@@ -10,8 +10,9 @@
 namespace Gedmo\Loggable;
 
 use Doctrine\Common\EventArgs;
-use Doctrine\ORM\Mapping\ClassMetadata;
+use Doctrine\ORM\Mapping\ClassMetadata as ORMClassMetadata;
 use Doctrine\Persistence\Event\LoadClassMetadataEventArgs;
+use Doctrine\Persistence\Mapping\ClassMetadata;
 use Doctrine\Persistence\ObjectManager;
 use Gedmo\Exception\InvalidArgumentException;
 use Gedmo\Loggable\Entity\LogEntry;
@@ -125,6 +126,7 @@ public function getSubscribedEvents()
      * Maps additional metadata
      *
      * @param LoadClassMetadataEventArgs $eventArgs
+     * @phpstan-param LoadClassMetadataEventArgs<ClassMetadata<object>, ObjectManager> $eventArgs
      *
      * @return void
      */
@@ -312,7 +314,7 @@ protected function createLogEntry($action, $object, LoggableAdapter $ea)
         if ($config = $this->getConfiguration($om, $meta->getName())) {
             $logEntryClass = $this->getLogEntryClass($ea, $meta->getName());
             $logEntryMeta = $om->getClassMetadata($logEntryClass);
-            /** @var LogEntryInterface $logEntry */
+            /** @var LogEntryInterface<T> $logEntry */
             $logEntry = $logEntryMeta->newInstance();
 
             $logEntry->setAction($action);
@@ -322,7 +324,7 @@ protected function createLogEntry($action, $object, LoggableAdapter $ea)
 
             // check for the availability of the primary key
             $uow = $om->getUnitOfWork();
-            if (LogEntryInterface::ACTION_CREATE === $action && ($ea->isPostInsertGenerator($meta) || ($meta instanceof ClassMetadata && $meta->isIdentifierComposite))) {
+            if (LogEntryInterface::ACTION_CREATE === $action && ($ea->isPostInsertGenerator($meta) || ($meta instanceof ORMClassMetadata && $meta->isIdentifierComposite))) {
                 $this->pendingLogEntryInserts[spl_object_id($object)] = $logEntry;
             } else {
                 $logEntry->setObjectId($wrapped->getIdentifier(false, true));
diff --git a/src/Mapping/Driver/AbstractAnnotationDriver.php b/src/Mapping/Driver/AbstractAnnotationDriver.php
@@ -79,6 +79,7 @@ public function setOriginalDriver($driver)
      * @param ClassMetadata $meta
      *
      * @return \ReflectionClass
+     * @phpstan-return \ReflectionClass<object>
      */
     public function getMetaReflectionClass($meta)
     {
diff --git a/src/Mapping/Driver/AttributeAnnotationReader.php b/src/Mapping/Driver/AttributeAnnotationReader.php
@@ -37,6 +37,8 @@ public function __construct(AttributeReader $attributeReader, Reader $annotation
     }
 
     /**
+     * @phpstan-param \ReflectionClass<object> $class
+     *
      * @return Annotation[]
      */
     public function getClassAnnotations(\ReflectionClass $class): array
@@ -51,7 +53,10 @@ public function getClassAnnotations(\ReflectionClass $class): array
     }
 
     /**
-     * @param class-string<T> $annotationName the name of the annotation
+     * @param string $annotationName
+     *
+     * @phpstan-param \ReflectionClass<object> $class
+     * @phpstan-param class-string<T> $annotationName the name of the annotation
      *
      * @return T|null the Annotation or NULL, if the requested annotation does not exist
      *
diff --git a/src/Mapping/Driver/AttributeReader.php b/src/Mapping/Driver/AttributeReader.php
@@ -22,6 +22,8 @@ final class AttributeReader
     private $isRepeatableAttribute = [];
 
     /**
+     * @phpstan-param \ReflectionClass<object> $class
+     *
      * @return array<Annotation|Annotation[]>
      */
     public function getClassAnnotations(\ReflectionClass $class): array
@@ -30,6 +32,7 @@ public function getClassAnnotations(\ReflectionClass $class): array
     }
 
     /**
+     * @phpstan-param \ReflectionClass<object> $class
      * @phpstan-param class-string $annotationName
      *
      * @return Annotation|Annotation[]|null
@@ -58,11 +61,12 @@ public function getPropertyAnnotation(\ReflectionProperty $property, string $ann
     }
 
     /**
-     * @param array<\ReflectionAttribute> $attributes
+     * @param iterable<\ReflectionAttribute> $attributes
+     * @phpstan-param iterable<\ReflectionAttribute<object>> $attributes
      *
      * @return array<string, Annotation|Annotation[]>
      */
-    private function convertToAttributeInstances(array $attributes): array
+    private function convertToAttributeInstances(iterable $attributes): array
     {
         $instances = [];
 
diff --git a/src/ReferenceIntegrity/ReferenceIntegrityListener.php b/src/ReferenceIntegrity/ReferenceIntegrityListener.php
@@ -11,6 +11,8 @@
 
 use Doctrine\Common\EventArgs;
 use Doctrine\Persistence\Event\LoadClassMetadataEventArgs;
+use Doctrine\Persistence\Mapping\ClassMetadata;
+use Doctrine\Persistence\ObjectManager;
 use Gedmo\Exception\InvalidMappingException;
 use Gedmo\Exception\ReferenceIntegrityStrictException;
 use Gedmo\Mapping\MappedEventSubscriber;
@@ -40,6 +42,7 @@ public function getSubscribedEvents()
      * Maps additional metadata for the Document
      *
      * @param LoadClassMetadataEventArgs $eventArgs
+     * @phpstan-param LoadClassMetadataEventArgs<ClassMetadata<object>, ObjectManager> $eventArgs
      *
      * @return void
      */
diff --git a/src/References/ReferencesListener.php b/src/References/ReferencesListener.php
@@ -12,6 +12,7 @@
 use Doctrine\Common\Collections\ArrayCollection;
 use Doctrine\Common\EventArgs;
 use Doctrine\Persistence\Event\LoadClassMetadataEventArgs;
+use Doctrine\Persistence\Mapping\ClassMetadata;
 use Doctrine\Persistence\ObjectManager;
 use Gedmo\Mapping\MappedEventSubscriber;
 
@@ -61,6 +62,7 @@ public function __construct(array $managers = [])
 
     /**
      * @param LoadClassMetadataEventArgs $eventArgs
+     * @phpstan-param LoadClassMetadataEventArgs<ClassMetadata<object>, ObjectManager> $eventArgs
      *
      * @return void
      */
diff --git a/src/Sluggable/SluggableListener.php b/src/Sluggable/SluggableListener.php
@@ -11,6 +11,7 @@
 
 use Doctrine\Common\EventArgs;
 use Doctrine\Persistence\Event\LoadClassMetadataEventArgs;
+use Doctrine\Persistence\Mapping\ClassMetadata;
 use Doctrine\Persistence\ObjectManager;
 use Gedmo\Exception\InvalidArgumentException;
 use Gedmo\Mapping\MappedEventSubscriber;
@@ -225,6 +226,7 @@ public function removeManagedFilter($name)
      * Mapps additional metadata
      *
      * @param LoadClassMetadataEventArgs $eventArgs
+     * @phpstan-param LoadClassMetadataEventArgs<ClassMetadata<object>, ObjectManager> $eventArgs
      *
      * @return void
      */
diff --git a/src/SoftDeleteable/SoftDeleteableListener.php b/src/SoftDeleteable/SoftDeleteableListener.php
@@ -14,6 +14,8 @@
 use Doctrine\ODM\MongoDB\UnitOfWork as MongoDBUnitOfWork;
 use Doctrine\ORM\EntityManagerInterface;
 use Doctrine\Persistence\Event\LoadClassMetadataEventArgs;
+use Doctrine\Persistence\Mapping\ClassMetadata;
+use Doctrine\Persistence\ObjectManager;
 use Gedmo\Mapping\MappedEventSubscriber;
 
 /**
@@ -108,6 +110,7 @@ public function onFlush(EventArgs $args)
      * Maps additional metadata
      *
      * @param LoadClassMetadataEventArgs $eventArgs
+     * @phpstan-param LoadClassMetadataEventArgs<ClassMetadata<object>, ObjectManager> $eventArgs
      *
      * @return void
      */
diff --git a/src/Sortable/Entity/Repository/SortableRepository.php b/src/Sortable/Entity/Repository/SortableRepository.php
@@ -21,6 +21,8 @@
  * Sortable Repository
  *
  * @author Lukas Botsch <lukas.botsch@gmail.com>
+ *
+ * @phpstan-extends EntityRepository<object>
  */
 class SortableRepository extends EntityRepository
 {
diff --git a/src/Sortable/SortableListener.php b/src/Sortable/SortableListener.php
@@ -87,6 +87,7 @@ public function getSubscribedEvents()
      * Maps additional metadata
      *
      * @param LoadClassMetadataEventArgs $args
+     * @phpstan-param LoadClassMetadataEventArgs<ClassMetadata<object>, ObjectManager> $args
      *
      * @return void
      */
diff --git a/src/Tool/Wrapper/AbstractWrapper.php b/src/Tool/Wrapper/AbstractWrapper.php
@@ -60,7 +60,7 @@ abstract class AbstractWrapper implements WrapperInterface
      *
      * @throws UnsupportedObjectManagerException
      *
-     * @return WrapperInterface
+     * @return WrapperInterface<ClassMetadata>
      */
     public static function wrap($object, ObjectManager $om)
     {
diff --git a/src/Tool/WrapperInterface.php b/src/Tool/WrapperInterface.php
@@ -14,7 +14,7 @@
 /**
  * Interface for a wrapper of a managed object.
  *
- * @phpstan-template TClassMetadata of ClassMetadata
+ * @phpstan-template-covariant TClassMetadata of ClassMetadata
  *
  * @author Gediminas Morkevicius <gediminas.morkevicius@gmail.com>
  */
diff --git a/src/Translatable/Document/Repository/TranslationRepository.php b/src/Translatable/Document/Repository/TranslationRepository.php
@@ -27,6 +27,8 @@
  * to interact with translations.
  *
  * @author Gediminas Morkevicius <gediminas.morkevicius@gmail.com>
+ *
+ * @phpstan-extends DocumentRepository<object>
  */
 class TranslationRepository extends DocumentRepository
 {
diff --git a/src/Translatable/Entity/Repository/TranslationRepository.php b/src/Translatable/Entity/Repository/TranslationRepository.php
@@ -27,6 +27,8 @@
  * to interact with translations.
  *
  * @author Gediminas Morkevicius <gediminas.morkevicius@gmail.com>
+ *
+ * @phpstan-extends EntityRepository<object>
  */
 class TranslationRepository extends EntityRepository
 {
diff --git a/src/Translatable/Mapping/Event/TranslatableAdapter.php b/src/Translatable/Mapping/Event/TranslatableAdapter.php
@@ -9,6 +9,7 @@
 
 namespace Gedmo\Translatable\Mapping\Event;
 
+use Doctrine\Persistence\Mapping\ClassMetadata;
 use Gedmo\Mapping\Event\AdapterInterface;
 use Gedmo\Tool\Wrapper\AbstractWrapper;
 
@@ -61,6 +62,7 @@ public function loadTranslations($object, $translationClass, $locale, $objectCla
      * @param string $translationClass
      * @param string $objectClass
      *
+     * @phpstan-param AbstractWrapper<ClassMetadata<object>> $wrapped
      * @phpstan-param class-string $translationClass
      * @phpstan-param class-string $objectClass
      *
@@ -74,6 +76,7 @@ public function findTranslation(AbstractWrapper $wrapped, $locale, $field, $tran
      * @param string $transClass
      * @param string $objectClass
      *
+     * @phpstan-param AbstractWrapper<ClassMetadata<object>> $wrapped
      * @phpstan-param class-string $transClass
      * @phpstan-param class-string $objectClass
      *
diff --git a/src/Translatable/TranslatableListener.php b/src/Translatable/TranslatableListener.php
@@ -217,6 +217,7 @@ public function addPendingTranslationInsert($oid, $translation)
      * Maps additional metadata
      *
      * @param LoadClassMetadataEventArgs $eventArgs
+     * @phpstan-param LoadClassMetadataEventArgs<ClassMetadata<object>, ObjectManager> $eventArgs
      *
      * @return void
      */
diff --git a/src/Tree/Document/MongoDB/Repository/AbstractTreeRepository.php b/src/Tree/Document/MongoDB/Repository/AbstractTreeRepository.php
@@ -21,6 +21,9 @@
 use Gedmo\Tree\RepositoryUtilsInterface;
 use Gedmo\Tree\TreeListener;
 
+/**
+ * @phpstan-extends DocumentRepository<object>
+ */
 abstract class AbstractTreeRepository extends DocumentRepository implements RepositoryInterface
 {
     /**
diff --git a/src/Tree/Document/MongoDB/Repository/MaterializedPathRepository.php b/src/Tree/Document/MongoDB/Repository/MaterializedPathRepository.php
@@ -55,6 +55,8 @@ public function getTreeQuery($rootNode = null)
      * Get tree
      *
      * @param object|null $rootNode
+     *
+     * @phpstan-return Iterator<object>
      */
     public function getTree($rootNode = null): Iterator
     {
diff --git a/src/Tree/Entity/Repository/AbstractTreeRepository.php b/src/Tree/Entity/Repository/AbstractTreeRepository.php
@@ -22,6 +22,9 @@
 use Gedmo\Tree\RepositoryUtilsInterface;
 use Gedmo\Tree\TreeListener;
 
+/**
+ * @phpstan-extends EntityRepository<object>
+ */
 abstract class AbstractTreeRepository extends EntityRepository implements RepositoryInterface
 {
     /**
diff --git a/src/Tree/TreeListener.php b/src/Tree/TreeListener.php
@@ -13,6 +13,7 @@
 use Doctrine\ODM\MongoDB\DocumentManager;
 use Doctrine\ORM\EntityManagerInterface;
 use Doctrine\Persistence\Event\LoadClassMetadataEventArgs;
+use Doctrine\Persistence\Mapping\ClassMetadata;
 use Doctrine\Persistence\ObjectManager;
 use Gedmo\Exception\InvalidArgumentException;
 use Gedmo\Exception\UnexpectedValueException;
@@ -288,6 +289,8 @@ public function postRemove(EventArgs $args)
      *
      * @param LoadClassMetadataEventArgs $eventArgs
      *
+     * @phpstan-param LoadClassMetadataEventArgs<ClassMetadata<object>, ObjectManager> $eventArgs
+     *
      * @return void
      */
     public function loadClassMetadata(EventArgs $eventArgs)
diff --git a/src/Uploadable/UploadableListener.php b/src/Uploadable/UploadableListener.php
@@ -15,6 +15,7 @@
 use Doctrine\Persistence\Event\LoadClassMetadataEventArgs;
 use Doctrine\Persistence\Mapping\ClassMetadata;
 use Doctrine\Persistence\NotifyPropertyChanged;
+use Doctrine\Persistence\ObjectManager;
 use Gedmo\Exception\InvalidArgumentException;
 use Gedmo\Exception\UploadableCantWriteException;
 use Gedmo\Exception\UploadableCouldntGuessMimeTypeException;
@@ -523,6 +524,8 @@ public function doMoveFile($source, $dest, $isUploadedFile = true)
      *
      * @param LoadClassMetadataEventArgs $eventArgs
      *
+     * @phpstan-param LoadClassMetadataEventArgs<ClassMetadata<object>, ObjectManager> $eventArgs
+     *
      * @return void
      */
     public function loadClassMetadata(EventArgs $eventArgs)
diff --git a/tests/Gedmo/Mapping/Fixture/Yaml/Category.php b/tests/Gedmo/Mapping/Fixture/Yaml/Category.php
@@ -32,12 +32,12 @@ class Category extends BaseCategory
     private $slug;
 
     /**
-     * @var Collection<int, Category>
+     * @var Collection<int, self>
      */
     private $children;
 
     /**
-     * @var Category
+     * @var self
      */
     private $parent;
 
@@ -82,16 +82,13 @@ public function getSlug(): string
         return $this->slug;
     }
 
-    /**
-     * @param Category $children
-     */
     public function addChildren(self $children): void
     {
         $this->children[] = $children;
     }
 
     /**
-     * @return Collection $children
+     * @return Collection<int, self> $children
      */
     public function getChildren()
     {
@@ -104,7 +101,7 @@ public function setParent(self $parent): void
     }
 
     /**
-     * @return Category $parent
+     * @return self $parent
      */
     public function getParent(): self
     {
diff --git a/tests/Gedmo/Mapping/Mock/Extension/Encoder/EncoderListener.php b/tests/Gedmo/Mapping/Mock/Extension/Encoder/EncoderListener.php
@@ -13,6 +13,8 @@
 
 use Doctrine\Common\EventArgs;
 use Doctrine\Persistence\Event\LoadClassMetadataEventArgs;
+use Doctrine\Persistence\Mapping\ClassMetadata;
+use Doctrine\Persistence\ObjectManager;
 use Gedmo\Mapping\Event\AdapterInterface as EventAdapterInterface;
 use Gedmo\Mapping\MappedEventSubscriber;
 
@@ -26,6 +28,9 @@ public function getSubscribedEvents(): array
         ];
     }
 
+    /**
+     * @phpstan-param LoadClassMetadataEventArgs<ClassMetadata<object>, ObjectManager> $args
+     */
     public function loadClassMetadata(LoadClassMetadataEventArgs $args): void
     {
         $ea = $this->getEventAdapter($args);
diff --git a/tests/Gedmo/Timestampable/Fixture/Article.php b/tests/Gedmo/Timestampable/Fixture/Article.php
@@ -186,6 +186,9 @@ public function addComment(Comment $comment): void
         $this->comments[] = $comment;
     }
 
+    /**
+     * @return Collection<int, Comment>
+     */
     public function getComments(): Collection
     {
         return $this->comments;
diff --git a/tests/Gedmo/Timestampable/Fixture/ArticleCarbon.php b/tests/Gedmo/Timestampable/Fixture/ArticleCarbon.php
diff --git a/tests/Gedmo/Translatable/TranslatableWithEmbeddedTest.php b/tests/Gedmo/Translatable/TranslatableWithEmbeddedTest.php
diff --git a/tests/Gedmo/Tree/Fixture/ForeignRootCategory.php b/tests/Gedmo/Tree/Fixture/ForeignRootCategory.php
diff --git a/tests/Gedmo/Tree/MaterializedPathODMMongoDBRepositoryTest.php b/tests/Gedmo/Tree/MaterializedPathODMMongoDBRepositoryTest.php

Original file line number	Diff line number	Diff line change
`@@ -47,6 +47,7 @@ public function getSubscribedEvents()`
`47`	`47`	`* Maps additional metadata for the object.`
`48`	`48`	`*`
`49`	`49`	`* @param LoadClassMetadataEventArgs $eventArgs`
	`50`	`+ * @phpstan-param LoadClassMetadataEventArgs<ClassMetadata<object>, ObjectManager> $eventArgs`
`50`	`51`	`*`
`51`	`52`	`* @return void`
`52`	`53`	`*/`
Original file line number	Diff line number	Diff line change
`@@ -79,6 +79,7 @@ public function setOriginalDriver($driver)`
`79`	`79`	`* @param ClassMetadata $meta`
`80`	`80`	`*`
`81`	`81`	`* @return \ReflectionClass`
	`82`	`+ * @phpstan-return \ReflectionClass<object>`
`82`	`83`	`*/`
`83`	`84`	`public function getMetaReflectionClass($meta)`
`84`	`85`	`{`
Original file line number	Diff line number	Diff line change
`@@ -37,6 +37,8 @@ public function __construct(AttributeReader $attributeReader, Reader $annotation`
`37`	`37`	`}`
`38`	`38`
`39`	`39`	`/**`
	`40`	`+ * @phpstan-param \ReflectionClass<object> $class`
	`41`	`+ *`
`40`	`42`	`* @return Annotation[]`
`41`	`43`	`*/`
`42`	`44`	`public function getClassAnnotations(\ReflectionClass $class): array`
`@@ -51,7 +53,10 @@ public function getClassAnnotations(\ReflectionClass $class): array`
`51`	`53`	`}`
`52`	`54`
`53`	`55`	`/**`
`54`		`- * @param class-string<T> $annotationName the name of the annotation`
	`56`	`+ * @param string $annotationName`
	`57`	`+ *`
	`58`	`+ * @phpstan-param \ReflectionClass<object> $class`
	`59`	`+ * @phpstan-param class-string<T> $annotationName the name of the annotation`
`55`	`60`	`*`
`56`	`61`	`* @return T\|null the Annotation or NULL, if the requested annotation does not exist`
`57`	`62`	`*`
Original file line number	Diff line number	Diff line change
`@@ -22,6 +22,8 @@ final class AttributeReader`
`22`	`22`	`private $isRepeatableAttribute = [];`
`23`	`23`
`24`	`24`	`/**`
	`25`	`+ * @phpstan-param \ReflectionClass<object> $class`
	`26`	`+ *`
`25`	`27`	`* @return array<Annotation\|Annotation[]>`
`26`	`28`	`*/`
`27`	`29`	`public function getClassAnnotations(\ReflectionClass $class): array`
`@@ -30,6 +32,7 @@ public function getClassAnnotations(\ReflectionClass $class): array`
`30`	`32`	`}`
`31`	`33`
`32`	`34`	`/**`
	`35`	`+ * @phpstan-param \ReflectionClass<object> $class`
`33`	`36`	`* @phpstan-param class-string $annotationName`
`34`	`37`	`*`
`35`	`38`	`* @return Annotation\|Annotation[]\|null`
`@@ -58,11 +61,12 @@ public function getPropertyAnnotation(\ReflectionProperty $property, string $ann`
`58`	`61`	`}`
`59`	`62`
`60`	`63`	`/**`
`61`		`- * @param array<\ReflectionAttribute> $attributes`
	`64`	`+ * @param iterable<\ReflectionAttribute> $attributes`
	`65`	`+ * @phpstan-param iterable<\ReflectionAttribute<object>> $attributes`
`62`	`66`	`*`
`63`	`67`	`* @return array<string, Annotation\|Annotation[]>`
`64`	`68`	`*/`
`65`		`- private function convertToAttributeInstances(array $attributes): array`
	`69`	`+ private function convertToAttributeInstances(iterable $attributes): array`
`66`	`70`	`{`
`67`	`71`	`$instances = [];`
`68`	`72`
Original file line number	Diff line number	Diff line change
`@@ -11,6 +11,7 @@`
`11`	`11`
`12`	`12`	`use Doctrine\Common\EventArgs;`
`13`	`13`	`use Doctrine\Persistence\Event\LoadClassMetadataEventArgs;`
	`14`	`+use Doctrine\Persistence\Mapping\ClassMetadata;`
`14`	`15`	`use Doctrine\Persistence\ObjectManager;`
`15`	`16`	`use Gedmo\Exception\InvalidArgumentException;`
`16`	`17`	`use Gedmo\Mapping\MappedEventSubscriber;`
`@@ -225,6 +226,7 @@ public function removeManagedFilter($name)`
`225`	`226`	`* Mapps additional metadata`
`226`	`227`	`*`
`227`	`228`	`* @param LoadClassMetadataEventArgs $eventArgs`
	`229`	`+ * @phpstan-param LoadClassMetadataEventArgs<ClassMetadata<object>, ObjectManager> $eventArgs`
`228`	`230`	`*`
`229`	`231`	`* @return void`
`230`	`232`	`*/`