The holds awaiting pickup page uses the old tablesorter plugin. It should use the DataTables plugin instead.
Created attachment 14794 [details] [review] Bug 9461 - Use DataTables on holds awaiting pickup report page Replace the tablesorter plugin with the DataTables plugin on the holds awaiting pickup report page. To test, open the holds awaiting pickup report (Circulation -> Holds awaiting pickup). Confirm that table sorting works correctly on all tables.
Created attachment 14858 [details] [review] Bug 9461 [Revised] Use DataTables on holds awaiting pickup report page Replace the tablesorter plugin with the DataTables plugin on the holds awaiting pickup report page. To test, open the holds awaiting pickup report (Circulation -> Holds awaiting pickup). Confirm that table sorting works correctly on all tables. Revision makes default result count (20) match options for number of pagination entries (10,20,50,100,all).
Created attachment 15182 [details] [review] Bug 9461 [Revised] Use DataTables on holds awaiting pickup report page Replace the tablesorter plugin with the DataTables plugin on the holds awaiting pickup report page. To test, open the holds awaiting pickup report (Circulation -> Holds awaiting pickup). Confirm that table sorting works correctly on all tables. Revision makes default result count (20) match options for number of pagination entries (10,20,50,100,all). Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
QA comment: replace tablesorter with datatable on the the holds awaiting page. No regression found. Marked as Passed QA.
Created attachment 15317 [details] [review] Bug 9461 [Revised] Use DataTables on holds awaiting pickup report page Replace the tablesorter plugin with the DataTables plugin on the holds awaiting pickup report page. To test, open the holds awaiting pickup report (Circulation -> Holds awaiting pickup). Confirm that table sorting works correctly on all tables. Revision makes default result count (20) match options for number of pagination entries (10,20,50,100,all). Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
This patch has been pushed to master.