fix double 'push tag' action feed (#15078)

Signed-off-by: a1012112796 <1012112796@qq.com>

diff --git a/modules/notification/action/action.go b/modules/notification/action/action.go
index 3530e48b1cebdd7e2356bb71407dd52a47f0037a..776bee6a0ca3e2b7185b8cfe5520babccede059f 100644 (file)
--- a/modules/notification/action/action.go
+++ b/modules/notification/action/action.go
@@ -332,7 +332,8 @@ func (a *actionNotifier) NotifyPushCommits(pusher *models.User, repo *models.Rep
 func (a *actionNotifier) NotifyCreateRef(doer *models.User, repo *models.Repository, refType, refFullName string) {
        opType := models.ActionCommitRepo
        if refType == "tag" {
-               opType = models.ActionPushTag
+               // has sent same action in `NotifyPushCommits`, so skip it.
+               return
        }
        if err := models.NotifyWatchers(&models.Action{
                ActUserID: doer.ID,
@@ -350,7 +351,8 @@ func (a *actionNotifier) NotifyCreateRef(doer *models.User, repo *models.Reposit
 func (a *actionNotifier) NotifyDeleteRef(doer *models.User, repo *models.Repository, refType, refFullName string) {
        opType := models.ActionDeleteBranch
        if refType == "tag" {
-               opType = models.ActionDeleteTag
+               // has sent same action in `NotifyPushCommits`, so skip it.
+               return
        }
        if err := models.NotifyWatchers(&models.Action{
                ActUserID: doer.ID,