Changeset 5033

Timestamp:

07/03/08 13:56:00 (6 months ago)

Author:

joonas.lehtinen@…

Message:

Corrections for #1867

Location:

trunk/src/com/itmill/toolkit/ui

Files:

• AbstractComponent.java (modified) (2 diffs)
• Form.java (modified) (3 diffs)

trunk/src/com/itmill/toolkit/ui/AbstractComponent.java

@@ -14 +14 @@
 
 import com.itmill.toolkit.Application;
-import com.itmill.toolkit.data.Validator.EmptyValueException;
 import com.itmill.toolkit.event.EventRouter;
 import com.itmill.toolkit.event.MethodEventSource;
-import com.itmill.toolkit.terminal.CompositeErrorMessage;
 import com.itmill.toolkit.terminal.ErrorMessage;
 import com.itmill.toolkit.terminal.PaintException;
@@ -627 +625 @@
                 final ErrorMessage error = getErrorMessage();
                 if (error != null) {
-                    // Do not display empty value errors for
-                    // empty required fields.
-                    if (!(error instanceof EmptyValueException ||
-                          (error instanceof CompositeErrorMessage &&
-                           ((CompositeErrorMessage)error).hasErrorMessageClass(EmptyValueException.class))))
-                        error.paint(target);
+                    error.paint(target);
                 }
             } else {

trunk/src/com/itmill/toolkit/ui/Form.java

@@ -16 +16 @@
 import com.itmill.toolkit.data.Validatable;
 import com.itmill.toolkit.data.Validator;
-import com.itmill.toolkit.data.Validator.EmptyValueException;
 import com.itmill.toolkit.data.Validator.InvalidValueException;
 import com.itmill.toolkit.data.util.BeanItem;
@@ -172 +171 @@
             formFooter.paint(target);
         }
-        
-        // AbstractComponent.paint() does not paint EmptyValueExceptions and
-        // filters them out, but Form wants to paint them, so we have to
-        // see if the error was skipped.
-        // Efficiency note: also AbstractComponent.paint() calls
-        // getErrorMessage(), which is a bit heavy call.
-        final ErrorMessage error = getErrorMessage();
-        if (error instanceof EmptyValueException ||
-            (error instanceof CompositeErrorMessage &&
-             ((CompositeErrorMessage)error).hasErrorMessageClass(EmptyValueException.class)))
-            error.paint(target);
     }
 
@@ -196 +184 @@
      */
     public ErrorMessage getErrorMessage() {
+
         // Reimplement the checking of validation error by using
         // getErrorMessage() recursively instead of validate().
         ErrorMessage validationError = null;
-        for (final Iterator i = propertyIds.iterator(); i.hasNext();) {
-            try {
-                AbstractComponent field = (AbstractComponent) fields.get(i
-                        .next());
-                validationError = field.getErrorMessage();
-                if (validationError != null) {
-                    // Skip empty errors
-                    if (validationError.toString().isEmpty())
-                        continue;
-                    break;
+        if (isValidationVisible()) {
+            for (final Iterator i = propertyIds.iterator(); i.hasNext();) {
+                try {
+                    AbstractComponent field = (AbstractComponent) fields.get(i
+                            .next());
+                    validationError = field.getErrorMessage();
+                    if (validationError != null) {
+                        // Skip empty errors
+                        if ("".equals(validationError.toString())) {
+                            continue;
+                        }
+                        break;
+                    }
+                } catch (ClassCastException ignored) {
                 }
-            } catch (ClassCastException ignored) {
-            }
-        }
-        
-        // The rest is reimplementation of the latter part of
-        // AbstractField.getErrorMessage()
-        
+            }
+        }
+
         // Check if there are any systems errors
         final ErrorMessage superError = super.getErrorMessage();