From 2d27456a2a5d519fefbd4ffebcd6a26df5e0ba24 Mon Sep 17 00:00:00 2001 From: 6543 <6543@obermui.de> Date: Mon, 24 Feb 2020 21:56:37 +0100 Subject: [PATCH 1/2] fix a pagination bug --- models/issue_watch.go | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/models/issue_watch.go b/models/issue_watch.go index 343ad16cde170..c4732d784e181 100644 --- a/models/issue_watch.go +++ b/models/issue_watch.go @@ -81,7 +81,7 @@ func GetIssueWatchers(issueID int64, listOptions ListOptions) (IssueWatchList, e return getIssueWatchers(x, issueID, listOptions) } -func getIssueWatchers(e Engine, issueID int64, listOptions ListOptions) (watches IssueWatchList, err error) { +func getIssueWatchers(e Engine, issueID int64, listOptions ListOptions) (IssueWatchList, error) { sess := e. Where("`issue_watch`.issue_id = ?", issueID). And("`issue_watch`.is_watching = ?", true). @@ -89,11 +89,13 @@ func getIssueWatchers(e Engine, issueID int64, listOptions ListOptions) (watches And("`user`.prohibit_login = ?", false). Join("INNER", "`user`", "`user`.id = `issue_watch`.user_id") - if listOptions.Page == 0 { + if listOptions.Page != 0 { sess = listOptions.setSessionPagination(sess) + watches := make([]*IssueWatch, 0, listOptions.PageSize) + return watches, sess.Find(&watches) } - err = sess.Find(&watches) - return + watches := make([]*IssueWatch, 0, 8) + return watches, sess.Find(&watches) } func removeIssueWatchersByRepoID(e Engine, userID int64, repoID int64) error { From fa62db9a65a39e6a77372d3f174b020188cf798a Mon Sep 17 00:00:00 2001 From: 6543 <6543@obermui.de> Date: Mon, 24 Feb 2020 22:13:23 +0100 Subject: [PATCH 2/2] fix pagination of ListAccessTokens --- models/token.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/models/token.go b/models/token.go index ce3218d8d6479..7ad9d41676efc 100644 --- a/models/token.go +++ b/models/token.go @@ -83,7 +83,7 @@ func ListAccessTokens(uid int64, listOptions ListOptions) ([]*AccessToken, error Where("uid=?", uid). Desc("id") - if listOptions.Page == 0 { + if listOptions.Page != 0 { sess = listOptions.setSessionPagination(sess) tokens := make([]*AccessToken, 0, listOptions.PageSize)