My favorites | Sign in
Project Home Downloads Wiki Issues Source
New issue   Search
for
  Advanced search   Search tips   Subscriptions
Issue 2878: REST API for changes - paging doesn't work
1 person starred this issue and may be notified of changes. Back to list
Status:  Released
Owner:  ----
Closed:  Sep 2014


Sign in to add a comment
 
Reported by buczkows...@gmail.com, Sep 2, 2014
************************************************************
***** NOTE: THIS BUG TRACKER IS FOR GERRIT CODE REVIEW *****
***** DO NOT SUBMIT BUGS FOR CHROME, ANDROID, INTERNAL *****
***** ISSUES WITH YOUR COMPANY'S GERRIT SETUP, ETC.    *****
***** THOSE ISSUE BELONG IN DIFFERENT ISSUE TRACKERS!  *****
************************************************************

Affected Version: 2.9

What steps will reproduce the problem?
1. Get last 3 merged changes
2. Try to get next 3 merged changes

What is the expected output? What do you see instead?
Expected result: 6 different changes received.
Instead of it there are only first 3 changes received - it seems that 'N' parameter is ignored

Please provide any additional information below.

Here is log from my test aplication:

calling: http://.../a/changes/?q=status:merged&n=3
+++ change sortkey: 002f83020000a950
+++ change sortkey: 002f82fb0000a94f
+++ change sortkey: 002f82f40000a948
calling: http://.../a/changes/?q=status:merged&N=002f82f40000a948&n=3
+++ change sortkey: 002f83020000a950
+++ change sortkey: 002f82fb0000a94f
+++ change sortkey: 002f82f40000a948

Sep 2, 2014
Project Member #1 david.pu...@sonymobile.com
Sortkey was replaced with the S operator [1].

Try it with

http://.../a/changes/?q=status:merged&n=3&S=3

[1] https://gerrit-review.googlesource.com/#/c/54509/
Status: AwaitingInformation
Sep 2, 2014
#2 buczkows...@gmail.com
Thanks a lot! You saved my day! It works! :)
Sep 2, 2014
Project Member #3 david.pu...@sonymobile.com
Uploaded a change to mention this in the 2.9 release notes.

https://gerrit-review.googlesource.com/59715

Status: ChangeUnderReview
Sep 10, 2014
Project Member #4 edwin.ke...@gmail.com
(No comment was entered for this change.)
Status: Submitted
Labels: FixedIn-2.9.1
Sep 11, 2014
Project Member #5 david.pu...@sonymobile.com
(No comment was entered for this change.)
Status: Released
Sign in to add a comment

Powered by Google Project Hosting