summaryrefslogtreecommitdiffstats
path: root/routers/api/v1/org
diff options
context:
space:
mode:
authorJohn Olheiser <42128690+jolheiser@users.noreply.github.com>2019-03-18 21:29:43 -0500
committertechknowlogick <matti@mdranta.net>2019-03-18 22:29:43 -0400
commitcac9e6e7605184f5679b1ebfbe5b5805191d9a53 (patch)
tree459a00974c8eff4f9cad8843b16d2a1f476c5912 /routers/api/v1/org
parentd10a668ffc4ed2a81a8a62ee78f0885ede713ddd (diff)
downloadgitea-cac9e6e7605184f5679b1ebfbe5b5805191d9a53.tar.gz
gitea-cac9e6e7605184f5679b1ebfbe5b5805191d9a53.zip
Updates to API 404 responses (#6077)
Diffstat (limited to 'routers/api/v1/org')
-rw-r--r--routers/api/v1/org/hook.go2
-rw-r--r--routers/api/v1/org/member.go6
-rw-r--r--routers/api/v1/org/team.go4
3 files changed, 6 insertions, 6 deletions
diff --git a/routers/api/v1/org/hook.go b/routers/api/v1/org/hook.go
index 84559a958e..5825de2c1c 100644
--- a/routers/api/v1/org/hook.go
+++ b/routers/api/v1/org/hook.go
@@ -164,7 +164,7 @@ func DeleteHook(ctx *context.APIContext) {
hookID := ctx.ParamsInt64(":id")
if err := models.DeleteWebhookByOrgID(org.ID, hookID); err != nil {
if models.IsErrWebhookNotExist(err) {
- ctx.Status(404)
+ ctx.NotFound()
} else {
ctx.Error(500, "DeleteWebhookByOrgID", err)
}
diff --git a/routers/api/v1/org/member.go b/routers/api/v1/org/member.go
index b66d86c660..33c906ec6a 100644
--- a/routers/api/v1/org/member.go
+++ b/routers/api/v1/org/member.go
@@ -135,11 +135,11 @@ func IsMember(ctx *context.APIContext) {
} else if userToCheckIsMember {
ctx.Status(204)
} else {
- ctx.Status(404)
+ ctx.NotFound()
}
return
} else if ctx.User.ID == userToCheck.ID {
- ctx.Status(404)
+ ctx.NotFound()
return
}
}
@@ -177,7 +177,7 @@ func IsPublicMember(ctx *context.APIContext) {
if userToCheck.IsPublicMember(ctx.Org.Organization.ID) {
ctx.Status(204)
} else {
- ctx.Status(404)
+ ctx.NotFound()
}
}
diff --git a/routers/api/v1/org/team.go b/routers/api/v1/org/team.go
index a1916db00b..ca3de9f04e 100644
--- a/routers/api/v1/org/team.go
+++ b/routers/api/v1/org/team.go
@@ -242,7 +242,7 @@ func GetTeamMembers(ctx *context.APIContext) {
ctx.Error(500, "IsOrganizationMember", err)
return
} else if !isMember {
- ctx.Status(404)
+ ctx.NotFound()
return
}
team := ctx.Org.Team
@@ -391,7 +391,7 @@ func getRepositoryByParams(ctx *context.APIContext) *models.Repository {
repo, err := models.GetRepositoryByName(ctx.Org.Team.OrgID, ctx.Params(":reponame"))
if err != nil {
if models.IsErrRepoNotExist(err) {
- ctx.Status(404)
+ ctx.NotFound()
} else {
ctx.Error(500, "GetRepositoryByName", err)
}