Attachment 22819 Details for Bug 11216 – Bug 11216 - Patron modification creates indefinite manual restriction

[patch] Bug 11216 - Patron modification creates indefinite manual restriction

Bug-11216---Patron-modification-creates-indefinite.patch (text/plain), 2.76 KB, created by Dobrica Pavlinusic on 2013-11-08 13:58:29 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Dobrica Pavlinusic

Created: 2013-11-08 13:58:29 UTC

Size: 2.76 KB

patch

obsolete

>From 2f76bb4476d5eae30d921922a9898d41fc543cd5 Mon Sep 17 00:00:00 2001
>From: Dobrica Pavlinusic <dpavlin@rot13.org>
>Date: Fri, 8 Nov 2013 14:50:38 +0100
>Subject: [PATCH] Bug 11216 - Patron modification creates indefinite manual restriction
>
>Editing partron data results in unintended adding of indefinite manual
>restriction to it. Reason for it is hidden field add_debarment which
>is wrongly initialized to 1 instead of 0. Also JavaScript code seems
>to want to toggle checkbox, and sice this field is hidden correct
>approach would be to change it's value to 1.
>
>Test scenario:
>
>1. edit patron data
>2. verify that debarrment is wrongly added
>3. remove debarrment
>3. apply this patch
>4. edit patron again
>5. verify that debarrment wasn't added
>6. intentionally add debarrment and verify that it's saved
>---
> .../prog/en/modules/members/memberentrygen.tt      |    6 +++---
> 1 files changed, 3 insertions(+), 3 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt
>index b158556..15ed9d7 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt
>@@ -1221,11 +1221,11 @@
>                 [% END %]
>                     <p><a href="#" id="add_manual_restriction">Add manual restriction</a></p>
>                     <fieldset id="manual_restriction_form">
>-                        <input type="hidden" id="add_debarment" name="add_debarment" value="1" />
>+                        <input type="hidden" id="add_debarment" name="add_debarment" value="0" />
>                         <legend>Add manual restriction</legend>
>                         <ol>
>-                            <li><label for="debarred_comment">Comment: </label><input type="text" id="debarred_comment" name="debarred_comment" onchange="$('#add_debarment').prop('checked', true);" /></li>
>-                            <li><label for="debarred_expiration">Expiration: </label><input name="debarred_expiration" id="debarred_expiration" size="10" readonly="readonly" value="" class="datepicker" onchange="$('#add_debarment').prop('checked', true);" />
>+                            <li><label for="debarred_comment">Comment: </label><input type="text" id="debarred_comment" name="debarred_comment" onchange="$('#add_debarment').val(1);" /></li>
>+                            <li><label for="debarred_expiration">Expiration: </label><input name="debarred_expiration" id="debarred_expiration" size="10" readonly="readonly" value="" class="datepicker" onchange="$('#add_debarment').val(1);" />
>                                     <a href='javascript:void(0)' onclick="$('#debarred_expiration').val('');">Clear date</a></li>
> 
>                         </ol>
>-- 
>1.7.2.5

Actions: View | Diff | Splinter Review

Attachments on bug 11216: 22819 | 22822 | 22826 | 22890 | 22894