[SCOUT] Make signup approval work again

The emails were broken because divisions can have multiple contacts now, and the list of signups did not properly filter on managed divisions.
roodjong · May 21, 2024 · a6b15fb · a6b15fb
1 parent a47bab0
commit a6b15fb
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 13 deletions.
diff --git a/src/Controller/Admin/DashboardController.php b/src/Controller/Admin/DashboardController.php
@@ -58,7 +58,7 @@ public function configureMenuItems(): iterable
             MenuItem::section('Administratie'),
             MenuItem::linkToCrud('Leden', 'fa fa-users', Member::class),
             MenuItem::linkToCrud('Steunleden', 'fa fa-users', SupportMember::class)->setPermission('ROLE_ADMIN'),
-            MenuItem::linkToCrud('Aanmeldingen', 'fa fa-user-plus', MembershipApplication::class)->setPermission('ROLE_ADMIN'),
+            MenuItem::linkToCrud('Aanmeldingen', 'fa fa-user-plus', MembershipApplication::class),
             MenuItem::linkToCrud('Groepen', 'fa fa-building', Division::class)->setPermission('ROLE_ADMIN'),
             MenuItem::linkToCrud('Lidmaatschapstypes', 'fa fa-building', MembershipStatus::class)->setPermission('ROLE_ADMIN'),
 

diff --git a/src/Controller/Admin/MembershipApplicationCrud.php b/src/Controller/Admin/MembershipApplicationCrud.php
@@ -40,11 +40,11 @@ public function __construct(MailerInterface $mailer, MollieApiClient $mollieApiC
     public function createIndexQueryBuilder(SearchDto $searchDto, EntityDto $entityDto, FieldCollection $fields, FilterCollection $filters): QueryBuilder
     {
         $response = $this->get(EntityRepository::class)->createQueryBuilder($searchDto, $entityDto, $fields, $filters);
-        if (in_array('ROLE_ADMIN', $this->getUser()->getRoles(), true)) {
-            return $response;
+
+        if (!in_array('ROLE_ADMIN', $this->getUser()->getRoles(), true)) {
+            $response->andWhere('entity.preferredDivision IN (:division)')->setParameter('division', $this->getUser()->getManagedDivisions());
         }
-        $division = $this->getUser()->getDivision();
-        $response->andWhere('entity.preferredDivision = :division')->setParameter('division', $division);
+
         return $response;
     }
 
@@ -157,11 +157,13 @@ public function acceptApplication(AdminContext $context)
                     ->from(new Address($noreply, $organizationName))
                     ->html(
                         $this->renderView($templatePrefix . 'email/html/contact_new_member.html.twig', [
+                            'contact' => $contact,
                             'member' => $member,
                         ]),
                     )
                     ->text(
                         $this->renderView($templatePrefix . 'email/text/contact_new_member.txt.twig', [
+                            'contact' => $contact,
                             'member' => $member,
                         ]),
                     );

diff --git a/templates/email/html/contact_new_member.html.twig b/templates/email/html/contact_new_member.html.twig
@@ -1,17 +1,15 @@
 {% extends 'email/html/layout.html.twig' %}
-{% set url = url('division', { division: member.division.name }) %}
 {% block content %}
 <p>
-    Beste {{ member.division.contact.firstName }} {{ member.division.contact.lastName }},
+    Beste {{ contact.getFullName }},
 </p>
 <p>
     Onlangs heeft een nieuw lid zich aangemeld bij {{ organisatienaam }}: {{ member.getFullName }}.
     Hun lidmaatschap is zojuist goedgekeurd door het bestuur van {{ organisatienaam }}. Bij het aanmelden heeft dit lid aangegeven
     zich graag aan te sluiten bij de groep {{ member.division.name }}.
 </p>
 <p>
-    Hun gegevens zijn toegevoegd aan het ledenoverzicht in het ledenadministratiesysteem. Klik op de onderstaande link om het te openen.
-    <a href="{{ url }}">{{ url }}</a>
+    Hun gegevens zijn toegevoegd aan het ledenoverzicht in het ledenadministratiesysteem.
 </p>
 <p>
     Kameraadschappelijke groet,

diff --git a/templates/email/text/contact_new_member.txt.twig b/templates/email/text/contact_new_member.txt.twig
@@ -1,13 +1,11 @@
 r% extends 'email/text/layout.txt.twig' %}
-{% set url = url('division', { division: member.division.name }) %}
 {% block content %}
-Beste {{ member.division.contact.firstName }} {{ member.division.contact.lastName }},
+Beste {{ contact.getFullName }},
 
 Onlangs heeft een nieuw lid zich aangemeld bij {{ organisatienaam }}: {{ member.getFullName }}.
 Hun lidmaatschap is zojuist goedgekeurd door het bestuur van {{ organisatienaam }}. Bij het aanmelden heeft dit lid aangegeven zich graag aan te sluiten bij de groep {{ member.division.name }}.
 
-Hun gegevens zijn toegevoegd aan het ledenoverzicht in het ledenadministratiesysteem. Klik op de onderstaande link om het te openen.
-{{ url }}
+Hun gegevens zijn toegevoegd aan het ledenoverzicht in het ledenadministratiesysteem.
 
 Kameraadschappelijke groet,