refactor: remove ViewModels.WorkingCopy.StashAllAsync method

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

Author: love-linger

src/ViewModels/Repository.cs

@@ -1112,8 +1112,14 @@ public void SetBranchFilterMode(BranchTreeNode node, Models.FilterMode mode, boo
 
         public async Task StashAllAsync(bool autoStart)
         {
-            if (_workingCopy != null)
-                await _workingCopy.StashAllAsync(autoStart);
+            if (!CanCreatePopup())
+                return;
+
+            var popup = new StashChanges(this, null);
+            if (autoStart)
+                await ShowAndStartPopupAsync(popup);
+            else
+                ShowPopup(popup);
         }
 
         public async Task SkipMergeAsync()

src/ViewModels/StashChanges.cs

@@ -15,7 +15,7 @@ public string Message
 
         public bool HasSelectedFiles
         {
-            get;
+            get => _changes != null;
         }
 
         public bool IncludeUntracked
@@ -43,11 +43,10 @@ public int ChangesAfterStashing
             set => _repo.Settings.ChangesAfterStashing = value;
         }
 
-        public StashChanges(Repository repo, List<Models.Change> changes, bool hasSelectedFiles)
+        public StashChanges(Repository repo, List<Models.Change> selectedChanges)
         {
             _repo = repo;
-            _changes = changes;
-            HasSelectedFiles = hasSelectedFiles;
+            _changes = selectedChanges;
         }
 
         public override async Task<bool> Sure()
@@ -62,7 +61,7 @@ public override async Task<bool> Sure()
             var keepIndex = mode == DealWithChangesAfterStashing.KeepIndex;
             bool succ;
 
-            if (!HasSelectedFiles)
+            if (_changes == null)
             {
                 if (OnlyStaged)
                 {
@@ -74,8 +73,12 @@ public override async Task<bool> Sure()
                     }
                     else
                     {
+                        var all = await new Commands.QueryLocalChanges(_repo.FullPath, false)
+                            .Use(log)
+                            .GetResultAsync();
+
                         var staged = new List<Models.Change>();
-                        foreach (var c in _changes)
+                        foreach (var c in all)
                         {
                             if (c.Index != Models.ChangeState.None && c.Index != Models.ChangeState.Untracked)
                                 staged.Add(c);

src/ViewModels/WorkingCopy.cs

@@ -342,17 +342,6 @@ public void OpenWithDefaultEditor(Models.Change c)
                 Native.OS.OpenWithDefaultEditor(absPath);
         }
 
-        public async Task StashAllAsync(bool autoStart)
-        {
-            if (!_repo.CanCreatePopup())
-                return;
-
-            if (autoStart)
-                await _repo.ShowAndStartPopupAsync(new StashChanges(_repo, _cached, false));
-            else
-                _repo.ShowPopup(new StashChanges(_repo, _cached, false));
-        }
-
         public async Task StageChangesAsync(List<Models.Change> changes, Models.Change next)
         {
             var canStaged = await GetCanStageChangesAsync(changes);

src/Views/WorkingCopy.axaml.cs

@@ -372,7 +372,7 @@ private ContextMenu CreateContextMenuForUnstagedChanges(ViewModels.WorkingCopy v
                     stash.Click += (_, e) =>
                     {
                         if (repo.CanCreatePopup())
-                            repo.ShowPopup(new ViewModels.StashChanges(repo, selectedUnstaged, true));
+                            repo.ShowPopup(new ViewModels.StashChanges(repo, selectedUnstaged));
 
                         e.Handled = true;
                     };
@@ -769,7 +769,7 @@ private ContextMenu CreateContextMenuForUnstagedChanges(ViewModels.WorkingCopy v
                 stash.Click += (_, e) =>
                 {
                     if (repo.CanCreatePopup())
-                        repo.ShowPopup(new ViewModels.StashChanges(repo, selectedUnstaged, true));
+                        repo.ShowPopup(new ViewModels.StashChanges(repo, selectedUnstaged));
 
                     e.Handled = true;
                 };
@@ -956,7 +956,7 @@ public ContextMenu CreateContextMenuForStagedChanges(ViewModels.WorkingCopy vm,
                 stash.Click += (_, e) =>
                 {
                     if (repo.CanCreatePopup())
-                        repo.ShowPopup(new ViewModels.StashChanges(repo, selectedStaged, true));
+                        repo.ShowPopup(new ViewModels.StashChanges(repo, selectedStaged));
 
                     e.Handled = true;
                 };
@@ -1164,7 +1164,7 @@ public ContextMenu CreateContextMenuForStagedChanges(ViewModels.WorkingCopy vm,
                 stash.Click += (_, e) =>
                 {
                     if (repo.CanCreatePopup())
-                        repo.ShowPopup(new ViewModels.StashChanges(repo, selectedStaged, true));
+                        repo.ShowPopup(new ViewModels.StashChanges(repo, selectedStaged));
 
                     e.Handled = true;
                 };