diff options
author | Yarden Shoham <git@yardenshoham.com> | 2023-05-17 03:12:37 +0300 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-05-17 08:12:37 +0800 |
commit | 0a3c4d4a595cc7e12462dde393ed64186260f26b (patch) | |
tree | 5c98ba3ef7b0b85e3ba2af61e7b33ae27d9b9261 /models | |
parent | 584c0789fa72f5e21167ba5f1370b7918e604247 (diff) | |
download | gitea-0a3c4d4a595cc7e12462dde393ed64186260f26b.tar.gz gitea-0a3c4d4a595cc7e12462dde393ed64186260f26b.zip |
Fix team members API endpoint pagination (#24754)
Now it's 1-based instead of 0-based
- Fixes #24747
### Before

### After

## :warning: BREAKING :warning:
Previous API consumers may have relied on the 0-based pagination of this
endpoint. The page numbering now starts at 1, as documented.
Signed-off-by: Yarden Shoham <git@yardenshoham.com>
Diffstat (limited to 'models')
-rw-r--r-- | models/organization/team_user.go | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/models/organization/team_user.go b/models/organization/team_user.go index 816daf3d34..768dc24c50 100644 --- a/models/organization/team_user.go +++ b/models/organization/team_user.go @@ -63,8 +63,8 @@ func GetTeamMembers(ctx context.Context, opts *SearchMembersOptions) ([]*user_mo Where(builder.Eq{"team_id": opts.TeamID}), ) } - if opts.PageSize > 0 && opts.Page > -1 { - sess = sess.Limit(opts.PageSize, opts.Page*opts.PageSize) + if opts.PageSize > 0 && opts.Page > 0 { + sess = sess.Limit(opts.PageSize, (opts.Page-1)*opts.PageSize) } if err := sess.OrderBy("full_name, name").Find(&members); err != nil { return nil, err |