Bug 9436

Summary: Use DataTables on cities administration page
Product: Koha Reporter: Owen Leonard <oleonard>
Component: TemplatesAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Jonathan Druart <jonathan.druart>
Severity: enhancement    
Priority: P5 - low CC: bgkriegel, jonathan.druart
Version: Main   
Hardware: All   
OS: All   
GIT URL: Change sponsored?: ---
Patch complexity: Trivial patch Documentation contact:
Documentation submission: Text to go in the release notes:
Version(s) released in:
Circulation function:
Attachments: Bug 9436 - Use DataTables on cities administration page
Bug 9436 - Use DataTables on cities administration page
Bug 9436 - Use DataTables on cities administration page
Bug 9436 [Revised] Use DataTables on cities administration page
[SIGNED-OFF] Bug 9436 [Revised] Use DataTables on cities administration page
Bug 9436 [Revised] Use DataTables on cities administration page
Bug 9436 [Revised] Use DataTables on cities administration page

Description Owen Leonard 2013-01-20 00:47:53 UTC
The cities administration page uses the old tablesorter plugin. It should use the DataTables plugin instead.
Comment 1 Owen Leonard 2013-01-20 00:57:36 UTC Comment hidden (obsolete)
Comment 2 Jonathan Druart 2013-01-22 15:41:30 UTC Comment hidden (obsolete)
Comment 3 Jonathan Druart 2013-01-28 08:07:55 UTC Comment hidden (obsolete)
Comment 4 Owen Leonard 2013-02-26 15:58:34 UTC Comment hidden (obsolete)
Comment 5 Bernardo Gonzalez Kriegel 2013-02-27 23:27:24 UTC Comment hidden (obsolete)
Comment 6 Jonathan Druart 2013-03-05 09:28:52 UTC
QA comment:

This patch replaces tablesorter with datatable on the cities admin page.
The pagination is not "four_button" I don't know if it is deliberate.

Marked as Passed QA.
Comment 7 Jonathan Druart 2013-03-05 09:29:11 UTC Comment hidden (obsolete)
Comment 8 Jared Camins-Esakov 2013-03-07 15:48:07 UTC
This conflicts with the Bootstrap button for creating a new city.
Comment 9 Owen Leonard 2013-03-14 19:58:08 UTC
Created attachment 16139 [details] [review]
Bug 9436 [Revised] Use DataTables on cities administration page

Replace the tablesorter plugin with the DataTables plugin on the
cities administration page.

To test, open the cities administration page (Administration
-> Cities and towns). Confirm that table sorting works correctly.

Revision corrects template path to datatables assets.
Further revision corrects conflict with Bug 9630

Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com>

Comment: Sorting works. No errors.
Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Amended patch: Add the aLengthMenu key for DataTables + delete an unused
div block (used for pager)
Comment 10 Owen Leonard 2013-03-14 19:58:49 UTC
Please update the status if you think it's incorrect to return this to "Passed QA."
Comment 11 Jared Camins-Esakov 2013-03-15 23:24:47 UTC
This patch has been pushed to master.