Bug 9476

Summary: Use Datatables on Koha news 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: gmcharlt, jonathan.druart
Version: master   
Hardware: All   
OS: All   
URL: /cgi-bin/koha/tools/koha-news.pl
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 9476 - Use Datatables on Koha news page
Bug 9476 - Use Datatables on Koha news page
Bug 9476 [Revised] Use Datatables on Koha news page
Bug 9476: Use Datatables on Koha news page
Bug 9476: Use Datatables on Koha news page

Description Owen Leonard 2013-01-24 01:37:32 UTC
The Koha news page uses the old tablesorter plugin. It should use the DataTables plugin instead.
Comment 1 Owen Leonard 2013-01-24 01:44:23 UTC Comment hidden (obsolete)
Comment 2 Galen Charlton 2013-02-06 04:41:31 UTC
Patch doesn't apply to master -- please rebase and resubmit.
Comment 3 Owen Leonard 2013-02-06 14:08:18 UTC Comment hidden (obsolete)
Comment 4 Galen Charlton 2013-02-06 15:22:21 UTC
With the revised patch, I get the following error:

Template process failed: file error - parse error - /home/gmc/koha/koha-tmpl/intranet-tmpl/prog/en/modules/tools/koha-news.tt line 202: unexpected end of input at /home/gmc/koha/C4/Templates.pm line 132.

Looks like the "[% IF ( opac_news_count ) %]" introduced by the patch isn't closed.
Comment 5 Owen Leonard 2013-02-06 15:58:24 UTC Comment hidden (obsolete)
Comment 6 Galen Charlton 2013-02-06 16:32:41 UTC Comment hidden (obsolete)
Comment 7 Jonathan Druart 2013-02-07 11:00:27 UTC
QA comment:

Replace tablesorter with datatable on the koha-news tool page.
Works great.

Marked as Passed QA.
Comment 8 Jonathan Druart 2013-02-07 11:00:41 UTC
Created attachment 15131 [details] [review]
Bug 9476: Use Datatables on Koha news page

Replace the tablesorter plugin with the DataTables plugin on the
Koha news page.

To test, open the Koha news page (Tools -> News). Confirm that table sorting
works correctly.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Comment 9 Jared Camins-Esakov 2013-02-07 12:40:07 UTC
This patch has been pushed to master.