fix(typesense): remove builder limit assignment on paginate (#858)

tharropoulos · web-flow · commit 71502b10998c · 2024-09-04T09:16:00.000-05:00
- Remove assignment of `perPage` to builder's limit attribute
- Allows for accurate total record count beyond `perPage` value
- Addresses interaction issue between `Builder` and `TypesenseEngine`
- Ensures correct limit setting for Typesense server queries
diff --git a/src/Engines/TypesenseEngine.php b/src/Engines/TypesenseEngine.php
@@ -205,8 +205,6 @@ public function search(Builder $builder)
      */
     public function paginate(Builder $builder, $perPage, $page)
     {
-        $builder->take($builder->limit ?? $perPage);
-
         return $this->performSearch(
             $builder,
             $this->buildSearchParameters($builder, $page, $perPage)

Original file line number	Diff line number	Diff line change
`@@ -205,8 +205,6 @@ public function search(Builder $builder)`
`205`	`205`	`*/`
`206`	`206`	`public function paginate(Builder $builder, $perPage, $page)`
`207`	`207`	`{`
`208`		`- $builder->take($builder->limit ?? $perPage);`
`209`		`-`
`210`	`208`	`return $this->performSearch(`
`211`	`209`	`$builder,`
`212`	`210`	`$this->buildSearchParameters($builder, $page, $perPage)`