Fixes #3 - _getCacheInternalId() is too generic

Signed-off-by: Frank Brückner <[email protected]>

src/Adapter/DbSelect.php

@@ -8,14 +8,15 @@
 
 namespace Laminas\Paginator\Adapter;
 
+use JsonSerializable;
 use Laminas\Db\Adapter\Adapter;
 use Laminas\Db\ResultSet\ResultSet;
 use Laminas\Db\ResultSet\ResultSetInterface;
 use Laminas\Db\Sql\Expression;
 use Laminas\Db\Sql\Select;
 use Laminas\Db\Sql\Sql;
 
-class DbSelect implements AdapterInterface
+class DbSelect implements AdapterInterface, JsonSerializable
 {
     const ROW_COUNT_COLUMN_NAME = 'C';
 
@@ -150,4 +151,17 @@ protected function getSelectCount()
 
         return $countSelect;
     }
+
+    /**
+     * @internal
+     */
+    public function jsonSerialize()
+    {
+        return [
+            'select'       => $this->sql->buildSqlString($this->select),
+            'count_select' => $this->sql->buildSqlString(
+                $this->getSelectCount()
+            ),
+        ];
+    }
 }

test/PaginatorTest.php

@@ -934,6 +934,56 @@ public function testGetCacheIdWithInheritedClass()
         $this->assertNotEquals($firstOutputGetCacheInternalId, $secondOutputGetCacheInternalId);
     }
 
+    public function testDbSelectAdapterShouldProduceValidCacheId()
+    {
+        // Create first interal cache ID
+        $paginator                    = new Paginator\Paginator(
+            new TestAsset\TestDbSelectAdapter(
+                (new Sql\Select('table1'))
+                    ->where('id = 1')
+                    ->where("foo = 'bar'"),
+                new DbAdapter\Adapter(
+                    new DbAdapter\Driver\Pdo\Pdo(
+                        new DbAdapter\Driver\Pdo\Connection([])
+                    )
+                )
+            )
+        );
+        $reflectionGetCacheInternalId = new ReflectionMethod(
+            $paginator,
+            '_getCacheInternalId'
+        );
+        $reflectionGetCacheInternalId->setAccessible(true);
+        $firstCacheId = $reflectionGetCacheInternalId->invoke(
+            $paginator
+        );
+
+        // Create second internal cache ID
+        $paginator                    = new Paginator\Paginator(
+            new TestAsset\TestDbSelectAdapter(
+                (new Sql\Select('table2'))
+                    ->where('id = 2')
+                    ->where("foo = 'bar'"),
+                new DbAdapter\Adapter(
+                    new DbAdapter\Driver\Pdo\Pdo(
+                        new DbAdapter\Driver\Pdo\Connection([])
+                    )
+                )
+            )
+        );
+        $reflectionGetCacheInternalId = new ReflectionMethod(
+            $paginator,
+            '_getCacheInternalId'
+        );
+        $reflectionGetCacheInternalId->setAccessible(true);
+        $secondCacheIde = $reflectionGetCacheInternalId->invoke(
+            $paginator
+        );
+
+        // Test
+        $this->assertNotEquals($firstCacheId, $secondCacheIde);
+    }
+
     public function testAcceptsComplexAdapters()
     {
         $paginator = new Paginator\Paginator(

test/TestAsset/TestDbSelectAdapter.php

@@ -0,0 +1,30 @@
+<?php
+
+/**
+ * @see       https://github.com/laminas/laminas-paginator for the canonical source repository
+ * @copyright https://github.com/laminas/laminas-paginator/blob/master/COPYRIGHT.md
+ * @license   https://github.com/laminas/laminas-paginator/blob/master/LICENSE.md New BSD License
+ */
+
+namespace LaminasTest\Paginator\TestAsset;
+
+use Laminas\Paginator\Adapter\DbSelect;
+
+class TestDbSelectAdapter extends DbSelect
+{
+    /**
+     * @inheritDoc
+     */
+    public function count()
+    {
+        return 10;
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function getItems($pageNumber, $itemCountPerPage)
+    {
+        return range(1, 10);
+    }
+}