Diff of /myfaces/impl/trunk/src/java/org/apache/myfaces/lifecycle/LifecycleImpl.java

Parent Directory | Revision Log | Patch

--- myfaces/impl/trunk/src/java/org/apache/myfaces/lifecycle/LifecycleImpl.java	2005/11/25 00:03:17	348837
+++ myfaces/impl/trunk/src/java/org/apache/myfaces/lifecycle/LifecycleImpl.java	2005/11/25 00:28:55	348838
@@ -453,6 +453,16 @@ public class LifecycleImpl
     }
 
 
+    /**
+     * Walk the component tree, executing any component-bindings to reattach
+     * components to their backing beans. Also, any UIInput component is
+     * marked as Valid.
+     * <p>
+     *  Note that this method effectively breaks encapsulation; instead of
+     *  asking each component to update itself and its children, this
+     * method just reaches into each component. That makes it impossible
+     * for any component to customise its behaviour at this point.
+     */
     private static void recursivelyHandleComponentReferencesAndSetValid(FacesContext facesContext,
                                                                         UIComponent root)
     {
@@ -469,10 +479,10 @@ public class LifecycleImpl
             if (component instanceof UIInput)
             {
                 ((UIInput)component).setValid(true);
-        }
+            }
 
             recursivelyHandleComponentReferencesAndSetValid(facesContext, component);
-    }
+        }
     }
 
     public void addPhaseListener(PhaseListener phaseListener)