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();
+ }
});
}