From 67cc7bf4ed238ec68042ed177afbc0a1e9bbcbd8 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Bj=C3=B6rn=20Fromme?= <fromme@dreipunktnull.com>
Date: Thu, 19 Oct 2023 15:49:26 +0200
Subject: [PATCH] Feat: Add popup window to display teamer info per application

---
 .../Admin/Application/InfoController.php      | 23 ++++++++++++++
 .../Admin/Teamer/InfoController.php           | 20 ++++++++++++
 src/Controller/Common/DownloadController.php  |  8 +++--
 src/Entity/Application.php                    |  4 +--
 .../_assignment_requirements_info.html.twig   | 24 +++++++-------
 templates/admin/application/index.html.twig   | 14 +++++++--
 templates/admin/application/info.html.twig    | 31 +++++++++++++++++++
 templates/admin/document/index.html.twig      |  3 +-
 8 files changed, 107 insertions(+), 20 deletions(-)
 create mode 100644 src/Controller/Admin/Application/InfoController.php
 create mode 100644 src/Controller/Admin/Teamer/InfoController.php
 create mode 100644 templates/admin/application/info.html.twig

diff --git a/src/Controller/Admin/Application/InfoController.php b/src/Controller/Admin/Application/InfoController.php
new file mode 100644
index 0000000..1f97361
--- /dev/null
+++ b/src/Controller/Admin/Application/InfoController.php
@@ -0,0 +1,23 @@
+<?php
+
+namespace App\Controller\Admin\Application;
+
+use App\Entity\Application;
+use App\Model\AjaxModalResponseDto;
+use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;
+use Symfony\Component\HttpFoundation\JsonResponse;
+use Symfony\Component\Routing\Annotation\Route;
+
+class InfoController extends AbstractController
+{
+    #[Route('/admin/application/info/{uuid}', name: 'app_admin_application_info')]
+    public function index(Application $application): JsonResponse
+    {
+        $response = new AjaxModalResponseDto();
+        $response->setContent($this->renderView('admin/application/info.html.twig', [
+            'application' => $application,
+        ]));
+
+        return $this->json($response);
+    }
+}
\ No newline at end of file
diff --git a/src/Controller/Admin/Teamer/InfoController.php b/src/Controller/Admin/Teamer/InfoController.php
new file mode 100644
index 0000000..a02f115
--- /dev/null
+++ b/src/Controller/Admin/Teamer/InfoController.php
@@ -0,0 +1,20 @@
+<?php
+
+namespace App\Controller\Admin\Teamer;
+
+use App\Entity\Teamer;
+use App\Model\AjaxModalResponseDto;
+use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;
+use Symfony\Component\HttpFoundation\JsonResponse;
+use Symfony\Component\Routing\Annotation\Route;
+
+class InfoController extends AbstractController
+{
+    #[Route('/admin/teamer/info/{uuid}', name: 'app_admin_teamer_info')]
+    public function index(Teamer $teamer): JsonResponse
+    {
+        $response = new AjaxModalResponseDto();
+
+        return $this->json($response);
+    }
+}
\ No newline at end of file
diff --git a/src/Controller/Common/DownloadController.php b/src/Controller/Common/DownloadController.php
index 78cf40c..015eb1a 100644
--- a/src/Controller/Common/DownloadController.php
+++ b/src/Controller/Common/DownloadController.php
@@ -5,7 +5,9 @@ namespace App\Controller\Common;
 use App\Entity\Upload;
 use App\Service\Upload\UploadHandler;
 use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;
+use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpFoundation\Response;
+use Symfony\Component\HttpFoundation\ResponseHeaderBag;
 use Symfony\Component\Routing\Annotation\Route;
 use Symfony\Component\Security\Http\Attribute\IsGranted;
 
@@ -17,11 +19,13 @@ class DownloadController extends AbstractController
 
     #[Route('/download/{uuid}', name: 'app_common_download')]
     #[IsGranted('DOWNLOAD', subject: 'upload')]
-    public function index(Upload $upload): Response
+    public function index(Upload $upload, Request $request): Response
     {
         $path = $this->uploadHandler->getUploadFilepath($upload);
         $originalFilename = $upload->getOriginalFilename();
+        $inline = (bool) $request->get('inline');
+        $disposition = $inline ? ResponseHeaderBag::DISPOSITION_INLINE : ResponseHeaderBag::DISPOSITION_ATTACHMENT;
 
-        return $this->file($path, $originalFilename);
+        return $this->file($path, $originalFilename, $disposition);
     }
 }
\ No newline at end of file
diff --git a/src/Entity/Application.php b/src/Entity/Application.php
index bc4e539..3b342d7 100644
--- a/src/Entity/Application.php
+++ b/src/Entity/Application.php
@@ -129,7 +129,7 @@ class Application implements TimestampableEntityInterface
         return $teamer->hasPickup($pickupId);
     }
 
-    public function getCompletedTrainingsCount(): int
+    public function matchesRequiredTrainings(): bool
     {
         $count = 0;
         $requiredTrainings = $this->getAssignment()->getJobProfile()->getRequiredTrainings();
@@ -141,6 +141,6 @@ class Application implements TimestampableEntityInterface
             }
         }
 
-        return $count;
+        return $count === $requiredTrainings->count();
     }
 }
