Attachment 110112 Details for Bug 26151 – Bug 26151: Remove the use of jquery.checkboxes plugin from suggestions management page

[patch] Bug 26151: Remove the use of jquery.checkboxes plugin from suggestions management page

Bug-26151-Remove-the-use-of-jquerycheckboxes-plugi.patch (text/plain), 2.28 KB, created by Brandon J on 2020-09-15 14:35:33 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Brandon J

Created: 2020-09-15 14:35:33 UTC

Size: 2.28 KB

patch

obsolete

>From 809f6cf77ca9318686bb54ba2e7af366297fc75f Mon Sep 17 00:00:00 2001
>From: Owen Leonard <oleonard@myacpl.org>
>Date: Wed, 5 Aug 2020 17:27:58 +0000
>Subject: [PATCH] Bug 26151: Remove the use of jquery.checkboxes plugin from
> suggestions management page
>
>This patch removes the use of the jquery.checkboxes plugin from the
>suggestions management page and replaces it with custom JS.
>
>To test, apply the patch and go to Acquisitions -> Suggestions
>
>Under each tab ("Accepted," "Pending," etc.), test that the "Check all"
>and "Uncheck all" controls work correctly and only affect the table on
>that tab.
>
>Signed-off-by: Brandon J <brandon.jimenez@inLibro.com>
>---
> .../prog/en/modules/suggestion/suggestion.tt         | 12 ++++++------
> 1 file changed, 6 insertions(+), 6 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/suggestion/suggestion.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/suggestion/suggestion.tt
>index ede593e118..a06d1cd0f1 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/suggestion/suggestion.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/suggestion/suggestion.tt
>@@ -1013,8 +1013,6 @@
>     [% IF op == 'else' %]
>         [% INCLUDE 'datatables.inc' %]
>         [% INCLUDE 'columns_settings.inc' %]
>-        [% Asset.js("lib/jquery/plugins/jquery.checkboxes.min.js") | $raw %]
>-
>         <script>
>             $(document).ready(function() {
>                 $('#suggestiontabs').tabs();
>@@ -1039,13 +1037,15 @@
> 
>                 $(".checkall").click(function(e){
>                     e.preventDefault();
>-                    $(this).parents('form').checkCheckboxes();
>-                    return false;
>+                    $(this).parents('form').find("input:checkbox").each(function(){
>+                        $(this).prop("checked", true);
>+                    });
>                 });
>                 $(".uncheckall").click(function(e){
>                     e.preventDefault();
>-                    $(this).parents('form').unCheckCheckboxes();
>-                    return false;
>+                    $(this).parents('form').find("input:checkbox").each(function(){
>+                        $(this).prop("checked", false);
>+                    });
>                 });
>                 $(".other_reason").hide();
>                 $("select[name='reason']").change(function(){
>-- 
>2.17.1

Actions: View | Diff | Splinter Review

Attachments on bug 26151: 107854 | 110112 | 110125