Fix bug about ListOptions and stars/watchers pagnation (#14556) (#14573)

* Fix bug about ListOptions and stars/watchers pagnation

* fix unit test
This commit is contained in:
6543 2021-02-05 22:11:15 +01:00 committed by GitHub
parent da7a525c5c
commit d0174d45ed
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 11 additions and 7 deletions

View File

@ -18,7 +18,7 @@ func TestGetCommitStatuses(t *testing.T) {
sha1 := "1234123412341234123412341234123412341234" sha1 := "1234123412341234123412341234123412341234"
statuses, maxResults, err := GetCommitStatuses(repo1, sha1, &CommitStatusOptions{}) statuses, maxResults, err := GetCommitStatuses(repo1, sha1, &CommitStatusOptions{ListOptions: ListOptions{Page: 1, PageSize: 50}})
assert.NoError(t, err) assert.NoError(t, err)
assert.Equal(t, int(maxResults), 5) assert.Equal(t, int(maxResults), 5)
assert.Len(t, statuses, 5) assert.Len(t, statuses, 5)

View File

@ -16,13 +16,13 @@ type ListOptions struct {
Page int // start from 1 Page int // start from 1
} }
func (opts ListOptions) getPaginatedSession() *xorm.Session { func (opts *ListOptions) getPaginatedSession() *xorm.Session {
opts.setDefaultValues() opts.setDefaultValues()
return x.Limit(opts.PageSize, (opts.Page-1)*opts.PageSize) return x.Limit(opts.PageSize, (opts.Page-1)*opts.PageSize)
} }
func (opts ListOptions) setSessionPagination(sess *xorm.Session) *xorm.Session { func (opts *ListOptions) setSessionPagination(sess *xorm.Session) *xorm.Session {
opts.setDefaultValues() opts.setDefaultValues()
if opts.PageSize <= 0 { if opts.PageSize <= 0 {
@ -31,21 +31,21 @@ func (opts ListOptions) setSessionPagination(sess *xorm.Session) *xorm.Session {
return sess.Limit(opts.PageSize, (opts.Page-1)*opts.PageSize) return sess.Limit(opts.PageSize, (opts.Page-1)*opts.PageSize)
} }
func (opts ListOptions) setEnginePagination(e Engine) Engine { func (opts *ListOptions) setEnginePagination(e Engine) Engine {
opts.setDefaultValues() opts.setDefaultValues()
return e.Limit(opts.PageSize, (opts.Page-1)*opts.PageSize) return e.Limit(opts.PageSize, (opts.Page-1)*opts.PageSize)
} }
// GetStartEnd returns the start and end of the ListOptions // GetStartEnd returns the start and end of the ListOptions
func (opts ListOptions) GetStartEnd() (start, end int) { func (opts *ListOptions) GetStartEnd() (start, end int) {
opts.setDefaultValues() opts.setDefaultValues()
start = (opts.Page - 1) * opts.PageSize start = (opts.Page - 1) * opts.PageSize
end = start + opts.Page end = start + opts.Page
return return
} }
func (opts ListOptions) setDefaultValues() { func (opts *ListOptions) setDefaultValues() {
if opts.PageSize <= 0 { if opts.PageSize <= 0 {
opts.PageSize = setting.API.DefaultPagingNum opts.PageSize = setting.API.DefaultPagingNum
} }

View File

@ -692,7 +692,10 @@ func RenderUserCards(ctx *context.Context, total int, getter func(opts models.Li
pager := context.NewPagination(total, models.ItemsPerPage, page, 5) pager := context.NewPagination(total, models.ItemsPerPage, page, 5)
ctx.Data["Page"] = pager ctx.Data["Page"] = pager
items, err := getter(models.ListOptions{Page: pager.Paginater.Current()}) items, err := getter(models.ListOptions{
Page: pager.Paginater.Current(),
PageSize: models.ItemsPerPage,
})
if err != nil { if err != nil {
ctx.ServerError("getter", err) ctx.ServerError("getter", err)
return return
@ -723,6 +726,7 @@ func Stars(ctx *context.Context) {
func Forks(ctx *context.Context) { func Forks(ctx *context.Context) {
ctx.Data["Title"] = ctx.Tr("repos.forks") ctx.Data["Title"] = ctx.Tr("repos.forks")
// TODO: need pagination
forks, err := ctx.Repo.Repository.GetForks(models.ListOptions{}) forks, err := ctx.Repo.Repository.GetForks(models.ListOptions{})
if err != nil { if err != nil {
ctx.ServerError("GetForks", err) ctx.ServerError("GetForks", err)