Attachment 68719 Details for Bug 19539 – Bug 19539: Fix column index shift in cirulation rules

[patch] Bug 19539: Fix column index shift in cirulation rules

Bug-19539-Fix-column-index-shift-in-cirulation-rul.patch (text/plain), 1.79 KB, created by Tomás Cohen Arazi (tcohen) on 2017-10-27 14:16:04 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Tomás Cohen Arazi (tcohen)

Created: 2017-10-27 14:16:04 UTC

Size: 1.79 KB

patch

obsolete

>From 2d955c54f318bbba0d9bf2bf7e72b721bcd92237 Mon Sep 17 00:00:00 2001
>From: Tomas Cohen Arazi <tomascohen@theke.io>
>Date: Fri, 27 Oct 2017 11:12:57 -0300
>Subject: [PATCH] Bug 19539: Fix column index shift in cirulation rules
>
>This patch fixes an index shift introduced by bug 18857 when it introduced
>a new column.
>
>To test:
>- On the Circulation and fine rules page
>- Empty the value of 'Current on-site checkouts allowed' and save
>- Notice 'Unlimited' shows on the saved rule.
>- Click edit
>=> FAIL: the 'Unlimited' string displays instead of the empty string on the editing row.
>- Apply this patch
>- Re-open the circ rules page
>- Click edit
>=> SUCCESS: An empty string fills the 'Unlimited' fields
>- Sign off :-D
>---
> koha-tmpl/intranet-tmpl/prog/en/modules/admin/smart-rules.tt | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/smart-rules.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/smart-rules.tt
>index 5bc44ba..27fcf6e 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/smart-rules.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/smart-rules.tt
>@@ -87,7 +87,7 @@ $(document).ready(function() {
>                         // Remove potential previous input added
>                         $(current_column).find("input").remove();
>                         $(current_column).append("<input type='hidden' name='"+name+"' value='"+val+"' />");
>-                    } else if ( i == 2 || i == 3 ) {
>+                    } else if ( i == 3 || i == 4 ) {
>                         // If the value is not an integer for "Current checkouts allowed" or "Current on-site checkouts allowed"
>                         // The value is "Unlimited" (or an equivalent translated string)
>                         // an it should be set to an empty string
>-- 
>2.7.4

Actions: View | Diff | Splinter Review

Attachments on bug 19539: 68719 | 68789 | 68846 | 68847 | 68879 | 68880 | 68883