From 4dd81da1efc767dee5d7c876c3a3988a4167a85c Mon Sep 17 00:00:00 2001
From: Julie Lenaerts <julielenaerts@gmail.com>
Date: Fri, 27 Jan 2023 11:23:29 +0100
Subject: [PATCH] FIX [routes][redirect] fix route naming and redirection after
 delete action

---
 .../Controller/AbsenceController.php               | 14 ++++++--------
 .../Resources/views/Menu/absence.html.twig         |  2 +-
 .../Routing/MenuBuilder/UserMenuBuilder.php        |  2 +-
 3 files changed, 8 insertions(+), 10 deletions(-)

diff --git a/src/Bundle/ChillMainBundle/Controller/AbsenceController.php b/src/Bundle/ChillMainBundle/Controller/AbsenceController.php
index 9c3dc679d..dcd5e2337 100644
--- a/src/Bundle/ChillMainBundle/Controller/AbsenceController.php
+++ b/src/Bundle/ChillMainBundle/Controller/AbsenceController.php
@@ -21,7 +21,7 @@ class AbsenceController extends AbstractController
     /**
      * @Route(
      *     "/{_locale}/absence",
-     *     name="chill_absence_user",
+     *     name="chill_main_user_absence_index",
      *     methods={"GET", "POST"}
      * )
      */
@@ -37,7 +37,7 @@ class AbsenceController extends AbstractController
             $em->persist($user);
             $em->flush();
 
-            return $this->redirect($this->generateUrl('chill_absence_user'));
+            return $this->redirect($this->generateUrl('chill_main_user_absence_index'));
         }
 
         return $this->render('@ChillMain/Menu/absence.html.twig', [
@@ -49,8 +49,8 @@ class AbsenceController extends AbstractController
     /**
      * @Route(
      *     "/{_locale}/absence/unset",
-     *     name="chill_unset_absence_user",
-     *     methods={"GET", "POST", "DELETE"}
+     *     name="chill_main_user_absence_unset",
+     *     methods={"GET", "POST"}
      * )
      */
     public function unsetAbsence(Request $request)
@@ -63,9 +63,7 @@ class AbsenceController extends AbstractController
         $em->persist($user);
         $em->flush();
 
-        return $this->render('@ChillMain/Menu/absence.html.twig', [
-            'user' => $user,
-            'form' => $form->createView(),
-        ]);
+        return $this->redirect($this->generateUrl('chill_main_user_absence_index'));
+        
     }
 }
diff --git a/src/Bundle/ChillMainBundle/Resources/views/Menu/absence.html.twig b/src/Bundle/ChillMainBundle/Resources/views/Menu/absence.html.twig
index 887894ae9..3258b5ac1 100644
--- a/src/Bundle/ChillMainBundle/Resources/views/Menu/absence.html.twig
+++ b/src/Bundle/ChillMainBundle/Resources/views/Menu/absence.html.twig
@@ -14,7 +14,7 @@
                 <p>{{ 'absence.You are listed as absent, as of'|trans }} {{ user.absenceStart|format_date('long') }}</p>
                 <ul class="record_actions sticky-form-buttons">
                     <li>
-                        <a href="{{ path('chill_unset_absence_user') }}"
+                        <a href="{{ path('chill_main_user_absence_unset') }}"
                            class="btn btn-delete">{{ 'absence.Unset absence'|trans }}</a>
                     </li>
                 </ul>
diff --git a/src/Bundle/ChillMainBundle/Routing/MenuBuilder/UserMenuBuilder.php b/src/Bundle/ChillMainBundle/Routing/MenuBuilder/UserMenuBuilder.php
index e91aa040b..ea5ed2062 100644
--- a/src/Bundle/ChillMainBundle/Routing/MenuBuilder/UserMenuBuilder.php
+++ b/src/Bundle/ChillMainBundle/Routing/MenuBuilder/UserMenuBuilder.php
@@ -81,7 +81,7 @@ class UserMenuBuilder implements LocalMenuBuilderInterface
             //TODO add an icon? How exactly? For example a clock icon...
             $menu
                 ->addChild($this->translator->trans('absence.Set absence date'), [
-                    'route' => 'chill_absence_user',
+                    'route' => 'chill_main_user_absence_index',
                 ])
                 ->setExtras([
                     'order' => -8888888,