You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by sv...@apache.org on 2013/06/05 11:11:39 UTC

[2/2] git commit: WICKET-5218 move child.configure() to #onConfigure()

WICKET-5218 move child.configure() to #onConfigure()

Project: http://git-wip-us.apache.org/repos/asf/wicket/repo
Commit: http://git-wip-us.apache.org/repos/asf/wicket/commit/02305741
Tree: http://git-wip-us.apache.org/repos/asf/wicket/tree/02305741
Diff: http://git-wip-us.apache.org/repos/asf/wicket/diff/02305741

Branch: refs/heads/master
Commit: 02305741273ded4b8de8f3fb6280475e4f68930b
Parents: ab4a0e3
Author: svenmeier <sv...@meiers.net>
Authored: Wed Jun 5 11:10:56 2013 +0200
Committer: svenmeier <sv...@meiers.net>
Committed: Wed Jun 5 11:10:56 2013 +0200

----------------------------------------------------------------------
 .../markup/html/basic/EnclosureContainer.java      |    8 ++++++--
 1 files changed, 6 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/wicket/blob/02305741/wicket-core/src/main/java/org/apache/wicket/markup/html/basic/EnclosureContainer.java
----------------------------------------------------------------------
diff --git a/wicket-core/src/main/java/org/apache/wicket/markup/html/basic/EnclosureContainer.java b/wicket-core/src/main/java/org/apache/wicket/markup/html/basic/EnclosureContainer.java
index 3f7959a..09845e1 100644
--- a/wicket-core/src/main/java/org/apache/wicket/markup/html/basic/EnclosureContainer.java
+++ b/wicket-core/src/main/java/org/apache/wicket/markup/html/basic/EnclosureContainer.java
@@ -71,10 +71,14 @@ public class EnclosureContainer extends WebMarkupContainer
 		setRenderBodyOnly(true);
 	}
 
+	/**
+	 * Overriden to set the visibility depending on childs {@link #determineVisibility()}.
+	 */
 	@Override
-	public boolean isVisible()
+	protected void onConfigure()
 	{
 		child.configure();
-		return child.determineVisibility();
+
+		setVisible(child.determineVisibility());
 	}
 }