Bug 26215: Remove the use of jquery.checkboxes plugin from Z39.50 search pages
authorOwen Leonard <oleonard@myacpl.org>
Fri, 14 Aug 2020 15:18:37 +0000 (15:18 +0000)
committerJonathan Druart <jonathan.druart@bugs.koha-community.org>
Tue, 18 Aug 2020 13:45:49 +0000 (15:45 +0200)
This patch removes the use of the jquery.checkboxes plugin from three
Z39.50 search templates: Acquisitions, Authorities, and Cataloging.

To test, apply the patch and test the controls for selecting and
deselecting Z39.50 search targets on the following pages:

 - In Acquisitions: Locate a vendor and add to basket -> From an
   external source.
 - In Authorities: Click "New authority" -> "New from Z39.50/SRU."
 - In Cataloging: Click "New from Z39.50/SRU."

Signed-off-by: Amit Gupta <amit.gupta@informaticsglobal.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>

koha-tmpl/intranet-tmpl/prog/en/modules/acqui/z3950_search.tt
koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/z3950_auth_search.tt
koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/z3950_search.tt
koha-tmpl/intranet-tmpl/prog/js/z3950_search.js

index f3aab65..f8d2ee7 100644 (file)
@@ -226,7 +226,6 @@ tr.selected { background-color : #FFFFCC; } tr.selected td { background-color :
 [% MACRO jsinclude BLOCK %]
     [% Asset.js("js/acquisitions-menu.js") | $raw %]
     [% INCLUDE 'datatables.inc' %]
-    [% Asset.js("lib/jquery/plugins/jquery.checkboxes.min.js") | $raw %]
     [% Asset.js("js/z3950_search.js") | $raw %]
     <script>
         $(document).ready(function(){
index b66e9d6..7335ac6 100644 (file)
 [% IF ( numberpending ) %]<h3 align="center">Still [% numberpending | html %] servers to search</h3>[% END %]
 
 [% MACRO jsinclude BLOCK %]
-    [% Asset.js("lib/jquery/plugins/jquery.checkboxes.min.js") | $raw %]
     [% INCLUDE 'datatables.inc' %]
     [% Asset.js("js/z3950_search.js") | $raw %]
     <script>
index 78e31de..5481a8c 100644 (file)
         [% END %]
 
 [% MACRO jsinclude BLOCK %]
-    [% Asset.js("lib/jquery/plugins/jquery.checkboxes.min.js") | $raw %]
     [% Asset.js("js/z3950_search.js") | $raw %]
     [% INCLUDE 'datatables.inc' %]
     [% INCLUDE 'columns_settings.inc' %]
index 7ee94e5..368aacb 100644 (file)
@@ -13,13 +13,13 @@ function Import(Breeding, recordid, AuthType, FrameworkCode) {
 
 $( document ).ready( function() {
 
-    $( "#CheckAll" ).click( function() {
-        $( ".checkboxed" ).checkCheckboxes();
-        return false;
+    $( "#CheckAll" ).click( function(e) {
+        e.preventDefault();
+        $( ".checkboxed input:checkbox" ).prop("checked", true);
     });
-    $( "#CheckNone" ).click( function() {
-        $( ".checkboxed" ).unCheckCheckboxes();
-        return false;
+    $( "#CheckNone" ).click( function(e) {
+        e.preventDefault();
+        $( ".checkboxed input:checkbox" ).prop("checked", false);
     });
 
     $( ".submit" ).on( "click", function() {