You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by kn...@apache.org on 2007/05/31 22:27:31 UTC

svn commit: r543244 - in /incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket: MarkupContainer.java settings/IDebugSettings.java settings/Settings.java

Author: knopp
Date: Thu May 31 13:27:30 2007
New Revision: 543244

URL: http://svn.apache.org/viewvc?view=rev&rev=543244
Log:
Added debugging information about markup container classes

Modified:
    incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/MarkupContainer.java
    incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/settings/IDebugSettings.java
    incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/settings/Settings.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=543244&r1=543243&r2=543244
==============================================================================
--- 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 Thu May 31 13:27:30 2007
@@ -618,7 +618,23 @@
 			setIgnoreAttributeModifier(true);
 			renderComponentTag(associatedMarkupOpenTag);
 			associatedMarkupStream.next();
+			
+			if (getApplication().getDebugSettings().isOutputMarkupContainerClassName()) 
+			{
+				getResponse().write("<!-- MARKUP FOR ");
+				getResponse().write(getClass().getName());
+				getResponse().write(" BEGIN -->");
+			}
+				
 			renderComponentTagBody(associatedMarkupStream, associatedMarkupOpenTag);
+			
+			if (getApplication().getDebugSettings().isOutputMarkupContainerClassName())
+			{
+				getResponse().write("<!-- MARKUP FOR ");
+				getResponse().write(getClass().getName());
+				getResponse().write(" END -->");
+			}
+				
 			renderClosingComponentTag(associatedMarkupStream, associatedMarkupOpenTag, false);
 			setMarkupStream(originalMarkupStream);
 		}

Modified: incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/settings/IDebugSettings.java
URL: http://svn.apache.org/viewvc/incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/settings/IDebugSettings.java?view=diff&rev=543244&r1=543243&r2=543244
==============================================================================
--- incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/settings/IDebugSettings.java (original)
+++ incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/settings/IDebugSettings.java Thu May 31 13:27:30 2007
@@ -60,4 +60,22 @@
 	 * 
 	 */
 	boolean isAjaxDebugModeEnabled();
+	
+	/**
+	 * Enables wrapping output of markup container in html comments that
+	 * contain markup container's class name. (Useful for determining which
+	 * part of page belongs to which markup file).
+	 * 
+	 * @param enable
+	 */
+	void setOutputMarkupContainerClassName(boolean enable);
+	
+	/**
+	 * Returns whether the output of markup container's should be wrapped
+	 * by comments containing the container's class name.
+	 * 
+	 * @return true if the markup container's class name should be written
+	 *         to response
+	 */
+	public boolean isOutputMarkupContainerClassName();
 }

Modified: incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/settings/Settings.java
URL: http://svn.apache.org/viewvc/incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/settings/Settings.java?view=diff&rev=543244&r1=543243&r2=543244
==============================================================================
--- incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/settings/Settings.java (original)
+++ incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/settings/Settings.java Thu May 31 13:27:30 2007
@@ -292,6 +292,11 @@
 	private boolean stripJavascriptCommentsAndWhitespace;
 
 	/**
+	 * Whether the container's class name should be printed to response (in a html comment).
+	 */
+	private boolean outputMarkupContainerClassName = false;
+	
+	/**
 	 * Create the application settings, carrying out any necessary
 	 * initialisations.
 	 * 
@@ -1246,5 +1251,21 @@
 	public void setStripJavascriptCommentsAndWhitespace(boolean value)
 	{
 		stripJavascriptCommentsAndWhitespace = value;
+	}
+	
+	/**
+	 * @see org.apache.wicket.settings.IDebugSettings#setOutputMarkupContainerClassName(boolean)
+	 */
+	public void setOutputMarkupContainerClassName(boolean enable)
+	{
+		outputMarkupContainerClassName = enable;
+	}
+
+	/**
+	 * @see org.apache.wicket.settings.IDebugSettings#isOutputMarkupContainerClassName()
+	 */
+	public boolean isOutputMarkupContainerClassName()
+	{
+		return outputMarkupContainerClassName;
 	}
 }