Thread: [limesurvey-developers] Email Validation added.
The leading Open Source survey tool
Brought to you by:
c_schmitz
From: Harsha H. <har...@gm...> - 2010-01-19 19:35:39
|
HI Carsten I managed to find the problem of validating emails in common.php.First it does not pass (! # $ % & * + = ? ^ ` { | } ~) anywhere after calling sanitize_email method. It removes all the characters not in the regex with null. But according to RFC5322 those are valid characters for local part of email addresses. So i modified the sanitize_email method to include those characters. But i dont know whether it is ok as sanitize.php resides in classes/core package.Pl Let me know if it has any concern. Then i used newest RFC 3696 Parser to validate email addresses and found those characters are stored in the database without any issue. Now it allows all the possible characters ( ! # $ % & * + = ? ^ ` { | } ~ ) in the local part of the email addresses :) Patch is attached. Please have a look and let me know any issue. http://bugs.limesurvey.org/view.php?id=3901 Cheers Harsha |