You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by iv...@apache.org on 2007/05/10 00:13:59 UTC

svn commit: r536688 - /incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/MarkupContainer.java

Author: ivaynberg
Date: Wed May  9 15:13:56 2007
New Revision: 536688

URL: http://svn.apache.org/viewvc?view=rev&rev=536688
Log:
WICKET-548 comments

Modified:
    incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/MarkupContainer.java

Modified: incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/MarkupContainer.java
URL: http://svn.apache.org/viewvc/incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/MarkupContainer.java?view=diff&rev=536688&r1=536687&r2=536688
==============================================================================
--- incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/MarkupContainer.java (original)
+++ incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/MarkupContainer.java Wed May  9 15:13:56 2007
@@ -858,6 +858,10 @@
 		final Page page = findPage();
 		if (page != null && page.isAttached())
 		{
+			// if page is not null and the page has already been attached,
+			// attach the component. we only attach if the page has been
+			// attached because at some point the page must be attached and the
+			// call would cascade down anyways.
 			component.attach();
 		}