diff options
author | yp05327 | 2023-02-24 14:18:52 +0900 |
---|---|---|
committer | GitHub | 2023-02-24 13:18:52 +0800 |
commit | 5b87c05a95a3eab1cf7f9a9f23052fee9eedd9ac (patch) | |
tree | 2ab60e33e5961b8d5a2774ff8c5127f1ac76c559 /models | |
parent | 0bc8bb3cc4f003e70bfee75863b74c2243c6d23c (diff) |
improve FindProjects (#23085)
I found `FindAndCount` which can `Find` and `Count` in the same time
Maybe it is better to use it in `FindProjects`
---------
Co-authored-by: Lunny Xiao <xiaolunwen@gmail.com>
Diffstat (limited to 'models')
-rw-r--r-- | models/project/project.go | 13 |
1 files changed, 3 insertions, 10 deletions
diff --git a/models/project/project.go b/models/project/project.go index 931ef4467..46b5c07c4 100644 --- a/models/project/project.go +++ b/models/project/project.go @@ -217,16 +217,8 @@ func CountProjects(ctx context.Context, opts SearchOptions) (int64, error) { // FindProjects returns a list of all projects that have been created in the repository func FindProjects(ctx context.Context, opts SearchOptions) ([]*Project, int64, error) { - e := db.GetEngine(ctx) + e := db.GetEngine(ctx).Where(opts.toConds()) projects := make([]*Project, 0, setting.UI.IssuePagingNum) - cond := opts.toConds() - - count, err := e.Where(cond).Count(new(Project)) - if err != nil { - return nil, 0, fmt.Errorf("Count: %w", err) - } - - e = e.Where(cond) if opts.Page > 0 { e = e.Limit(setting.UI.IssuePagingNum, (opts.Page-1)*setting.UI.IssuePagingNum) @@ -243,7 +235,8 @@ func FindProjects(ctx context.Context, opts SearchOptions) ([]*Project, int64, e e.Asc("created_unix") } - return projects, count, e.Find(&projects) + count, err := e.FindAndCount(&projects) + return projects, count, err } // NewProject creates a new Project |