Use dedicated repository method to fetch single mod list (#308)

This will: - title
ArmaForces · Jan 8, 2024 · 9ce7f74 · 9ce7f74
2 parents 00dad38 + 64004b2
commit 9ce7f74
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 4 deletions.
diff --git a/src/Api/Controller/GetModListByNameAction.php b/src/Api/Controller/GetModListByNameAction.php
@@ -17,9 +17,7 @@ public function __construct(
 
     public function __invoke(string $name): ?ModList
     {
-        $modList = $this->modListRepository->findOneBy([
-            'name' => $name,
-        ]);
+        $modList = $this->modListRepository->findOneByName($name);
 
         if (!$modList) {
             throw new NotFoundHttpException('Not Found');

diff --git a/src/Controller/ModListPublic/SelectAction.php b/src/Controller/ModListPublic/SelectAction.php
@@ -37,7 +37,7 @@ public function __invoke(): Response
 
         $nextMissionModList = null;
         if ($nextMission) {
-            $nextMissionModList = $this->modListRepository->findOneBy(['name' => $nextMission->getModlist()]);
+            $nextMissionModList = $this->modListRepository->findOneByName($nextMission->getModlist());
         }
 
         return $this->render('mod_list_public/select.html.twig', [

diff --git a/src/Repository/ModList/ModListRepository.php b/src/Repository/ModList/ModListRepository.php
@@ -20,4 +20,9 @@ public function __construct(ManagerRegistry $registry)
     {
         parent::__construct($registry, ModList::class);
     }
+
+    public function findOneByName(string $name): ?ModList
+    {
+        return $this->findOneBy(['name' => $name]);
+    }
 }