You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@turbine.apache.org by se...@apache.org on 2003/11/10 04:07:13 UTC
cvs commit: jakarta-turbine-fulcrum/intake/src/java/org/apache/fulcrum/intake/validator DefaultValidator.java
seade 2003/11/09 19:07:13
Modified: intake/src/java/org/apache/fulcrum/intake/validator
DefaultValidator.java
Log:
Porting changes from Turbine: Use StringUtils. Removed redundant "else".
Revision Changes Path
1.2 +4 -4 jakarta-turbine-fulcrum/intake/src/java/org/apache/fulcrum/intake/validator/DefaultValidator.java
Index: DefaultValidator.java
===================================================================
RCS file: /home/cvs/jakarta-turbine-fulcrum/intake/src/java/org/apache/fulcrum/intake/validator/DefaultValidator.java,v
retrieving revision 1.1
retrieving revision 1.2
diff -u -r1.1 -r1.2
--- DefaultValidator.java 26 Sep 2003 13:52:25 -0000 1.1
+++ DefaultValidator.java 10 Nov 2003 03:07:13 -0000 1.2
@@ -56,6 +56,7 @@
import java.util.Map;
+import org.apache.commons.lang.StringUtils;
import org.apache.commons.logging.Log;
import org.apache.commons.logging.LogFactory;
@@ -195,12 +196,11 @@
public void assertValidity(String testValue)
throws ValidationException
{
- if (!required && (testValue == null || testValue.length() == 0))
+ if (!required && StringUtils.isEmpty(testValue))
{
return;
}
- else if (required
- && (testValue == null || testValue.length() == 0))
+ if (required && StringUtils.isEmpty(testValue))
{
errorMessage = requiredMessage;
throw new ValidationException(requiredMessage);
---------------------------------------------------------------------
To unsubscribe, e-mail: turbine-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: turbine-dev-help@jakarta.apache.org