upstream/harbor
1
0
Fork 0
mirror of https://github.com/goharbor/harbor synced 2025-04-27 22:07:18 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix pagination for users and users search apis

Browse Source 
Signed-off-by: He Weiwei <hweiwei@vmware.com>
This commit is contained in:
He Weiwei 2019-03-21 13:03:18 +08:00
parent cd33d312fb
commit 79235fffd1
2 changed files with 11 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/common/dao/dao_test.go
View File

@ -318,6 +318,11 @@ func TestListUsers(t *testing.T) {
if users2[0].Username != username {
t.Errorf("The username in result list does not match, expected: %s, actual: %s", username, users2[0].Username)
}
users3, err := ListUsers(&models.UserQuery{Username: username, Pagination: &models.Pagination{Page: 2, Size: 1}})
if len(users3) != 0 {
t.Errorf("Expect no user in list, but the acutal length is %d, the list: %+v", len(users3), users3)
}
}
func TestResetUserPassword(t *testing.T) {

9
src/common/dao/user.go
View File

@ -106,10 +106,13 @@ func GetTotalOfUsers(query *models.UserQuery) (int64, error) {
// ListUsers lists all users according to different conditions.
func ListUsers(query *models.UserQuery) ([]models.User, error) {
qs := userQueryConditions(query)
if query != nil && query.Pagination != nil {
offset := (query.Pagination.Page - 1) * query.Pagination.Size
qs = qs.Offset(offset).Limit(query.Pagination.Size)
}
users := []models.User{}
_, err := userQueryConditions(query).Limit(-1).
OrderBy("username").
All(&users)
_, err := qs.OrderBy("username").All(&users)
return users, err
}