From 69984e501cacf258c273c16df77e96e5420f4bbd Mon Sep 17 00:00:00 2001
From: Kemal Zebari <kemalzebra@gmail.com>
Date: Mon, 13 Jan 2025 00:21:31 -0800
Subject: [PATCH] PR feedback

---
 routers/web/repo/release.go      |  2 +-
 routers/web/repo/release_test.go | 18 ++++++++++++++++--
 2 files changed, 17 insertions(+), 3 deletions(-)

diff --git a/routers/web/repo/release.go b/routers/web/repo/release.go
index 585c0417416d8..21d8726f761c3 100644
--- a/routers/web/repo/release.go
+++ b/routers/web/repo/release.go
@@ -489,7 +489,7 @@ func NewReleasePost(ctx *context.Context) {
 			return
 		}
 	} else {
-		if len(form.TagOnly) > 0 && rel.IsTag {
+		if form.TagOnly != "" && rel.IsTag {
 			ctx.Data["Err_TagName"] = true
 			ctx.RenderWithErr(ctx.Tr("repo.release.tag_name_already_exist"), tplReleaseNew, &form)
 			return
diff --git a/routers/web/repo/release_test.go b/routers/web/repo/release_test.go
index 7ebea4c3fbe30..9f79c6f501b7e 100644
--- a/routers/web/repo/release_test.go
+++ b/routers/web/repo/release_test.go
@@ -22,12 +22,13 @@ func TestNewReleasePost(t *testing.T) {
 		RepoID  int64
 		UserID  int64
 		TagName string
+		IsTag   bool
 		Form    forms.NewReleaseForm
 	}{
-		{
+		{ // pre-existing tag
 			RepoID:  1,
 			UserID:  2,
-			TagName: "v1.1", // pre-existing tag
+			TagName: "v1.1",
 			Form: forms.NewReleaseForm{
 				TagName: "newtag",
 				Target:  "master",
@@ -35,6 +36,18 @@ func TestNewReleasePost(t *testing.T) {
 				Content: "content",
 			},
 		},
+		{ // creating a new tag when there's already a pre-existing tag
+			RepoID:  1,
+			UserID:  2,
+			TagName: "delete-tag",
+			IsTag:   true,
+			Form: forms.NewReleaseForm{
+				TagName: "delete-tag",
+				Target:  "master",
+				Title:   "delete-tag",
+				TagOnly: "1",
+			},
+		},
 		{
 			RepoID:  1,
 			UserID:  2,
@@ -62,6 +75,7 @@ func TestNewReleasePost(t *testing.T) {
 			Target:      testCase.Form.Target,
 			Title:       testCase.Form.Title,
 			Note:        testCase.Form.Content,
+			IsTag:       true,
 		}, unittest.Cond("is_draft=?", len(testCase.Form.Draft) > 0))
 		ctx.Repo.GitRepo.Close()
 	}