Created attachment 53989 [details]
Record 21-40 (Page 2) -- Actual: Record 2-21
Created attachment 53990 [details]
Record 41 (Page 3) -- Actual: Record 3-22
Created attachment 54639 [details] [review] Bug 17048: Fix pagination offset for authority searches At the intranet, the pagination has been broken by bug 12478 (Elastic Search). There was a confusion between the offset and the page number. At the OPAC, it is broken since bug 2735 which has mimicked the intranet script. Test plan: Search for a term which will return more than 1 page of results. Click on the second page => Without this patch, the first result of the second page is the second result of the first page => With this patch applied, the offset will be corrected I cannot reproduce the bug. Can someone check to see if this patch is still necessary? (In reply to Aleisha Amohia from comment #6) > I cannot reproduce the bug. Can someone check to see if this patch is still > necessary? Yes it is. Did you see the screenshots? They explain very well the issue. Created attachment 54788 [details] [review] [SIGNED-OFF] Bug 17048: Fix pagination offset for authority searches At the intranet, the pagination has been broken by bug 12478 (Elastic Search). There was a confusion between the offset and the page number. At the OPAC, it is broken since bug 2735 which has mimicked the intranet script. Test plan: Search for a term which will return more than 1 page of results. Click on the second page => Without this patch, the first result of the second page is the second result of the first page => With this patch applied, the offset will be corrected Signed-off-by: Aleisha Amohia <aleishaamohia@hotmail.com> (In reply to Jonathan Druart from comment #7) > Yes it is. Did you see the screenshots? They explain very well the issue. You're right sorry, I was trying to reproduce it in the wrong place. Tested in the right place this time and patch works well. :) *** Bug 17192 has been marked as a duplicate of this bug. *** Created attachment 54955 [details] [review] [PASSED QA] Bug 17048: Fix pagination offset for authority searches At the intranet, the pagination has been broken by bug 12478 (Elastic Search). There was a confusion between the offset and the page number. At the OPAC, it is broken since bug 2735 which has mimicked the intranet script. Test plan: Search for a term which will return more than 1 page of results. Click on the second page => Without this patch, the first result of the second page is the second result of the first page => With this patch applied, the offset will be corrected Signed-off-by: Aleisha Amohia <aleishaamohia@hotmail.com> Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de> Pushed to master for 16.11, thanks Jonathan! Pushed in 16.05. Will be in 16.05.04. *** Bug 17370 has been marked as a duplicate of this bug. *** This problem persist in master and 18.05 (In reply to Héctor Eduardo Castro Avalos from comment #15) > This problem persist in master and 18.05 Please open a new bug report (linked to this one), provide the exact version you are using as well as the search engine you are using (zebra/es). |
Created attachment 53988 [details] Record 1-20 (Page 1) Authority search result list page scrolling not working properly. The ">" (single right arrow) doesn't start from the end of previous page. For example, after displayed record 1-20. Click ">", you should get record 21-40, but, it actually showing record 2-21! See screen shots attached.