9ca245ad96
This PR uses `db.ListOptions` instead of `Paginor` to make the code simpler. And it also fixed the performance problem when viewing /pulls or /issues. Before the counting in fact will also do the search. --------- Co-authored-by: Jason Song <i@wolfogre.com> Co-authored-by: silverwind <me@silverwind.io> (cherry picked from commit 3f26fe2fa2c7141c9e622297e50a70f3e0003e4d)
34 lines
929 B
Go
34 lines
929 B
Go
// Copyright 2023 The Gitea Authors. All rights reserved.
|
|
// SPDX-License-Identifier: MIT
|
|
|
|
package internal
|
|
|
|
import (
|
|
"math"
|
|
|
|
"code.gitea.io/gitea/models/db"
|
|
)
|
|
|
|
// ParsePaginator parses a db.Paginator into a skip and limit
|
|
func ParsePaginator(paginator *db.ListOptions, max ...int) (int, int) {
|
|
// Use a very large number to indicate no limit
|
|
unlimited := math.MaxInt32
|
|
if len(max) > 0 {
|
|
// Some indexer engines have a limit on the page size, respect that
|
|
unlimited = max[0]
|
|
}
|
|
|
|
if paginator == nil || paginator.IsListAll() {
|
|
// It shouldn't happen. In actual usage scenarios, there should not be requests to search all.
|
|
// But if it does happen, respect it and return "unlimited".
|
|
// And it's also useful for testing.
|
|
return 0, unlimited
|
|
}
|
|
|
|
if paginator.PageSize == 0 {
|
|
// Do not return any results when searching, it's used to get the total count only.
|
|
return 0, 0
|
|
}
|
|
|
|
return paginator.GetSkipTake()
|
|
}
|