Merge pull request #329 from KnpLabs/fix/order-by-walker-signature

fix: order by walker signature

phpstan.neon

@@ -2,5 +2,3 @@ parameters:
     level: 5
     paths:
         - src
-includes:
-    - phpstan-baseline.neon

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

@@ -34,7 +34,7 @@ class OrderByWalker extends TreeWalkerAdapter
      * Walks down a SelectStatement AST node, modifying it to
      * sort the query like requested by url
      */
-    public function walkSelectStatement(SelectStatement $AST): string
+    public function walkSelectStatement(SelectStatement $AST): void
     {
         $query = $this->_getQuery();
         $fields = (array)$query->getHint(self::HINT_PAGINATOR_SORT_FIELD);
@@ -90,7 +90,5 @@ public function walkSelectStatement(SelectStatement $AST): string
                 $AST->orderByClause = new OrderByClause([$orderByItem]);
             }
         }
-
-        return '';
     }
 }