Bug 26713

Summary: Add pagination to bottom of saved SQL reports table
Product: Koha Reporter: Barbara Johnson <barbara.johnson>
Component: ReportsAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: andrewfh, caroline.cyr-la-rose, david, fridolin.somers, lisettepalouse+koha, victor
Version: master   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
This enhancement adds a second pagination menu to the bottom of saved SQL reports tables.
Version(s) released in:
21.05.00,20.11.02,20.05.09,19.11.15
Attachments: Bug 26713: Add pagination to bottom of saved SQL reports table
Bug 26713: Add pagination to bottom of saved SQL reports table
Bug 26713: Add pagination to bottom of saved SQL reports table

Description Barbara Johnson 2020-10-16 16:07:23 UTC
It would be a big timesaver to add pagination to the bottom of the saved SQL reports table so that you didn't have to scroll back to the top to page through the results.
Comment 1 Owen Leonard 2020-12-03 12:15:16 UTC
Created attachment 114143 [details] [review]
Bug 26713: Add pagination to bottom of saved SQL reports table

This patch adds a second pagination menu to the bottom of the report
results table.

To test, apply the patch and go to Reports.

- Run a report which will return multiple pages of results (greater
  than 20).
- The results should display with a pagination bar at the top and bottom
  of the table.
- Confirm that both pagination menus work correctly.
Comment 2 David Nind 2020-12-07 17:39:03 UTC
Created attachment 114240 [details] [review]
Bug 26713: Add pagination to bottom of saved SQL reports table

This patch adds a second pagination menu to the bottom of the report
results table.

To test, apply the patch and go to Reports.

- Run a report which will return multiple pages of results (greater
  than 20).
- The results should display with a pagination bar at the top and bottom
  of the table.
- Confirm that both pagination menus work correctly.

Signed-off-by: David Nind <david@davidnind.com>
Comment 3 David Nind 2020-12-07 17:39:44 UTC
Testing notes:

- Used this query: select * from borrowers
Comment 4 Katrin Fischer 2021-01-10 22:15:33 UTC
Created attachment 114997 [details] [review]
Bug 26713: Add pagination to bottom of saved SQL reports table

This patch adds a second pagination menu to the bottom of the report
results table.

To test, apply the patch and go to Reports.

- Run a report which will return multiple pages of results (greater
  than 20).
- The results should display with a pagination bar at the top and bottom
  of the table.
- Confirm that both pagination menus work correctly.

Signed-off-by: David Nind <david@davidnind.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 5 Jonathan Druart 2021-01-12 15:48:04 UTC
Pushed to master for 21.05, thanks to everybody involved!
Comment 6 Fridolin Somers 2021-01-18 14:37:55 UTC
Small yet very useful enhancement.
I choose to backport.

Pushed to 20.11.x for 20.11.02
Comment 7 Andrew Fuerste-Henry 2021-01-29 14:42:36 UTC
As Fridolin says, very small and handy enhancement. Pushed to 20.05.x for 20.05.09
Comment 8 Victor Grousset/tuxayo 2021-02-01 18:25:16 UTC
Backported: Pushed to 19.11.x branch for 19.11.15