fix: Fixed AjaxNodesExplorerController with SearchResultsInterface

roadiz · Mar 19, 2024 · 06e510d · 06e510d
1 parent 77a1bef
commit 06e510d
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/src/SearchEngine/SearchResultsInterface.php b/src/SearchEngine/SearchResultsInterface.php
@@ -4,6 +4,9 @@
 
 namespace RZ\Roadiz\CoreBundle\SearchEngine;
 
+/**
+ * @extends \Iterator<SolrSearchResultItem>
+ */
 interface SearchResultsInterface extends \Iterator
 {
     public function getResultCount(): int;

diff --git a/src/SearchEngine/SolrSearchResults.php b/src/SearchEngine/SolrSearchResults.php
@@ -129,10 +129,7 @@ protected function getHydratedItem(array $item): mixed
                         DocumentTranslation::class,
                         $item[SolariumDocumentTranslation::IDENTIFIER_KEY]
                     );
-                    if (null === $documentTranslation) {
-                        return null;
-                    }
-                    return $documentTranslation->getDocument();
+                    return $documentTranslation?->getDocument();
             }
         }
 
@@ -143,11 +140,11 @@ protected function getHydratedItem(array $item): mixed
      * Return the current element
      *
      * @link https://php.net/manual/en/iterator.current.php
-     * @return mixed Can return any type.
+     * @return SolrSearchResultItem
      * @since 5.0
      */
     #[\ReturnTypeWillChange]
-    public function current(): mixed
+    public function current(): SolrSearchResultItem
     {
         return $this->getResultItems()[$this->position];
     }