Remove unused comment actions

Author: lunny

models/issue.go

@@ -721,13 +721,10 @@ func (issue *Issue) ChangeTitle(doer *User, oldTitle string) (err error) {
 		OldTitle: oldTitle,
 		NewTitle: issue.Title,
 	}
-	comment, err := createCommentWithNoAction(sess, opts)
+	_, err = createCommentWithNoAction(sess, opts)
 	if err != nil {
 		return fmt.Errorf("createComment: %v", err)
 	}
-	if err = sendCreateCommentAction(sess, opts, comment); err != nil {
-		return err
-	}
 	if err = issue.addCrossReferences(sess, doer, true); err != nil {
 		return err
 	}
@@ -753,13 +750,10 @@ func AddDeletePRBranchComment(doer *User, repo *Repository, issueID int64, branc
 		Issue:     issue,
 		CommitSHA: branchName,
 	}
-	comment, err := createCommentWithNoAction(sess, opts)
+	_, err = createCommentWithNoAction(sess, opts)
 	if err != nil {
 		return err
 	}
-	if err = sendCreateCommentAction(sess, opts, comment); err != nil {
-		return err
-	}
 
 	return sess.Commit()
 }
@@ -899,14 +893,10 @@ func newIssue(e *xorm.Session, doer *User, opts NewIssueOptions) (err error) {
 			OldMilestoneID: 0,
 			MilestoneID:    opts.Issue.MilestoneID,
 		}
-		comment, err := createCommentWithNoAction(e, opts)
+		_, err = createCommentWithNoAction(e, opts)
 		if err != nil {
 			return err
 		}
-
-		if err = sendCreateCommentAction(e, opts, comment); err != nil {
-			return err
-		}
 	}
 
 	if opts.IsPull {

models/issue_assignees.go

@@ -144,9 +144,6 @@ func (issue *Issue) toggleAssignee(sess *xorm.Session, doer *User, assigneeID in
 	if err != nil {
 		return false, nil, fmt.Errorf("createComment: %v", err)
 	}
-	if err = sendCreateCommentAction(sess, opts, comment); err != nil {
-		return false, nil, err
-	}
 
 	// if pull request is in the middle of creation - don't call webhook
 	if isCreate {

models/issue_comment.go

@@ -678,7 +678,7 @@ func createDeadlineComment(e *xorm.Session, doer *User, issue *Issue, newDeadlin
 	if err != nil {
 		return nil, err
 	}
-	return comment, sendCreateCommentAction(e, opts, comment)
+	return comment, nil
 }
 
 // Creates issue dependency comment
@@ -699,13 +699,10 @@ func createIssueDependencyComment(e *xorm.Session, doer *User, issue *Issue, dep
 		Issue:            issue,
 		DependentIssueID: dependentIssue.ID,
 	}
-	comment, err := createCommentWithNoAction(e, opts)
+	_, err = createCommentWithNoAction(e, opts)
 	if err != nil {
 		return
 	}
-	if err = sendCreateCommentAction(e, opts, comment); err != nil {
-		return err
-	}
 
 	opts = &CreateCommentOptions{
 		Type:             cType,
@@ -714,13 +711,10 @@ func createIssueDependencyComment(e *xorm.Session, doer *User, issue *Issue, dep
 		Issue:            dependentIssue,
 		DependentIssueID: issue.ID,
 	}
-	comment, err = createCommentWithNoAction(e, opts)
+	_, err = createCommentWithNoAction(e, opts)
 	if err != nil {
 		return
 	}
-	if err = sendCreateCommentAction(e, opts, comment); err != nil {
-		return err
-	}
 
 	return
 }

models/issue_label.go

@@ -414,13 +414,10 @@ func newIssueLabel(e *xorm.Session, issue *Issue, label *Label, doer *User) (err
 		Label:   label,
 		Content: "1",
 	}
-	comment, err := createCommentWithNoAction(e, opts)
+	_, err = createCommentWithNoAction(e, opts)
 	if err != nil {
 		return err
 	}
-	if err = sendCreateCommentAction(e, opts, comment); err != nil {
-		return err
-	}
 
 	return updateLabel(e, label)
 }
@@ -494,13 +491,10 @@ func deleteIssueLabel(e *xorm.Session, issue *Issue, label *Label, doer *User) (
 		Issue: issue,
 		Label: label,
 	}
-	comment, err := createCommentWithNoAction(e, opts)
+	_, err = createCommentWithNoAction(e, opts)
 	if err != nil {
 		return err
 	}
-	if err = sendCreateCommentAction(e, opts, comment); err != nil {
-		return err
-	}
 
 	return updateLabel(e, label)
 }

models/issue_lock.go

@@ -52,14 +52,10 @@ func updateIssueLock(opts *IssueLockOptions, lock bool) error {
 		Type:    commentType,
 		Content: opts.Reason,
 	}
-	comment, err := createCommentWithNoAction(sess, opt)
+	_, err := createCommentWithNoAction(sess, opt)
 	if err != nil {
 		return err
 	}
 
-	if err = sendCreateCommentAction(sess, opt, comment); err != nil {
-		return err
-	}
-
 	return sess.Commit()
 }

models/issue_milestone.go

@@ -394,14 +394,10 @@ func changeMilestoneAssign(e *xorm.Session, doer *User, issue *Issue, oldMilesto
 			OldMilestoneID: oldMilestoneID,
 			MilestoneID:    issue.MilestoneID,
 		}
-		comment, err := createCommentWithNoAction(e, opts)
+		_, err := createCommentWithNoAction(e, opts)
 		if err != nil {
 			return err
 		}
-
-		if err := sendCreateCommentAction(e, opts, comment); err != nil {
-			return err
-		}
 	}
 
 	return nil