You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@myfaces.apache.org by lu...@apache.org on 2008/08/21 06:31:09 UTC

svn commit: r687554 - /myfaces/commons/trunk/myfaces-commons-validators/src/main/java/org/apache/myfaces/commons/validator/AbstractCompareToValidator.java

Author: lu4242
Date: Wed Aug 20 21:31:09 2008
New Revision: 687554

URL: http://svn.apache.org/viewvc?rev=687554&view=rev
Log:
remove unnecessary check for null value (checked before and if is null just stop validation)

Modified:
    myfaces/commons/trunk/myfaces-commons-validators/src/main/java/org/apache/myfaces/commons/validator/AbstractCompareToValidator.java

Modified: myfaces/commons/trunk/myfaces-commons-validators/src/main/java/org/apache/myfaces/commons/validator/AbstractCompareToValidator.java
URL: http://svn.apache.org/viewvc/myfaces/commons/trunk/myfaces-commons-validators/src/main/java/org/apache/myfaces/commons/validator/AbstractCompareToValidator.java?rev=687554&r1=687553&r2=687554&view=diff
==============================================================================
--- myfaces/commons/trunk/myfaces-commons-validators/src/main/java/org/apache/myfaces/commons/validator/AbstractCompareToValidator.java (original)
+++ myfaces/commons/trunk/myfaces-commons-validators/src/main/java/org/apache/myfaces/commons/validator/AbstractCompareToValidator.java Wed Aug 20 21:31:09 2008
@@ -298,7 +298,7 @@
                 value.toString(),
                 (alternateOperatorName == null) ? nameForOperator(operator) : alternateOperatorName,
                 foreignComponent.getId(),
-                (foreignValue == null) ? foreignComponent.getId() : foreignValue.toString()
+                foreignValue.toString()
         };
 
         String message = getMessage();