fix: when upstream is null show CommitAndPush and stop auto push (#955) (#956)

Author: gadfly3173

src/ViewModels/LauncherPage.cs

@@ -59,15 +59,17 @@ public bool CanCreatePopup()
         public void StartPopup(Popup popup)
         {
             Popup = popup;
-            ProcessPopup();
+            ProcessPopup(true);
         }
 
-        public async void ProcessPopup()
+        public async void ProcessPopup(bool autoCheck = false)
         {
             if (_popup != null)
             {
                 if (!_popup.Check())
                     return;
+                if (autoCheck && !_popup.AutoCheck())
+                    return;
 
                 _popup.InProgress = true;
                 var task = _popup.Sure();

src/ViewModels/Popup.cs

@@ -37,6 +37,11 @@ public bool Check()
             return !HasErrors;
         }
 
+        public virtual bool AutoCheck()
+        {
+            return true;
+        }
+
         public virtual Task<bool> Sure()
         {
             return null;

src/ViewModels/Push.cs

@@ -152,6 +152,11 @@ public Push(Repository repo, Models.Branch localBranch)
             View = new Views.Push() { DataContext = this };
         }
 
+        public override bool AutoCheck()
+        {
+            return !string.IsNullOrEmpty(_selectedRemoteBranch?.Head);
+        }
+
         public override Task<bool> Sure()
         {
             _repo.SetWatcherEnabled(false);

src/ViewModels/Repository.cs

@@ -937,9 +937,6 @@ public void RefreshBranches()
                 CurrentBranch = branches.Find(x => x.IsCurrent);
                 LocalBranchTrees = builder.Locals;
                 RemoteBranchTrees = builder.Remotes;
-
-                if (_workingCopy != null)
-                    _workingCopy.CanCommitWithPush = _currentBranch != null && !string.IsNullOrEmpty(_currentBranch.Upstream);
             });
         }
 

src/ViewModels/WorkingCopy.cs

@@ -26,12 +26,6 @@ public bool IncludeUntracked
             }
         }
 
-        public bool CanCommitWithPush
-        {
-            get => _canCommitWithPush;
-            set => SetProperty(ref _canCommitWithPush, value);
-        }
-
         public bool HasUnsolvedConflicts
         {
             get => _hasUnsolvedConflicts;
@@ -1625,7 +1619,6 @@ private bool IsChanged(List<Models.Change> old, List<Models.Change> cur)
         private bool _isUnstaging = false;
         private bool _isCommitting = false;
         private bool _useAmend = false;
-        private bool _canCommitWithPush = false;
         private List<Models.Change> _cached = [];
         private List<Models.Change> _unstaged = [];
         private List<Models.Change> _staged = [];

src/Views/WorkingCopy.axaml

@@ -375,7 +375,6 @@
           <Button.IsVisible>
             <MultiBinding Converter="{x:Static BoolConverters.And}">
               <Binding Path="UseAmend" Converter="{x:Static BoolConverters.Not}"/>
-              <Binding Path="CanCommitWithPush"/>
               <Binding Path="InProgressContext" Converter="{x:Static ObjectConverters.IsNull}"/>
             </MultiBinding>
           </Button.IsVisible>