fix: add a new interface CanStartDirectly and move this check to StartPopup (#956)

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

Author: love-linger

src/ViewModels/LauncherPage.cs

@@ -59,7 +59,9 @@ public bool CanCreatePopup()
         public void StartPopup(Popup popup)
         {
             Popup = popup;
-            ProcessPopup();
+
+            if (popup.CanStartDirectly())
+                ProcessPopup();
         }
 
         public async void ProcessPopup()

src/ViewModels/Popup.cs

@@ -29,14 +29,19 @@ public string ProgressDescription
         }
 
         [UnconditionalSuppressMessage("AssemblyLoadTrimming", "IL2026:RequiresUnreferencedCode")]
-        public virtual bool Check()
+        public bool Check()
         {
             if (HasErrors)
                 return false;
             ValidateAllProperties();
             return !HasErrors;
         }
 
+        public virtual bool CanStartDirectly()
+        {
+            return true;
+        }
+
         public virtual Task<bool> Sure()
         {
             return null;

src/ViewModels/Push.cs

@@ -152,9 +152,9 @@ public Push(Repository repo, Models.Branch localBranch)
             View = new Views.Push() { DataContext = this };
         }
 
-        public override bool Check()
+        public override bool CanStartDirectly()
         {
-            return base.Check() && !string.IsNullOrEmpty(_selectedRemoteBranch?.Head);
+            return !string.IsNullOrEmpty(_selectedRemoteBranch?.Head);
         }
 
         public override Task<bool> Sure()