Add generics for the event subscriber configuration arrays and adapters

Author: mbabker

phpstan-baseline.neon

@@ -280,21 +280,6 @@ parameters:
 			count: 1
 			path: src/References/ReferencesListener.php
 
-		-
-			message: "#^Call to an undefined method Gedmo\\\\Mapping\\\\Event\\\\AdapterInterface\\:\\:extractIdentifier\\(\\)\\.$#"
-			count: 2
-			path: src/References/ReferencesListener.php
-
-		-
-			message: "#^Call to an undefined method Gedmo\\\\Mapping\\\\Event\\\\AdapterInterface\\:\\:getIdentifier\\(\\)\\.$#"
-			count: 1
-			path: src/References/ReferencesListener.php
-
-		-
-			message: "#^Call to an undefined method Gedmo\\\\Mapping\\\\Event\\\\AdapterInterface\\:\\:getSingleReference\\(\\)\\.$#"
-			count: 1
-			path: src/References/ReferencesListener.php
-
 		-
 			message: "#^Call to an undefined method Doctrine\\\\Persistence\\\\Mapping\\\\ClassMetadata\\<object\\>\\:\\:getReflectionProperty\\(\\)\\.$#"
 			count: 2
@@ -395,11 +380,6 @@ parameters:
 			count: 1
 			path: src/SoftDeleteable/Query/TreeWalker/SoftDeleteableWalker.php
 
-		-
-			message: "#^Call to an undefined method Gedmo\\\\Mapping\\\\Event\\\\AdapterInterface\\:\\:getDateValue\\(\\)\\.$#"
-			count: 1
-			path: src/SoftDeleteable/SoftDeleteableListener.php
-
 		-
 			message: "#^Access to offset 'inherited' on an unknown class Doctrine\\\\ODM\\\\MongoDB\\\\Mapping\\\\AssociationFieldMapping\\.$#"
 			count: 1

src/AbstractTrackingListener.php

@@ -28,6 +28,11 @@
 /**
  * The AbstractTrackingListener provides generic functions for all listeners.
  *
+ * @phpstan-template TConfig of array
+ * @phpstan-template TEventAdapter of AdapterInterface
+ *
+ * @phpstan-extends MappedEventSubscriber<TConfig, TEventAdapter>
+ *
  * @author Gediminas Morkevicius <[email protected]>
  */
 abstract class AbstractTrackingListener extends MappedEventSubscriber

src/Blameable/BlameableListener.php

@@ -11,12 +11,15 @@
 
 use Doctrine\Persistence\Mapping\ClassMetadata;
 use Gedmo\AbstractTrackingListener;
