You cannot select more than 25 topics Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
gitea/services
delvh 675c14aba6
Ignore error when retrieving changed PR review files (#21487) (#21524)
When a PR reviewer reviewed a file on a commit that was later gc'ed,
they would always get a `500` response from then on when loading the PR.
This PR simply ignores that error and instead marks all files as
unchanged.
This approach was chosen as the only feasible option without diving into
**a lot** of error handling.

Fixes #21392
Backport of #21487

Co-authored-by: Lunny Xiao <xiaolunwen@gmail.com>
Co-authored-by: wxiaoguang <wxiaoguang@gmail.com>
2 years ago
..
agit move agit dependcy (#19998) 3 years ago
asymkey Move issues related files into models/issues (#19931) 3 years ago
attachment Move almost all functions' parameter db.Engine to context.Context (#19748) 3 years ago
auth EscapeFilter the group dn membership (#20200) (#20254) 3 years ago
automerge Move issues related files into models/issues (#19931) 3 years ago
comments Move issues related files into models/issues (#19931) 3 years ago
context Move almost all functions' parameter db.Engine to context.Context (#19748) 3 years ago
cron Move some code into models/git (#19879) 3 years ago
externalaccount Refactor auth package (#17962) 3 years ago
forms Move issues related files into models/issues (#19931) 3 years ago
gitdiff Ignore error when retrieving changed PR review files (#21487) (#21524) 2 years ago
issue Check if project has the same repository id with issue when assign project to issue (#20133) (#20188) 3 years ago
lfs Move some code into models/git (#19879) 3 years ago
mailer Move issues related files into models/issues (#19931) 3 years ago
migrations Foreign ID conflicts if ID is 0 for each item (#21271) (#21272) 2 years ago
mirror Always try to fetch repo for mirrors (#19975) 3 years ago
org Move almost all functions' parameter db.Engine to context.Context (#19748) 3 years ago
packages Use correct context to get package content (#20673) (#20676) 3 years ago
pull Do DB update after merge in hammer context (#21401) (#21416) 2 years ago
release Add more checks in migration code (#21011) (#21050) 3 years ago
repository Fix incorrect notification commit url (#21479) (#21483) 2 years ago
task Use a more general (and faster) method to sanitize URLs with credentials (#19239) 3 years ago
user Move almost all functions' parameter db.Engine to context.Context (#19748) 3 years ago
webhook Display total commit count in hook message (#21400) (#21481) 2 years ago
wiki Move some helper files out of models (#19355) 3 years ago