diff options
| author | zeripath <art27@cantab.net> | 2019-07-25 11:55:06 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2019-07-25 11:55:06 +0100 |
| commit | 734aa965088fb72330f4ad744c187c614cce3d8b (patch) | |
| tree | 96c97d3333b54e327a2ff5b6a5549b9567157eff /routers/api/v1/repo/status.go | |
| parent | 5efd3630bc21d4b0ba6ff492d16d4c7e2814dd1f (diff) | |
| download | gitea-734aa965088fb72330f4ad744c187c614cce3d8b.tar.gz gitea-734aa965088fb72330f4ad744c187c614cce3d8b.zip | |
Fix #7136: Add paging and extend API GetCommitStatuses (#7141)
* Fix #7136: Add paging and extend API GetCommitStatuses
* update swagger
* Update routers/api/v1/repo/status.go
Co-Authored-By: techknowlogick <matti@mdranta.net>
* Update routers/api/v1/repo/status.go
Co-Authored-By: techknowlogick <matti@mdranta.net>
* Update routers/api/v1/repo/status.go
Co-Authored-By: techknowlogick <matti@mdranta.net>
* Apply suggestions from code review
Diffstat (limited to 'routers/api/v1/repo/status.go')
| -rw-r--r-- | routers/api/v1/repo/status.go | 51 |
1 files changed, 46 insertions, 5 deletions
diff --git a/routers/api/v1/repo/status.go b/routers/api/v1/repo/status.go index 1aa4f4e1a1..e4afb599ee 100644 --- a/routers/api/v1/repo/status.go +++ b/routers/api/v1/repo/status.go @@ -89,6 +89,23 @@ func GetCommitStatuses(ctx *context.APIContext) { // description: sha of the commit // type: string // required: true + // - name: page + // in: query + // description: page number of results + // type: integer + // required: false + // - name: sort + // in: query + // description: type of sort + // type: string + // enum: [oldest, recentupdate, leastupdate, leastindex, highestindex] + // required: false + // - name: state + // in: query + // description: type of state + // type: string + // enum: [pending, success, error, failure, warning] + // required: false // responses: // "200": // "$ref": "#/responses/StatusList" @@ -118,6 +135,23 @@ func GetCommitStatusesByRef(ctx *context.APIContext) { // description: name of branch/tag/commit // type: string // required: true + // - name: page + // in: query + // description: page number of results + // type: integer + // required: false + // - name: sort + // in: query + // description: type of sort + // type: string + // enum: [oldest, recentupdate, leastupdate, leastindex, highestindex] + // required: false + // - name: state + // in: query + // description: type of state + // type: string + // enum: [pending, success, error, failure, warning] + // required: false // responses: // "200": // "$ref": "#/responses/StatusList" @@ -131,11 +165,13 @@ func getCommitStatuses(ctx *context.APIContext, sha string) { } repo := ctx.Repo.Repository - page := ctx.ParamsInt("page") - - statuses, err := models.GetCommitStatuses(repo, sha, page) + statuses, _, err := models.GetCommitStatuses(repo, sha, &models.CommitStatusOptions{ + Page: ctx.QueryInt("page"), + SortType: ctx.QueryTrim("sort"), + State: ctx.QueryTrim("state"), + }) if err != nil { - ctx.Error(500, "GetCommitStatuses", fmt.Errorf("GetCommitStatuses[%s, %s, %d]: %v", repo.FullName(), sha, page, err)) + ctx.Error(500, "GetCommitStatuses", fmt.Errorf("GetCommitStatuses[%s, %s, %d]: %v", repo.FullName(), sha, ctx.QueryInt("page"), err)) return } @@ -180,6 +216,11 @@ func GetCombinedCommitStatusByRef(ctx *context.APIContext) { // description: name of branch/tag/commit // type: string // required: true + // - name: page + // in: query + // description: page number of results + // type: integer + // required: false // responses: // "200": // "$ref": "#/responses/Status" @@ -190,7 +231,7 @@ func GetCombinedCommitStatusByRef(ctx *context.APIContext) { } repo := ctx.Repo.Repository - page := ctx.ParamsInt("page") + page := ctx.QueryInt("page") statuses, err := models.GetLatestCommitStatus(repo, sha, page) if err != nil { |