diff options
author | Michael Muré <batolettre@gmail.com> | 2018-08-15 14:53:50 +0200 |
---|---|---|
committer | Michael Muré <batolettre@gmail.com> | 2018-08-15 14:53:50 +0200 |
commit | bc1fb34c24e985b7a9d50c559d7a42726a3a2007 (patch) | |
tree | 2cfc71c61d8dd491462ec77ab8826e50c1728181 /graphql/connections/gen_bug.go | |
parent | 5c568a362b73cf163b06bc7371982f9a7ceaaf29 (diff) | |
download | git-bug-bc1fb34c24e985b7a9d50c559d7a42726a3a2007.tar.gz git-bug-bc1fb34c24e985b7a9d50c559d7a42726a3a2007.zip |
graphql: fix two bugs in the connection code
1) totalCount was incorrect when not in the first page
2) pageInfo.Has{Previous,Next}Page was incorrect when using before or after
Diffstat (limited to 'graphql/connections/gen_bug.go')
-rw-r--r-- | graphql/connections/gen_bug.go | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/graphql/connections/gen_bug.go b/graphql/connections/gen_bug.go index fc9de067..000ff109 100644 --- a/graphql/connections/gen_bug.go +++ b/graphql/connections/gen_bug.go @@ -27,6 +27,7 @@ func StringCon(source []string, edgeMaker StringEdgeMaker, conMaker StringConMak var edges []LazyBugEdge var cursors []string var pageInfo models.PageInfo + var totalCount = len(source) emptyCon, _ := conMaker(edges, nodes, pageInfo, 0) @@ -39,6 +40,7 @@ func StringCon(source []string, edgeMaker StringEdgeMaker, conMaker StringConMak // remove all previous element including the "after" one source = source[i+1:] offset = i + 1 + pageInfo.HasPreviousPage = true break } } @@ -50,6 +52,7 @@ func StringCon(source []string, edgeMaker StringEdgeMaker, conMaker StringConMak if edge.GetCursor() == *input.Before { // remove all after element including the "before" one + pageInfo.HasNextPage = true break } @@ -103,5 +106,5 @@ func StringCon(source []string, edgeMaker StringEdgeMaker, conMaker StringConMak pageInfo.EndCursor = cursors[len(cursors)-1] } - return conMaker(edges, nodes, pageInfo, len(source)) + return conMaker(edges, nodes, pageInfo, totalCount) } |