Fixed reset language on login redirect

dyatlov-a · Sep 29, 2024 · 4413430 · 4413430
1 parent cb7d170
commit 4413430
Show file tree

Hide file tree

Showing 3 changed files with 23 additions and 8 deletions.
diff --git a/src/Inc.TeamAssistant.Gateway/Bff/AccountsController.cs b/src/Inc.TeamAssistant.Gateway/Bff/AccountsController.cs
@@ -67,11 +67,15 @@ public async Task<IActionResult> LoginAsSuperUser(string? returnUrl)
     }
 
     [HttpGet("logout")]
-    public async Task<IActionResult> Logout(string? returnUrl)
+    public async Task<IActionResult> Logout(string? languageCode)
     {
         await HttpContext.SignOutAsync(ApplicationContext.AuthenticationScheme);
 
-        return Redirect(DetectTargetUrl(returnUrl));
+        var path = string.IsNullOrWhiteSpace(languageCode)
+            ? "/"
+            : $"/{languageCode}/";
+
+        return Redirect(path);
     }
 
     private string DetectTargetUrl(string? returnUrl)

diff --git a/src/Inc.TeamAssistant.WebUI/Features/Layouts/MainNavbar.razor b/src/Inc.TeamAssistant.WebUI/Features/Layouts/MainNavbar.razor
@@ -18,7 +18,7 @@
                     <br/>
                     @person.Username
                 </span>
-                <a href="@LogoutUrl" class="link link_light navbar__item">
+                <a href="@_logoutUrl" class="link link_light navbar__item">
                     @Resources[Messages.Navigation_Logout]
                 </a>
             </Authorized>
@@ -43,11 +43,16 @@
     private IDisposable? _routerScope;
 
     private string MainUrl => NavRouter.CreateRoute(null);
-    private string LogoutUrl => $"accounts/logout?returnUrl={NavRouter.CurrentUrl}";
+    private string _logoutUrl = string.Empty;
 
     protected override void OnInitialized()
     {
+        var languageId = NavRouter.GetCurrentLanguage();
+
         _routerScope = NavRouter.OnRouteChanged(StateHasChanged);
+        _logoutUrl = languageId is null
+            ? "accounts/logout"
+            : $"accounts/logout?languageCode={languageId.Value}";
     }
 
     public void Dispose() => _routerScope?.Dispose();

diff --git a/src/Inc.TeamAssistant.WebUI/Routing/NavRouter.cs b/src/Inc.TeamAssistant.WebUI/Routing/NavRouter.cs
@@ -1,7 +1,6 @@
 using Inc.TeamAssistant.Primitives.Languages;
 using Inc.TeamAssistant.WebUI.Contracts;
 using Inc.TeamAssistant.WebUI.Extensions;
-using Inc.TeamAssistant.WebUI.Services.ClientCore;
 using Microsoft.AspNetCore.Components;
 using Microsoft.AspNetCore.Components.Routing;
 using Microsoft.JSInterop;
@@ -39,13 +38,20 @@ public string GetRouteSegment()
         return routeWithoutLanguage;
     }
 
+    public LanguageId? GetCurrentLanguage()
+    {
+        var languageContext = _renderContext.GetLanguageContext();
+
+        return languageContext.Selected
+            ? languageContext.CurrentLanguage
+            : null;
+    }
+
     public NavRoute CreateRoute(string? routeSegment)
     {
         var link = string.IsNullOrWhiteSpace(routeSegment) ? "/" : routeSegment;
-        var languageContext = _renderContext.GetLanguageContext();
-        var selectedLanguage = languageContext.Selected ? languageContext.CurrentLanguage : null;
 
-        return new NavRoute(selectedLanguage, link);
+        return new NavRoute(GetCurrentLanguage(), link);
     }
 
     public IDisposable OnRouteChanged(Action onRouteChanged)