Merge pull request #29 from MacFJA/fix-paginated-response

Fix PaginatedResponse

Author: MacFJA

CHANGELOG.md

@@ -11,6 +11,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Summarize Search option enabled by default (without possibility to disable it) ([Issue#26])
 - Highlight Search option enabled by default (without possibility to disable it) ([Issue#26])
 - Fix error when parsing response from Search with NOCONTENT flag
+- Make PaginatedResponse `valid` iterator function work
 
 ## [2.1.0]
 

src/Redis/Response/PaginatedResponse.php

@@ -109,14 +109,19 @@ public function key()
             return 0;
         }
 
-        return (int) floor(($this->lastCommand->getOffset() ?? 0) / $this->getPageSize());
+        return (int) floor(($this->lastCommand->getOffset() ?? $this->requestedOffset ?? 0) / $this->getPageSize());
     }
 
     public function valid()
     {
-        return $this->requestedOffset >= 0
-            && $this->requestedSize > 0
-            && $this->requestedOffset < $this->totalCount;
+        return (
+                null === $this->requestedOffset
+                && null === $this->requestedSize
+            ) || (
+                $this->requestedOffset >= 0
+                && $this->requestedSize > 0
+                && $this->requestedOffset < $this->totalCount
+            );
     }
 
     public function rewind(): void