feat: Enhance Protected Names functionality with pagination and additional game type display

Author: frasermolyneux

src/XtremeIdiots.Portal.Web/Controllers/ProtectedNamesController.cs

@@ -54,17 +54,37 @@ public async Task<IActionResult> Index(CancellationToken cancellationToken = def
             if (authResult is not null)
                 return authResult;
 
-            var protectedNamesResponse = await repositoryApiClient.Players.V1.GetProtectedNames(0, 1000).ConfigureAwait(false);
+            const int protectedNamesPageSize = 500;
+            var protectedNames = new List<ProtectedNameDto>();
+            var skipEntries = 0;
 
-            if (!protectedNamesResponse.IsSuccess || protectedNamesResponse.Result?.Data?.Items is null)
+            while (true)
             {
-                Logger.LogWarning("Failed to retrieve protected names for user {UserId}", User.XtremeIdiotsId());
-                return RedirectToAction(nameof(ErrorsController.Display), nameof(ErrorsController), new { id = 500 });
+                var protectedNamesResponse = await repositoryApiClient.Players.V1
+                    .GetProtectedNames(skipEntries, protectedNamesPageSize)
+                    .ConfigureAwait(false);
+
+                if (!protectedNamesResponse.IsSuccess || protectedNamesResponse.Result?.Data?.Items is null)
+                {
+                    Logger.LogWarning("Failed to retrieve protected names for user {UserId}", User.XtremeIdiotsId());
+                    return RedirectToAction(nameof(ErrorsController.Display), nameof(ErrorsController), new { id = 500 });
+                }
+
+                var page = protectedNamesResponse.Result.Data.Items.ToList();
+                if (page.Count == 0)
+                    break;
+
+                protectedNames.AddRange(page);
+
+                if (page.Count < protectedNamesPageSize)
+                    break;
+
+                skipEntries += protectedNamesPageSize;
             }
 
             var model = new ProtectedNamesViewModel
             {
-                ProtectedNames = [.. protectedNamesResponse.Result.Data.Items]
+                ProtectedNames = protectedNames
             };
 
             return View(model);

src/XtremeIdiots.Portal.Web/Views/ProtectedNames/Index.cshtml

@@ -26,7 +26,7 @@
             <h5>All Protected Names</h5>
         </div>
         <div class="ibox-content">
-            <partial name="_ProtectedNamesTable" model="@(Model?.ProtectedNames ?? new List<ProtectedNameDto>())" />
+            <partial name="_ProtectedNamesTable" model="@(Model?.ProtectedNames ?? Array.Empty<ProtectedNameDto>())" />
         </div>
     </div>
 </div>

src/XtremeIdiots.Portal.Web/Views/ProtectedNames/Report.cshtml

@@ -39,6 +39,13 @@
                                 </a>
                             </dd>
                         </div>
+                        <div class="detail-field">
+                            <dt class="detail-label">Owner Game</dt>
+                            <dd class="detail-value">
+                                <game-type-icon game="@Model.Report.OwningPlayer.GameType"></game-type-icon>
+                                <span>@Model.Report.OwningPlayer.GameType</span>
+                            </dd>
+                        </div>
                     </dl>
                 }
             </div>

src/XtremeIdiots.Portal.Web/Views/Shared/_ProtectedNamesTable.cshtml

@@ -16,6 +16,7 @@
             <tr>
                 <th>Name</th>
                 <th>Owner</th>
+                <th>Owner Game</th>
                 <th>Created</th>
                 <th>Created By</th>
                 <th>Actions</th>
@@ -31,6 +32,10 @@
                             View Player Profile
                         </a>
                     </td>
+                    <td>
+                        <game-type-icon game="@protectedName.OwnerGameType"></game-type-icon>
+                        <span>@protectedName.OwnerGameType</span>
+                    </td>
                     <td><time user-time utc="@protectedName.CreatedOn"></time></td>
                     <td>@protectedName.CreatedByUserProfile?.DisplayName</td>
                     <td>

src/XtremeIdiots.Portal.Web/XtremeIdiots.Portal.Web.csproj

@@ -48,8 +48,8 @@
     <PackageReference Include="MX.Api.Abstractions" Version="2.3.31" />
     <PackageReference Include="MX.Api.Client" Version="2.3.31" />
     <PackageReference Include="XtremeIdiots.Portal.Integrations.Servers.Api.Client.V1" Version="2.1.138" />
-    <PackageReference Include="XtremeIdiots.Portal.Repository.Abstractions.V1" Version="2.1.220" />
-    <PackageReference Include="XtremeIdiots.Portal.Repository.Api.Client.V1" Version="2.1.220" />
+    <PackageReference Include="XtremeIdiots.Portal.Repository.Abstractions.V1" Version="2.1.230" />
+    <PackageReference Include="XtremeIdiots.Portal.Repository.Api.Client.V1" Version="2.1.230" />
     <PackageReference Include="MX.GeoLocation.Api.Client.V1" Version="1.2.39" />
   </ItemGroup>
 

src/XtremeIdiots.Portal.Web/wwwroot/js/protected-names-table.js

@@ -13,13 +13,14 @@ $(document).ready(function () {
             autoWidth: false,
             paging: true,
             pageLength: 25,
-            order: [[2, 'desc']], // Created date desc (column index 2)
+            order: [[3, 'desc']], // Created date desc (column index 3)
             columnDefs: [
                 { targets: 0, responsivePriority: 1 }, // Name - always visible
                 { targets: 1, responsivePriority: 3 }, // Owner/Player - medium priority
-                { targets: 2, responsivePriority: 4 }, // Created - lower priority
-                { targets: 3, responsivePriority: 5 }, // Created By - lowest priority
-                { targets: 4, responsivePriority: 2 }  // Actions - high priority
+                { targets: 2, responsivePriority: 4 }, // Owner game - lower priority
+                { targets: 3, responsivePriority: 5 }, // Created - lower priority
+                { targets: 4, responsivePriority: 6 }, // Created By - lowest priority
+                { targets: 5, responsivePriority: 2 }  // Actions - high priority
             ],
             language: {
                 search: '<i class="fa fa-search" aria-hidden="true"></i>',