Bug 9467 - Use DataTables on catalog by item type report page
Summary: Use DataTables on catalog by item type report page
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Templates (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement (vote)
Assignee: Owen Leonard
QA Contact: Jonathan Druart
URL: /cgi-bin/koha/reports/manager.pl?repo...
Keywords:
Depends on:
Blocks:
 
Reported: 2013-01-23 19:21 UTC by Owen Leonard
Modified: 2014-05-26 21:04 UTC (History)
2 users (show)

See Also:
Change sponsored?: ---
Patch complexity: Trivial patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
Bug 9467 - Use DataTables on catalog by item type report page (1.90 KB, patch)
2013-01-23 19:24 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 9467 - Use DataTables on catalog by item type report page (1.91 KB, patch)
2013-02-08 17:12 UTC, Kyle M Hall
Details | Diff | Splinter Review
Bug 9467 - Use DataTables on catalog by item type report page (1.98 KB, patch)
2013-02-14 13:54 UTC, Jonathan Druart
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Owen Leonard 2013-01-23 19:21:24 UTC
The catalog by item type report uses the old tablesorter plugin. It should use the DataTables plugin instead.
Comment 1 Owen Leonard 2013-01-23 19:24:29 UTC Comment hidden (obsolete)
Comment 2 Kyle M Hall 2013-02-08 17:12:01 UTC Comment hidden (obsolete)
Comment 3 Jonathan Druart 2013-02-14 13:54:22 UTC
QA comment:

Replace tablesorter with datatable on the item type report page.
Add a test in order not to apply the plugin to a missing element.

Marked as Passed QA.
Comment 4 Jonathan Druart 2013-02-14 13:54:34 UTC
Created attachment 15319 [details] [review]
Bug 9467 - Use DataTables on catalog by item type report page

Replace the tablesorter plugin with the DataTables plugin on the
catalog by item type report page.

To test, run the catalog by item type report (Reports -> Catalog by
item type). Confirm that table sorting works correctly.

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Comment 5 Jared Camins-Esakov 2013-02-15 13:36:37 UTC
This patch has been pushed to master.