From 4716b704d4e66b2cdec01283d57cb650dcb8c14d Mon Sep 17 00:00:00 2001
From: Rockstar Developer <5191402+rockstardev@users.noreply.github.com>
Date: Fri, 19 Jul 2019 02:50:17 -0500
Subject: [PATCH] Paging on List Users page for big hosting providers like Esky
 (#905)

---
 BTCPayServer/Controllers/ServerController.cs  |  15 +--
 .../Models/ServerViewModels/UsersViewModel.cs |  25 ++---
 BTCPayServer/Views/Server/ListUsers.cshtml    | 102 +++++++++++++-----
 3 files changed, 90 insertions(+), 52 deletions(-)

diff --git a/BTCPayServer/Controllers/ServerController.cs b/BTCPayServer/Controllers/ServerController.cs
index f3059d420..8692b10c3 100644
--- a/BTCPayServer/Controllers/ServerController.cs
+++ b/BTCPayServer/Controllers/ServerController.cs
@@ -154,17 +154,20 @@ namespace BTCPayServer.Controllers
         }
 
         [Route("server/users")]
-        public IActionResult ListUsers()
+        public IActionResult ListUsers(int skip = 0, int count = 50)
         {
             var users = new UsersViewModel();
             users.StatusMessage = StatusMessage;
-            users.Users
-                = _UserManager.Users.Select(u => new UsersViewModel.UserViewModel()
+            users.Users = _UserManager.Users.Skip(skip).Take(count)
+                .Select(u => new UsersViewModel.UserViewModel
                 {
                     Name = u.UserName,
                     Email = u.Email,
                     Id = u.Id
                 }).ToList();
+            users.Skip = skip;
+            users.Count = count;
+            users.Total = _UserManager.Users.Count();
             return View(users);
         }
 
@@ -479,7 +482,7 @@ namespace BTCPayServer.Controllers
             if (command.StartsWith("remove-domain", StringComparison.InvariantCultureIgnoreCase))
             {
                 ModelState.Clear();
-                var index = int.Parse(command.Substring(command.IndexOf(":",StringComparison.InvariantCultureIgnoreCase) + 1),  CultureInfo.InvariantCulture);
+                var index = int.Parse(command.Substring(command.IndexOf(":", StringComparison.InvariantCultureIgnoreCase) + 1), CultureInfo.InvariantCulture);
                 settings.DomainToAppMapping.RemoveAt(index);
                 return View(settings);
             }
@@ -569,7 +572,7 @@ namespace BTCPayServer.Controllers
             var storageSettings = await _SettingsRepository.GetSettingAsync<StorageSettings>();
             result.ExternalStorageServices.Add(new ServicesViewModel.OtherExternalService()
             {
-                Name = storageSettings == null? "Not set": storageSettings.Provider.ToString(),
+                Name = storageSettings == null ? "Not set" : storageSettings.Provider.ToString(),
                 Link = Url.Action("Storage")
             });
             return View(result);
@@ -577,7 +580,7 @@ namespace BTCPayServer.Controllers
 
         private async Task<List<SelectListItem>> GetAppSelectList()
         {
-// load display app dropdown
+            // load display app dropdown
             using (var ctx = _ContextFactory.CreateContext())
             {
                 var userId = _UserManager.GetUserId(base.User);
diff --git a/BTCPayServer/Models/ServerViewModels/UsersViewModel.cs b/BTCPayServer/Models/ServerViewModels/UsersViewModel.cs
index 8dd79cbbc..c87eb53fc 100644
--- a/BTCPayServer/Models/ServerViewModels/UsersViewModel.cs
+++ b/BTCPayServer/Models/ServerViewModels/UsersViewModel.cs
@@ -10,25 +10,16 @@ namespace BTCPayServer.Models.ServerViewModels
         public class UserViewModel
         {
             public string Id { get; set; }
-            public string Name
-            {
-                get; set;
-            }
-            public string Email
-            {
-                get; set;
-            }
+            public string Name { get; set; }
+            public string Email { get; set; }
         }
+        
+        public int Skip { get; set; }
+        public int Count { get; set; }
+        public int Total { get; set; }
+        public string StatusMessage { get; set; }
 
-        public string StatusMessage
-        {
-            get; set;
-        }
-
-        public List<UserViewModel> Users
-        {
-            get; set;
-        } = new List<UserViewModel>();
+        public List<UserViewModel> Users { get; set; } = new List<UserViewModel>();
     }
 
 }
diff --git a/BTCPayServer/Views/Server/ListUsers.cshtml b/BTCPayServer/Views/Server/ListUsers.cshtml
index 5149430d3..ab24e4a25 100644
--- a/BTCPayServer/Views/Server/ListUsers.cshtml
+++ b/BTCPayServer/Views/Server/ListUsers.cshtml
@@ -6,40 +6,84 @@
 <partial name="_StatusMessage" for="StatusMessage" />
 
 <div class="row">
-    <div class="col">
-
-    </div>
+    <div class="col align-self-center">Total Users: @Model.Total</div>
     <div class="col text-right">
-        <a 
-            asp-controller="Account" 
-            asp-action="Register" 
-            asp-route-returnUrl="@Context.Request.GetCurrentPath()" 
-            asp-route-logon="false" 
-            class="btn btn-primary" 
-            role="button">
+        <a asp-controller="Account"
+           asp-action="Register"
+           asp-route-returnUrl="@Context.Request.GetCurrentPath()"
+           asp-route-logon="false"
+           class="btn btn-primary"
+           role="button">
             <span class="fa fa-plus"></span> Add User
         </a>
     </div>
 </div>
 
-<div id="users">
-    <table class="table table-sm table-responsive-md">
-        <thead>
+<table class="table table-sm table-responsive-md">
+    <thead>
+        <tr>
+            <th>Name</th>
+            <th>Email</th>
+            <th>Actions</th>
+        </tr>
+    </thead>
+    <tbody>
+        @foreach (var user in Model.Users)
+        {
             <tr>
-                <th>Name</th>
-                <th>Email</th>
-                <th>Actions</th>
+                <td>@user.Name</td>
+                <td>@user.Email</td>
+                <td><a asp-action="User" asp-route-userId="@user.Id">Edit</a> <span> - </span> <a asp-action="DeleteUser" asp-route-userId="@user.Id">Remove</a></td>
             </tr>
-        </thead>
-        <tbody>
-            @foreach (var user in Model.Users)
-            {
-                <tr>
-                    <td>@user.Name</td>
-                    <td>@user.Email</td>
-                    <td><a asp-action="User" asp-route-userId="@user.Id">Edit</a> <span> - </span> <a asp-action="DeleteUser" asp-route-userId="@user.Id">Remove</a></td>
-                </tr>
-            }
-        </tbody>
-    </table>
-</div>
+        }
+    </tbody>
+</table>
+
+<nav aria-label="..." class="w-100">
+    <ul class="pagination float-left">
+        <li class="page-item @(Model.Skip == 0 ? "disabled" : null)">
+            <a class="page-link" tabindex="-1" href="@listUsers(-1, Model.Count)">&laquo;</a>
+        </li>
+        <li class="page-item disabled">
+            <span class="page-link">@(Model.Skip + 1) to @(Model.Skip + Model.Users.Count) of @Model.Total</span>
+        </li>
+        <li class="page-item @(Model.Total > (Model.Skip + Model.Users.Count) ? null : "disabled")">
+            <a class="page-link" href="@listUsers(1, Model.Count)">&raquo;</a>
+        </li>
+    </ul>
+    <ul class="pagination float-right">
+        <li class="page-item disabled">
+            <span class="page-link">Page Size:</span>
+        </li>
+        <li class="page-item @(Model.Count == 50 ? "active" : null)">
+            <a class="page-link" href="@listUsers(0, 50)">50</a>
+        </li>
+        <li class="page-item @(Model.Count == 100 ? "active" : null)">
+            <a class="page-link" href="@listUsers(0, 100)">100</a>
+        </li>
+        <li class="page-item @(Model.Count == 250 ? "active" : null)">
+            <a class="page-link" href="@listUsers(0, 250)">250</a>
+        </li>
+        <li class="page-item @(Model.Count == 500 ? "active" : null)">
+            <a class="page-link" href="@listUsers(0, 500)">500</a>
+        </li>
+    </ul>
+</nav>
+@{
+    string listUsers(int prevNext, int count)
+    {
+        var skip = Model.Skip;
+        if (prevNext == -1)
+            skip = Math.Max(0, Model.Skip - Model.Count);
+        else if (prevNext == 1)
+            skip = Model.Skip + count;
+
+        var act = Url.Action("ListUsers", new
+        {
+            skip = skip,
+            count = count,
+        });
+
+        return act;
+    }
+}