Browse Source

Removed HTTP 500 error on the release page when a user deleted their account

tags/v0.9.99
Martin Hartkorn 8 years ago
parent
commit
6b3e47b103
1 changed files with 12 additions and 4 deletions
  1. 12
    4
      routers/repo/release.go

+ 12
- 4
routers/repo/release.go View File

@@ -45,8 +45,12 @@ func Releases(ctx *middleware.Context) {
if rel.TagName == rawTag {
rel.Publisher, err = models.GetUserByID(rel.PublisherID)
if err != nil {
ctx.Handle(500, "GetUserByID", err)
return
if models.IsErrUserNotExist(err) {
rel.Publisher = models.NewFakeUser()
} else {
ctx.Handle(500, "GetUserByID", err)
return
}
}
// FIXME: duplicated code.
// Get corresponding target if it's not the current branch.
@@ -105,8 +109,12 @@ func Releases(ctx *middleware.Context) {

rel.Publisher, err = models.GetUserByID(rel.PublisherID)
if err != nil {
ctx.Handle(500, "GetUserByID", err)
return
if models.IsErrUserNotExist(err) {
rel.Publisher = models.NewFakeUser()
} else {
ctx.Handle(500, "GetUserByID", err)
return
}
}
// FIXME: duplicated code.
// Get corresponding target if it's not the current branch.

Loading…
Cancel
Save