Use nameof() instead of strings

This commit is contained in:
Umar Bolatov
2020-07-20 20:02:14 -07:00
parent 36e3eeecaa
commit 25527ec1dd
2 changed files with 13 additions and 13 deletions

View File

@@ -60,11 +60,11 @@ namespace BTCPayServer.Controllers
{
switch (sortOrderColumn)
{
case "Name":
case nameof(app.AppName):
return app.AppName;
case "Store":
case nameof(app.StoreName):
return app.StoreName;
case "AppType":
case nameof(app.AppType):
return app.AppType;
default:
return app.Id;

View File

@@ -34,28 +34,28 @@
<th>
<a
asp-action="ListApps"
asp-route-sortOrder="@(ViewData["StoreSortOrder"] ?? "asc")"
asp-route-sortOrderColumn="Store"
asp-route-sortOrder="@(ViewData["StoreNameSortOrder"] ?? "asc")"
asp-route-sortOrderColumn="StoreName"
class="text-nowrap"
title="@((string)ViewData["StoreSortOrder"] == "desc" ? "Sort by descending..." : "Sort by ascending...")"
title="@((string)ViewData["StoreNameSortOrder"] == "desc" ? "Sort by descending..." : "Sort by ascending...")"
>
Store @if (ViewData["StoreSortOrder"] != null)
Store @if (ViewData["StoreNameSortOrder"] != null)
{
<span class="fa @((string)ViewData["StoreSortOrder"] == "asc" ? "fa-arrow-down": "fa-arrow-up")" />
<span class="fa @((string)ViewData["StoreNameSortOrder"] == "asc" ? "fa-arrow-down": "fa-arrow-up")" />
}
</a>
</th>
<th>
<a
asp-action="ListApps"
asp-route-sortOrder="@(ViewData["NameSortOrder"] ?? "asc")"
asp-route-sortOrderColumn="Name"
asp-route-sortOrder="@(ViewData["AppNameSortOrder"] ?? "asc")"
asp-route-sortOrderColumn="AppName"
class="text-nowrap"
title="@((string)ViewData["NameSortOrder"] == "desc" ? "Sort by descending..." : "Sort by ascending...")"
title="@((string)ViewData["AppNameSortOrder"] == "desc" ? "Sort by descending..." : "Sort by ascending...")"
>
Name @if (ViewData["NameSortOrder"] != null)
Name @if (ViewData["AppNameSortOrder"] != null)
{
<span class="fa @((string)ViewData["NameSortOrder"] == "asc" ? "fa-arrow-down": "fa-arrow-up")" />
<span class="fa @((string)ViewData["AppNameSortOrder"] == "asc" ? "fa-arrow-down": "fa-arrow-up")" />
}
</a>
</th>