Use the original field name to reduce changes

Author: lunny

models/issues/issue_search.go

@@ -21,7 +21,7 @@ import (
 
 // IssuesOptions represents options of an issue.
 type IssuesOptions struct { //nolint
-	*db.ListOptions
+	Paginator          *db.ListOptions
 	RepoIDs            []int64 // overwrites RepoCond if the length is not 0
 	AllPublic          bool    // include also all public repositories
 	RepoCond           builder.Cond
@@ -100,15 +100,15 @@ func applySorts(sess *xorm.Session, sortType string, priorityRepoID int64) {
 }
 
 func applyLimit(sess *xorm.Session, opts *IssuesOptions) *xorm.Session {
-	if opts.ListOptions.IsListAll() {
+	if opts.Paginator == nil || opts.Paginator.IsListAll() {
 		return sess
 	}
 
 	start := 0
-	if opts.ListOptions.Page > 0 {
-		start = (opts.ListOptions.Page - 1) * opts.ListOptions.PageSize
+	if opts.Paginator.Page > 1 {
+		start = (opts.Paginator.Page - 1) * opts.Paginator.PageSize
 	}
-	sess.Limit(opts.ListOptions.PageSize, start)
+	sess.Limit(opts.Paginator.PageSize, start)
 
 	return sess
 }

models/issues/issue_test.go

@@ -164,7 +164,7 @@ func TestIssues(t *testing.T) {
 			issues_model.IssuesOptions{
 				RepoCond: builder.In("repo_id", 1, 3),
 				SortType: "oldest",
-				ListOptions: &db.ListOptions{
+				Paginator: &db.ListOptions{
 					Page:     1,
 					PageSize: 4,
 				},
@@ -174,7 +174,7 @@ func TestIssues(t *testing.T) {
 		{
 			issues_model.IssuesOptions{
 				LabelIDs: []int64{1},
-				ListOptions: &db.ListOptions{
+				Paginator: &db.ListOptions{
 					Page:     1,
 					PageSize: 4,
 				},
@@ -184,7 +184,7 @@ func TestIssues(t *testing.T) {
 		{
 			issues_model.IssuesOptions{
 				LabelIDs: []int64{1, 2},
-				ListOptions: &db.ListOptions{
+				Paginator: &db.ListOptions{
 					Page:     1,
 					PageSize: 4,
 				},
@@ -326,7 +326,7 @@ func TestCorrectIssueStats(t *testing.T) {
 
 	// Now we will get all issueID's that match the "Bugs are nasty" query.
 	issues, err := issues_model.Issues(context.TODO(), &issues_model.IssuesOptions{
-		ListOptions: &db.ListOptions{
+		Paginator: &db.ListOptions{
 			PageSize: issueAmount,
 		},
 		RepoIDs: []int64{1},

modules/indexer/internal/paginator.go

@@ -10,25 +10,25 @@ import (
 )
 
 // ParsePaginator parses a db.Paginator into a skip and limit
-func ParsePaginator(listOptions *db.ListOptions, max ...int) (int, int) {
+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 listOptions == nil || listOptions.IsListAll() {
+	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 listOptions.PageSize == 0 {
+	if paginator.PageSize == 0 {
 		// Do not return any results when searching, it's used to get the total count only.
 		return 0, 0
 	}
 
-	return listOptions.GetSkipTake()
+	return paginator.GetSkipTake()
 }

modules/indexer/issues/bleve/bleve.go

@@ -266,7 +266,7 @@ func (b *Indexer) Search(ctx context.Context, options *internal.SearchOptions) (
 		indexerQuery = bleve.NewMatchAllQuery()
 	}
 
-	skip, limit := indexer_internal.ParsePaginator(options.ListOptions)
+	skip, limit := indexer_internal.ParsePaginator(options.Paginator)
 	search := bleve.NewSearchRequestOptions(indexerQuery, limit, skip, false)
 
 	if options.SortBy == "" {

modules/indexer/issues/db/db.go

@@ -79,7 +79,7 @@ func (i *Indexer) Search(ctx context.Context, options *internal.SearchOptions) (
 	}
 
 	// If pagesize == 0, return total count only. It's a special case for search count.
-	if options.ListOptions != nil && options.ListOptions.PageSize == 0 {
+	if options.Paginator != nil && options.Paginator.PageSize == 0 {
 		total, err := issue_model.CountIssues(ctx, opt, cond)
 		if err != nil {
 			return nil, err

modules/indexer/issues/db/options.go

@@ -50,7 +50,7 @@ func ToDBOptions(ctx context.Context, options *internal.SearchOptions) (*issue_m
 	}
 
 	opts := &issue_model.IssuesOptions{
-		ListOptions:        options.ListOptions,
+		Paginator:          options.Paginator,
 		RepoIDs:            options.RepoIDs,
 		AllPublic:          options.AllPublic,
 		RepoCond:           nil,

modules/indexer/issues/dboptions.go

@@ -65,7 +65,7 @@ func ToSearchOptions(keyword string, opts *issues_model.IssuesOptions) *SearchOp
 		searchOpt.UpdatedBeforeUnix = optional.Some(opts.UpdatedBeforeUnix)
 	}
 
-	searchOpt.ListOptions = opts.ListOptions
+	searchOpt.Paginator = opts.Paginator
 
 	switch opts.SortType {
 	case "":

modules/indexer/issues/elasticsearch/elasticsearch.go

@@ -248,7 +248,7 @@ func (b *Indexer) Search(ctx context.Context, options *internal.SearchOptions) (
 	// TODO: make it configurable since it's configurable in elasticsearch
 	const maxPageSize = 10000
 
-	skip, limit := indexer_internal.ParsePaginator(options.ListOptions, maxPageSize)
+	skip, limit := indexer_internal.ParsePaginator(options.Paginator, maxPageSize)
 	searchResult, err := b.inner.Client.Search().
 		Index(b.inner.VersionedIndexName()).
 		Query(query).

modules/indexer/issues/indexer.go

@@ -308,7 +308,7 @@ func SearchIssues(ctx context.Context, opts *SearchOptions) ([]int64, int64, err
 
 // CountIssues counts issues by options. It is a shortcut of SearchIssues(ctx, opts) but only returns the total count.
 func CountIssues(ctx context.Context, opts *SearchOptions) (int64, error) {
-	opts = opts.Copy(func(options *SearchOptions) { options.ListOptions = &db_model.ListOptions{PageSize: 0} })
+	opts = opts.Copy(func(options *SearchOptions) { options.Paginator = &db_model.ListOptions{PageSize: 0} })
 
 	_, total, err := SearchIssues(ctx, opts)
 	return total, err

modules/indexer/issues/indexer_test.go

@@ -397,7 +397,7 @@ func searchIssueWithPaginator(t *testing.T) {
 	}{
 		{
 			SearchOptions{
-				ListOptions: &db.ListOptions{
+				Paginator: &db.ListOptions{
 					PageSize: 5,
 				},
 			},

modules/indexer/issues/internal/model.go

@@ -106,7 +106,7 @@ type SearchOptions struct {
 	UpdatedAfterUnix  optional.Option[int64]
 	UpdatedBeforeUnix optional.Option[int64]
 
-	*db.ListOptions
+	Paginator *db.ListOptions
 
 	SortBy SortBy // sort by field
 }