Merge pull request #150 from mulcek/patch-2

DBALQueryBuilderSubscriber orderBy query part is removed at count query
KnpLabs · Dec 6, 2016 · c91dddb · c91dddb
2 parents 6cedcd4 + 1ff05ba
commit c91dddb
Showing 1 changed file with 10 additions and 4 deletions.
diff --git a/src/Knp/Component/Pager/Event/Subscriber/Paginate/Doctrine/DBALQueryBuilderSubscriber.php b/src/Knp/Component/Pager/Event/Subscriber/Paginate/Doctrine/DBALQueryBuilderSubscriber.php
@@ -19,11 +19,17 @@ public function items(ItemsEvent $event)
             /** @var $target QueryBuilder */
             $target = $event->target;
 
-            // get the query
-            $sql = $target->getSQL();
-
             // count results
             $qb = clone $target;
+
+            //reset count orderBy since it can break query and slow it down
+            $qb
+                ->resetQueryPart('orderBy')
+            ;
+
+            // get the query
+            $sql = $qb->getSQL();
+
             $qb
                 ->resetQueryParts()
                 ->select('count(*) as cnt')
@@ -60,4 +66,4 @@ public static function getSubscribedEvents()
             'knp_pager.items' => array('items', 10 /*make sure to transform before any further modifications*/)
         );
     }
-}
+}