Refactor jump history handling to ensure ordered retrieval in Overvie…

…w component (#328)
pfh59 · Dec 19, 2024 · ba0732d · ba0732d
1 parent 2da929c
commit ba0732d
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/src/WHMapper/Pages/Mapper/LinkInfos/Overview.cs b/src/WHMapper/Pages/Mapper/LinkInfos/Overview.cs
@@ -30,7 +30,7 @@ public partial class Overview : ComponentBase
         private string FirstJumpLogCharacterName { get; set; } = string.Empty;
         private string FirstJumpLogShipName { get; set; } = string.Empty;
         private DateTime? FirstJumpLogDate { get; set; } = null;
-
+        
         private string LastJumpLogCharacterName{ get; set; } = string.Empty;
         private string LastJumpLogShipName { get; set; } = string.Empty;
         private DateTime? LastJumpLogDate { get; set; } = null;
@@ -62,7 +62,9 @@ private async Task Restore()
                                 SystemLink = await DbSystemLink.GetById(CurrentSystemLink.Id);
                                 if (SystemLink != null)
                                 {
-                                        var firstJump = SystemLink.JumpHistory.FirstOrDefault();
+                                        var JumpHistory = SystemLink.JumpHistory.OrderBy(x => x.JumpDate).ToList();
+
+                                        var firstJump = JumpHistory.FirstOrDefault();
                                         if(firstJump!=null)
                                         {
                                                 _showing=true;
@@ -71,7 +73,7 @@ private async Task Restore()
                                                 FirstJumpLogShipName = await GetJumpLogShipName(firstJump);
                                         }
 
-                                        var lastJump = SystemLink.JumpHistory.LastOrDefault();
+                                        var lastJump = JumpHistory.LastOrDefault();
                                         if(lastJump!=null)
                                         {      
                                                 _showing=true;

diff --git a/src/WHMapper/Pages/Mapper/LinkInfos/Overview.razor b/src/WHMapper/Pages/Mapper/LinkInfos/Overview.razor
@@ -74,7 +74,7 @@
                 </MudStack>
             </MudStack>
         </MudStack>
-        <MudTable Items="@SystemLink?.JumpHistory" Height="calc(100%)" Outlined="true" Dense="true" Hover="false" Bordered=true Striped=true 
+        <MudTable Items="@SystemLink?.JumpHistory.OrderBy(x=>x.JumpDate)" Height="325px" Outlined="true" Dense="true" Hover="false" Bordered=true Striped=true 
             AllowUnsorted=false SortLabel="Sort By"
             Virtualize="true" Loading=@_isLoading LoadingProgressColor="Color.Info">
             <HeaderContent>