realaravinth/forgejo-federation
1
0
Fork 0
Code Issues Pull requests 6 Projects Releases Packages Wiki Activity

Remove unneeded database loads (#814)

Browse source 
Remove unnecessary calls to repo.GetOwner() in context handlers
This commit is contained in:
Ethan Koenig 2017-02-02 07:33:56 -05:00 committed by Lunny Xiao
parent d7d094bd8a
commit 73d05a51e3
2 changed files with 2 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
modules/context/repo.go
View file

@ -185,10 +185,8 @@ func RepoAssignment(args ...bool) macaron.Handler {
ctx.Handle(500, "GetRepositoryByName", err) ctx.Handle(500, "GetRepositoryByName", err)
} }
return return
} else if err = repo.GetOwner(); err != nil {
ctx.Handle(500, "GetOwner", err)
return
} }
repo.Owner = owner
// Admin has super access. // Admin has super access.
if ctx.IsSigned && ctx.User.IsAdmin { if ctx.IsSigned && ctx.User.IsAdmin {

4
routers/api/v1/api.go
View file

@ -57,10 +57,8 @@ func repoAssignment() macaron.Handler {
ctx.Error(500, "GetRepositoryByName", err) ctx.Error(500, "GetRepositoryByName", err)
} }
return return
} else if err = repo.GetOwner(); err != nil {
ctx.Error(500, "GetOwner", err)
return
} }
repo.Owner = owner
if ctx.IsSigned && ctx.User.IsAdmin { if ctx.IsSigned && ctx.User.IsAdmin {
ctx.Repo.AccessMode = models.AccessModeOwner ctx.Repo.AccessMode = models.AccessModeOwner