ClearOS Bug Tracker

Main | My View | View Issues | Change Log | Roadmap | Repositories
  

View Revisions: Issue #22701 [ Back to Issue ]
Summary 0022701: Using multiple notification e-mails addresses gives an invalid e-mail address message
Revision 2019-01-08 14:10 by NickH
Description If you set up the Events with multiple addresses in the "Email to Notify" field you get an invalid e-mail address error.

I believe it is because the explode function in /usr/clearos/apps/events/libraries/Events.php is leaving a trailing space in the e-mail field. If you trim the "$email" parameter before testing, for validity then it works, so change:

if (!preg_match("/^([a-zA-Z0-9])+([a-zA-Z0-9\._-])*@([a-zA-Z0-9_-])+([a-zA-Z0-9\._-]+)+$/", $email))

to:

if (!preg_match("/^([a-zA-Z0-9])+([a-zA-Z0-9\._-])*@([a-zA-Z0-9_-])+([a-zA-Z0-9\._-]+)+$/", trim($email)))
Revision 2019-01-08 14:12 by NickH
Description If you set up the Events with multiple addresses in the "Email to Notify" field you get an invalid e-mail address error.

I believe it is because the explode function in /usr/clearos/apps/events/libraries/Events.php is leaving a trailing space in the e-mail field. If you trim the "$email" parameter before testing, for validity then it works, so change line 1028 from:

if (!preg_match("/^([a-zA-Z0-9])+([a-zA-Z0-9\._-])*@([a-zA-Z0-9_-])+([a-zA-Z0-9\._-]+)+$/", $email))

to:

if (!preg_match("/^([a-zA-Z0-9])+([a-zA-Z0-9\._-])*@([a-zA-Z0-9_-])+([a-zA-Z0-9\._-]+)+$/", trim($email)))
Revision 2019-01-15 03:21 by NickH
Description If you set up the Events with multiple addresses in the "Email to Notify" field you get an invalid e-mail address error.

I believe it is because the explode function in /usr/clearos/apps/events/libraries/Events.php is leaving a trailing space in the e-mail field. If you trim the "$email" parameter before testing, for validity then it works, so change line 1027 from:

if (!preg_match("/^([a-zA-Z0-9])+([a-zA-Z0-9\._-])*@([a-zA-Z0-9_-])+([a-zA-Z0-9\._-]+)+$/", $email))

to:

if (!preg_match("/^([a-zA-Z0-9])+([a-zA-Z0-9\._-])*@([a-zA-Z0-9_-])+([a-zA-Z0-9\._-]+)+$/", trim($email)))

Sitemap

Foundation

Company

Partners

Purchase

Contact Us