Attachment 108282 Details for Bug 26212 – Bug 26212: Remove the use of jquery.checkboxes plugin from pending offline circulations

[patch] Bug 26212: Remove the use of jquery.checkboxes plugin from pending offline circulations

Bug-26212-Remove-the-use-of-jquerycheckboxes-plugi.patch (text/plain), 3.94 KB, created by Owen Leonard on 2020-08-14 14:01:50 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Owen Leonard

Created: 2020-08-14 14:01:50 UTC

Size: 3.94 KB

patch

obsolete

>From 7ac54e7363a988c9e312f2d5dc5f3ef34c6470c9 Mon Sep 17 00:00:00 2001
>From: Owen Leonard <oleonard@myacpl.org>
>Date: Fri, 14 Aug 2020 13:39:17 +0000
>Subject: [PATCH] Bug 26212: Remove the use of jquery.checkboxes plugin from
> pending offline circulations
>
>This patch removes the use of the jquery.checkboxes plugin from the
>pending offline circulations page, replacing it with in-page JS.
>
>Unrelated: The JS-based creation of the "select all" and "clear all"
>controls has been replaced with HTML links. Labels are added to the
>results table to make checkbox selection easier.
>
>To test, apply the patch and go to Circulation.
>
>- Upload an offline circulation file (.koc) with multiple transactions.
>  If you don't want to generate this file using the desktop application
>  or Firefox add-on you can create a text file like this (tab
>  separated):
>
>  Version=1.0    Generator=koct-firefox    GeneratorVersion=0.1
>  2020-08-14 13:24:21    issue    0109132    33000000093476
>
>- Go to Pending offline circulation actions. You should see the list of
>  transactions you uploaded.
>- Test that the "Check all" and "Uncheck all" controls work correctly.
>---
> .../prog/en/modules/offline_circ/list.tt           | 27 ++++++++++++----------
> 1 file changed, 15 insertions(+), 12 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/offline_circ/list.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/offline_circ/list.tt
>index 451b8f223d..414e1aeb88 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/offline_circ/list.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/offline_circ/list.tt
>@@ -21,8 +21,11 @@
> 	    [% IF ( pending_operations ) %]
> 
> 	        <form>
>-             <p><span class="checkall"></span> |
>-             <span class="clearall"></span></p>
>+                <p>
>+                    <a id="CheckAll" href="#"><i class="fa fa-check"></i> Check all</a>
>+                    <a id="CheckNone" href="#"><i class="fa fa-remove"></i> Uncheck all</a>
>+                </p>
>+
> 	        <table id="operations">
> 	            <thead>
> 		            <tr>
>@@ -37,8 +40,10 @@
> 		        <tbody>
> 		            [% FOREACH operation IN pending_operations %]
> 		                <tr class="oc-[% operation.action | html %]">
>-		                    <td><input type="checkbox" name="operationid" value="[% operation.operationid | html %]" /></td>
>-			                <td>[% operation.timestamp | html %]</td>
>+                            <td><input type="checkbox" name="operationid" id="operationid[% operation.operationid | html %]" value="[% operation.operationid | html %]" /></td>
>+                            <td>
>+                                <label for="operationid[% operation.operationid | html %]">[% operation.timestamp | html %]</label>
>+                            </td>
>                             <td>
>                                 [% SWITCH ( operation.action ) -%]
>                                     [%   CASE "issue" -%]
>@@ -93,15 +98,13 @@
>     <script type="text/javascript">
>         $(document).ready(function() {
> 
>-        $("span.clearall").html("<a id=\"CheckNone\" href=\"/cgi-bin/koha/offline_circ/list.pl\">"+_("Uncheck all")+"<\/a>");
>-        $("span.checkall").html("<a id=\"CheckAll\" href=\"/cgi-bin/koha/offline_circ/list.pl\">"+_("Check all")+"<\/a>");
>-            $('#CheckNone').click(function() {
>-                $("#operations").unCheckCheckboxes();
>-                return false;
>+            $('#CheckNone').click(function(e) {
>+                e.preventDefault();
>+                $("#operations input:checkbox").prop("checked", false );
>             });
>-            $('#CheckAll').click(function() {
>-                $("#operations").checkCheckboxes();
>-                return false;
>+            $('#CheckAll').click(function(e) {
>+                e.preventDefault();
>+                $("#operations input:checkbox").prop("checked", true );
>             });
>             $('#process,#delete').click(function() {
>                 var action = $(this).attr("id");
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 26212: 108282 | 108284 | 108413 | 108418