Fix using bundle configured nbResults for search

Author: norkunas

composer.json

@@ -43,6 +43,7 @@
     "symfony/doctrine-bridge": "^4.4 || ^5.0 || ^6.0",
     "symfony/http-client": "^4.4 || ^5.0 || ^6.0",
     "symfony/phpunit-bridge": "^4.4 || ^5.0 || ^6.0",
+    "symfony/var-exporter": "^4.4 || ^5.0 || ^6.0",
     "symfony/yaml": "^4.4 || ^5.0 || ^6.0"
   },
   "autoload": {

src/DependencyInjection/Configuration.php

@@ -22,10 +22,10 @@ public function getConfigTreeBuilder(): TreeBuilder
                 ->scalarNode('prefix')
                     ->defaultNull()
                 ->end()
-                ->scalarNode('nbResults')
+                ->integerNode('nbResults')
                     ->defaultValue(20)
                 ->end()
-                ->scalarNode('batchSize')
+                ->integerNode('batchSize')
                     ->defaultValue(500)
                 ->end()
                 ->arrayNode('doctrineSubscribedEvents')

src/Services/MeilisearchService.php

@@ -155,7 +155,7 @@ public function search(
     ): array {
         $this->assertIsSearchable($className);
 
-        $ids = $this->engine->search($query, $this->searchableAs($className), $searchParams);
+        $ids = $this->engine->search($query, $this->searchableAs($className), $searchParams + ['limit' => $this->configuration['nbResults']]);
         $results = [];
 
         // Check if the engine returns results in "hits" key

tests/Integration/SearchTest.php

@@ -10,7 +10,6 @@
 use Meilisearch\Bundle\Tests\Entity\Post;
 use Meilisearch\Bundle\Tests\Entity\Tag;
 use Meilisearch\Endpoints\Indexes;
-use Meilisearch\Exceptions\ApiException;
 use Symfony\Bundle\FrameworkBundle\Console\Application;
 use Symfony\Component\Console\Tester\CommandTester;
 
@@ -26,10 +25,6 @@ class SearchTest extends BaseKernelTestCase
     protected Application $application;
     protected Indexes $index;
 
-    /**
-     * @throws ApiException
-     * @throws \Exception
-     */
     protected function setUp(): void
     {
         parent::setUp();
@@ -115,8 +110,20 @@ public function testSearchPagination(): void
         $this->assertEqualsCanonicalizing(array_slice($testDataTitles, 2, 2), $resultTitles);
     }
 
-    protected function tearDown(): void
+    public function testSearchNbResults(): void
     {
-        parent::tearDown();
+        for ($i = 0; $i < 15; ++$i) {
+            $this->createPost();
+        }
+
+        $command = $this->application->find('meilisearch:import');
+        $commandTester = new CommandTester($command);
+        $commandTester->execute([
+            '--indices' => $this->index->getUid(),
+        ]);
+
+        $results = $this->searchService->search($this->objectManager, Post::class, 'test');
+
+        $this->assertCount(12, $results);
     }
 }

tests/Kernel.php

@@ -6,6 +6,7 @@
 
 use Doctrine\Bundle\DoctrineBundle\ConnectionFactory;
 use Doctrine\Bundle\DoctrineBundle\DoctrineBundle;
+use Doctrine\ORM\Configuration;
 use Meilisearch\Bundle\MeilisearchBundle;
 use Symfony\Bundle\FrameworkBundle\FrameworkBundle;
 use Symfony\Bundle\FrameworkBundle\Kernel\MicroKernelTrait;
@@ -40,5 +41,13 @@ protected function configureContainer(ContainerBuilder $container, LoaderInterfa
                 ],
             ]);
         }
+        // @phpstan-ignore-next-line
+        if (method_exists(Configuration::class, 'setLazyGhostObjectEnabled') && Kernel::VERSION_ID >= 60100) {
+            $container->prependExtensionConfig('doctrine', [
+                'orm' => [
+                    'enable_lazy_ghost_objects' => true,
+                ],
+            ]);
+        }
     }
 }