Bug 26153: Remove the use of jquery.checkboxes plugin from items lost report
This patch removes the use of the jquery.checkboxes plugin from the
lost items report page and replaces it with custom JS.
To test, apply the patch and go to Reports -> Lost items.
- Run the report with parameters which will return multiple results.
- On the results page Create a CSV export profile if there's none.
- On the results page test the "Select all" and "Clear all"
links at the top of the results. The controls should work correctly.
Signed-off-by: Didier Gautheron <didier.gautheron@biblibre.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
(cherry picked from commit e24a294f39)
Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
This commit is contained in:
parent
df5c9d0725
commit
2ea51ab86e
|
|
@ -183,7 +183,6 @@
|
|||
[% Asset.js("lib/jquery/plugins/jquery.dataTables.columnFilter.js") | $raw %]
|
||||
[% INCLUDE 'columns_settings.inc' %]
|
||||
[% Asset.js("js/table_filters.js") | $raw %]
|
||||
[% Asset.js("lib/jquery/plugins/jquery.checkboxes.min.js") | $raw %]
|
||||
<script>
|
||||
$(document).ready(function() {
|
||||
var columns_settings = [% ColumnsSettings.GetColumns( 'reports', 'lostitems', 'lostitems-table', 'json' ) | $raw %];
|
||||
|
|
@ -233,13 +232,17 @@
|
|||
|
||||
$(".SelectAll").on("click",function(e){
|
||||
e.preventDefault();
|
||||
$("#lostitems-table").checkCheckboxes();
|
||||
$("#lostitems-table input:checkbox").each(function(){
|
||||
$(this).prop("checked", true);
|
||||
});
|
||||
itemSelectionBuildActionLinks();
|
||||
});
|
||||
|
||||
$(".ClearAll").on("click",function(e){
|
||||
e.preventDefault();
|
||||
$("#lostitems-table").unCheckCheckboxes();
|
||||
$("#lostitems-table input:checkbox").each(function(){
|
||||
$(this).prop("checked", false);
|
||||
});
|
||||
itemSelectionBuildActionLinks();
|
||||
});
|
||||
});
|
||||
|
|
|
|||
Loading…
Reference in New Issue