Skip to content

Commit a5409aa

Browse files
authored
Merge pull request #543 from aspnetboilerplate/localization-controller-local-url
Remove url encode when constructing via Url.Action
2 parents efb58e2 + 5282fd8 commit a5409aa

File tree

2 files changed

+15
-10
lines changed
  • aspnet-core/src/AbpCompanyName.AbpProjectName.Web.Mvc/Views/Shared/Components

2 files changed

+15
-10
lines changed

aspnet-core/src/AbpCompanyName.AbpProjectName.Web.Mvc/Views/Shared/Components/AccountLanguages/Default.cshtml

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,16 @@
1-
@using System.Text.Encodings.Web
2-
@using AbpCompanyName.AbpProjectName.Web.Views.Shared.Components.AccountLanguages
1+
@using AbpCompanyName.AbpProjectName.Web.Views.Shared.Components.AccountLanguages
32
@model LanguageSelectionViewModel
43
@if (Model.Languages.Count > 1)
54
{
65
<div class="text-center">
7-
@foreach (var language in Model.Languages)
6+
@foreach (var languageInfo in Model.Languages)
87
{
9-
<a href="~/AbpLocalization/ChangeCulture?cultureName=@(language.Name)&returnUrl=@(UrlEncoder.Default.Encode(Context.Request.Path + Context.Request.QueryString))">
10-
<span class="@(language.Name == Model.CurrentLanguage.Name ? "current-language-icon" : "")" title="@language.DisplayName">
11-
<i class="@language.Icon"></i>
8+
<a href="@Url.Action("ChangeCulture", "AbpLocalization", new {
9+
cultureName = languageInfo.Name,
10+
returnUrl = Context.Request.Path + Context.Request.QueryString
11+
})">
12+
<span class="@(languageInfo.Name == Model.CurrentLanguage.Name ? "current-language-icon" : "")" title="@languageInfo.DisplayName">
13+
<i class="@languageInfo.Icon"></i>
1214
</span>
1315
</a>
1416
}

aspnet-core/src/AbpCompanyName.AbpProjectName.Web.Mvc/Views/Shared/Components/RightNavbarLanguageSwitch/Default.cshtml

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,12 +7,15 @@
77
<span class="d-none d-md-inline-block">@Model.CurrentLanguage.DisplayName</span>
88
</a>
99
<div class="dropdown-menu dropdown-menu-right p-0">
10-
@foreach (var language in Model.Languages)
10+
@foreach (var languageInfo in Model.Languages)
1111
{
12-
if (language.Name != Model.CurrentLanguage.Name)
12+
if (languageInfo.Name != Model.CurrentLanguage.Name)
1313
{
14-
<a class="dropdown-item" href="@(ApplicationPath)AbpLocalization/ChangeCulture?cultureName=@(language.Name)&returnUrl=@(Context.Request.Path)">
15-
<i class="mr-2 @language.Icon"></i> @language.DisplayName
14+
<a class="dropdown-item" href="@Url.Action("ChangeCulture", "AbpLocalization", new {
15+
cultureName = languageInfo.Name,
16+
returnUrl = Context.Request.Path + Context.Request.QueryString
17+
})">
18+
<i class="mr-2 @languageInfo.Icon"></i> @languageInfo.DisplayName
1619
</a>
1720
}
1821
}

0 commit comments

Comments
 (0)