You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by aw...@apache.org on 2007/05/22 22:39:05 UTC

svn commit: r540741 - in /myfaces/trinidad/trunk/trinidad/trinidad-api/src/main/java/org/apache/myfaces/trinidad/validator: DoubleRangeValidator.java LongRangeValidator.java

Author: awiner
Date: Tue May 22 13:39:04 2007
New Revision: 540741

URL: http://svn.apache.org/viewvc?view=rev&rev=540741
Log:
TRINIDAD-34:  tr:longRangeValidator/tr:doubleRangeValidator: Logic for showing NOT_IN_RANGE is wrong

Modified:
    myfaces/trinidad/trunk/trinidad/trinidad-api/src/main/java/org/apache/myfaces/trinidad/validator/DoubleRangeValidator.java
    myfaces/trinidad/trunk/trinidad/trinidad-api/src/main/java/org/apache/myfaces/trinidad/validator/LongRangeValidator.java

Modified: myfaces/trinidad/trunk/trinidad/trinidad-api/src/main/java/org/apache/myfaces/trinidad/validator/DoubleRangeValidator.java
URL: http://svn.apache.org/viewvc/myfaces/trinidad/trunk/trinidad/trinidad-api/src/main/java/org/apache/myfaces/trinidad/validator/DoubleRangeValidator.java?view=diff&rev=540741&r1=540740&r2=540741
==============================================================================
--- myfaces/trinidad/trunk/trinidad/trinidad-api/src/main/java/org/apache/myfaces/trinidad/validator/DoubleRangeValidator.java (original)
+++ myfaces/trinidad/trunk/trinidad/trinidad-api/src/main/java/org/apache/myfaces/trinidad/validator/DoubleRangeValidator.java Tue May 22 13:39:04 2007
@@ -319,7 +319,7 @@
         
         if (doubleValue > max)
         {
-          if (min == Double.MIN_VALUE)//the default...
+          if (min != Double.MIN_VALUE)//the default...
           {
              throw new ValidatorException
                         (_getNotInRangeMessage(context, component, value, min, max));
@@ -333,7 +333,7 @@
 
         if (doubleValue < min)
         {
-          if (max == Double.MAX_VALUE)//the default...
+          if (max != Double.MAX_VALUE)//the default...
           {
             throw new ValidatorException
                         (_getNotInRangeMessage(context, component, value, min, max));

Modified: myfaces/trinidad/trunk/trinidad/trinidad-api/src/main/java/org/apache/myfaces/trinidad/validator/LongRangeValidator.java
URL: http://svn.apache.org/viewvc/myfaces/trinidad/trunk/trinidad/trinidad-api/src/main/java/org/apache/myfaces/trinidad/validator/LongRangeValidator.java?view=diff&rev=540741&r1=540740&r2=540741
==============================================================================
--- myfaces/trinidad/trunk/trinidad/trinidad-api/src/main/java/org/apache/myfaces/trinidad/validator/LongRangeValidator.java (original)
+++ myfaces/trinidad/trunk/trinidad/trinidad-api/src/main/java/org/apache/myfaces/trinidad/validator/LongRangeValidator.java Tue May 22 13:39:04 2007
@@ -321,7 +321,7 @@
         
         if (longValue > max)
         {
-          if (min == Long.MIN_VALUE)//the default...
+          if (min != Long.MIN_VALUE)//the default...
           {
              throw new ValidatorException
                         (_getNotInRangeMessage(context, component, value, IntegerUtils.getString(min), IntegerUtils.getString(max)));
@@ -335,7 +335,7 @@
 
         if (longValue < min)
         {
-          if (max == Long.MAX_VALUE)//the default...
+          if (max != Long.MAX_VALUE)//the default...
           {
             throw new ValidatorException
                         (_getNotInRangeMessage(context, component, value, IntegerUtils.getString(min), IntegerUtils.getString(max)));