You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by ja...@apache.org on 2006/11/03 14:58:43 UTC

svn commit: r470822 - /incubator/wicket/trunk/wicket/src/main/java/wicket/markup/html/form/Form.java

Author: janne
Date: Fri Nov  3 05:58:42 2006
New Revision: 470822

URL: http://svn.apache.org/viewvc?view=rev&rev=470822
Log:
If the IFormProcessingListener.processChildren() prevents the validation of the component also the model update has to be skipped.

Modified:
    incubator/wicket/trunk/wicket/src/main/java/wicket/markup/html/form/Form.java

Modified: incubator/wicket/trunk/wicket/src/main/java/wicket/markup/html/form/Form.java
URL: http://svn.apache.org/viewvc/incubator/wicket/trunk/wicket/src/main/java/wicket/markup/html/form/Form.java?view=diff&rev=470822&r1=470821&r2=470822
==============================================================================
--- incubator/wicket/trunk/wicket/src/main/java/wicket/markup/html/form/Form.java (original)
+++ incubator/wicket/trunk/wicket/src/main/java/wicket/markup/html/form/Form.java Fri Nov  3 05:58:42 2006
@@ -979,19 +979,14 @@
 	 */
 	protected final void updateFormComponentModels()
 	{
-		visitFormComponents(new FormComponent.AbstractVisitor()
+		visitFormComponents(new ValidationVisitor() 
 		{
 			@Override
-			public void onFormComponent(final FormComponent formComponent)
+			public void validate(FormComponent formComponent)
 			{
-				// Only update the component when it is visible and valid
-				if (formComponent.isVisibleInHierarchy() && formComponent.isEnabled()
-						&& formComponent.isValid() && formComponent.isEnableAllowed())
-				{
-					// Potentially update the model
-					formComponent.updateModel();
-				}
-			}
+				// Potentially update the model
+				formComponent.updateModel();
+			}		
 		});
 	}