Skip to content

Commit 3e6bb3e

Browse files
clementvtrdmuchafm
andcommitted
fix: order by walker signature
Co-authored-by: muchafm <[email protected]>
1 parent ac70448 commit 3e6bb3e

File tree

1 file changed

+1
-3
lines changed
  • src/Knp/Component/Pager/Event/Subscriber/Sortable/Doctrine/ORM/Query

1 file changed

+1
-3
lines changed

src/Knp/Component/Pager/Event/Subscriber/Sortable/Doctrine/ORM/Query/OrderByWalker.php

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ class OrderByWalker extends TreeWalkerAdapter
3434
* Walks down a SelectStatement AST node, modifying it to
3535
* sort the query like requested by url
3636
*/
37-
public function walkSelectStatement(SelectStatement $AST): string
37+
public function walkSelectStatement(SelectStatement $AST): void
3838
{
3939
$query = $this->_getQuery();
4040
$fields = (array)$query->getHint(self::HINT_PAGINATOR_SORT_FIELD);
@@ -90,7 +90,5 @@ public function walkSelectStatement(SelectStatement $AST): string
9090
$AST->orderByClause = new OrderByClause([$orderByItem]);
9191
}
9292
}
93-
94-
return '';
9593
}
9694
}

0 commit comments

Comments
 (0)