From 6de6c69a2e5befd7e01be30eef667609da953b7e Mon Sep 17 00:00:00 2001
From: Joonas Lehtinen <joonas.lehtinen@itmill.com>
Date: Mon, 19 May 2008 10:25:55 +0000
Subject: [PATCH] Fixed known problems for #736 with testcase [4544]

svn changeset:4545/svn branch:trunk
---
 src/com/itmill/toolkit/ui/Form.java | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/src/com/itmill/toolkit/ui/Form.java b/src/com/itmill/toolkit/ui/Form.java
index bb6ba54dff..42c2cf52ac 100644
--- a/src/com/itmill/toolkit/ui/Form.java
+++ b/src/com/itmill/toolkit/ui/Form.java
@@ -119,6 +119,7 @@ public class Form extends AbstractField implements Item.Editor, Buffered, Item,
      */
     public Form() {
         this(null);
+        setValidationVisible(false);
     }
 
     /**
@@ -143,6 +144,8 @@ public class Form extends AbstractField implements Item.Editor, Buffered, Item,
         super();
         setLayout(formLayout);
         setFieldFactory(fieldFactory);
+        setValidationVisible(false);
+
     }
 
     /* Documented in interface */
@@ -164,6 +167,11 @@ public class Form extends AbstractField implements Item.Editor, Buffered, Item,
 
         LinkedList problems = null;
 
+        // Only commit on valid state if so requested
+        if (!isInvalidCommitted() && !isValid()) {
+            return;
+        }
+
         // Try to commit all
         for (final Iterator i = propertyIds.iterator(); i.hasNext();) {
             try {
-- 
2.39.5