Attachment 14261 Details for Bug 9289 – Bug 9289 - Adjacent checkbox behavior broken on circulation.pl

[patch] Bug 9289 - Adjacent checkbox behavior broken on circulation.pl

Bug-9289---Adjacent-checkbox-behavior-broken-on-ci.patch (text/plain), 1.58 KB, created by Jonathan Druart on 2012-12-26 10:48:22 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jonathan Druart

Created: 2012-12-26 10:48:22 UTC

Size: 1.58 KB

patch

obsolete

>From d3ddf469fcd08cdc36c0d0609823eb921e19ffe8 Mon Sep 17 00:00:00 2001
>From: Owen Leonard <oleonard@myacpl.org>
>Date: Fri, 14 Dec 2012 12:45:04 -0500
>Subject: [PATCH] Bug 9289 - Adjacent checkbox behavior broken on
> circulation.pl
>
>This patch fixes the behavior of adjacent checkboxes in the list of
>checkouts on circulation.pl. Checkboxes in both the "renew" and "check
>in" columns should not be able to be checked at the same time.
>
>A line was commented out (by me) which was necessary to this working
>correctly. I'm guessing it was done in debugging and left by mistake.
>
>To test, load a patron for checkout who has items checked out. You
>should not be able to check boxes in both the "renew" and "check in"
>columns in the same row, whether you click the checkbox itself or the
>containing table cell.
>
>Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
>---
> koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt |    2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt
>index c860ed6..25dabca 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt
>@@ -130,7 +130,7 @@ var allcheckboxes = $(".checkboxed");
>                     $(this).removeAttr("checked");
>                 } else {
>                     $(this).attr("checked","checked");
>-                  //  radioCheckBox($(this));
>+                    radioCheckBox($(this));
>                 }
>            });
>         }
>-- 
>1.7.10.4

Actions: View | Diff | Splinter Review

Attachments on bug 9289: 14151 | 14250 | 14261