Attachment 120919 Details for Bug 28313 – Bug 28313: Add street type to alternate address in OPAC

[patch] Bug 28313: Add street type to alternate address in OPAC

Bug-28313-Add-street-type-to-alternate-address-in-.patch (text/plain), 4.85 KB, created by David Nind on 2021-05-13 11:28:18 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: David Nind

Created: 2021-05-13 11:28:18 UTC

Size: 4.85 KB

patch

obsolete

>From 83a58700a3bd5d3e9a29e879a391d21c4b22efc8 Mon Sep 17 00:00:00 2001
>From: Lucas Gass <lucas@bywatersolutions.com>
>Date: Mon, 10 May 2021 21:16:59 +0000
>Subject: [PATCH] Bug 28313: Add street type to alternate address in OPAC
>
>1. Add some values to the ROADTYPE authorized value
>2. Go to PatronSelfRegistrationBorrowerUnwantedField and make sure B_streettype is NOT checked
>3. Go the self reg form, no alternate address street type field
>4. Apply patch
>5. Check the self reg form, B_streettype should be there with all the options defined in ROADTYPE a.v.
>6. Check the self modification form, B_street type should be there.
>7. Make sure PatronSelfRegistrationBorrowerUnwantedField and PatronSelfModificationBorrowerUnwantedField hide the field correctly.
>8. Make sure PatronSelfRegistrationBorrowerMandatoryField properly makes the field required.
>
>Signed-off-by: David Nind <david@davidnind.com>
>---
> .../bootstrap/en/modules/opac-memberentry.tt        | 21 ++++++++++++++++++++-
> 1 file changed, 20 insertions(+), 1 deletion(-)
>
>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 2a97dff6e7..c452dc37e2 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-memberentry.tt
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-memberentry.tt
>@@ -190,7 +190,7 @@
> 
>                 <form method="post" action="/cgi-bin/koha/opac-memberentry.pl" id="memberentry-form" autocomplete="off">
> 
>-                [% FOREACH field = ['streetnumber' 'streettype'  'cardnumber' 'branchcode' 'categorycode' 'title' 'surname' 'firstname' 'dateofbirth' 'initials' 'othernames' 'address' 'address2' 'city' 'state' 'zipcode' 'country' 'phone' 'phonepro' 'mobile' 'email' 'emailpro' 'fax' 'B_address' 'B_address2' 'B_city' 'B_state' 'B_zipcode' 'B_country' 'B_phone' 'B_email' 'contactnote' 'altcontactsurname' 'altcontactfirstname' 'altcontactaddress1' 'altcontactaddress2' 'altcontactaddress3' 'altcontactstate' 'altcontactzipcode' 'altcontactcountry' 'altcontactphone' 'password' ] %]
>+                [% FOREACH field = ['streetnumber' 'streettype'  'cardnumber' 'branchcode' 'categorycode' 'title' 'surname' 'firstname' 'dateofbirth' 'initials' 'othernames' 'address' 'address2' 'city' 'state' 'zipcode' 'country' 'phone' 'phonepro' 'mobile' 'email' 'emailpro' 'fax' 'B_streettype' 'B_address' 'B_address2' 'B_city' 'B_state' 'B_zipcode' 'B_country' 'B_phone' 'B_email' 'contactnote' 'altcontactsurname' 'altcontactfirstname' 'altcontactaddress1' 'altcontactaddress2' 'altcontactaddress3' 'altcontactstate' 'altcontactzipcode' 'altcontactcountry' 'altcontactphone' 'password' ] %]
>                     [% IF mandatory.defined( field ) %]
>                         [% SET required.$field = 'required' %]
>                     [% END %]
>@@ -592,6 +592,25 @@
>                                     <legend id="alternateaddress_legend">Alternate address</legend>
> 
>                                     <ol>
>+                                        [% IF roadtypes %]
>+                                            [% UNLESS hidden.defined('B_streettype') %]
>+                                                <li>
>+                                                    <label for="borrower_B_streettype" class="[% required.B_streettype | html %]">Street type:</label>
>+
>+                                                    <select name="borrower_B_streettype" name="borrower_B_streettype" class="[% required.B_streettype | html %]">
>+                                                        <option value=""></option>
>+                                                        [% FOR roadtype IN roadtypes %]
>+                                                            [% IF roadtype.authorised_value == patron.streettype %]
>+                                                                <option value="[% roadtype.authorised_value | html %]" selected="selected">[% roadtype.lib | html %]</option>
>+                                                            [% ELSE %]
>+                                                                <option value="[% roadtype.authorised_value | html %]">[% roadtype.lib | html %]</option>
>+                                                            [% END %]
>+                                                        [% END %]
>+                                                    </select>
>+                                                    <div class="required_label [% required.B_streettype | html %]">Required</div>
>+                                                </li>
>+                                            [% END %]
>+                                        [% END %]
>                                         [% UNLESS hidden.defined('B_address') %]
>                                             <li>
>                                                 <label for="borrower_B_address" class="[% required.B_address | html %]">Address:</label>
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 28313: 120813 | 120919 | 121592 | 122213