feat(pagination): add pagination to list controllers

fix merge conflicts

composer update to fix composer synchro between lock and json

fix(ci): remove --configuration for phpstan as it is deprecated

Author: olivmai

.github/workflows/ci.yml

@@ -102,7 +102,7 @@ jobs:
               if: always()
 
             - name: Run PHPStan
-              run: vendor/bin/phpstan analyse --configuration phpstan.neon --level max --no-progress . --error-format=checkstyle | cs2pr
+              run: vendor/bin/phpstan analyse --level max --no-progress . --error-format=checkstyle | cs2pr
               if: always()
 
             - name: Bootstrap database

composer.json

@@ -11,6 +11,7 @@
     "beberlei/assert": "^3.2",
     "doctrine/annotations": "^1.8",
     "doctrine/doctrine-fixtures-bundle": "^3.3",
+    "knplabs/knp-paginator-bundle": "^5.2",
     "friendsofsymfony/jsrouting-bundle": "^2.5",
     "martin-georgiev/postgresql-for-doctrine": "^1.3",
     "odolbeau/phone-number-bundle": "^3.0",

composer.lock

@@ -21,6 +21,7 @@
     Fidry\AliceDataFixtures\Bridge\Symfony\FidryAliceDataFixturesBundle::class => ['dev' => true, 'test' => true, 'panther' => true],
     Hautelook\AliceBundle\HautelookAliceBundle::class => ['dev' => true, 'test' => true, 'panther' => true],
     Misd\PhoneNumberBundle\MisdPhoneNumberBundle::class => ['all' => true],
+    Knp\Bundle\PaginatorBundle\KnpPaginatorBundle::class => ['all' => true],
     Symfony\Bundle\DebugBundle\DebugBundle::class => ['dev' => true, 'test' => true],
     FOS\JsRoutingBundle\FOSJsRoutingBundle::class => ['all' => true],
 ];

config/bundles.php

@@ -0,0 +1,3 @@
+knp_paginator:
+    template:
+        pagination: '@KnpPaginator/Pagination/twitter_bootstrap_v4_pagination.html.twig'

config/packages/knp_paginator.yaml

@@ -1,6 +1,7 @@
 # Put parameters here that don't need to change on each machine where the app is deployed
 # https://symfony.com/doc/current/best_practices/configuration.html#application-related-configuration
 parameters:
+    app.pagination_default_limit: 30
     app.most_important_skills_limit: 6 # used for separing data in the planning
     app.most_important_skills_to_display: [ch_vpsp, ch_vl] # used for displaying a badge near the user name
     app.slot_interval: '%env(APP_SLOT_INTERVAL)%'

config/parameters.yaml

@@ -1,5 +1,64 @@
 # This file is the entry point to configure your own services.
 # Files in the packages/ subdirectory configure your dependencies.
+# Put parameters here that don't need to change on each machine where the app is deployed
+# https://symfony.com/doc/current/best_practices/configuration.html#application-related-configuration
+parameters:
+    app.pagination_default_limit: 30
+    app.most_important_skills_limit: 6 # used for separing data in the planning
+    app.most_important_skills_to_display: [ch_vpsp, ch_vl] # used for displaying a badge near the user name
+    app.slot_interval: '%env(APP_SLOT_INTERVAL)%'
+    app.available_skills:
+        # Caution: the order matters for the planning display
+        ci_bspp:
+            label: CI Réseau BSPP
+            includes: [ci_samu]
+        ci_samu:
+            label: CI Réseau Samu
+            includes: [ci]
+        ch_vpsp:
+            label: Chauffeur VPSP
+            includes: [pse2, ch_vl]
+        ci:
+            label: CI
+            includes: [pse2]
+        pse2:
+            label: PSE2
+            includes: [pse1]
+        pse1:
+            label: PSE1
+            includes: [psc1]
+        ch_vl:
+            label: Chauffeur VL
+            includes: [benevole]
+        infirmier:
+            label: Infirmier.e local.e
+            includes: [benevole]
+        ce_maraude:
+            label: Chef.fe d'équipe maraudes
+            includes: [solidar]
+        solidar:
+            label: Solidar
+            includes: [maraudeur]
+        maraudeur:
+            label: Maraudeur.se
+            includes: [benevole]
+        tsa_coreg:
+            label: TSA / Coreg
+            includes: [pse2]
+        logisticien:
+            label: Logisticien.ne Croix-Rouge
+            includes: [benevole]
+        resp_perm_soc:
+            label: Responsable Permanence Sociale
+            includes: [benevole]
+        anim_fle:
+            label: Animateur.rice de cours de FLE
+            includes: [benevole]
+        psc1:
+            label: PSC1
+            includes: [benevole]
+        benevole:
+            label: Bénévole
 
 services:
     # default configuration for services in *this* file

config/services.yaml

