Revert "Fix getTotalCount() method" (#96)

Co-authored-by: Jordan Massart <[email protected]>
meilisearch · Feb 3, 2021 · 93bd366 · 93bd366
1 parent 802635d
commit 93bd366
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/Engines/MeilisearchEngine.php b/src/Engines/MeilisearchEngine.php
@@ -200,7 +200,7 @@ public function map(Builder $builder, $results, $model)
      */
     public function getTotalCount($results)
     {
-        return count($results['hits']);
+        return $results['nbHits'];
     }
 
     /**

diff --git a/tests/MeilisearchEngineTest.php b/tests/MeilisearchEngineTest.php
@@ -138,7 +138,7 @@ public function mapIdsReturnsEmptyCollectionIfNoHits()
         $engine = new MeilisearchEngine($client);
 
         $results = $engine->mapIds([
-            'hits' => [],
+            'nbHits' => 0, 'hits' => [],
         ]);
 
         $this->assertEquals(0, count($results));
@@ -156,7 +156,7 @@ public function mapCorrectlyMapsResultsToModels()
         $builder = m::mock(Builder::class);
 
         $results = $engine->map($builder, [
-            'hits' => [
+            'nbHits' => 1, 'hits' => [
                 ['id' => 1],
             ],
         ], $model);
@@ -182,7 +182,7 @@ public function mapMethodRespectsOrder()
         $builder = m::mock(Builder::class);
 
         $results = $engine->map($builder, [
-            'hits' => [
+            'nbHits' => 4, 'hits' => [
                 ['id' => 1],
                 ['id' => 2],
                 ['id' => 4],
-Original file line number
+Diff line change
@@ Expand Up / @@ -200,7 +200,7 @@ public function map(Builder $builder, $results, $model) @@
          */
         public function getTotalCount($results)
         {
-            return count($results['hits']);
+            return $results['nbHits'];
         }
         /**
@@ Expand Down @@