Fix deprecations

.php-cs-fixer.dist.php

@@ -16,6 +16,7 @@
                 'import_functions' => false,
                 'import_constants' => false,
             ],
+            'no_superfluous_phpdoc_tags' => false,
         ]
 )
     ->setRiskyAllowed(true)

config/services.xml

@@ -12,9 +12,6 @@
                  class="Meilisearch\Bundle\EventListener\DoctrineEventSubscriber"
                  public="true">
             <argument type="service" id="search.service"/>
-            <argument type="collection"/> <!-- doctrine subscribed events -->
-            <tag name="doctrine.event_subscriber"/>
-            <tag name="doctrine_mongodb.odm.event_subscriber"/>
         </service>
 
         <service id="search.client" class="Meilisearch\Client" public="true" lazy="true">

src/Collection.php

@@ -17,9 +17,6 @@ final class Collection implements \ArrayAccess, \Countable, \IteratorAggregate
      */
     private array $items;
 
-    /**
-     * @param mixed $items
-     */
     public function __construct($items = [])
     {
         $this->items = $this->getArrayableItems($items);
@@ -30,12 +27,6 @@ public function all(): array
         return $this->items;
     }
 
-    /**
-     * @param mixed $key
-     * @param mixed $default
-     *
-     * @return mixed
-     */
     public function get($key, $default = null)
     {
         if (array_key_exists($key, $this->items)) {
@@ -151,10 +142,6 @@ public function unique($key = null, bool $strict = false)
 
     /**
      * Get the first item from the collection passing the given truth test.
-     *
-     * @param mixed $default
-     *
-     * @return mixed
      */
     public function first(callable $callback = null, $default = null)
     {
@@ -181,27 +168,18 @@ public function first(callable $callback = null, $default = null)
      * Get the first item by the given key value pair.
      *
      * @param string $key
-     * @param mixed  $operator
-     * @param mixed  $value
-     *
-     * @return mixed
      */
     public function firstWhere($key, $operator = null, $value = null)
     {
         return $this->first($this->operatorForWhere(...func_get_args()));
     }
 
-    /**
-     * @param mixed $offset
-     */
     public function offsetExists($offset): bool
     {
         return isset($this->items[$offset]);
     }
 
     /**
-     * @param mixed $offset
-     *
      * @return mixed
      */
     #[\ReturnTypeWillChange]
@@ -210,10 +188,6 @@ public function offsetGet($offset)
         return $this->items[$offset];
     }
 
-    /**
-     * @param mixed $offset
-     * @param mixed $value
-     */
     public function offsetSet($offset, $value): void
     {
         if (is_null($offset)) {
@@ -223,9 +197,6 @@ public function offsetSet($offset, $value): void
         }
     }
 
-    /**
-     * @param mixed $offset
-     */
     public function offsetUnset($offset): void
     {
         unset($this->items[$offset]);
@@ -241,9 +212,6 @@ public function getIterator(): \ArrayIterator
         return new \ArrayIterator($this->items);
     }
 
-    /**
-     * @param mixed $items
-     */
     private function getArrayableItems($items): array
     {
         if (is_array($items)) {
@@ -269,9 +237,6 @@ private function getArrayableItems($items): array
         return (array) $items;
     }
 
-    /**
-     * @param mixed $value
-     */
     private function useAsCallable($value): bool
     {
         return !is_string($value) && is_callable($value);
@@ -292,11 +257,7 @@ private function valueRetriever($value)
     }
 
     /**
-     * @param mixed                 $target
      * @param string|array|int|null $key
-     * @param mixed                 $default
-     *
-     * @return mixed
      */
     private static function getDeepData($target, $key, $default = null)
     {
@@ -358,9 +319,6 @@ public static function arrayCollapse(iterable $array): array
         return array_merge([], ...$results);
     }
 
-    /**
-     * @param mixed $value
-     */
     public static function accessible($value): bool
     {
         return is_array($value) || $value instanceof \ArrayAccess;
@@ -381,10 +339,7 @@ private static function existsInArray($array, $key): bool
         return array_key_exists($key, $array);
     }
 
-    /**
-     * @param mixed $value
-     */
-    private function operatorForWhere(string $key, ?string $operator = null, $value = null): \Closure
+    private function operatorForWhere(string $key, string $operator = null, $value = null): \Closure
     {
         if (1 === func_num_args()) {
             $value = true;

src/DependencyInjection/MeilisearchExtension.php

@@ -19,9 +19,6 @@
  */
 final class MeilisearchExtension extends Extension
 {
-    /**
-     * {@inheritdoc}
-     */
     public function load(array $configs, ContainerBuilder $container): void
     {
         $loader = new Loader\XmlFileLoader($container, new FileLocator(__DIR__.'/../../config'));
@@ -39,7 +36,12 @@ public function load(array $configs, ContainerBuilder $container): void
         $container->setParameter('meili_symfony_version', MeilisearchBundle::qualifiedVersion());
 
         if (\count($doctrineEvents = $config['doctrineSubscribedEvents']) > 0) {
-            $container->getDefinition('search.search_indexer_subscriber')->setArgument(1, $doctrineEvents);
+            $subscriber = $container->getDefinition('search.search_indexer_subscriber');
+
+            foreach ($doctrineEvents as $event) {
+                $subscriber->addTag('doctrine.event_listener', ['event' => $event]);
+                $subscriber->addTag('doctrine_mongodb.odm.event_listener', ['event' => $event]);
+            }
         } else {
             $container->removeDefinition('search.search_indexer_subscriber');
         }

src/EventListener/DoctrineEventSubscriber.php

@@ -4,24 +4,16 @@
 
 namespace Meilisearch\Bundle\EventListener;
 
-use Doctrine\Common\EventSubscriber;
 use Doctrine\Persistence\Event\LifecycleEventArgs;
 use Meilisearch\Bundle\SearchService;
 
-final class DoctrineEventSubscriber implements EventSubscriber
+final class DoctrineEventSubscriber
 {
     private SearchService $searchService;
-    private array $subscribedEvents;
 
-    public function __construct(SearchService $searchService, array $subscribedEvents)
+    public function __construct(SearchService $searchService)
     {
         $this->searchService = $searchService;
-        $this->subscribedEvents = $subscribedEvents;
-    }
-
-    public function getSubscribedEvents(): array
-    {
-        return $this->subscribedEvents;
     }
 
     public function postUpdate(LifecycleEventArgs $args): void

src/Model/Aggregator.php

@@ -78,10 +78,7 @@ public static function getEntityClassFromObjectID(string $objectId): string
         throw new EntityNotFoundInObjectID("Entity class from ObjectID $objectId not found.");
     }
 
-    /**
-     * {@inheritdoc}
-     */
-    public function normalize(NormalizerInterface $normalizer, ?string $format = null, array $context = []): array
+    public function normalize(NormalizerInterface $normalizer, string $format = null, array $context = []): array
     {
         return array_merge(['objectID' => $this->objectID], $normalizer->normalize($this->entity, $format, $context));
     }

src/SearchableEntity.php

@@ -43,7 +43,7 @@ public function __construct(
         string $indexUid,
         $entity,
         ClassMetadata $entityMetadata,
-        ?NormalizerInterface $normalizer = null,
+        NormalizerInterface $normalizer = null,
         array $extra = []
     ) {
         $this->indexUid = $indexUid;

src/Services/MeilisearchService.php

@@ -46,9 +46,6 @@ public function __construct(NormalizerInterface $normalizer, Engine $engine, arr
         $this->setIndexIfMapping();
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function isSearchable($className): bool
     {
         if (is_object($className)) {
@@ -68,9 +65,6 @@ public function getConfiguration(): Collection
         return $this->configuration;
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function searchableAs(string $className): string
     {
         $indexes = new Collection($this->getConfiguration()->get('indices'));
@@ -185,9 +179,6 @@ public function search(
         return $results;
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function rawSearch(
         string $className,
         string $query = '',

tests/Entity/Link.php

@@ -91,9 +91,6 @@ public function setIsSponsored(bool $isSponsored): Link
         return $this;
     }
 
-    /**
-     * {@inheritDoc}
-     */
     public function normalize(NormalizerInterface $normalizer, $format = null, array $context = []): array
     {
         if (Searchable::NORMALIZATION_FORMAT === $format) {

tests/Entity/Page.php

@@ -27,7 +27,7 @@ class Page
     #[ORM\Id]
     #[ORM\GeneratedValue(strategy: 'NONE')]
     #[ORM\Column(type: Types::OBJECT)]
-    private $id = null;
+    private $id;
 
     /**
      * @ORM\Column(type="string", nullable=true)

tests/Entity/Tag.php

@@ -88,8 +88,6 @@ public function setPublic(bool $public): Tag
     }
 
     /**
-     * {@inheritDoc}
-     *
      * @throws ExceptionInterface
      */
     public function normalize(NormalizerInterface $normalizer, $format = null, array $context = []): array

tests/Integration/EventListener/DoctrineEventSubscriberTest.php

@@ -4,7 +4,7 @@
 
 namespace Meilisearch\Bundle\Tests\Integration\EventListener;
 
-use Doctrine\ORM\Event\LifecycleEventArgs;
+use Doctrine\Persistence\Event\LifecycleEventArgs;
 use Meilisearch\Bundle\EventListener\DoctrineEventSubscriber;
 use Meilisearch\Bundle\Tests\BaseKernelTestCase;
 use Meilisearch\Bundle\Tests\Entity\Page;
@@ -35,7 +35,7 @@ public function testPostPersist(): void
 
         $eventArgs = new LifecycleEventArgs($post, $this->entityManager);
 
-        $subscriber = new DoctrineEventSubscriber($this->searchService, []);
+        $subscriber = new DoctrineEventSubscriber($this->searchService);
         $subscriber->postPersist($eventArgs);
 
         $this->waitForAllTasks();
@@ -53,7 +53,7 @@ public function testPostPersistWithObjectId(): void
 
         $eventArgs = new LifecycleEventArgs($page, $this->entityManager);
 
-        $subscriber = new DoctrineEventSubscriber($this->searchService, []);
+        $subscriber = new DoctrineEventSubscriber($this->searchService);
         $subscriber->postPersist($eventArgs);
 
         $this->waitForAllTasks();
@@ -74,7 +74,7 @@ public function testPostUpdate(): void
 
         $eventArgs = new LifecycleEventArgs($post, $this->entityManager);
 
-        $subscriber = new DoctrineEventSubscriber($this->searchService, []);
+        $subscriber = new DoctrineEventSubscriber($this->searchService);
         $subscriber->postUpdate($eventArgs);
 
         $this->waitForAllTasks();
@@ -92,7 +92,7 @@ public function testPostUpdateWithObjectId(): void
 
         $eventArgs = new LifecycleEventArgs($page, $this->entityManager);
 
-        $subscriber = new DoctrineEventSubscriber($this->searchService, []);
+        $subscriber = new DoctrineEventSubscriber($this->searchService);
         $subscriber->postUpdate($eventArgs);
 
         $this->waitForAllTasks();
@@ -113,7 +113,7 @@ public function testPreRemove(): void
 
         $eventArgs = new LifecycleEventArgs($post, $this->entityManager);
 
-        $subscriber = new DoctrineEventSubscriber($this->searchService, []);
+        $subscriber = new DoctrineEventSubscriber($this->searchService);
         $subscriber->postPersist($eventArgs);
 
         $this->waitForAllTasks();
@@ -138,7 +138,7 @@ public function testPreRemoveWithObjectId(): void
 
         $eventArgs = new LifecycleEventArgs($page, $this->entityManager);
 
-        $subscriber = new DoctrineEventSubscriber($this->searchService, []);
+        $subscriber = new DoctrineEventSubscriber($this->searchService);
         $subscriber->postPersist($eventArgs);
 
         $this->waitForAllTasks();

tests/Integration/SearchTest.php

@@ -29,8 +29,6 @@ class SearchTest extends BaseKernelTestCase
     protected Indexes $index;
 
     /**
-     * {@inheritDoc}
-     *
      * @throws ApiException
      * @throws \Exception
      */

tests/Kernel.php

@@ -17,8 +17,6 @@
 class Kernel extends HttpKernel
 {
     /**
-     * {@inheritDoc}
-     *
      * @return array<int, BundleInterface>
      */
     public function registerBundles(): array

tests/Unit/ConfigurationTest.php

@@ -14,9 +14,6 @@ class ConfigurationTest extends KernelTestCase
 {
     /**
      * @dataProvider dataTestConfigurationTree
-     *
-     * @param mixed $inputConfig
-     * @param mixed $expectedConfig
      */
     public function testConfigurationTree($inputConfig, $expectedConfig): void
     {

tests/baseline-ignore

@@ -3,3 +3,8 @@
 %The "Symfony\\Component\\HttpClient\\HttplugClient" class implements "Http\\Message\\RequestFactory" that is deprecated since version 1.1, use Psr\\Http\\Message\\RequestFactoryInterface instead.%
 %The "Symfony\\Component\\HttpClient\\HttplugClient" class implements "Http\\Message\\StreamFactory" that is deprecated since version 1.1, use Psr\\Http\\Message\\StreamFactoryInterface instead.%
 %The "Symfony\\Component\\HttpClient\\HttplugClient" class implements "Http\\Message\\UriFactory" that is deprecated since version 1.1, use Psr\\Http\\Message\\UriFactoryInterface instead.%
+%Doctrine\\DBAL\\Platforms\\AbstractPlatform::usesSequenceEmulatedIdentityColumns is deprecated. \(AbstractPlatform.php:\d+ called by ClassMetadataFactory.php:\d+, https://github.com/doctrine/dbal/pull/5513, package doctrine/dbal\)%
+%Column::setCustomSchemaOptions\(\) is deprecated. Use setPlatformOptions\(\) instead. \(Column.php:\d+ called by Column.php:\d+, https://github.com/doctrine/dbal/pull/5476, package doctrine/dbal\)%
+%SqlitePlatform::canEmulateSchemas\(\) is deprecated. \(SqlitePlatform.php:\d+ called by SchemaTool.php:\d+, https://github.com/doctrine/dbal/pull/4805, package doctrine/dbal\)%
+%Doctrine\\DBAL\\Schema\\Table::getPrimaryKeyColumns is deprecated. Use getPrimaryKey\(\) and Index::getColumns\(\) instead. \(Table.php:\d+ called by Table.php:\d+, https://github.com/doctrine/dbal/pull/5731, package doctrine/dbal\)%
+%The annotation mapping driver is deprecated and will be removed in Doctrine ORM 3.0, please migrate to the attribute or XML driver. \(AnnotationDriver.php:\d+ called by getDoctrine_Orm_DefaultAnnotationMetadataDriverService.php:20, https://github.com/doctrine/orm/issues/10098, package doctrine/orm\)%

tests/config/config.yaml

@@ -12,7 +12,8 @@ doctrine:
                 path: '%kernel.cache_dir%/test.sqlite'
     orm:
         auto_generate_proxy_classes: true
-        naming_strategy: doctrine.orm.naming_strategy.underscore
+        validate_xml_mapping: true
+        naming_strategy: doctrine.orm.naming_strategy.underscore_number_aware
         auto_mapping: true
         mappings:
             App: