Bug 12195 - Remove duplicated plugin jquery.dataTables.min.js
Summary: Remove duplicated plugin jquery.dataTables.min.js
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Templates (show other bugs)
Version: Main
Hardware: All All
: P5 - low normal (vote)
Assignee: Owen Leonard
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-05-06 15:02 UTC by Owen Leonard
Modified: 2014-12-07 20:07 UTC (History)
1 user (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 12195 - Remove duplicated plugin jquery.dataTables.min.js (71.64 KB, patch)
2014-05-06 15:10 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 12195 - Remove duplicated plugin jquery.dataTables.min.js (71.65 KB, patch)
2014-05-06 15:28 UTC, Galen Charlton
Details | Diff | Splinter Review
Bug 12195 - Remove duplicated plugin jquery.dataTables.min.js (71.71 KB, patch)
2014-05-08 15:33 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-05-06 15:02:00 UTC
Now that all staff client templates use the new datatables include file the datatables plugin located in prog/en/lib/jquery/plugins can be removed.
Comment 1 Owen Leonard 2014-05-06 15:10:24 UTC Comment hidden (obsolete)
Comment 2 Galen Charlton 2014-05-06 15:28:47 UTC Comment hidden (obsolete)
Comment 3 Jonathan Druart 2014-05-08 15:33:21 UTC
Created attachment 28106 [details] [review]
Bug 12195 - Remove duplicated plugin jquery.dataTables.min.js

All templates in the staff client now load the DataTables jquery plugin
through a single include file, datatables.inc, which points to a copy of
the plugin in intranet-tmpl/lib/jquery/plugins. This patch removes the
unused copy of the plugin found in intranet-tmpl/prog/en/lib/jquery/plugins.

To test, search the Koha source for references to
jquery.dataTables.min.js. There should be no files which point to the
redundant file.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Comment 4 Galen Charlton 2014-05-08 16:42:52 UTC
Pushed to master.  Thanks, Owen!