Attachment 108021 Details for Bug 26010 – Bug 26010: Remove the use of jquery.checkboxes plugin from staff interface cart

[patch] Bug 26010: Remove the use of jquery.checkboxes plugin from staff interface cart

Bug-26010-Remove-the-use-of-jquerycheckboxes-plugi.patch (text/plain), 2.90 KB, created by Katrin Fischer on 2020-08-10 20:56:48 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Katrin Fischer

Created: 2020-08-10 20:56:48 UTC

Size: 2.90 KB

patch

obsolete

>From 4611de8c0ff005c9d81352f4d90382444d4b247d Mon Sep 17 00:00:00 2001
>From: Owen Leonard <oleonard@myacpl.org>
>Date: Fri, 17 Jul 2020 17:29:15 +0000
>Subject: [PATCH] Bug 26010: Remove the use of jquery.checkboxes plugin from
> staff interface cart
>
>This patch removes the use of the jquery.checkboxes plugin from the
>staff interface cart and replaces its functionality with "plain" jQuery.
>
>To test, apply the patch and add some items to the Cart in the staff
>interface.
>
>- Open the Cart window in the staff interface by clicking the "Cart"
>  link in the header.
>- Test that the "Select all" and "Clear all" links work correctly to
>  check and uncheck all checkboxes.
>- Test that the "Remove" or "Place hold" controls work correctly,
>  applying to only the checked checkboxes whether they were checked
>  using "Select all" or by manually checking them.
>
>Signed-off-by: Sally <sally.healey@cheshiresharedservices.gov.uk>
>
>Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
>---
> .../intranet-tmpl/prog/en/modules/basket/basket.tt |  1 -
> koha-tmpl/intranet-tmpl/prog/js/cart.js            | 29 ++++++++++------------
> 2 files changed, 13 insertions(+), 17 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/basket/basket.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/basket/basket.tt
>index d110dcbe52..05036622a5 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/basket/basket.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/basket/basket.tt
>@@ -322,7 +322,6 @@
>     [% INCLUDE js_includes.inc %]
>     [% INCLUDE 'datatables.inc' %]
>     [% Asset.js("js/cart.js") | $raw %]
>-    [% Asset.js("lib/jquery/plugins/jquery.checkboxes.min.js") | $raw %]
> 
>     </body>
> </html>
>diff --git a/koha-tmpl/intranet-tmpl/prog/js/cart.js b/koha-tmpl/intranet-tmpl/prog/js/cart.js
>index 923ed3242a..fbf97fb6fe 100644
>--- a/koha-tmpl/intranet-tmpl/prog/js/cart.js
>+++ b/koha-tmpl/intranet-tmpl/prog/js/cart.js
>@@ -66,24 +66,21 @@ function batchModify(){
> 
> $(document).ready(function(){
>     $("#items-popover").popover();
>-    $("#CheckAll").click(function(){
>-    var checked = [];
>-    $("#bookbag_form").checkCheckboxes("*", true).each(
>-        function() {
>-            $(this).change();
>-        }
>-    );
>-        return false;
>+
>+    $("#CheckAll").click(function (e) {
>+        e.preventDefault();
>+        $(".select_record").each(function () {
>+            $(this).prop("checked", true).change();
>+        });
>     });
>-    $("#CheckNone").click(function(){
>-    var checked = [];
>-    $("#bookbag_form").unCheckCheckboxes("*",true).each(
>-        function() {
>-            $(this).change();
>-        }
>-    );
>-        return false;
>+
>+    $("#CheckNone").click(function (e) {
>+        e.preventDefault();
>+        $(".select_record").each(function () {
>+            $(this).prop("checked", false).change();
>+        });
>     });
>+
>     $(".holdsep").text("| ");
>     $(".hold").text(_("Place hold"));
>     $("#downloadcartc").empty();
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 26010: 107045 | 107647 | 108021