Merge pull request #394 from itk-dev/feature/SUPP0RT-1234-logout-func…

…tionality

#1234: Added logout functionality

CHANGELOG.md

@@ -10,6 +10,8 @@ about writing changes to this log.
 
 ## [Unreleased]
 
+- [PR-394](https://github.com/itk-dev/naevnssekretariatet/pull/394)
+  Added logout functionality.
 - [PR-393](https://github.com/itk-dev/naevnssekretariatet/pull/393)
   Fixed `BoardRole` crud search property.
 - [PR-392](https://github.com/itk-dev/naevnssekretariatet/pull/392)

config/packages/security.yaml

@@ -19,6 +19,8 @@ security:
                 - ItkDev\OpenIdConnectBundle\Security\CliLoginTokenAuthenticator
                 - App\Security\AzureAdLoginAuthenticator
             entry_point: App\Security\AzureAdLoginAuthenticator
+            logout:
+                path: app_logout
 
             # activate different ways to authenticate
             # https://symfony.com/doc/current/security.html#firewalls-authentication

src/Controller/Admin/DashboardController.php

@@ -86,7 +86,7 @@ public function configureActions(): Actions
     public function configureUserMenu(UserInterface $user): UserMenu
     {
         // See AbstractDashboardController configureUserMenu method, here with unnecessary features such as logout removed
-        return UserMenu::new()
+        return parent::configureUserMenu($user)
             ->displayUserName()
             ->setName($user->getName())
         ;

src/Controller/DefaultController.php

@@ -92,4 +92,13 @@ public function redirectToUserSettings(AdminUrlGenerator $urlGenerator): Respons
 
         return $this->redirect($url);
     }
+
+    /**
+     * @Route("/logout", name="app_logout")
+     */
+    public function logout(): void
+    {
+        // This will never be called so can be blank,
+        // cf. https://symfony.com/doc/5.x/security.html#logging-out
+    }
 }

templates/navbar/_menu.html.twig

@@ -29,6 +29,7 @@
         <div class="dropdown-menu dropdown-menu-right" aria-labelledby="user-menu">
             {% if is_granted('ROLE_CASEWORKER') or is_granted('ROLE_ADMINISTRATION') %}
                 <a class="dropdown-item" href="{{ path('user_settings') }}">{% trans %}User settings{% endtrans %}</a>
+                <a class="dropdown-item" href="{{ path('app_logout') }}">{% trans %}Log out{% endtrans %}</a>
             {% endif %}
         </div>
     </li>

translations/navbar+intl-icu.da.xlf

@@ -25,6 +25,10 @@
         <source>Create new</source>
         <target>Opret ny</target>
       </trans-unit>
+      <trans-unit id="SWFhRVF" resname="Log out">
+        <source>Log out</source>
+        <target state="translated">Log ud</target>
+      </trans-unit>
     </body>
   </file>
 </xliff>

translations/navbar+intl-icu.en.xlf

@@ -25,6 +25,10 @@
         <source>User settings</source>
         <target>User settings</target>
       </trans-unit>
+      <trans-unit id="SWFhRVF" resname="Log out">
+        <source>Log out</source>
+        <target>Log out</target>
+      </trans-unit>
     </body>
   </file>
 </xliff>