Minor improve on error handling

tvon · Feb 10, 2017 · f5dc436 · f5dc436
1 parent d9d329b
commit f5dc436
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 4 deletions.
diff --git a/models/pull.go b/models/pull.go
@@ -364,7 +364,7 @@ func (pr *PullRequest) testPatch() (err error) {
 	log.Trace("PullRequest[%d].testPatch (patchPath): %s", pr.ID, patchPath)
 
 	if err := pr.BaseRepo.UpdateLocalCopyBranch(pr.BaseBranch); err != nil {
-		return fmt.Errorf("UpdateLocalCopy: %v", err)
+		return fmt.Errorf("UpdateLocalCopy [%d]: %v", pr.BaseRepoID, err)
 	}
 
 	pr.Status = PULL_REQUEST_STATUS_CHECKING

diff --git a/modules/context/context.go b/modules/context/context.go
@@ -118,9 +118,6 @@ func (ctx *Context) NotFoundOrServerError(title string, errck func(error) bool,
 }
 
 func (ctx *Context) HandleText(status int, title string) {
-	if (status/100 == 4) || (status/100 == 5) {
-		log.Error(4, "%s", title)
-	}
 	ctx.PlainText(status, []byte(title))
 }