You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ofbiz.apache.org by "Jacopo Cappellato (JIRA)" <ji...@apache.org> on 2007/08/09 21:55:42 UTC

[jira] Assigned: (OFBIZ-668) UtilValidate.isEmail has incorrect logic

     [ https://issues.apache.org/jira/browse/OFBIZ-668?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jacopo Cappellato reassigned OFBIZ-668:
---------------------------------------

    Assignee: Jacopo Cappellato

> UtilValidate.isEmail has incorrect logic
> ----------------------------------------
>
>                 Key: OFBIZ-668
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-668
>             Project: OFBiz
>          Issue Type: Bug
>          Components: framework
>    Affects Versions: SVN trunk
>         Environment: all
>            Reporter: John Martin
>            Assignee: Jacopo Cappellato
>            Priority: Minor
>
> Two problems:
> 1. The isEmail(String email, boolean requireDot) test for an empty/null string and will return true.  Obviously both these conditions should return false.
> 2. the isEmail(String email) default is to not test for a dot in the domain.  Being that most tests would be for real email address (not something like admin@localhost), that the default should be true.
> I'm also suggesting that we wrapper the Java Commons email validator class which is much more precise
>  org.apache.commons.validator.EmailValidator
> http://www.koders.com/java/fid2F1364A91DBBCED0B3D0DB88F5AA0499FD29A77F.aspx
> Any thoughts?
> Will implement after comments.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.