Skip to content

Commit ed85c6a

Browse files
author
Bjorn Van Acker
committed
Use MapQueryParameters instead of relying on the request->query to get the page number
1 parent 6c5ffdb commit ed85c6a

File tree

1 file changed

+5
-3
lines changed

1 file changed

+5
-3
lines changed

src/Controller/User/Admin/OverviewController.php

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@
99
use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;
1010
use Symfony\Component\HttpFoundation\Request;
1111
use Symfony\Component\HttpFoundation\Response;
12+
use Symfony\Component\HttpKernel\Attribute\MapQueryParameter;
1213
use Symfony\Component\Routing\Annotation\Route;
1314

1415
class OverviewController extends AbstractController
@@ -17,7 +18,9 @@ class OverviewController extends AbstractController
1718
#[Breadcrumb('users')]
1819
public function __invoke(
1920
Request $request,
20-
UserRepository $userRepository
21+
UserRepository $userRepository,
22+
#[MapQueryParameter]
23+
?int $page = 1
2124
): Response {
2225
$form = $this->createForm(
2326
FilterType::class,
@@ -27,8 +30,7 @@ public function __invoke(
2730
$form->handleRequest($request);
2831

2932
$paginatedUsers = $userRepository->getAllFilteredUsers($form->getData());
30-
31-
$paginatedUsers->paginate($request->query->getInt('page', 1));
33+
$paginatedUsers->paginate($page);
3234

3335
return $this->render('user/admin/overview.html.twig', [
3436
'form' => $form->createView(),

0 commit comments

Comments
 (0)