Ignore ErrIssueNotExist on comment.LoadDepIssueDetails()

6543 · 6543 · commit 7c7eb9e58230 · 2020-08-30T18:19:29.000+02:00
diff --git a/routers/repo/issue.go b/routers/repo/issue.go
@@ -1079,8 +1079,10 @@ func ViewIssue(ctx *context.Context) {
 			}
 		} else if comment.Type == models.CommentTypeRemoveDependency || comment.Type == models.CommentTypeAddDependency {
 			if err = comment.LoadDepIssueDetails(); err != nil {
-				ctx.ServerError("LoadDepIssueDetails", err)
-				return
+				if !models.IsErrIssueNotExist(err) {
+					ctx.ServerError("LoadDepIssueDetails", err)
+					return
+				}
 			}
 		} else if comment.Type == models.CommentTypeCode || comment.Type == models.CommentTypeReview {
 			comment.RenderedContent = string(markdown.Render([]byte(comment.Content), ctx.Repo.RepoLink,

Original file line number	Diff line number	Diff line change
`@@ -1079,8 +1079,10 @@ func ViewIssue(ctx *context.Context) {`
`1079`	`1079`	`}`
`1080`	`1080`	`} else if comment.Type == models.CommentTypeRemoveDependency \|\| comment.Type == models.CommentTypeAddDependency {`
`1081`	`1081`	`if err = comment.LoadDepIssueDetails(); err != nil {`
`1082`		`- ctx.ServerError("LoadDepIssueDetails", err)`
`1083`		`- return`
	`1082`	`+ if !models.IsErrIssueNotExist(err) {`
	`1083`	`+ ctx.ServerError("LoadDepIssueDetails", err)`
	`1084`	`+ return`
	`1085`	`+ }`
`1084`	`1086`	`}`
`1085`	`1087`	`} else if comment.Type == models.CommentTypeCode \|\| comment.Type == models.CommentTypeReview {`
`1086`	`1088`	`comment.RenderedContent = string(markdown.Render([]byte(comment.Content), ctx.Repo.RepoLink,`