diff --git a/templates/_partials/_assignment_requirements_info.html.twig b/templates/_partials/_assignment_requirements_info.html.twig
index 06327d4..472fc99 100644
--- a/templates/_partials/_assignment_requirements_info.html.twig
+++ b/templates/_partials/_assignment_requirements_info.html.twig
@@ -1,24 +1,24 @@
-<ul class="list-disc pl-4 pb-4">
-    {% if assignment.pickup and assignment.effectivePickupDate %}
-        <li>
-            <div class="flex items-center space-x-2">
-                <span>Busbegleitung ab {{ assignment.pickup|bpn_pickup_label }} am {{ assignment.effectivePickupDate|date('d.m.Y') }}</span>
+<ul class="pb-8 divide-y divide-gray-200">
+{% if assignment.pickup and assignment.effectivePickupDate %}
+        <li class="py-2">
+            <div class="flex items-center justify-between">
+                <span class="flex-1">Busbegleitung {{ assignment.pickup|bpn_pickup_label }} am {{ assignment.effectivePickupDate|date('d.m.Y') }}</span>
                 {% if teamer.hasPickup(assignment.pickup) %}
-                    {{ icon('check', 'w-5 h-5 text-emerald-500') }}
+                    {{ icon('check', 'w-5 h-5 text-green-500 shrink-0') }}
                 {% else %}
-                    {{ icon('close', 'w-5 h-5 text-red-500') }}
+                    {{ icon('close', 'w-5 h-5 text-red-500 shrink-0') }}
                 {% endif %}
             </div>
         </li>
     {% endif %}
     {% for training in assignment.jobProfile.requiredTrainings %}
-        <li>
-            <div class="flex items-center space-x-2">
-                <span>{{ training.name }} absolviert</span>
+        <li class="py-2">
+            <div class="flex items-center justify-between">
+                <span class="flex-1">{{ training.name }} absolviert</span>
                 {% if teamer.trainingAttendance(training) %}
-                    {{ icon('check', 'w-5 h-5 text-emerald-500') }}
+                    {{ icon('check', 'w-5 h-5 text-green-500 shrink-0') }}
                 {% else %}
-                    {{ icon('close', 'w-5 h-5 text-red-500') }}
+                    {{ icon('close', 'w-5 h-5 text-red-500 shrink-0') }}
                 {% endif %}
             </div>
         </li>
diff --git a/templates/admin/application/index.html.twig b/templates/admin/application/index.html.twig
index 1073b37..5f17543 100644
--- a/templates/admin/application/index.html.twig
+++ b/templates/admin/application/index.html.twig
@@ -64,7 +64,15 @@
                                     </a>
                                 </td>
                                 <td>
-                                    <div class="flex items-center justify-end space-x-1">
+                                    <div class="flex items-center justify-end space-x-1"
+                                         role="button"
+                                         title="Bewerber:innen-Info anzeigen"
+                                         aria-label="Bewerber:innen-Info anzeigen"
+                                         {{ stimulus_controller('modal-button', [], [], {'ajax-modal': '#ajax-modal'}) }}
+                                         {{ stimulus_action('modal-button', 'ajax', 'click', {
+                                             'title': 'Bewerber:innen-Info',
+                                             'url': path('app_admin_application_info', { 'uuid': application.uuid })
+                                         }) }}>
                                         {% if assignment.pickup %}
                                             {% if application.matchesPickup %}
                                                 {{ icon('bus', 'w-5 h-5 text-green-500 shrink-0') }}
@@ -73,8 +81,8 @@
                                             {% endif %}
                                         {% endif %}
                                         {% set requiredTrainingsCount = assignment.jobProfile.requiredTrainings|length %}
-                                        {% if requiredTrainingsCount > 0 %}
-                                            <span>{{ application.completedTrainingsCount }}/{{ requiredTrainingsCount }}</span>
+                                        {% if not application.matchesRequiredTrainings  %}
+                                            {{ icon('alert', 'w-5 h-5 text-red-500 shrink-0') }}
                                         {% endif %}
                                     </div>
                                 </td>
diff --git a/templates/admin/application/info.html.twig b/templates/admin/application/info.html.twig
new file mode 100644
index 0000000..13cffdf
--- /dev/null
+++ b/templates/admin/application/info.html.twig
@@ -0,0 +1,31 @@
+<h3 class="text-xl font-bold pb-4">
+    {{ application.teamer }}
+</h3>
+{% include '_partials/_assignment_requirements_info.html.twig' with {
+    'assignment': application.assignment,
+    'teamer': application.teamer
+} %}
+<ul class="pb-8 divide-y divide-gray-200">
+    {% for type in ['ski', 'snowboard'] %}
+        <li class="py-2">
+            <div class="flex items-center justify-between">
+                <span class="flex-1">{{ type|license_label }}</span>
+                {% set license = application.teamer.licenseOfType(type) %}
+                {% if license %}
+                    <div class="flex items-center space-x-2">
+                        <span>{{ license.date|date('m/Y') }}</span>
+                        {% if is_granted('DOWNLOAD', license.certificate) %}
+                            <a href="{{ path('app_common_download', { 'uuid': license.certificate.uuid, 'inline': true }) }}"
+                               target="_blank"
+                               title="Download Nachweis">
+                                {{ icon('download', 'w-4 h-4') }}
+                            </a>
+                        {% endif %}
+                    </div>
+                {% else %}
+                    {{ icon('minus', 'w-4 h-4') }}
+                {% endif %}
+            </div>
+        </li>
+    {% endfor %}
+</ul>
diff --git a/templates/admin/document/index.html.twig b/templates/admin/document/index.html.twig
index 55bb1d2..ed6b7c6 100644
--- a/templates/admin/document/index.html.twig
+++ b/templates/admin/document/index.html.twig
@@ -71,7 +71,8 @@
                                                 {{ icon('edit') }}
                                             </button>
                                         {% endif %}
-                                        <a href="{{ path('app_common_download', { 'uuid': upload.uuid }) }}"
+                                        <a href="{{ path('app_common_download', { 'uuid': upload.uuid, 'inline': true }) }}"
+                                           target="_blank"
                                            title="Download">
                                             {{ icon('download') }}
                                         </a>