+use Gedmo\Blameable\Mapping\Event\BlameableAdapter;
 use Gedmo\Exception\InvalidArgumentException;
 
 /**
  * The Blameable listener handles the update of
  * dates on creation and update.
  *
+ * @phpstan-extends AbstractTrackingListener<array, BlameableAdapter>
+ *
  * @author Gediminas Morkevicius <[email protected]>
  *
  * @final since gedmo/doctrine-extensions 3.11

src/IpTraceable/IpTraceableListener.php

@@ -12,12 +12,15 @@
 use Doctrine\Persistence\Mapping\ClassMetadata;
 use Gedmo\AbstractTrackingListener;
 use Gedmo\Exception\InvalidArgumentException;
+use Gedmo\IpTraceable\Mapping\Event\IpTraceableAdapter;
 use Gedmo\Mapping\Event\AdapterInterface;
 
 /**
  * The IpTraceable listener handles the update of
  * IPs on creation and update.
  *
+ * @phpstan-extends AbstractTrackingListener<array, IpTraceableAdapter>
+ *
  * @author Pierre-Charles Bertineau <[email protected]>
  *
  * @final since gedmo/doctrine-extensions 3.11

src/Loggable/LoggableListener.php

@@ -17,7 +17,6 @@
 use Doctrine\Persistence\Mapping\ClassMetadata;
 use Doctrine\Persistence\ObjectManager;
 use Gedmo\Exception\InvalidArgumentException;
-use Gedmo\Loggable\Entity\LogEntry;
 use Gedmo\Loggable\Mapping\Event\LoggableAdapter;
 use Gedmo\Mapping\MappedEventSubscriber;
 use Gedmo\Tool\Wrapper\AbstractWrapper;
@@ -35,11 +34,9 @@
  *   versioned?: string[],
  * }
  *
- * @phpstan-method LoggableConfiguration getConfiguration(ObjectManager $objectManager, $class)
- *
- * @method LoggableAdapter getEventAdapter(EventArgs $args)
- *
  * @phpstan-template T of Loggable|object
+ *
+ * @phpstan-extends MappedEventSubscriber<LoggableConfiguration, LoggableAdapter>
  */
 class LoggableListener extends MappedEventSubscriber
 {

src/Mapping/MappedEventSubscriber.php

@@ -27,9 +27,6 @@
 use Gedmo\Mapping\Driver\AttributeReader;
 use Gedmo\Mapping\Event\AdapterInterface;
 use Gedmo\Mapping\Event\ClockAwareAdapterInterface;
-use Gedmo\ReferenceIntegrity\Mapping\Validator as ReferenceIntegrityValidator;
-use Gedmo\Uploadable\FilenameGenerator\FilenameGeneratorInterface;
-use Gedmo\Uploadable\Mapping\Validator as MappingValidator;
 use Psr\Cache\CacheItemPoolInterface;
 use Psr\Clock\ClockInterface;
 use Symfony\Component\Cache\Adapter\ArrayAdapter;
@@ -43,6 +40,9 @@
  * all extensions who maps additional metadata through
  * extended drivers
  *
+ * @phpstan-template TConfig of array
+ * @phpstan-template TEventAdapter of AdapterInterface
+ *
  * @author Gediminas Morkevicius <[email protected]>
  */
 abstract class MappedEventSubscriber implements EventSubscriber
@@ -115,24 +115,7 @@ public function __construct()
      *
      * @return array<string, mixed>
      *
-     * @phpstan-return array{
-     *  useObjectClass?: class-string,
-     *  referenceIntegrity?: array<string, array<string, value-of<ReferenceIntegrityValidator::INTEGRITY_ACTIONS>>>,
-     *  filePathField?: string,
-     *  uploadable?: bool,
-     *  fileNameField?: string,
-     *  allowOverwrite?: bool,
-     *  appendNumber?: bool,
-     *  maxSize?: float,
-     *  path?: string,
-     *  pathMethod?: string,
-     *  allowedTypes?: string[],
-     *  disallowedTypes?: string[],
-     *  filenameGenerator?: MappingValidator::FILENAME_GENERATOR_*|class-string<FilenameGeneratorInterface>,
-     *  fileMimeTypeField?: string,
-     *  fileSizeField?: string,
-     *  callback?: string,
-     * }
+     * @phpstan-return TConfig
      */
     public function getConfiguration(ObjectManager $objectManager, $class)
     {
@@ -273,6 +256,8 @@ public function loadMetadataForObjectClass(ObjectManager $objectManager, $metada
      * @throws InvalidArgumentException if event is not recognized
      *
      * @return AdapterInterface
+     *
+     * @phpstan-return TEventAdapter
      */
     protected function getEventAdapter(EventArgs $args)
     {

src/ReferenceIntegrity/ReferenceIntegrityListener.php

@@ -16,12 +16,15 @@
 use Doctrine\Persistence\ObjectManager;
 use Gedmo\Exception\InvalidMappingException;
 use Gedmo\Exception\ReferenceIntegrityStrictException;
+use Gedmo\Mapping\Event\AdapterInterface;
 use Gedmo\Mapping\MappedEventSubscriber;
 use Gedmo\ReferenceIntegrity\Mapping\Validator;
 
 /**
  * The ReferenceIntegrity listener handles the reference integrity on related documents
  *
+ * @phpstan-extends MappedEventSubscriber<array, AdapterInterface>
+ *
  * @author Evert Harmeling <[email protected]>
  *
  * @final since gedmo/doctrine-extensions 3.11

src/References/ReferencesListener.php

@@ -16,6 +16,7 @@
 use Doctrine\Persistence\Mapping\ClassMetadata;
 use Doctrine\Persistence\ObjectManager;
 use Gedmo\Mapping\MappedEventSubscriber;
+use Gedmo\References\Mapping\Event\ReferencesAdapter;
 
 /**
  * Listener for loading and persisting cross database references.
@@ -39,7 +40,7 @@
  *   useObjectClass?: class-string,
  * }
  *
- * @phpstan-method ReferencesConfiguration getConfiguration(ObjectManager $objectManager, $class)
+ * @phpstan-extends MappedEventSubscriber<ReferencesConfiguration, ReferencesAdapter>
  *
  * @final since gedmo/doctrine-extensions 3.11
  */

src/Sluggable/SluggableListener.php

@@ -65,9 +65,7 @@
  *   useObjectClass?: class-string,
  * }
  *
- * @phpstan-method SluggableConfiguration getConfiguration(ObjectManager $objectManager, $class)
- *
- * @method SluggableAdapter getEventAdapter(EventArgs $args)
+ * @phpstan-extends MappedEventSubscriber<SluggableConfiguration, SluggableAdapter>
  */
 class SluggableListener extends MappedEventSubscriber
 {

src/SoftDeleteable/SoftDeleteableListener.php

@@ -23,10 +23,13 @@
 use Gedmo\Mapping\MappedEventSubscriber;
 use Gedmo\SoftDeleteable\Event\PostSoftDeleteEventArgs;
 use Gedmo\SoftDeleteable\Event\PreSoftDeleteEventArgs;
+use Gedmo\SoftDeleteable\Mapping\Event\SoftDeleteableAdapter;
 
 /**
  * SoftDeleteable listener
  *
+ * @phpstan-extends MappedEventSubscriber<array, SoftDeleteableAdapter>
+ *
  * @author Gustavo Falco <[email protected]>
  * @author Gediminas Morkevicius <[email protected]>
  *