Refactor controllers using ADR pattern (#272)

This will:
- title
- change router and security annotations to attributes

src/Controller/Dlc/CreateAction.php

@@ -0,0 +1,47 @@
+<?php
+
+declare(strict_types=1);
+
+namespace App\Controller\Dlc;
+
+use App\Form\DataTransformerRegistry;
+use App\Form\Dlc\DlcFormType;
+use App\Form\Dlc\Dto\DlcFormDto;
+use App\Security\Enum\PermissionsEnum;
+use Doctrine\ORM\EntityManagerInterface;
+use Sensio\Bundle\FrameworkExtraBundle\Configuration\IsGranted;
+use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;
+use Symfony\Component\HttpFoundation\Request;
+use Symfony\Component\HttpFoundation\Response;
+use Symfony\Component\Routing\Annotation\Route;
+
+class CreateAction extends AbstractController
+{
+    public function __construct(
+        private EntityManagerInterface $entityManager,
+        private DataTransformerRegistry $dataTransformerRegistry
+    ) {
+    }
+
+    #[Route('/dlc/create', name: 'app_dlc_create')]
+    #[IsGranted(PermissionsEnum::DLC_CREATE)]
+    public function __invoke(Request $request): Response
+    {
+        $dlcFormDto = new DlcFormDto();
+        $form = $this->createForm(DlcFormType::class, $dlcFormDto);
+        $form->handleRequest($request);
+
+        if ($form->isSubmitted() && $form->isValid()) {
+            $dlc = $this->dataTransformerRegistry->transformToEntity($dlcFormDto);
+
+            $this->entityManager->persist($dlc);
+            $this->entityManager->flush();
+
+            return $this->redirectToRoute('app_dlc_list');
+        }
+
+        return $this->render('dlc/form.html.twig', [
+            'form' => $form->createView(),
+        ]);
+    }
+}

src/Controller/Dlc/DeleteAction.php

@@ -0,0 +1,31 @@
+<?php
+
+declare(strict_types=1);
+
+namespace App\Controller\Dlc;
+
+use App\Entity\Dlc\Dlc;
+use App\Security\Enum\PermissionsEnum;
+use Doctrine\ORM\EntityManagerInterface;
+use Sensio\Bundle\FrameworkExtraBundle\Configuration\IsGranted;
+use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;
+use Symfony\Component\HttpFoundation\Response;
+use Symfony\Component\Routing\Annotation\Route;
+
+class DeleteAction extends AbstractController
+{
+    public function __construct(
+        private EntityManagerInterface $entityManager,
+    ) {
+    }
+
+    #[Route('/dlc/{id}/delete', name: 'app_dlc_delete')]
+    #[IsGranted(PermissionsEnum::DLC_DELETE, 'dlc')]
+    public function __invoke(Dlc $dlc): Response
+    {
+        $this->entityManager->remove($dlc);
+        $this->entityManager->flush();
+
+        return $this->redirectToRoute('app_dlc_list');
+    }
+}

src/Controller/Dlc/ListAction.php

@@ -0,0 +1,31 @@
+<?php
+
+declare(strict_types=1);
+
+namespace App\Controller\Dlc;
+
+use App\Repository\Dlc\DlcRepository;
+use App\Security\Enum\PermissionsEnum;
+use Sensio\Bundle\FrameworkExtraBundle\Configuration\IsGranted;
+use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;
+use Symfony\Component\HttpFoundation\Response;
+use Symfony\Component\Routing\Annotation\Route;
+
+class ListAction extends AbstractController
+{
+    public function __construct(
+        private DlcRepository $dlcRepository
+    ) {
+    }
+
+    #[Route('/dlc/list', name: 'app_dlc_list')]
+    #[IsGranted(PermissionsEnum::DLC_LIST)]
+    public function __invoke(): Response
+    {
+        $dlcs = $this->dlcRepository->findBy([], ['name' => 'ASC']);
+
+        return $this->render('dlc/list.html.twig', [
+            'dlcs' => $dlcs,
+        ]);
+    }
+}

src/Controller/Dlc/UpdateAction.php

@@ -0,0 +1,47 @@
+<?php
+
+declare(strict_types=1);
+
+namespace App\Controller\Dlc;
+
+use App\Entity\Dlc\Dlc;
+use App\Form\DataTransformerRegistry;
+use App\Form\Dlc\DlcFormType;
+use App\Form\Dlc\Dto\DlcFormDto;
+use App\Security\Enum\PermissionsEnum;
+use Doctrine\ORM\EntityManagerInterface;
+use Sensio\Bundle\FrameworkExtraBundle\Configuration\IsGranted;
+use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;
+use Symfony\Component\HttpFoundation\Request;
+use Symfony\Component\HttpFoundation\Response;
+use Symfony\Component\Routing\Annotation\Route;
+
+class UpdateAction extends AbstractController
+{
+    public function __construct(
+        private EntityManagerInterface $entityManager,
+        private DataTransformerRegistry $dataTransformerRegistry
+    ) {
+    }
+
+    #[Route('/dlc/{id}/update', name: 'app_dlc_update')]
+    #[IsGranted(PermissionsEnum::DLC_UPDATE, 'dlc')]
+    public function __invoke(Request $request, Dlc $dlc): Response
+    {
+        $dlcFormDto = $this->dataTransformerRegistry->transformFromEntity(new DlcFormDto(), $dlc);
+        $form = $this->createForm(DlcFormType::class, $dlcFormDto);
+        $form->handleRequest($request);
+
+        if ($form->isSubmitted() && $form->isValid()) {
+            $this->dataTransformerRegistry->transformToEntity($dlcFormDto, $dlc);
+
+            $this->entityManager->flush();
+
+            return $this->redirectToRoute('app_dlc_list');
+        }
+
+        return $this->render('dlc/form.html.twig', [
+            'form' => $form->createView(),
+        ]);
+    }
+}

src/Controller/Home/IndexAction.php

@@ -0,0 +1,35 @@
+<?php
+
+declare(strict_types=1);
+
+namespace App\Controller\Home;
+
+use App\Service\Mission\MissionClientInterface;
+use Psr\Log\LoggerInterface;
+use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;
+use Symfony\Component\HttpFoundation\Response;
+use Symfony\Component\Routing\Annotation\Route;
+
+class IndexAction extends AbstractController
+{
+    public function __construct(
+        private MissionClientInterface $missionClient,
+        private LoggerInterface $logger
+    ) {
+    }
+
+    #[Route('/', name: 'app_home_index')]
+    public function __invoke(): Response
+    {
+        try {
+            $nearestMission = $this->missionClient->getNextUpcomingMission();
+        } catch (\Exception $ex) {
+            $this->logger->warning('Could not fetch nearest mission', ['ex' => $ex]);
+            $nearestMission = null;
+        }
+
+        return $this->render('home/index/index.html.twig', [
+            'nearestMission' => $nearestMission,
+        ]);
+    }
+}

src/Controller/Home/JoinUsAction.php

@@ -0,0 +1,18 @@
+<?php
+
+declare(strict_types=1);
+
+namespace App\Controller\Home;
+
+use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;
+use Symfony\Component\HttpFoundation\Response;
+use Symfony\Component\Routing\Annotation\Route;
+
+class JoinUsAction extends AbstractController
+{
+    #[Route('/join-us', name: 'app_home_join_us')]
+    public function __invoke(): Response
+    {
+        return $this->render('home/join_us/join_us.html.twig');
+    }
+}

src/Controller/HomeController.php → src/Controller/Home/MissionsAction.php

@@ -2,54 +2,24 @@
 
 declare(strict_types=1);
 
-namespace App\Controller;
+namespace App\Controller\Home;
 
 use App\Service\Mission\MissionClientInterface;
 use Psr\Log\LoggerInterface;
 use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;
 use Symfony\Component\HttpFoundation\Response;
 use Symfony\Component\Routing\Annotation\Route;
 
-/**
- * @Route("/", name="app_home")
- */
-class HomeController extends AbstractController
+class MissionsAction extends AbstractController
 {
     public function __construct(
         private MissionClientInterface $missionClient,
         private LoggerInterface $logger
     ) {
     }
 
-    /**
-     * @Route("", name="_index")
-     */
-    public function indexAction(): Response
-    {
-        try {
-            $nearestMission = $this->missionClient->getNextUpcomingMission();
-        } catch (\Exception $ex) {
-            $this->logger->warning('Could not fetch nearest mission', ['ex' => $ex]);
-            $nearestMission = null;
-        }
-
-        return $this->render('home/index/index.html.twig', [
-            'nearestMission' => $nearestMission,
-        ]);
-    }
-
-    /**
-     * @Route("/join-us", name="_join_us")
-     */
-    public function joinUsAction(): Response
-    {
-        return $this->render('home/join_us/join_us.html.twig');
-    }
-
-    /**
-     * @Route("/missions", name="_missions")
-     */
-    public function missionsAction(): Response
+    #[Route('/missions', name: 'app_home_missions')]
+    public function __invoke(): Response
     {
         try {
             $upcomingMissions = $this->missionClient->getUpcomingMissions();