Use DEFAULT_PAGING_NUM instead of MAX_RESPONSE_ITEMS in ListOptions (go-gitea#11831)

CirnoT · techknowlogick · Yohann Delafollye · commit 26bb7245d2ff · 2020-07-31T14:30:02.000-04:00
Co-authored-by: techknowlogick &lt;techknowlogick@gitea.io&gt;
diff --git a/models/list_options.go b/models/list_options.go
@@ -38,7 +38,10 @@ func (opts ListOptions) setEnginePagination(e Engine) Engine {
 }
 
 func (opts ListOptions) setDefaultValues() {
-	if opts.PageSize <= 0 || opts.PageSize > setting.API.MaxResponseItems {
+	if opts.PageSize <= 0 {
+		opts.PageSize = setting.API.DefaultPagingNum
+	}
+	if opts.PageSize > setting.API.MaxResponseItems {
 		opts.PageSize = setting.API.MaxResponseItems
 	}
 	if opts.Page <= 0 {

Original file line number	Diff line number	Diff line change
`@@ -38,7 +38,10 @@ func (opts ListOptions) setEnginePagination(e Engine) Engine {`
`38`	`38`	`}`
`39`	`39`
`40`	`40`	`func (opts ListOptions) setDefaultValues() {`
`41`		`- if opts.PageSize <= 0 \|\| opts.PageSize > setting.API.MaxResponseItems {`
	`41`	`+ if opts.PageSize <= 0 {`
	`42`	`+ opts.PageSize = setting.API.DefaultPagingNum`
	`43`	`+ }`
	`44`	`+ if opts.PageSize > setting.API.MaxResponseItems {`
`42`	`45`	`opts.PageSize = setting.API.MaxResponseItems`
`43`	`46`	`}`
`44`	`47`	`if opts.Page <= 0 {`