style(core): fixs

meilisearch · Nov 24, 2020 · 9f8e368 · 9f8e368
1 parent bacd343
commit 9f8e368
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 10 deletions.
diff --git a/src/Endpoints/Indexes.php b/src/Endpoints/Indexes.php
@@ -129,7 +129,7 @@ public function search($query, array $searchParams = [], array $options = []): S
 
         $searchResult = new SearchResult($result);
 
-        if (array_key_exists('filteringHits', $options) && is_callable($options['filteringHits'])) {
+        if (\array_key_exists('filteringHits', $options) && \is_callable($options['filteringHits'])) {
             $searchResult = $searchResult->filter($options['filteringHits']);
         }
 

diff --git a/src/Search/SearchResult.php b/src/Search/SearchResult.php
@@ -4,11 +4,11 @@
 
 namespace MeiliSearch\Search;
 
+use function array_filter;
 use ArrayIterator;
+use function count;
 use Countable;
 use IteratorAggregate;
-use function array_filter;
-use function count;
 
 class SearchResult implements Countable, IteratorAggregate
 {
@@ -77,16 +77,14 @@ public function __construct(array $body)
      *
      * The method DOES not trigger a new search.
      *
-     * @param callable $callback
-     *
      * @return SearchResult
      */
     public function filter(callable $callback): self
     {
         $results = array_filter($this->hits, $callback, ARRAY_FILTER_USE_BOTH);
 
         $this->hits = $results;
-        $this->nbHits = count($results);
+        $this->nbHits = \count($results);
 
         return $this;
     }
@@ -121,7 +119,7 @@ public function getMatches(): int
 
     public function getNbHits(): int
     {
-        return count($this->hits);
+        return \count($this->hits);
     }
 
     public function getExhaustiveNbHits(): bool
@@ -159,7 +157,7 @@ public function toArray(): array
             'offset' => $this->offset,
             'limit' => $this->limit,
             'matches' => $this->nbHits,
-            'nbHits' => count($this->hits),
+            'nbHits' => \count($this->hits),
             'exhaustiveNbHits' => $this->exhaustiveNbHits,
             'processingTimeMs' => $this->processingTimeMs,
             'query' => $this->query,
@@ -175,6 +173,6 @@ public function getIterator(): ArrayIterator
 
     public function count(): int
     {
-        return count($this->hits);
+        return \count($this->hits);
     }
 }
diff --git a/tests/Search/SearchResultTest.php b/tests/Search/SearchResultTest.php
@@ -102,7 +102,7 @@ public function testSearchResultCanBeFiltered(): void
         ]);
 
         $filteredResults = $result->filter(function (array $hit, int $_): bool {
-            return strtoupper($hit['title']) === 'AMERICAN SNIPER';
+            return 'AMERICAN SNIPER' === strtoupper($hit['title']);
         });
 
         static::assertSame(1, $filteredResults->count());