Details
-
Type:
Improvement
-
Status:
Closed
-
Priority:
Minor
-
Resolution: Duplicate
-
Affects Version/s: 0.1.3, 0.1.4, 0.1.5
-
Fix Version/s: None
-
Component/s: Zend_Filter
-
Labels:None
Description
While Zend_Filter_Input::testEmail() is already advertised in the manual, said method relies on Zend_Filter::isEmail(), which currently contains nothing more than a TODO marker. I don't know what kind of mega-super-duper email validation stuff was originally planned for this, but considering the method is needed by methods that are documented as if they were working, wouldn't it be nice to at least put a simple RegEx in place here, so that the method at least works in 95% of all cases?
If nothing else, it saves first-time users from having to backtrack the source of the problem throughout the ZF source.
Here's a regex that works for me:
/^[a-zA-Z]([.]?([[:alnum:]_-]+)*)?@([[:alnum:]-_]\.)[a-zA-Z]{2,4}$/
Issue Links
| This issue duplicates: | ||||
| ZF-42 | Zend_Validate::isEmail() implementation (TRAC#34) |
|
|
|
There has been an issue opened for this since the Trac days of the ZF, including a patch:
http://framework.zend.com/issues/browse/ZF-42