Attachment 25558 Details for Bug 11809 – [Signed-off] Bug 11809 - OPAC patron details form shouldn't show 'Clear date' if field is required

[patch] [Signed-off] Bug 11809 - OPAC patron details form shouldn't show 'Clear date' if field is required

Signed-off-Bug-11809---OPAC-patron-details-form-sh.patch (text/plain), 4.06 KB, created by Marc Véron on 2014-02-23 19:49:04 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Marc Véron

Created: 2014-02-23 19:49:04 UTC

Size: 4.06 KB

patch

obsolete

>From 1c4ac42b76f6936da4c14f29ed237bc67c181746 Mon Sep 17 00:00:00 2001
>From: Owen Leonard <oleonard@myacpl.org>
>Date: Fri, 21 Feb 2014 10:35:40 -0500
>Subject: [PATCH] [Signed-off] Bug 11809 - OPAC patron details form shouldn't
> show 'Clear date' if field is required
>MIME-Version: 1.0
>Content-Type: text/plain; charset=UTF-8
>Content-Transfer-Encoding: 8bit
>
>On the OPAC's patron details form, if the date of birth field is
>required we shouldn't offer the user a "clear date." This patch removes
>the "clear date" link if the field is required.
>
>To test, configure the BorrowerMandatoryField system preference to
>require dateofbirth. Enable the OPACPatronDetails system preference and
>log in to the OPAC. View the "personal details" form and confirm that
>there is no "clear date" link next to the required date of birth field.
>
>Remove dateofbirth from the BorrowerMandatoryField system preference and
>re-test. The "clear date" link should appear.
>
>Test in both prog and Bootstrap themes.
>
>I tested with prog and bootstrap themes, adding/removing field 'dateofbirth'
>in sys pref PatronSelfRegistrationBorrowerMandatoryField.
>Patch behaves as expected.
>Signed-off-by: Marc VÃ©ron <veron@veron.ch>
>---
> koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-memberentry.tt |    4 +++-
> koha-tmpl/opac-tmpl/prog/en/modules/opac-memberentry.tt      |    4 +++-
> 2 files changed, 6 insertions(+), 2 deletions(-)
>
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-memberentry.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-memberentry.tt
>index efc210c..386c861 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-memberentry.tt
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-memberentry.tt
>@@ -161,7 +161,9 @@
>                                             <input type="text" id="borrower_dateofbirth" name="borrower_dateofbirth" value="[% borrower.dateofbirth | $KohaDates %]" size="10" />
> 
>                                             [% UNLESS action == 'edit' && !OPACPatronDetails %]
>-                                                <a href="#" style="font-size:85%;text-decoration:none;" onclick="document.getElementById('borrower_dateofbirth').value='';return false;">Clear date</a>
>+                                                [% UNLESS ( mandatory.defined('dateofbirth') ) %]
>+                                                    <a href="#" style="font-size:85%;text-decoration:none;" onclick="document.getElementById('borrower_dateofbirth').value='';return false;">Clear date</a>
>+                                                [% END %]
>                                             [% END %]
> 
>                                             [% IF mandatory.defined('dateofbirth') %]<span class="required">Required</span>[% END %]
>diff --git a/koha-tmpl/opac-tmpl/prog/en/modules/opac-memberentry.tt b/koha-tmpl/opac-tmpl/prog/en/modules/opac-memberentry.tt
>index 6913860..ec1fa71 100644
>--- a/koha-tmpl/opac-tmpl/prog/en/modules/opac-memberentry.tt
>+++ b/koha-tmpl/opac-tmpl/prog/en/modules/opac-memberentry.tt
>@@ -168,7 +168,9 @@
>                                                     <input type="text" id="borrower_dateofbirth" name="borrower_dateofbirth" value="[% borrower.dateofbirth | $KohaDates %]" size="10" />
> 
>                                                     [% UNLESS action == 'edit' && !OPACPatronDetails %]
>-                                                        <a href="#" style="font-size:85%;text-decoration:none;" onclick="document.getElementById('borrower_dateofbirth').value='';return false;">Clear date</a>
>+                                                        [% UNLESS ( mandatory.defined('dateofbirth') ) %]
>+                                                            <a href="#" style="font-size:85%;text-decoration:none;" onclick="document.getElementById('borrower_dateofbirth').value='';return false;">Clear date</a>
>+                                                        [% END %]
>                                                     [% END %]
> 
>                                                     [% IF mandatory.defined('dateofbirth') %]<span class="required">Required</span>[% END %]
>-- 
>1.7.10.4

Actions: View | Diff | Splinter Review

Attachments on bug 11809: 25544 | 25558 | 25563