[hibernate-issues] [Hibernate-JIRA] Issue Comment Edited: (HV-435) org.hibernate.validator.constraints.impl.PatternValidator rejecting empty strings

[ http://opensource.atlassian.com/projects/hibernate/browse/HV-435?page=com... ] Stephen Hiley edited comment on HV-435 at 2/14/11 6:26 PM: ----------------------------------------------------------- Yes, I implemented that already, but consider that a workaround. We are developing under a Spring/JSP framework, and so do not have the option of automatically converting empty strings to nulls. So I understand your position, but retaining the status quo will make this an ongoing pain point for us and many others. From a practical standpoint, I think permitting empty strings will be the better path. Thanks for your consideration! A case in point from your own framework where this has been successfully employed: {code} /** * Validate that the string is a valid URL. * * @author Hardy Ferentschik */ public class URLValidator implements ConstraintValidator<URL, String> { private String protocol; private String host; private int port; public void initialize(URL url) { this.protocol = url.protocol(); this.host = url.host(); this.port = url.port(); } public boolean isValid(String value, ConstraintValidatorContext constraintValidatorContext) { if ( value == null || value.length() == 0 ) { return true; } {code} was (Author: shiley65): Yes, I implemented that already, but consider that a workaround. We are developing under a Spring/JSP framework, and so do not have the option of automatically converting empty strings to nulls. So I understand your position, but retaining the status quo will make this an ongoing pain point for us and many others. From a practical standpoint, I think permitting empty strings will be the better path. Thanks for your consideration! -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://opensource.atlassian.com/projects/hibernate/secure/Administrators.... - For more information on JIRA, see: http://www.atlassian.com/software/jira