@@ -6,8 +6,10 @@
 
 use App\Entity\Organization;
 use App\Repository\AssetTypeRepository;
+use Knp\Component\Pager\PaginatorInterface;
 use Sensio\Bundle\FrameworkExtraBundle\Configuration\Security;
 use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;
+use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpFoundation\Response;
 use Symfony\Component\Routing\Annotation\Route;
 
@@ -18,19 +20,26 @@
 class AssetTypeListController extends AbstractController
 {
     private AssetTypeRepository $assetTypeRepository;
+    private PaginatorInterface $paginator;
 
-    public function __construct(AssetTypeRepository $assetTypeRepository)
+    public function __construct(AssetTypeRepository $assetTypeRepository, PaginatorInterface $paginator)
     {
         $this->assetTypeRepository = $assetTypeRepository;
+        $this->paginator = $paginator;
     }
 
-    public function __invoke(): Response
+    public function __invoke(Request $request): Response
     {
         /** @var Organization $organization */
         $organization = $this->getUser();
+        $assetTypes = $this->paginator->paginate(
+            $this->assetTypeRepository->findByOrganization($organization),
+            $request->query->getInt('page', 1),
+            $this->getParameter('app.pagination_default_limit')
+        );
 
         return $this->render('organization/assetType/list.html.twig', [
-            'assetTypes' => $this->assetTypeRepository->findByOrganization($organization),
+            'assetTypes' => $assetTypes,
         ]);
     }
 }

src/Controller/Organization/AssetType/AssetTypeListController.php

@@ -6,7 +6,9 @@
 
 use App\Entity\Organization;
 use App\Repository\OrganizationRepository;
+use Knp\Component\Pager\PaginatorInterface;
 use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;
+use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpFoundation\Response;
 use Symfony\Component\Routing\Annotation\Route;
 use Symfony\Component\Security\Core\Exception\AccessDeniedException;
@@ -17,20 +19,26 @@
 class ListController extends AbstractController
 {
     protected OrganizationRepository $organizationRepository;
+    private PaginatorInterface $paginator;
 
-    public function __construct(OrganizationRepository $organizationRepository)
+    public function __construct(OrganizationRepository $organizationRepository, PaginatorInterface $paginator)
     {
         $this->organizationRepository = $organizationRepository;
+        $this->paginator = $paginator;
     }
 
-    public function __invoke(): Response
+    public function __invoke(Request $request): Response
     {
         $organization = $this->getUser();
         if (!$organization instanceof Organization || !$organization->isParent()) {
             throw new AccessDeniedException();
         }
 
-        $organizations = $this->organizationRepository->findBy(['parent' => $organization], ['name' => 'ASC']);
+        $organizations = $this->paginator->paginate(
+            $this->organizationRepository->findChildrenQueryBuilder($organization),
+            $request->query->getInt('page', 1),
+            $this->getParameter('app.pagination_default_limit')
+        );
 
         return $this->render('organization/list.html.twig', [
             'organizations' => $organizations,

src/Controller/Organization/Children/ListController.php

@@ -8,6 +8,7 @@
 use App\Form\Factory\OrganizationSelectorFormFactory;
 use App\Repository\CommissionableAssetRepository;
 use App\Security\Voter\OrganizationVoter;
+use Knp\Component\Pager\PaginatorInterface;
 use Sensio\Bundle\FrameworkExtraBundle\Configuration\IsGranted;
 use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;
 use Symfony\Component\HttpFoundation\Request;
@@ -22,23 +23,30 @@ class AssetsListController extends AbstractController
 {
     private CommissionableAssetRepository $assetRepository;
     private OrganizationSelectorFormFactory $organizationSelectorFormFactory;
+    private PaginatorInterface $paginator;
 
-    public function __construct(CommissionableAssetRepository $assetRepository, OrganizationSelectorFormFactory $organizationSelectorFormFactory)
+    public function __construct(CommissionableAssetRepository $assetRepository, OrganizationSelectorFormFactory $organizationSelectorFormFactory, PaginatorInterface $paginator)
     {
         $this->assetRepository = $assetRepository;
         $this->organizationSelectorFormFactory = $organizationSelectorFormFactory;
+        $this->paginator = $paginator;
     }
 
     public function __invoke(Request $request, Organization $organization): Response
     {
         /** @var Organization $currentOrganization */
         $currentOrganization = $this->getUser();
+        $assets = $this->paginator->paginate(
+            $this->assetRepository->findByOrganizationAndChildrenQb($organization),
+            $request->query->getInt('page', 1),
+            $this->getParameter('app.pagination_default_limit')
+        );
 
         return $this->render(
             'organization/commissionable_asset/list.html.twig',
             [
                 'organization' => $organization,
-                'assets' => $this->assetRepository->findByOrganization($organization),
+                'assets' => $assets,
                 'organization_selector_form' => $this->organizationSelectorFormFactory->createForm(
                     $organization,
                     $currentOrganization