Fixed known problems for #736 with testcase [4544]

svn changeset:4545/svn branch:trunk

1 files changed, 8 insertions, 0 deletions

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 {