Fix deprecations

Author: norkunas

src/EventListener/DoctrineEventSubscriber.php

@@ -18,16 +18,25 @@ public function __construct(SearchService $searchService)
 
     public function postUpdate(LifecycleEventArgs $args): void
     {
+        if (!$this->searchService->isSearchable($args->getObject())) {
+            return;
+        }
         $this->searchService->index($args->getObjectManager(), $args->getObject());
     }
 
     public function postPersist(LifecycleEventArgs $args): void
     {
+        if (!$this->searchService->isSearchable($args->getObject())) {
+            return;
+        }
         $this->searchService->index($args->getObjectManager(), $args->getObject());
     }
 
     public function preRemove(LifecycleEventArgs $args): void
     {
+        if (!$this->searchService->isSearchable($args->getObject())) {
+            return;
+        }
         $this->searchService->remove($args->getObjectManager(), $args->getObject());
     }
 }

src/Services/MeilisearchService.php

@@ -5,6 +5,7 @@
 namespace Meilisearch\Bundle\Services;
 
 use Doctrine\Common\Util\ClassUtils;
+use Doctrine\ORM\Mapping\LegacyReflectionFields;
 use Doctrine\ORM\Proxy\DefaultProxyClassNameResolver;
 use Doctrine\Persistence\ObjectManager;
 use Meilisearch\Bundle\Collection;
@@ -374,6 +375,11 @@ private static function resolveClass(object $object): string
         static $resolver;
 
         $resolver ??= (function () {
+            // Native lazy objects compatibility
+            if (class_exists(LegacyReflectionFields::class)) {
+                return fn ($object) => $object::class;
+            }
+
             // Doctrine ORM v3+ compatibility
             if (class_exists(DefaultProxyClassNameResolver::class)) {
                 return fn (object $object) => DefaultProxyClassNameResolver::getClass($object);

tests/Integration/AggregatorTest.php

@@ -4,6 +4,7 @@
 
 namespace Meilisearch\Bundle\Tests\Integration;
 
+use Doctrine\ORM\Mapping\LegacyReflectionFields;
 use Doctrine\Persistence\Proxy;
 use Meilisearch\Bundle\Exception\EntityNotFoundInObjectID;
 use Meilisearch\Bundle\Exception\InvalidEntityForAggregator;
@@ -35,6 +36,10 @@ public function testConstructor(): void
 
     public function testAggregatorProxyClass(): void
     {
+        if (class_exists(LegacyReflectionFields::class)) {
+            $this->markTestSkipped('Skipping, because proxies are not wrapped anymore with lazy native objects.');
+        }
+
         $this->entityManager->persist($post = new Post());
         $this->entityManager->flush();
         $postId = $post->getId();

tests/Kernel.php

@@ -7,6 +7,7 @@
 use Doctrine\Bundle\DoctrineBundle\ConnectionFactory;
 use Doctrine\Bundle\DoctrineBundle\DoctrineBundle;
 use Doctrine\ORM\Configuration;
+use Doctrine\ORM\Mapping\LegacyReflectionFields;
 use Meilisearch\Bundle\MeilisearchBundle;
 use Symfony\Bridge\Doctrine\ArgumentResolver\EntityValueResolver;
 use Symfony\Bundle\FrameworkBundle\FrameworkBundle;
@@ -29,7 +30,11 @@ public function registerBundles(): iterable
     protected function configureContainer(ContainerBuilder $container, LoaderInterface $loader): void
     {
         if (PHP_VERSION_ID >= 80000) {
-            $loader->load(__DIR__.'/config/config.yaml');
+            if (class_exists(LegacyReflectionFields::class) && PHP_VERSION_ID >= 80400) {
+                $loader->load(__DIR__.'/config/config.yaml');
+            } else {
+                $loader->load(__DIR__.'/config/config_old_proxy.yaml');
+            }
         } else {
             $loader->load(__DIR__.'/config/config_php7.yaml');
         }

tests/baseline-ignore

@@ -1,3 +1,9 @@
 %Method "ArrayAccess::offsetGet\(\)" might add "mixed" as a native return type declaration in the future. Do the same in implementation "Meilisearch\\Contracts\\Data" now to avoid errors or add an explicit @return annotation to suppress this message.%
 %Since symfony/var-exporter 7.3: The "Symfony\\Component\\VarExporter\\LazyGhostTrait" trait is deprecated, use native lazy objects instead.%
 %Since symfony/var-exporter 7.3: Using ProxyHelper::generateLazyGhost\(\) is deprecated, use native lazy objects instead.%
+%Class "Doctrine\\ORM\\Proxy\\Autoloader" is deprecated. Use native lazy objects instead.%
+%Calling Doctrine\\ORM\\Configuration::setProxyDir is deprecated and will not be possible in Doctrine ORM 4.0%
+%Calling Doctrine\\ORM\\Configuration::getProxyDir is deprecated and will not be possible in Doctrine ORM 4.0%
+%Calling Doctrine\\ORM\\Configuration::setAutoGenerateProxyClasses is deprecated and will not be possible in Doctrine ORM 4.0%
+%Calling Doctrine\\ORM\\Configuration::getAutoGenerateProxyClasses is deprecated and will not be possible in Doctrine ORM 4.0%
+%Calling Doctrine\\ORM\\Configuration::setProxyNamespace is deprecated and will not be possible in Doctrine ORM 4.0%

tests/config/config.yaml

@@ -13,7 +13,8 @@ doctrine:
         types:
             dummy_object_id: Meilisearch\Bundle\Tests\Dbal\Type\DummyObjectIdType
     orm:
-        auto_generate_proxy_classes: true
+        enable_native_lazy_objects: true
+        auto_generate_proxy_classes: false
         report_fields_where_declared: true
         naming_strategy: doctrine.orm.naming_strategy.underscore_number_aware
         auto_mapping: true

tests/config/config_old_proxy.yaml

@@ -0,0 +1,26 @@
+framework:
+    test: true
+    secret: 67d829bf61dc5f87a73fd814e2c9f629
+    http_method_override: false
+
+doctrine:
+    dbal:
+        default_connection: default
+        connections:
+            default:
+                driver: pdo_sqlite
+                path: '%kernel.cache_dir%/test.sqlite'
+        types:
+            dummy_object_id: Meilisearch\Bundle\Tests\Dbal\Type\DummyObjectIdType
+    orm:
+        auto_generate_proxy_classes: true
+        report_fields_where_declared: true
+        naming_strategy: doctrine.orm.naming_strategy.underscore_number_aware
+        auto_mapping: true
+        mappings:
+            App:
+                is_bundle: false
+                type: attribute
+                dir: '%kernel.project_dir%/tests/Entity'
+                prefix: 'Meilisearch\Bundle\Tests\Entity'
+                alias: App