code_review: PR #1093

Merge deleting branch and tag on remote into `SourceGit.Commands.Push(repo, remote, refname, isDelete)`

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

Author: love-linger

src/Commands/Branch.cs

@@ -54,21 +54,14 @@ public static bool DeleteLocal(string repo, string name)
 
         public static bool DeleteRemote(string repo, string remote, string name)
         {
-            var cmd = new Command();
-            cmd.WorkingDirectory = repo;
-            cmd.Context = repo;
-
             bool exists = new Remote(repo).HasBranch(remote, name);
             if (exists)
-            {
-                cmd.SSHKey = new Config(repo).Get($"remote.{remote}.sshkey");
-                cmd.Args = $"push {remote} --delete refs/heads/{name}";
-            }
-            else
-            {
-                cmd.Args = $"branch -D -r {remote}/{name}";
-            }
+                return new Push(repo, remote, $"refs/heads/{name}", true).Exec();
 
+            var cmd = new Command();
+            cmd.WorkingDirectory = repo;
+            cmd.Context = repo;
+            cmd.Args = $"branch -D -r {remote}/{name}";
             return cmd.Exec();
         }
     }

src/Commands/Push.cs

@@ -26,7 +26,7 @@ public Push(string repo, string local, string remote, string remoteBranch, bool
             Args += $"{remote} {local}:{remoteBranch}";
         }
 
-        public Push(string repo, string remote, string tag, bool isDelete)
+        public Push(string repo, string remote, string refname, bool isDelete)
         {
             WorkingDirectory = repo;
             Context = repo;
@@ -36,7 +36,7 @@ public Push(string repo, string remote, string tag, bool isDelete)
             if (isDelete)
                 Args += "--delete ";
 
-            Args += $"{remote} refs/tags/{tag}";
+            Args += $"{remote} {refname}";
         }
 
         protected override void OnReadline(string line)

src/Commands/Tag.cs

@@ -48,9 +48,7 @@ public static bool Delete(string repo, string name, List<Models.Remote> remotes)
             if (remotes != null)
             {
                 foreach (var r in remotes)
-                {
-                    new Push(repo, r.Name, name, true).Exec();
-                }
+                    new Push(repo, r.Name, $"refs/tags/{name}", true).Exec();
             }
 
             return true;

src/ViewModels/CreateTag.cs

@@ -96,7 +96,7 @@ public override Task<bool> Sure()
                     foreach (var remote in remotes)
                     {
                         SetProgressDescription($"Pushing tag to remote {remote.Name} ...");
-                        new Commands.Push(_repo.FullPath, remote.Name, _tagName, false).Exec();
+                        new Commands.Push(_repo.FullPath, remote.Name, $"refs/tags/{_tagName}", false).Exec();
                     }
                 }
 

src/ViewModels/PushTag.cs

@@ -43,21 +43,22 @@ public override Task<bool> Sure()
 
             return Task.Run(() =>
             {
-                bool succ = true;
+                var succ = true;
+                var tag = $"refs/tags/{Target.Name}";
                 if (_pushAllRemotes)
                 {
                     foreach (var remote in _repo.Remotes)
                     {
                         SetProgressDescription($"Pushing tag to remote {remote.Name} ...");
-                        succ = new Commands.Push(_repo.FullPath, remote.Name, Target.Name, false).Exec();
+                        succ = new Commands.Push(_repo.FullPath, remote.Name, tag, false).Exec();
                         if (!succ)
                             break;
                     }
                 }
                 else
                 {
                     SetProgressDescription($"Pushing tag to remote {SelectedRemote.Name} ...");
-                    succ = new Commands.Push(_repo.FullPath, SelectedRemote.Name, Target.Name, false).Exec();
+                    succ = new Commands.Push(_repo.FullPath, SelectedRemote.Name, tag, false).Exec();
                 }
 
                 CallUIThread(() => _repo.SetWatcherEnabled(true));