From: Josh Stompro <stomproj@larl.org>
Date: Thu, 11 Jun 2015 01:12:10 +0000 (-0500)
Subject: LP#1400801 - Updated email validation regex for Org Unit admin gui
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=ec8383ef5b0c0c52f489dca55f402ca21ebc6851;p=evergreen%2Fpines.git

LP#1400801 - Updated email validation regex for Org Unit admin gui

Grabbed the example of a simplified RFC 5322 email regex from
http://www.regular-expressions.info/email.html and lightly modified it
for case sensitivity and removed the "&" character from the localpart allowed
characters, which causes a 500 server error for some reason.

This matches many more of the valid localpart special characters.

Signed-off-by: Josh Stompro <stomproj@larl.org>
Signed-off-by: Ben Shum <bshum@biblio.org>
---

diff --git a/Open-ILS/web/conify/global/actor/org_unit.html b/Open-ILS/web/conify/global/actor/org_unit.html
index 2c34a67560..851a9ec648 100644
--- a/Open-ILS/web/conify/global/actor/org_unit.html
+++ b/Open-ILS/web/conify/global/actor/org_unit.html
@@ -239,7 +239,7 @@
 							<tr>
 								<th>&conify.org_unit.editor_pane.main_email;</th>
 								<td>
-									<span id="editor_pane_email" dojoType="dijit.form.ValidationTextBox" jsId="editor_pane_email" regExp="^\w+(?:\.\w+)*\@\w+(?:\.\w+)+$">
+									<span id="editor_pane_email" dojoType="dijit.form.ValidationTextBox" jsId="editor_pane_email" regExp="[a-zA-Z0-9!#$%'*+/=?^_`{|}~-]+(?:\.[a-zA-Z0-9!#$%'*+/=?^_`{|}~-]+)*@(?:[a-zA-Z0-9](?:[a-zA-Z0-9-]*[a-zA-Z0-9])?\.)+[a-zA-Z0-9](?:[a-zA-Z0-9-]*[a-zA-Z0-9])?">
 										<script type="dojo/connect" event="onChange">
 											if (current_ou) ou_list_store.setValue( current_ou, "email", this.getValue() );
 										</script>