diff options
| author | Antoine GIRARD <sapk@users.noreply.github.com> | 2017-07-06 15:30:19 +0200 |
|---|---|---|
| committer | Lunny Xiao <xiaolunwen@gmail.com> | 2017-07-06 21:30:19 +0800 |
| commit | 30787e48f219b23701f660ba0b99b326ab82e997 (patch) | |
| tree | 005dfef495149e81e78ad31c251cc3a3e2709870 /routers/api/v1/api.go | |
| parent | 2ef33b53381f68dc2af88eabfe5f35c3dbc94412 (diff) | |
| download | gitea-30787e48f219b23701f660ba0b99b326ab82e997.tar.gz gitea-30787e48f219b23701f660ba0b99b326ab82e997.zip | |
Improve org error handling (#2117)
* Improve ErrOrgNotExist type
Return new error type
Use good error check
Use new method to check error
Update tests
* Fix unchanged method name report
Diffstat (limited to 'routers/api/v1/api.go')
| -rw-r--r-- | routers/api/v1/api.go | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/routers/api/v1/api.go b/routers/api/v1/api.go index 56bb3b0f3b..14a8d59855 100644 --- a/routers/api/v1/api.go +++ b/routers/api/v1/api.go @@ -208,12 +208,12 @@ func orgAssignment(args ...bool) macaron.Handler { var err error if assignOrg { - ctx.Org.Organization, err = models.GetUserByName(ctx.Params(":orgname")) + ctx.Org.Organization, err = models.GetOrgByName(ctx.Params(":orgname")) if err != nil { - if models.IsErrUserNotExist(err) { + if models.IsErrOrgNotExist(err) { ctx.Status(404) } else { - ctx.Error(500, "GetUserByName", err) + ctx.Error(500, "GetOrgByName", err) } return } |