Fix hook check logic

Signed-off-by: Chengwei Yang <yangchengwei@qiyi.com>
This commit is contained in:
Chengwei Yang 2015-09-01 17:44:30 +08:00
parent 370e53da1e
commit 8bfc155aee

View file

@ -67,11 +67,11 @@ func PostHook(c *gin.Context) {
log.Warnf("ignoring hook. repo %s has no owner.", repo.FullName) log.Warnf("ignoring hook. repo %s has no owner.", repo.FullName)
c.Writer.WriteHeader(204) c.Writer.WriteHeader(204)
return return
case !repo.Hooks.Push && hook.PullRequest != nil: case !repo.Hooks.Push && hook.Commit != nil:
log.Infof("ignoring hook. repo %s is disabled.", repo.FullName) log.Infof("ignoring hook. repo %s is disabled.", repo.FullName)
c.Writer.WriteHeader(204) c.Writer.WriteHeader(204)
return return
case !repo.Hooks.PullRequest && hook.PullRequest == nil: case !repo.Hooks.PullRequest && hook.PullRequest != nil:
log.Warnf("ignoring hook. repo %s is disabled for pull requests.", repo.FullName) log.Warnf("ignoring hook. repo %s is disabled for pull requests.", repo.FullName)
c.Writer.WriteHeader(204) c.Writer.WriteHeader(204)
return return