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/03/01 02:44:15 UTC

svn commit: r513098 - /incubator/wicket/branches/wicket-1.x/wicket/src/main/java/wicket/Component.java

Author: ivaynberg
Date: Wed Feb 28 17:44:14 2007
New Revision: 513098

URL: http://svn.apache.org/viewvc?view=rev&rev=513098
Log:
add visibility safeguard for header contributions

Modified:
    incubator/wicket/branches/wicket-1.x/wicket/src/main/java/wicket/Component.java

Modified: incubator/wicket/branches/wicket-1.x/wicket/src/main/java/wicket/Component.java
URL: http://svn.apache.org/viewvc/incubator/wicket/branches/wicket-1.x/wicket/src/main/java/wicket/Component.java?view=diff&rev=513098&r1=513097&r2=513098
==============================================================================
--- incubator/wicket/branches/wicket-1.x/wicket/src/main/java/wicket/Component.java (original)
+++ incubator/wicket/branches/wicket-1.x/wicket/src/main/java/wicket/Component.java Wed Feb 28 17:44:14 2007
@@ -1764,17 +1764,20 @@
 	 */
 	public void renderHead(final HtmlHeaderContainer container)
 	{
-		// Ask all behaviors if they have something to contribute to the
-		// header or body onLoad tag.
-		if (this.behaviors != null)
+		if (isVisible())
 		{
-			final Iterator iter = this.behaviors.iterator();
-			while (iter.hasNext())
+			// Ask all behaviors if they have something to contribute to the
+			// header or body onLoad tag.
+			if (this.behaviors != null)
 			{
-				IBehavior behavior = (IBehavior)iter.next();
-				if (behavior instanceof IHeaderContributor)
+				final Iterator iter = this.behaviors.iterator();
+				while (iter.hasNext())
 				{
-					((IHeaderContributor)behavior).renderHead(container.getHeaderResponse());
+					IBehavior behavior = (IBehavior)iter.next();
+					if (behavior instanceof IHeaderContributor)
+					{
+						((IHeaderContributor)behavior).renderHead(container.getHeaderResponse());
+					}
 				}
 			}
 		}
@@ -2259,8 +2262,8 @@
 	}
 
 	/**
-	 * Gets a URL for the listener interface on a behaviour
-	 * (e.g. IBehaviorListener on AjaxPagingNavigationBehavior).
+	 * Gets a URL for the listener interface on a behaviour (e.g.
+	 * IBehaviorListener on AjaxPagingNavigationBehavior).
 	 * 
 	 * @param behaviour
 	 *            The behaviour that the URL should point to
@@ -2268,7 +2271,8 @@
 	 *            The listener interface that the URL should call
 	 * @return The URL
 	 */
-	public final CharSequence urlFor(final IBehavior behaviour, final RequestListenerInterface listener)
+	public final CharSequence urlFor(final IBehavior behaviour,
+			final RequestListenerInterface listener)
 	{
 		return getRequestCycle().urlFor(this, behaviour, listener);
 	}