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/04/20 00:23:56 UTC

svn commit: r530577 - /incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/border/MarkupComponentBorder.java

Author: ivaynberg
Date: Thu Apr 19 15:23:56 2007
New Revision: 530577

URL: http://svn.apache.org/viewvc?view=rev&rev=530577
Log:
todo

Modified:
    incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/border/MarkupComponentBorder.java

Modified: incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/border/MarkupComponentBorder.java
URL: http://svn.apache.org/viewvc/incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/border/MarkupComponentBorder.java?view=diff&rev=530577&r1=530576&r2=530577
==============================================================================
--- incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/border/MarkupComponentBorder.java (original)
+++ incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/border/MarkupComponentBorder.java Thu Apr 19 15:23:56 2007
@@ -36,9 +36,9 @@
 import org.apache.wicket.util.resource.locator.IResourceStreamLocator;
 
 /**
- * This is a IComponentBorder implementation that can be used if you have
- * markup that should be around a component. It works just like {@link Border}
- * so you have to have a <wicket:border>HTML before<wicket:body/>HTML after</wicket:border>
+ * This is a IComponentBorder implementation that can be used if you have markup
+ * that should be around a component. It works just like {@link Border} so you
+ * have to have a <wicket:border>HTML before<wicket:body/>HTML after</wicket:border>
  * in the html of your subclass.
  * 
  * @author jcompagner
@@ -163,6 +163,10 @@
 	{
 		final String markupType = getMarkupType(owner);
 
+		// TODO we need to expose this functionality for any class not just for
+		// markupcontainers in markupcache so we dont have to replicate this
+		// logic here
+
 		// Get locator to search for the resource
 		final IResourceStreamLocator locator = Application.get().getResourceSettings()
 				.getResourceStreamLocator();
@@ -184,9 +188,10 @@
 			// Did we find it already?
 			if (resourceStream != null)
 			{
-				ContainerInfo ci = new ContainerInfo(containerClass, locale, style,null, markupType);
+				ContainerInfo ci = new ContainerInfo(containerClass, locale, style, null,
+						markupType);
 				markupResourceStream = new MarkupResourceStream(resourceStream, ci, containerClass);
-				break; 
+				break;
 			}
 
 			// Walk up the class hierarchy one level, if markup has not