Browse Source

fix duplicated webhook when creating issue with assignees (#7681)

tags/v1.11.0-dev
Lunny Xiao GitHub 6 years ago
parent
commit
fdd2bd5245
1 changed files with 6 additions and 3 deletions
  1. +6
    -3
      models/issue_assignees.go

+ 6
- 3
models/issue_assignees.go View File

@@ -142,11 +142,15 @@ func (issue *Issue) ChangeAssignee(doer *User, assigneeID int64) (err error) {
return err
}

return sess.Commit()
if err := sess.Commit(); err != nil {
return err
}

go HookQueue.Add(issue.RepoID)
return nil
}

func (issue *Issue) changeAssignee(sess *xorm.Session, doer *User, assigneeID int64, isCreate bool) (err error) {

// Update the assignee
removed, err := updateIssueAssignee(sess, issue, assigneeID)
if err != nil {
@@ -209,7 +213,6 @@ func (issue *Issue) changeAssignee(sess *xorm.Session, doer *User, assigneeID in
return nil
}
}
go HookQueue.Add(issue.RepoID)
return nil
}



Loading…
Cancel
Save