From 9e9ca66467116e9079a2639c00e9e623aca23015 Mon Sep 17 00:00:00 2001 From: ᴜɴᴋɴᴡᴏɴ Date: Mon, 16 Mar 2020 01:22:27 +0800 Subject: refactor: unify error handling in routing layer --- internal/route/api/v1/admin/org_repo.go | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) (limited to 'internal/route/api/v1/admin/org_repo.go') diff --git a/internal/route/api/v1/admin/org_repo.go b/internal/route/api/v1/admin/org_repo.go index b17b1462..63a638fa 100644 --- a/internal/route/api/v1/admin/org_repo.go +++ b/internal/route/api/v1/admin/org_repo.go @@ -7,13 +7,12 @@ package admin import ( "gogs.io/gogs/internal/context" "gogs.io/gogs/internal/db" - "gogs.io/gogs/internal/db/errors" ) func GetRepositoryByParams(c *context.APIContext) *db.Repository { repo, err := db.GetRepositoryByName(c.Org.Team.OrgID, c.Params(":reponame")) if err != nil { - c.NotFoundOrServerError("GetRepositoryByName", errors.IsRepoNotExist, err) + c.NotFoundOrError(err, "get repository by name") return nil } return repo @@ -25,7 +24,7 @@ func AddTeamRepository(c *context.APIContext) { return } if err := c.Org.Team.AddRepository(repo); err != nil { - c.ServerError("AddRepository", err) + c.Error(err, "add repository") return } @@ -38,7 +37,7 @@ func RemoveTeamRepository(c *context.APIContext) { return } if err := c.Org.Team.RemoveRepository(repo.ID); err != nil { - c.ServerError("RemoveRepository", err) + c.Error(err, "remove repository") return } -- cgit v1.2.3