fix: LIMIT & OFFSET are not supported with replacing select count(*)

rekalogika · Sep 7, 2024 · 7118770 · 7118770
1 parent b07d6df
commit 7118770
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,9 @@
 # Changelog
 
+# 0.17.1
+
+* fix: LIMIT & OFFSET are not supported with replacing select count(*)
+
 # 0.17.0
 
 * feat: DBAL adapter

diff --git a/packages/rekapager-doctrine-dbal-adapter/src/QueryBuilderAdapter.php b/packages/rekapager-doctrine-dbal-adapter/src/QueryBuilderAdapter.php
@@ -422,7 +422,11 @@ private function doCount(
 
         $sql = $queryBuilder->getSQL();
 
-        if (!str_contains(strtoupper($sql), 'GROUP BY')) {
+        if (
+            !str_contains(strtoupper($sql), ' GROUP BY ') &&
+            !str_contains(strtoupper($sql), ' LIMIT ') &&
+            !str_contains(strtoupper($sql), ' OFFSET ')
+        ) {
             return $this->doCountWithReplacingSelect($queryBuilder);
         }