refactor: it's not necessary to store untracked file list for selected stash

Signed-off-by: leo <[email protected]>

Author: love-linger

src/Models/Stash.cs

@@ -10,7 +10,6 @@ public class Stash
         public List<string> Parents { get; set; } = [];
         public ulong Time { get; set; } = 0;
         public string Message { get; set; } = "";
-        public bool HasUntracked => Parents.Count == 3;
 
         public string TimeStr => DateTime.UnixEpoch.AddSeconds(Time).ToLocalTime().ToString(DateTimeFormat.Actived.DateTime);
     }

src/ViewModels/StashesPage.cs

@@ -59,24 +59,16 @@ public Models.Stash SelectedStash
                         Task.Run(() =>
                         {
                             var changes = new Commands.CompareRevisions(_repo.FullPath, $"{value.SHA}^", value.SHA).Result();
-                            var untracked = new HashSet<string>();
-                            if (value.HasUntracked)
+                            if (value.Parents.Count == 3)
                             {
-                                var untrackedChanges = new Commands.CompareRevisions(_repo.FullPath, "4b825dc642cb6eb9a060e54bf8d69288fbee4904", value.Parents[2]).Result();
-                                foreach (var c in untrackedChanges)
-                                {
-                                    untracked.Add(c.Path);
+                                var untracked = new Commands.CompareRevisions(_repo.FullPath, "4b825dc642cb6eb9a060e54bf8d69288fbee4904", value.Parents[2]).Result();
+                                foreach (var c in untracked)
                                     changes.Add(c);
-                                }
-                            }
 
-                            changes.Sort((l, r) => Models.NumericSort.Compare(l.Path, r.Path));
+                                changes.Sort((l, r) => string.Compare(l.Path, r.Path, StringComparison.Ordinal));
+                            }
 
-                            Dispatcher.UIThread.Invoke(() =>
-                            {
-                                Changes = changes;
-                                _untrackedChanges = untracked;
-                            });
+                            Dispatcher.UIThread.Invoke(() => Changes = changes);
                         });
                     }
                 }
@@ -102,7 +94,7 @@ public Models.Change SelectedChange
                 {
                     if (value == null)
                         DiffContext = null;
-                    else if (_untrackedChanges.Contains(value.Path))
+                    else if (value.Index == Models.ChangeState.Added && _selectedStash.Parents.Count == 3)
                         DiffContext = new DiffContext(_repo.FullPath, new Models.DiffOption("4b825dc642cb6eb9a060e54bf8d69288fbee4904", _selectedStash.Parents[2], value), _diffContext);
                     else
                         DiffContext = new DiffContext(_repo.FullPath, new Models.DiffOption(_selectedStash.Parents[0], _selectedStash.SHA, value), _diffContext);
@@ -178,7 +170,7 @@ public ContextMenu MakeContextMenu(Models.Stash stash)
                     var opts = new List<Models.DiffOption>();
                     foreach (var c in _changes)
                     {
-                        if (_untrackedChanges.Contains(c.Path))
+                        if (c.Index == Models.ChangeState.Added && _selectedStash.Parents.Count == 3)
                             opts.Add(new Models.DiffOption("4b825dc642cb6eb9a060e54bf8d69288fbee4904", _selectedStash.Parents[2], c));
                         else
                             opts.Add(new Models.DiffOption(_selectedStash.Parents[0], _selectedStash.SHA, c));
@@ -303,7 +295,6 @@ private void RefreshVisible()
         private List<Models.Stash> _visibleStashes = new List<Models.Stash>();
         private string _searchFilter = string.Empty;
         private Models.Stash _selectedStash = null;
-        private HashSet<string> _untrackedChanges = new HashSet<string>();
         private List<Models.Change> _changes = null;
         private Models.Change _selectedChange = null;
         private DiffContext _diffContext = null;