[open-ils-commits] [GIT] Evergreen ILS branch rel_3_2 updated. fc0264346bacae7ccafee34c0155c13e53d55229

[Evergreen Git]

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "Evergreen ILS".

The branch, rel_3_2 has been updated
       via  fc0264346bacae7ccafee34c0155c13e53d55229 (commit)
      from  72ff895e86ddc16dcd5df42578d477517169fa59 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit fc0264346bacae7ccafee34c0155c13e53d55229
Author: Jason Boyer <jboyer at library.in.gov>
Date:   Thu Nov 15 11:43:30 2018 -0500

    LP1755625: Multiple emails in patron editor
    
    Even though the HTML5 "email" input type allows multiple
    addresses to be separated by commas there is nothing that
    can be done to signal to Angular.js that this is ok; it
    will only ever accept a single address. This patch forces
    the type of the input tag to plain text so Angular will
    rely on any supplied regex to verify the entry.
    
    The ui.patron.edit.au.email.regex (Regex for email field on patron
    registration) library setting can be used to control whether
    one or multiple email addresses will be permitted on the
    patron registration form.
    
    To allow multiple addresses, that setting can be:
    
    ^(?:(?:\b[^@,\s]+@[^@,\s]+\.[^@.,\s]+\b)(?:,\s?(?!$)|$))*$
    
    To allow only a single address, use:
    
    ^(?:\b[^@,\s]+@[^@,\s]+\.[^@.,\s]+\b)$
    
    To test
    -------
    [1] In the web staff client, try to register a new patron, entering
        multiple email addresseses in the email field separated by
        commas.
    [2] Note that the email field displays as invalid and the
        record cannot be saved.
    [3] Apply the patch and repeat step #1, This time, the email
        field should accept multiple addresses.
    [4] Set the Regex for email field on patron registration library
        setting to the multiple-email form listed above. Reload
        the patron registration form and verify that it will
        continue to accept multiple email addresses but will not
        accept entry that doesn't look like an email address.
    [5] Set the library setting to the single-address value. Reload
        the patron registration form; this time it should accept
        a valid email address but not multiple ones.
    
    Signed-off-by: Jason Boyer <jboyer at library.in.gov>
    Signed-off-by: Galen Charlton <gmc at equinoxinitiative.org>
    Signed-off-by: Cesar Velez <cesar.velez at equinoxinitiative.org>

diff --git a/Open-ILS/src/templates/staff/circ/patron/t_edit.tt2 b/Open-ILS/src/templates/staff/circ/patron/t_edit.tt2
index 3a502a7..3f95036 100644
--- a/Open-ILS/src/templates/staff/circ/patron/t_edit.tt2
+++ b/Open-ILS/src/templates/staff/circ/patron/t_edit.tt2
@@ -113,7 +113,9 @@ MACRO draw_form_input(cls, field, path, type, disable) BLOCK;
 %]
   <div class="col-md-3 reg-field-input">
     <input 
-      type="[% type %]" 
+      [% IF type == "email" %]type="text"
+      [% ELSE %]type="[% type %]"
+      [% END %]
       class="form-control" 
       name="[% model %]"
       ng-change="field_modified()" 

-----------------------------------------------------------------------

Summary of changes:
 .../src/templates/staff/circ/patron/t_edit.tt2     |    4 +++-
 1 files changed, 3 insertions(+), 1 deletions(-)


hooks/post-receive
-- 
Evergreen ILS