Bug 11739 - Use new DataTables include in notices template
Summary: Use new DataTables include in notices template
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Templates (show other bugs)
Version: Main
Hardware: All All
: P5 - low normal
Assignee: Owen Leonard
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks: 11720
  Show dependency treegraph
 
Reported: 2014-02-10 20:05 UTC by Owen Leonard
Modified: 2014-12-07 20:05 UTC (History)
3 users (show)

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


Attachments
Bug 11739 - Use new DataTables include in notices template (1.83 KB, patch)
2014-02-10 20:10 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 11739 - Use new DataTables include in notices template (1.84 KB, patch)
2014-02-11 02:49 UTC, Biblibre Sandboxes
Details | Diff | Splinter Review
Bug 11739 - Use new DataTables include in notices template (1.91 KB, patch)
2014-02-11 09:35 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 2014-02-10 20:05:41 UTC
The Notices and Slips template should be updated to use the new DataTables include file introduced by Bug 10649.
Comment 1 Owen Leonard 2014-02-10 20:10:28 UTC Comment hidden (obsolete)
Comment 2 Biblibre Sandboxes 2014-02-11 02:49:13 UTC
Patch tested with a sandbox, by merlissia <manuelimel@gmail.com>
Comment 3 Biblibre Sandboxes 2014-02-11 02:49:31 UTC Comment hidden (obsolete)
Comment 4 Jonathan Druart 2014-02-11 09:35:48 UTC
Created attachment 25194 [details] [review]
Bug 11739 - Use new DataTables include in notices template

Bug 10649 introduced a new include file for adding DataTables-related
JavaScript assets. This patch adds use of this include file to the
Notices & Slips page.

To test, view the Notices & Slips page (tools/letter.pl). Confirm that
the table of notices is sorted correctly.

Signed-off-by: merlissia <manuelimel@gmail.com>
Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Comment 5 Galen Charlton 2014-02-13 03:20:26 UTC
Pushed to master.  Thanks, Owen!
Comment 6 Fridolin Somers 2014-05-19 14:00:57 UTC
Pushed to 3.14.x, will be in 3.14.07.