Fixed #1811: Validation logic is not logical: isRequired == false should not imply...

svn changeset:4874/svn branch:trunk

diff --git a/src/com/itmill/toolkit/ui/AbstractField.java b/src/com/itmill/toolkit/ui/AbstractField.java
index 72b85468cb7936b30a67d413c4d3a21f11f6e0fb..86fdb2bb4dbd359b0d4ef87f219a1554716debf8 100644 (file)
--- a/src/com/itmill/toolkit/ui/AbstractField.java
+++ b/src/com/itmill/toolkit/ui/AbstractField.java
@@ -595,10 +595,6 @@ public abstract class AbstractField extends AbstractComponent implements Field,
             if (isEmpty()) {
                 return false;
             }
-        } else {
-            if (isEmpty()) {
-                return true;
-            }
         }
 
         if (validators == null) {
@@ -626,10 +622,6 @@ public abstract class AbstractField extends AbstractComponent implements Field,
             if (isEmpty()) {
                 throw new Validator.InvalidValueException("");
             }
-        } else {
-            if (isEmpty()) {
-                return;
-            }
         }
 
         // If there is no validator, there can not be any errors