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

svn commit: r584925 [7/34] - in /wicket/trunk/jdk-1.4/wicket/src: main/java/org/apache/wicket/ main/java/org/apache/wicket/ajax/ main/java/org/apache/wicket/ajax/calldecorator/ main/java/org/apache/wicket/ajax/form/ main/java/org/apache/wicket/ajax/mar...

Modified: wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/image/resource/RenderedDynamicImageResource.java
URL: http://svn.apache.org/viewvc/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/image/resource/RenderedDynamicImageResource.java?rev=584925&r1=584924&r2=584925&view=diff
==============================================================================
--- wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/image/resource/RenderedDynamicImageResource.java (original)
+++ wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/image/resource/RenderedDynamicImageResource.java Mon Oct 15 14:21:25 2007
@@ -24,17 +24,16 @@
 
 
 /**
- * A DynamicImageResource subclass that allows easy rendering of regeneratable
- * (unbuffered) dynamic images. A RenderedDynamicImageResource implements the
- * abstract method render(Graphics2D) to create/re-create a given image
- * on-the-fly. When a RenderedDynamicImageResource is serialized, the image
- * state is transient, which means it will disappear when the resource is sent
- * over the wire and then will be recreated when required.
+ * A DynamicImageResource subclass that allows easy rendering of regeneratable (unbuffered) dynamic
+ * images. A RenderedDynamicImageResource implements the abstract method render(Graphics2D) to
+ * create/re-create a given image on-the-fly. When a RenderedDynamicImageResource is serialized, the
+ * image state is transient, which means it will disappear when the resource is sent over the wire
+ * and then will be recreated when required.
  * <p>
- * The format of the image (and therefore the resource's extension) can be
- * specified with setFormat(String). The default format is "PNG" because JPEG is
- * lossy and makes generated images look bad and GIF has patent issues.
- *
+ * The format of the image (and therefore the resource's extension) can be specified with
+ * setFormat(String). The default format is "PNG" because JPEG is lossy and makes generated images
+ * look bad and GIF has patent issues.
+ * 
  * @see org.apache.wicket.markup.html.image.resource.DefaultButtonImageResource
  * @see org.apache.wicket.markup.html.image.resource.DefaultButtonImageResourceFactory
  * @author Jonathan Locke
@@ -59,7 +58,7 @@
 
 	/**
 	 * Constructor.
-	 *
+	 * 
 	 * @param width
 	 *            Width of image
 	 * @param height
@@ -73,7 +72,7 @@
 
 	/**
 	 * Constructor.
-	 *
+	 * 
 	 * @param width
 	 *            Width of image
 	 * @param height
@@ -114,7 +113,7 @@
 
 	/**
 	 * Causes the image to be redrawn the next time its requested.
-	 *
+	 * 
 	 * @see org.apache.wicket.Resource#invalidate()
 	 */
 	public synchronized void invalidate()
@@ -171,7 +170,7 @@
 
 	/**
 	 * Renders this image
-	 *
+	 * 
 	 * @return The image data
 	 */
 	protected byte[] render()
@@ -188,12 +187,11 @@
 
 	/**
 	 * Override this method to provide your rendering code
-	 *
+	 * 
 	 * @param graphics
 	 *            The graphics context to render on
-	 * @return True if the image was rendered. False if the image size was
-	 *         changed by the rendering implementation and the image should be
-	 *         re-rendered at the new size.
+	 * @return True if the image was rendered. False if the image size was changed by the rendering
+	 *         implementation and the image should be re-rendered at the new size.
 	 */
 	protected abstract boolean render(Graphics2D graphics);
 }

Modified: wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/include/Include.java
URL: http://svn.apache.org/viewvc/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/include/Include.java?rev=584925&r1=584924&r2=584925&view=diff
==============================================================================
--- wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/include/Include.java (original)
+++ wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/include/Include.java Mon Oct 15 14:21:25 2007
@@ -33,40 +33,35 @@
 
 /**
  * <p>
- * Component that includes/ renders the import result of an URL, much like JSP
- * include.
+ * Component that includes/ renders the import result of an URL, much like JSP include.
  * </p>
  * <p>
- * Use this to integrate non-Wicket locations in your page. <strong>This
- * component is NOT meant for integrating more Wicket sources as a means of
- * quick and dirty page composition. Use Panels, Borders and (Markup)inheritance
- * for page composition instead.</strong>
+ * Use this to integrate non-Wicket locations in your page. <strong>This component is NOT meant for
+ * integrating more Wicket sources as a means of quick and dirty page composition. Use Panels,
+ * Borders and (Markup)inheritance for page composition instead.</strong>
  * </p>
  * <p>
- * You can feed this component the URL directly, or use a model that should
- * deliver a valid URL. You can both use absolute (e.g.
- * http://www.theserverside.com/) and relative (e.g. mydir/mypage.html) urls.
- * This component will try to resolve relative urls to resources in the same
- * webapplication.
+ * You can feed this component the URL directly, or use a model that should deliver a valid URL. You
+ * can both use absolute (e.g. http://www.theserverside.com/) and relative (e.g. mydir/mypage.html)
+ * urls. This component will try to resolve relative urls to resources in the same webapplication.
  * </p>
  * <p>
- * The following example shows how to integrate a header and footer, coming
- * from a plain HTML source on the same server is integrated using this
- * component. The files footer.html and header.html would be located in the web
- * application root directory
+ * The following example shows how to integrate a header and footer, coming from a plain HTML source
+ * on the same server is integrated using this component. The files footer.html and header.html
+ * would be located in the web application root directory
  * </p>
  * <p>
  * Java:
- *
+ * 
  * <pre>
  *   ...
  * 	add(new Include(&quot;header&quot;, &quot;header.html&quot;));
  * 	add(new Include(&quot;footer&quot;, &quot;footer.html&quot;));
  *   ...
  * </pre>
- *
+ * 
  * Html:
- *
+ * 
  * <pre>
  *   ...
  * 	&lt;div&gt;
@@ -76,9 +71,9 @@
  * 	&lt;/div&gt;
  *   ...
  * </pre>
- *
+ * 
  * </p>
- *
+ * 
  * @author Eelco Hillenius
  */
 public class Include extends WebComponent
@@ -92,22 +87,19 @@
 	 * <p>
 	 * RFC 1738 says the following:
 	 * </p>
-	 * <blockquote>Scheme names consist of a sequence of characters. The lower
-	 * case letters "a"--"z", digits, and the characters plus ("+"), period
-	 * ("."), and hyphen ("-") are allowed. For resiliency, programs
-	 * interpreting URLs should treat upper case letters as equivalent to lower
-	 * case in scheme names (e.g., allow "HTTP" as well as "http").
-	 * </blockquote>
+	 * <blockquote>Scheme names consist of a sequence of characters. The lower case letters
+	 * "a"--"z", digits, and the characters plus ("+"), period ("."), and hyphen ("-") are allowed.
+	 * For resiliency, programs interpreting URLs should treat upper case letters as equivalent to
+	 * lower case in scheme names (e.g., allow "HTTP" as well as "http"). </blockquote>
 	 * <p>
-	 * We treat as absolute any URL that begins with such a scheme name,
-	 * followed by a colon.
+	 * We treat as absolute any URL that begins with such a scheme name, followed by a colon.
 	 * </p>
 	 */
 	private static final String VALID_SCHEME_CHARS = "abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789+.-";
 
 	/**
 	 * Construct.
-	 *
+	 * 
 	 * @param id
 	 *            component id
 	 */
@@ -118,7 +110,7 @@
 
 	/**
 	 * Construct.
-	 *
+	 * 
 	 * @param id
 	 *            component id
 	 * @param model
@@ -131,7 +123,7 @@
 
 	/**
 	 * Construct.
-	 *
+	 * 
 	 * @param id
 	 *            component id
 	 * @param modelObject
@@ -144,7 +136,7 @@
 
 	/**
 	 * Imports the contents of the url of the model object.
-	 *
+	 * 
 	 * @return the imported contents
 	 */
 	protected String importAsString()
@@ -175,7 +167,7 @@
 
 	/**
 	 * Gets whether the given url is absolute (<tt>true</tt>) or relative (<tt>false</tt>).
-	 *
+	 * 
 	 * @param url
 	 *            the url
 	 * @return whether the given url is absolute (<tt>true</tt>) or relative (<tt>false</tt>)
@@ -210,7 +202,7 @@
 
 	/**
 	 * Imports from a relative url.
-	 *
+	 * 
 	 * @param url
 	 *            the url to import
 	 * @return the imported url's contents
@@ -236,7 +228,7 @@
 
 	/**
 	 * Imports from an absolute url.
-	 *
+	 * 
 	 * @param url
 	 *            the url to import
 	 * @return the imported url's contents
@@ -255,7 +247,7 @@
 
 	/**
 	 * Imports the contents from the given url.
-	 *
+	 * 
 	 * @param url
 	 *            the url
 	 * @return the imported contents

Modified: wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/internal/HeaderResponse.java
URL: http://svn.apache.org/viewvc/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/internal/HeaderResponse.java?rev=584925&r1=584924&r2=584925&view=diff
==============================================================================
--- wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/internal/HeaderResponse.java (original)
+++ wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/internal/HeaderResponse.java Mon Oct 15 14:21:25 2007
@@ -258,17 +258,18 @@
 	{
 		if (!closed)
 		{
-			List token = Arrays.asList(new Object[] { "javascript-event", target, event, javascript });
+			List token = Arrays
+					.asList(new Object[] { "javascript-event", target, event, javascript });
 			if (wasRendered(token) == false)
 			{
 				renderJavascriptReference(WicketEventReference.INSTANCE);
-				JavascriptUtils.writeJavascript(getResponse(), "Wicket.Event.add(" + target + ", \"" +
-						event + "\", function() { " + javascript + ";});");
+				JavascriptUtils.writeJavascript(getResponse(), "Wicket.Event.add(" + target +
+						", \"" + event + "\", function() { " + javascript + ";});");
 				markRendered(token);
 			}
 		}
 	}
-	
+
 	/**
 	 * @see org.apache.wicket.markup.html.IHeaderResponse#close()
 	 */
@@ -292,11 +293,12 @@
 	{
 		return closed;
 	}
-	
+
 	/**
 	 * Once the HeaderResponse is closed, no output may be written to it anymore. To enforce that,
-	 * the {@link #getResponse()} is defined final in this class and will return a NullResponse instance once closed or otherwise
-	 * the Response provided by this method.
+	 * the {@link #getResponse()} is defined final in this class and will return a NullResponse
+	 * instance once closed or otherwise the Response provided by this method.
+	 * 
 	 * @return Response
 	 */
 	protected abstract Response getRealResponse();

Modified: wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/internal/HtmlHeaderContainer.java
URL: http://svn.apache.org/viewvc/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/internal/HtmlHeaderContainer.java?rev=584925&r1=584924&r2=584925&view=diff
==============================================================================
--- wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/internal/HtmlHeaderContainer.java (original)
+++ wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/internal/HtmlHeaderContainer.java Mon Oct 15 14:21:25 2007
@@ -33,37 +33,34 @@
 
 
 /**
- * The HtmlHeaderContainer is automatically created and added to the component
- * hierarchy by a HtmlHeaderResolver instance. HtmlHeaderContainer tries to
- * handle/render the &gt;head&gt; tag and its body. However depending on the
- * parent component, the behavior must be different. E.g. if parent component is
- * a Page all components of the page's hierarchy must be asked if they have
- * something to contribute to the &lt;head&gt; section of the html response. If
- * yes, it must <b>immediately</b> be rendered.
+ * The HtmlHeaderContainer is automatically created and added to the component hierarchy by a
+ * HtmlHeaderResolver instance. HtmlHeaderContainer tries to handle/render the &gt;head&gt; tag and
+ * its body. However depending on the parent component, the behavior must be different. E.g. if
+ * parent component is a Page all components of the page's hierarchy must be asked if they have
+ * something to contribute to the &lt;head&gt; section of the html response. If yes, it must
+ * <b>immediately</b> be rendered.
  * <p>
- * &lt;head&gt; regions may contain additional wicket components, which can be
- * added by means of add(Component) as usual.
+ * &lt;head&gt; regions may contain additional wicket components, which can be added by means of
+ * add(Component) as usual.
  * <p>
- * &lt;wicket:head&gt; tags are handled by simple WebMarkupContainers also
- * created by a HtmlHeaderResolver.
+ * &lt;wicket:head&gt; tags are handled by simple WebMarkupContainers also created by a
+ * HtmlHeaderResolver.
  * <p>
  * <ul>
  * <li> &lt;head&gt; will be inserted in output automatically if required</li>
- * <li> &lt;head&gt; is <b>not</b> a wicket specific tag and you must use add()
- * to add components referenced in body of the head tag</li>
- * <li> &lt;head&gt; is supported by panels, borders and inherited markup, but
- * is <b>not</b> copied to the output. They are for previewability only (except
- * on Pages)</li>
- * <li> &lt;wicket:head&gt; does not make sense in page markup (but does in
- * inherited page markup)</li>
- * <li> &lt;wicket:head&gt; makes sense in Panels, Borders and inherited markup
- * (of Panels, Borders and Pages)</li>
- * <li> components within &lt;wicket:head&gt; must be added by means of add(),
- * like always with Wicket. No difference.</li>
- * <li> &lt;wicket:head&gt; and it's content is copied to the output. Components
- * contained in &lt;org.apache.wicket.head&gt; are rendered as usual</li>
+ * <li> &lt;head&gt; is <b>not</b> a wicket specific tag and you must use add() to add components
+ * referenced in body of the head tag</li>
+ * <li> &lt;head&gt; is supported by panels, borders and inherited markup, but is <b>not</b> copied
+ * to the output. They are for previewability only (except on Pages)</li>
+ * <li> &lt;wicket:head&gt; does not make sense in page markup (but does in inherited page markup)</li>
+ * <li> &lt;wicket:head&gt; makes sense in Panels, Borders and inherited markup (of Panels, Borders
+ * and Pages)</li>
+ * <li> components within &lt;wicket:head&gt; must be added by means of add(), like always with
+ * Wicket. No difference.</li>
+ * <li> &lt;wicket:head&gt; and it's content is copied to the output. Components contained in
+ * &lt;org.apache.wicket.head&gt; are rendered as usual</li>
  * </ul>
- *
+ * 
  * @author Juergen Donnerstag
  */
 public class HtmlHeaderContainer extends WebMarkupContainer
@@ -71,23 +68,21 @@
 	private static final long serialVersionUID = 1L;
 
 	/**
-	 * wicket:head tags (components) must only be added once. To allow for a
-	 * little bit more control, each wicket:head has an associated scope which
-	 * by default is equal to the java class name directly associated with the
-	 * markup which contains the wicket:head. It can be modified by means of the
-	 * scope attribute.
+	 * wicket:head tags (components) must only be added once. To allow for a little bit more
+	 * control, each wicket:head has an associated scope which by default is equal to the java class
+	 * name directly associated with the markup which contains the wicket:head. It can be modified
+	 * by means of the scope attribute.
 	 */
 	private transient Map renderedComponentsPerScope;
 
 	/**
-	 * Header response that is responsible for filtering duplicate
-	 * contributions.
+	 * Header response that is responsible for filtering duplicate contributions.
 	 */
 	private transient IHeaderResponse headerResponse = null;
 
 	/**
 	 * Construct
-	 *
+	 * 
 	 * @see Component#Component(String)
 	 */
 	public HtmlHeaderContainer(final String id)
@@ -104,10 +99,10 @@
 	}
 
 	/**
-	 * First render the body of the component. And if it is the header component
-	 * of a Page (compared to a Panel or Border), than get the header sections
-	 * from all component in the hierarchy and render them as well.
-	 *
+	 * First render the body of the component. And if it is the header component of a Page (compared
+	 * to a Panel or Border), than get the header sections from all component in the hierarchy and
+	 * render them as well.
+	 * 
 	 * @see org.apache.wicket.MarkupContainer#onComponentTagBody(org.apache.wicket.markup.MarkupStream,
 	 *      org.apache.wicket.markup.ComponentTag)
 	 */
@@ -186,21 +181,21 @@
 	}
 
 	/**
-	 * Ask all child components of the Page if they have something to contribute
-	 * to the &lt;head&gt; section of the HTML output. Every component
-	 * interested must implement IHeaderContributor.
+	 * Ask all child components of the Page if they have something to contribute to the &lt;head&gt;
+	 * section of the HTML output. Every component interested must implement IHeaderContributor.
 	 * <p>
-	 * Note: HtmlHeaderContainer will be removed from the component hierarchy at
-	 * the end of the request (@see #onEndRequest()) and thus can not transport
-	 * status from one request to the next. This is true for all components
-	 * added to the header.
-	 *
+	 * Note: HtmlHeaderContainer will be removed from the component hierarchy at the end of the
+	 * request (@see #onEndRequest()) and thus can not transport status from one request to the
+	 * next. This is true for all components added to the header.
+	 * 
 	 * @param page
-	 *            Usually it is the page object, but there might also be that a WebMarkupContainer has been attached to the &lt;html&gt; tag
+	 *            Usually it is the page object, but there might also be that a WebMarkupContainer
+	 *            has been attached to the &lt;html&gt; tag
 	 * @param container
 	 *            The header component container
 	 */
-	private final void renderHeaderSections(final MarkupContainer page, final HtmlHeaderContainer container)
+	private final void renderHeaderSections(final MarkupContainer page,
+			final HtmlHeaderContainer container)
 	{
 		page.renderHead(container);
 		// Make sure all Components interested in contributing to the header
@@ -235,7 +230,7 @@
 
 	/**
 	 * Check if the header component is ok to render within the scope given.
-	 *
+	 * 
 	 * @param scope
 	 *            The scope of the header component
 	 * @param id
@@ -279,29 +274,29 @@
 
 	/**
 	 * Factory method for creating header response
-	 *
+	 * 
 	 * @return new header response
 	 */
 	protected IHeaderResponse newHeaderResponse()
 	{
 		IHeaderResponse headerResponse = RequestContext.get().getHeaderResponse();
-		if ( headerResponse == null )
+		if (headerResponse == null)
 		{
 			// no (portlet) headerResponse override, create a default one
-			headerResponse =
-				new HeaderResponse() {
-					protected Response getRealResponse()
-					{
-						return HtmlHeaderContainer.this.getResponse();
-					}
-				};
+			headerResponse = new HeaderResponse()
+			{
+				protected Response getRealResponse()
+				{
+					return HtmlHeaderContainer.this.getResponse();
+				}
+			};
 		}
 		return headerResponse;
 	}
 
 	/**
 	 * Returns the header response.
-	 *
+	 * 
 	 * @return header response
 	 */
 	public IHeaderResponse getHeaderResponse()

Modified: wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/link/AbstractLink.java
URL: http://svn.apache.org/viewvc/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/link/AbstractLink.java?rev=584925&r1=584924&r2=584925&view=diff
==============================================================================
--- wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/link/AbstractLink.java (original)
+++ wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/link/AbstractLink.java Mon Oct 15 14:21:25 2007
@@ -58,21 +58,18 @@
 	}
 
 	/**
-	 * Simple insertion string to allow disabled links to look like <i>Disabled
-	 * link </i>.
+	 * Simple insertion string to allow disabled links to look like <i>Disabled link </i>.
 	 */
 	private String beforeDisabledLink;
 
 	/**
-	 * Simple insertion string to allow disabled links to look like <i>Disabled
-	 * link </i>.
+	 * Simple insertion string to allow disabled links to look like <i>Disabled link </i>.
 	 */
 	private String afterDisabledLink;
 
 
 	/**
-	 * Sets the insertion string to allow disabled links to look like
-	 * <i>Disabled link </i>.
+	 * Sets the insertion string to allow disabled links to look like <i>Disabled link </i>.
 	 * 
 	 * @param afterDisabledLink
 	 *            The insertion string
@@ -88,8 +85,7 @@
 	}
 
 	/**
-	 * Gets the insertion string to allow disabled links to look like
-	 * <i>Disabled link </i>.
+	 * Gets the insertion string to allow disabled links to look like <i>Disabled link </i>.
 	 * 
 	 * @return The insertion string
 	 */
@@ -99,8 +95,7 @@
 	}
 
 	/**
-	 * Sets the insertion string to allow disabled links to look like
-	 * <i>Disabled link </i>.
+	 * Sets the insertion string to allow disabled links to look like <i>Disabled link </i>.
 	 * 
 	 * @param beforeDisabledLink
 	 *            The insertion string
@@ -129,8 +124,7 @@
 	}
 
 	/**
-	 * Gets the insertion string to allow disabled links to look like
-	 * <i>Disabled link </i>.
+	 * Gets the insertion string to allow disabled links to look like <i>Disabled link </i>.
 	 * 
 	 * @return The insertion string
 	 */
@@ -140,8 +134,8 @@
 	}
 
 	/**
-	 * Helper methods that both checks whether the link is enabled and whether
-	 * the action ENABLE is allowed.
+	 * Helper methods that both checks whether the link is enabled and whether the action ENABLE is
+	 * allowed.
 	 * 
 	 * @return whether the link should be rendered as enabled
 	 */
@@ -157,8 +151,7 @@
 	 *            the markup stream
 	 * @param openTag
 	 *            the open part of this tag
-	 * @see org.apache.wicket.Component#onComponentTagBody(MarkupStream,
-	 *      ComponentTag)
+	 * @see org.apache.wicket.Component#onComponentTagBody(MarkupStream, ComponentTag)
 	 */
 	protected void onComponentTagBody(final MarkupStream markupStream, final ComponentTag openTag)
 	{
@@ -181,16 +174,16 @@
 	/**
 	 * Alters the tag so that the link renders as disabled.
 	 * 
-	 * This method is meant to be called from
-	 * {@link #onComponentTag(ComponentTag)} method of the derived class.
+	 * This method is meant to be called from {@link #onComponentTag(ComponentTag)} method of the
+	 * derived class.
 	 * 
 	 * @param tag
 	 */
 	protected void disableLink(final ComponentTag tag)
 	{
 		// if the tag is an anchor proper
-		if (tag.getName().equalsIgnoreCase("a") || tag.getName().equalsIgnoreCase("link")
-				|| tag.getName().equalsIgnoreCase("area"))
+		if (tag.getName().equalsIgnoreCase("a") || tag.getName().equalsIgnoreCase("link") ||
+				tag.getName().equalsIgnoreCase("area"))
 		{
 			// Change anchor link to span tag
 			tag.setName("span");
@@ -201,8 +194,8 @@
 			tag.remove("onclick");
 		}
 		// if the tag is a button or input
-		else if ("button".equalsIgnoreCase(tag.getName())
-				|| "input".equalsIgnoreCase(tag.getName()))
+		else if ("button".equalsIgnoreCase(tag.getName()) ||
+				"input".equalsIgnoreCase(tag.getName()))
 		{
 			tag.put("disabled", "disabled");
 		}

Modified: wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/link/BookmarkablePageLink.java
URL: http://svn.apache.org/viewvc/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/link/BookmarkablePageLink.java?rev=584925&r1=584924&r2=584925&view=diff
==============================================================================
--- wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/link/BookmarkablePageLink.java (original)
+++ wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/link/BookmarkablePageLink.java Mon Oct 15 14:21:25 2007
@@ -25,8 +25,7 @@
 import org.apache.wicket.util.lang.Classes;
 
 /**
- * Renders a stable link which can be cached in a web browser and used at a
- * later time.
+ * Renders a stable link which can be cached in a web browser and used at a later time.
  * 
  * @author Jonathan Locke
  */
@@ -64,8 +63,7 @@
 	 * @param pageClass
 	 *            The class of page to link to
 	 * @param parameters
-	 *            The parameters to pass to the new page when the link is
-	 *            clicked
+	 *            The parameters to pass to the new page when the link is clicked
 	 */
 	public BookmarkablePageLink(final String id, final Class pageClass,
 			final PageParameters parameters)
@@ -77,8 +75,8 @@
 		}
 		else if (!Page.class.isAssignableFrom(pageClass))
 		{
-			throw new IllegalArgumentException("Page class must be derived from "
-					+ Page.class.getName());
+			throw new IllegalArgumentException("Page class must be derived from " +
+					Page.class.getName());
 		}
 		this.pageClassName = pageClass.getName();
 		this.parameters = parameters;
@@ -127,9 +125,8 @@
 	}
 
 	/**
-	 * THIS METHOD IS NOT USED! Bookmarkable links do not have a click handler.
-	 * It is here to satisfy the interface only, as bookmarkable links will be
-	 * dispatched by the handling servlet.
+	 * THIS METHOD IS NOT USED! Bookmarkable links do not have a click handler. It is here to
+	 * satisfy the interface only, as bookmarkable links will be dispatched by the handling servlet.
 	 * 
 	 * @see org.apache.wicket.markup.html.link.Link#onClick()
 	 */

Modified: wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/link/IPageLink.java
URL: http://svn.apache.org/viewvc/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/link/IPageLink.java?rev=584925&r1=584924&r2=584925&view=diff
==============================================================================
--- wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/link/IPageLink.java (original)
+++ wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/link/IPageLink.java Mon Oct 15 14:21:25 2007
@@ -20,20 +20,18 @@
 import org.apache.wicket.Page;
 
 /**
- * Interface that is used to implement delayed page linking. The getPage()
- * method returns an instance of Page when a link is actually clicked (thus
- * avoiding the need to create a destination Page object for every link on a
- * given Page in advance). The getPageIdentity() method returns the subclass of
- * Page that getPage() will return if and when it is called.
+ * Interface that is used to implement delayed page linking. The getPage() method returns an
+ * instance of Page when a link is actually clicked (thus avoiding the need to create a destination
+ * Page object for every link on a given Page in advance). The getPageIdentity() method returns the
+ * subclass of Page that getPage() will return if and when it is called.
  * <p>
- * This way of arranging things is useful in determining whether a link links to
- * a given page, which is in turn useful for deciding how to display the link
- * (because links in a navigation which link to a page itself are not useful and
- * generally should instead indicate where the user is in the navigation).
+ * This way of arranging things is useful in determining whether a link links to a given page, which
+ * is in turn useful for deciding how to display the link (because links in a navigation which link
+ * to a page itself are not useful and generally should instead indicate where the user is in the
+ * navigation).
  * <p>
- * To understand how getPageIdentity() is used in this way, take a look at the
- * Link.linksTo() method and its override in PageLink. Also, see the
- * documentation for getPageIdentity() below.
+ * To understand how getPageIdentity() is used in this way, take a look at the Link.linksTo() method
+ * and its override in PageLink. Also, see the documentation for getPageIdentity() below.
  * 
  * @see Link#linksTo(Page)
  * @see PageLink#linksTo(Page)
@@ -49,18 +47,16 @@
 	Page getPage();
 
 	/**
-	 * Gets the class of the destination page, which serves as a form of
-	 * identity that can be used to determine if a link is on the same Page that
-	 * it links to. When Pages are parameterized, the Link.linksTo() method
-	 * should be overridden instead.
+	 * Gets the class of the destination page, which serves as a form of identity that can be used
+	 * to determine if a link is on the same Page that it links to. When Pages are parameterized,
+	 * the Link.linksTo() method should be overridden instead.
 	 * <p>
-	 * A page's identity is important because links which are on the same page
-	 * that they link to often need to be displayed in a different way to
-	 * indicate that they are 'disabled' and don't go anywhere. Links can be
-	 * manually disabled by calling Link.setDisabled(). Links which have
-	 * setAutoEnable(true) will automatically enable or disable themselves
-	 * depending on whether or not Link.linksTo() returns true. The default
-	 * implementation of PageLink.linksTo() therefore looks like this:
+	 * A page's identity is important because links which are on the same page that they link to
+	 * often need to be displayed in a different way to indicate that they are 'disabled' and don't
+	 * go anywhere. Links can be manually disabled by calling Link.setDisabled(). Links which have
+	 * setAutoEnable(true) will automatically enable or disable themselves depending on whether or
+	 * not Link.linksTo() returns true. The default implementation of PageLink.linksTo() therefore
+	 * looks like this:
 	 * 
 	 * <pre>
 	 * private final IPageLink pageLink;

Modified: wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/link/ImageMap.java
URL: http://svn.apache.org/viewvc/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/link/ImageMap.java?rev=584925&r1=584924&r2=584925&view=diff
==============================================================================
--- wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/link/ImageMap.java (original)
+++ wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/link/ImageMap.java Mon Oct 15 14:21:25 2007
@@ -34,7 +34,7 @@
 public final class ImageMap extends WebMarkupContainer
 {
 	private static final long serialVersionUID = 1L;
-	
+
 	/** list of shape links. */
 	private final List shapeLinks = new ArrayList();
 
@@ -225,8 +225,7 @@
 		}
 
 		/**
-		 * The shape as a string using the given request cycle; will be used for
-		 * rendering.
+		 * The shape as a string using the given request cycle; will be used for rendering.
 		 * 
 		 * @return The shape as a string
 		 */
@@ -244,12 +243,17 @@
 				popupJavaScript = null;
 			}
 
-			return "<area shape=\"" + getType() + "\"" + " coords=\"" + getCoordinates() + "\""
-					+ " href=\"" + link.getURL() + "\""
-					// Output the markup ID if that was specified, so we can link tooltips, etc. to it.
-					+ (link.getOutputMarkupId() ? " id=\"" + link.getMarkupId() + "\"" : "")
-					+ ((popupJavaScript == null) ? "" : (" onClick = \"" + popupJavaScript + "\""))
-					+ ">";
+			return "<area shape=\"" + getType() + "\"" + " coords=\"" +
+					getCoordinates() +
+					"\"" +
+					" href=\"" +
+					link.getURL() +
+					"\""
+					// Output the markup ID if that was specified, so we can link tooltips, etc. to
+					// it.
+					+ (link.getOutputMarkupId() ? " id=\"" + link.getMarkupId() + "\"" : "") +
+					((popupJavaScript == null) ? "" : (" onClick = \"" + popupJavaScript + "\"")) +
+					">";
 		}
 
 		/**

Modified: wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/link/InternalFrame.java
URL: http://svn.apache.org/viewvc/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/link/InternalFrame.java?rev=584925&r1=584924&r2=584925&view=diff
==============================================================================
--- wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/link/InternalFrame.java (original)
+++ wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/link/InternalFrame.java Mon Oct 15 14:21:25 2007
@@ -26,16 +26,14 @@
 import org.apache.wicket.util.string.Strings;
 
 /**
- * Implementation of an <a
- * href="http://www.w3.org/TR/REC-html40/present/frames.html#h-16.5">inline
- * frame</a> component. Must be used with an iframe (&lt;iframe src...)
- * element. The src attribute will be generated.
+ * Implementation of an <a href="http://www.w3.org/TR/REC-html40/present/frames.html#h-16.5">inline
+ * frame</a> component. Must be used with an iframe (&lt;iframe src...) element. The src attribute
+ * will be generated.
  * 
  * @author Sven Meier
  * @author Ralf Ebert
  * 
- * @deprecated will be replaced by {@link InlineFrame} in Wicket 2.0 as that's a
- *             better name for it.
+ * @deprecated will be replaced by {@link InlineFrame} in Wicket 2.0 as that's a better name for it.
  */
 public class InternalFrame extends WebMarkupContainer implements ILinkListener
 {
@@ -45,15 +43,13 @@
 	private final IPageLink pageLink;
 
 	/**
-	 * The pagemap name where the page that will be created by this inline frame
-	 * will be created in.
+	 * The pagemap name where the page that will be created by this inline frame will be created in.
 	 */
 	private final String pageMapName;
 
 	/**
-	 * Constructs an inline frame that instantiates the given Page class when
-	 * the content of the inline frame is requested. The instantiated Page is
-	 * used to render a response to the user.
+	 * Constructs an inline frame that instantiates the given Page class when the content of the
+	 * inline frame is requested. The instantiated Page is used to render a response to the user.
 	 * 
 	 * @param id
 	 *            See Component
@@ -88,8 +84,8 @@
 	}
 
 	/**
-	 * This constructor is ideal if a Page object was passed in from a previous
-	 * Page. Construct an inline frame containing the given Page.
+	 * This constructor is ideal if a Page object was passed in from a previous Page. Construct an
+	 * inline frame containing the given Page.
 	 * 
 	 * @param id
 	 *            See component
@@ -120,19 +116,17 @@
 	/**
 	 * This constructor is ideal for constructing pages lazily.
 	 * 
-	 * Constructs an inline frame which invokes the getPage() method of the
-	 * IPageLink interface when the content of the inline frame is requested.
-	 * Whatever Page objects is returned by this method will be rendered back to
-	 * the user.
+	 * Constructs an inline frame which invokes the getPage() method of the IPageLink interface when
+	 * the content of the inline frame is requested. Whatever Page objects is returned by this
+	 * method will be rendered back to the user.
 	 * 
 	 * @param id
 	 *            See Component
 	 * @param pageMap
 	 *            the pagemap where the page of the inline frame must be in
 	 * @param pageLink
-	 *            An implementation of IPageLink which will create the page to
-	 *            be contained in the inline frame if and when the content is
-	 *            requested
+	 *            An implementation of IPageLink which will create the page to be contained in the
+	 *            inline frame if and when the content is requested
 	 */
 	public InternalFrame(final String id, final IPageMap pageMap, IPageLink pageLink)
 	{

Modified: wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/link/Link.java
URL: http://svn.apache.org/viewvc/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/link/Link.java?rev=584925&r1=584924&r2=584925&view=diff
==============================================================================
--- wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/link/Link.java (original)
+++ wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/link/Link.java Mon Oct 15 14:21:25 2007
@@ -27,14 +27,13 @@
 import org.apache.wicket.version.undo.Change;
 
 /**
- * Implementation of a hyperlink component. A link can be used with an anchor
- * (&lt;a href...) element or any element that supports the onclick javascript
- * event handler (such as buttons, td elements, etc). When used with an anchor,
- * a href attribute will be generated. When used with any other element, an
- * onclick javascript event handler attribute will be generated.
+ * Implementation of a hyperlink component. A link can be used with an anchor (&lt;a href...)
+ * element or any element that supports the onclick javascript event handler (such as buttons, td
+ * elements, etc). When used with an anchor, a href attribute will be generated. When used with any
+ * other element, an onclick javascript event handler attribute will be generated.
  * <p>
  * You can use a link like:
- *
+ * 
  * <pre>
  * add(new Link(&quot;myLink&quot;)
  * {
@@ -44,23 +43,23 @@
  *     }
  * );
  * </pre>
- *
+ * 
  * and in your HTML file:
- *
+ * 
  * <pre>
  *  &lt;a href=&quot;#&quot; wicket:id=&quot;myLink&quot;&gt;click here&lt;/a&gt;
  * </pre>
- *
+ * 
  * or:
- *
+ * 
  * <pre>
  *  &lt;td wicket:id=&quot;myLink&quot;&gt;my clickable column&lt;/td&gt;
  * </pre>
- *
+ * 
  * </p>
- * The following snippet shows how to pass a parameter from the Page creating
- * the Page to the Page responded by the Link.
- *
+ * The following snippet shows how to pass a parameter from the Page creating the Page to the Page
+ * responded by the Link.
+ * 
  * <pre>
  * add(new Link(&quot;link&quot;, listItem.getModel())
  * {
@@ -70,7 +69,7 @@
  *         setResponsePage(new MyPage(obj.getId(), ... ));
  *     }
  * </pre>
- *
+ * 
  * @author Jonathan Locke
  * @author Eelco Hillenius
  */
@@ -86,7 +85,7 @@
 
 		/**
 		 * Construct.
-		 *
+		 * 
 		 * @param anchor
 		 */
 		public AnchorChange(Component anchor)
@@ -103,24 +102,22 @@
 	private static final long serialVersionUID = 1L;
 
 	/**
-	 * An anchor (form 'http://server/app/etc#someAnchor') will be appended to
-	 * the link so that after this link executes, it will jump to the provided
-	 * anchor component's position. The provided anchor must either have the
-	 * {@link Component#getOutputMarkupId()} flag true, or it must be attached
-	 * to a &lt;a tag with a href attribute of more than one character starting
+	 * An anchor (form 'http://server/app/etc#someAnchor') will be appended to the link so that
+	 * after this link executes, it will jump to the provided anchor component's position. The
+	 * provided anchor must either have the {@link Component#getOutputMarkupId()} flag true, or it
+	 * must be attached to a &lt;a tag with a href attribute of more than one character starting
 	 * with '#' ('&lt;a href="#someAnchor" ... ').
 	 */
 	private Component anchor;
 
 	/**
-	 * True if link should automatically enable/disable based on current page;
-	 * false by default.
+	 * True if link should automatically enable/disable based on current page; false by default.
 	 */
 	private boolean autoEnable = false;
 
 	/**
-	 * The popup specification. If not-null, a javascript on-click event handler
-	 * will be generated that opens a new window using the popup properties.
+	 * The popup specification. If not-null, a javascript on-click event handler will be generated
+	 * that opens a new window using the popup properties.
 	 */
 	private PopupSettings popupSettings = null;
 
@@ -142,7 +139,7 @@
 
 	/**
 	 * Gets any anchor component.
-	 *
+	 * 
 	 * @return Any anchor component to jump to, might be null
 	 */
 	public Component getAnchor()
@@ -151,11 +148,9 @@
 	}
 
 	/**
-	 * Gets whether link should automatically enable/disable based on current
-	 * page.
-	 *
-	 * @return Whether this link should automatically enable/disable based on
-	 *         current page.
+	 * Gets whether link should automatically enable/disable based on current page.
+	 * 
+	 * @return Whether this link should automatically enable/disable based on current page.
 	 */
 	public final boolean getAutoEnable()
 	{
@@ -163,10 +158,9 @@
 	}
 
 	/**
-	 * Gets the popup specification. If not-null, a javascript on-click event
-	 * handler will be generated that opens a new window using the popup
-	 * properties.
-	 *
+	 * Gets the popup specification. If not-null, a javascript on-click event handler will be
+	 * generated that opens a new window using the popup properties.
+	 * 
 	 * @return the popup specification.
 	 */
 	public PopupSettings getPopupSettings()
@@ -199,13 +193,11 @@
 	public abstract void onClick();
 
 	/**
-	 * THIS METHOD IS NOT PART OF THE WICKET API. DO NOT ATTEMPT TO OVERRIDE OR
-	 * CALL IT.
-	 *
-	 * Called when a link is clicked. The implementation of this method is
-	 * currently to simply call onClick(), but this may be augmented in the
-	 * future.
-	 *
+	 * THIS METHOD IS NOT PART OF THE WICKET API. DO NOT ATTEMPT TO OVERRIDE OR CALL IT.
+	 * 
+	 * Called when a link is clicked. The implementation of this method is currently to simply call
+	 * onClick(), but this may be augmented in the future.
+	 * 
 	 * @see ILinkListener
 	 */
 	public final void onLinkClicked()
@@ -223,14 +215,13 @@
 	}
 
 	/**
-	 * Sets an anchor component. An anchor (form
-	 * 'http://server/app/etc#someAnchor') will be appended to the link so that
-	 * after this link executes, it will jump to the provided anchor component's
-	 * position. The provided anchor must either have the
-	 * {@link Component#getOutputMarkupId()} flag true, or it must be attached
-	 * to a &lt;a tag with a href attribute of more than one character starting
-	 * with '#' ('&lt;a href="#someAnchor" ... ').
-	 *
+	 * Sets an anchor component. An anchor (form 'http://server/app/etc#someAnchor') will be
+	 * appended to the link so that after this link executes, it will jump to the provided anchor
+	 * component's position. The provided anchor must either have the
+	 * {@link Component#getOutputMarkupId()} flag true, or it must be attached to a &lt;a tag with a
+	 * href attribute of more than one character starting with '#' ('&lt;a href="#someAnchor" ...
+	 * ').
+	 * 
 	 * @param anchor
 	 *            The anchor
 	 * @return this
@@ -243,12 +234,10 @@
 	}
 
 	/**
-	 * Sets whether this link should automatically enable/disable based on
-	 * current page.
-	 *
+	 * Sets whether this link should automatically enable/disable based on current page.
+	 * 
 	 * @param autoEnable
-	 *            whether this link should automatically enable/disable based on
-	 *            current page.
+	 *            whether this link should automatically enable/disable based on current page.
 	 * @return This
 	 */
 	public final Link setAutoEnable(final boolean autoEnable)
@@ -258,10 +247,9 @@
 	}
 
 	/**
-	 * Sets the popup specification. If not-null, a javascript on-click event
-	 * handler will be generated that opens a new window using the popup
-	 * properties.
-	 *
+	 * Sets the popup specification. If not-null, a javascript on-click event handler will be
+	 * generated that opens a new window using the popup properties.
+	 * 
 	 * @param popupSettings
 	 *            the popup specification.
 	 * @return This
@@ -273,23 +261,21 @@
 	}
 
 	/**
-	 * Appends any anchor to the url if the url is not null and the url does not
-	 * already contain an anchor (url.indexOf('#') != -1). This implementation
-	 * looks whether an anchor component was set, and if so, it will append the
-	 * markup id of that component. That markup id is gotten by either calling
-	 * {@link Component#getMarkupId()} if {@link Component#getOutputMarkupId()}
-	 * returns true, or if the anchor component does not output it's id, this
-	 * method will try to retrieve the id from the markup directly. If neither
-	 * is found, an {@link WicketRuntimeException exception} is thrown. If no
-	 * anchor component was set, but the link component is attached to a &lt;a
-	 * element, this method will append what is in the href attribute <i>if</i>
-	 * there is one, starts with a '#' and has more than one character.
+	 * Appends any anchor to the url if the url is not null and the url does not already contain an
+	 * anchor (url.indexOf('#') != -1). This implementation looks whether an anchor component was
+	 * set, and if so, it will append the markup id of that component. That markup id is gotten by
+	 * either calling {@link Component#getMarkupId()} if {@link Component#getOutputMarkupId()}
+	 * returns true, or if the anchor component does not output it's id, this method will try to
+	 * retrieve the id from the markup directly. If neither is found, an
+	 * {@link WicketRuntimeException exception} is thrown. If no anchor component was set, but the
+	 * link component is attached to a &lt;a element, this method will append what is in the href
+	 * attribute <i>if</i> there is one, starts with a '#' and has more than one character.
 	 * <p>
-	 * You can override this method, but it means that you have to take care of
-	 * whatever is done with any set anchor component yourself. You also have to
-	 * manually append the '#' at the right place.
+	 * You can override this method, but it means that you have to take care of whatever is done
+	 * with any set anchor component yourself. You also have to manually append the '#' at the right
+	 * place.
 	 * </p>
-	 *
+	 * 
 	 * @param tag
 	 *            The component tag
 	 * @param url
@@ -359,9 +345,8 @@
 	 * @param url
 	 *            The url for the link
 	 * @return Any onClick JavaScript that should be used
-	 * @deprecated this method will be removed by
-	 *             {@link #getOnClickScript(CharSequence)} shortly. Please
-	 *             override that method instead.
+	 * @deprecated this method will be removed by {@link #getOnClickScript(CharSequence)} shortly.
+	 *             Please override that method instead.
 	 */
 	protected String getOnClickScript(final String url)
 	{
@@ -370,7 +355,7 @@
 
 	/**
 	 * Gets the url to use for this link.
-	 *
+	 * 
 	 * @return The URL that this link links to
 	 */
 	protected CharSequence getURL()
@@ -380,7 +365,7 @@
 
 	/**
 	 * Whether this link refers to the given page.
-	 *
+	 * 
 	 * @param page
 	 *            A page
 	 * @return True if this link goes to the given page
@@ -392,7 +377,7 @@
 
 	/**
 	 * Handles this link's tag. OVERRIDES MUST CALL SUPER.
-	 *
+	 * 
 	 * @param tag
 	 *            the component tag
 	 * @see org.apache.wicket.Component#onComponentTag(ComponentTag)

Modified: wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/link/PageLink.java
URL: http://svn.apache.org/viewvc/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/link/PageLink.java?rev=584925&r1=584924&r2=584925&view=diff
==============================================================================
--- wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/link/PageLink.java (original)
+++ wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/link/PageLink.java Mon Oct 15 14:21:25 2007
@@ -19,13 +19,11 @@
 import org.apache.wicket.Page;
 
 /**
- * Links to a given page via an object implementing the IPageLink delayed
- * linking interface. PageLinks can be constructed directly with an IPageLink
- * interface or with a Page Class object. In the latter case, an IPageLink
- * implementation is provided which constructs a Page of the given class when
- * the link is clicked. A default no-args constructor must be available in this
- * case or a WicketRuntimeException will be thrown when Wicket fails to
- * instantiate the class.
+ * Links to a given page via an object implementing the IPageLink delayed linking interface.
+ * PageLinks can be constructed directly with an IPageLink interface or with a Page Class object. In
+ * the latter case, an IPageLink implementation is provided which constructs a Page of the given
+ * class when the link is clicked. A default no-args constructor must be available in this case or a
+ * WicketRuntimeException will be thrown when Wicket fails to instantiate the class.
  * 
  * @see IPageLink
  * @author Jonathan Locke
@@ -38,8 +36,8 @@
 	private final IPageLink pageLink;
 
 	/**
-	 * Constructs a link that instantiates the given Page class when the link is
-	 * clicked. The instantiated Page is used to render a response to the user.
+	 * Constructs a link that instantiates the given Page class when the link is clicked. The
+	 * instantiated Page is used to render a response to the user.
 	 * 
 	 * @param id
 	 *            See Component
@@ -76,16 +74,15 @@
 	/**
 	 * This constructor is ideal for constructing pages lazily.
 	 * 
-	 * Constructs a link which invokes the getPage() method of the IPageLink
-	 * interface when the link is clicked. Whatever Page objects is returned by
-	 * this method will be rendered back to the user.
+	 * Constructs a link which invokes the getPage() method of the IPageLink interface when the link
+	 * is clicked. Whatever Page objects is returned by this method will be rendered back to the
+	 * user.
 	 * 
 	 * @param id
 	 *            See Component
 	 * @param pageLink
-	 *            An implementation of IPageLink which will create the page
-	 *            linked to if and when this hyperlink is clicked at a later
-	 *            time.
+	 *            An implementation of IPageLink which will create the page linked to if and when
+	 *            this hyperlink is clicked at a later time.
 	 */
 	public PageLink(final String id, final IPageLink pageLink)
 	{
@@ -94,19 +91,17 @@
 	}
 
 	/**
-	 * This constructor is ideal if a Page object was passed in from a previous
-	 * Page. Construct a link to the Page. Warning: DO NOT use this for
-	 * constructing links to pages you didn't already have an instance of. This
-	 * constructor is strongly discouraged for anything other than linking back
-	 * to the same page.
+	 * This constructor is ideal if a Page object was passed in from a previous Page. Construct a
+	 * link to the Page. Warning: DO NOT use this for constructing links to pages you didn't already
+	 * have an instance of. This constructor is strongly discouraged for anything other than linking
+	 * back to the same page.
 	 * 
 	 * @param id
 	 *            See component
 	 * @param page
 	 *            The page
-	 * @deprecated rather than using this class/ constructor, use normal
-	 *             {@link Link links} and call setResponsePage in their
-	 *             {@link Link#onClick() onClick} methods.
+	 * @deprecated rather than using this class/ constructor, use normal {@link Link links} and call
+	 *             setResponsePage in their {@link Link#onClick() onClick} methods.
 	 */
 	public PageLink(final String id, final Page page)
 	{
@@ -130,8 +125,8 @@
 	}
 
 	/**
-	 * Returns true if the given page is of the same class as the (delayed)
-	 * destination of this page link.
+	 * Returns true if the given page is of the same class as the (delayed) destination of this page
+	 * link.
 	 * 
 	 * @see org.apache.wicket.markup.html.link.Link#linksTo(org.apache.wicket.Page)
 	 */
@@ -141,9 +136,8 @@
 	}
 
 	/**
-	 * Handles a link click by asking for a concrete Page instance through the
-	 * IPageLink.getPage() delayed linking interface. This call will normally
-	 * cause the destination page to be created.
+	 * Handles a link click by asking for a concrete Page instance through the IPageLink.getPage()
+	 * delayed linking interface. This call will normally cause the destination page to be created.
 	 * 
 	 * @see org.apache.wicket.markup.html.link.Link#onClick()
 	 */

Modified: wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/link/PopupSettings.java
URL: http://svn.apache.org/viewvc/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/link/PopupSettings.java?rev=584925&r1=584924&r2=584925&view=diff
==============================================================================
--- wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/link/PopupSettings.java (original)
+++ wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/link/PopupSettings.java Mon Oct 15 14:21:25 2007
@@ -26,18 +26,18 @@
 
 
 /**
- * A popup specification can be used as a property of the {@link Link}classes
- * to specify that the link should be rendered with an onClick javascript event
- * handler that opens a new window with the links' URL.
+ * A popup specification can be used as a property of the {@link Link}classes to specify that the
+ * link should be rendered with an onClick javascript event handler that opens a new window with the
+ * links' URL.
  * <p>
  * You can 'or' display flags together like this:
- *
+ * 
  * <pre>
  * new PopupSettings(PopupSettings.RESIZABLE | PopupSettings.SCROLLBARS);
  * </pre>
- *
+ * 
  * </p>
- *
+ * 
  * @author Jonathan Locke
  * @author Eelco Hillenius
  */
@@ -76,9 +76,8 @@
 	private int left = -1;
 
 	/**
-	 * The target to put in JavaScript. This implementation simply refers to the
-	 * href element, but clients may want to override this (e.g. when the HTML
-	 * element is not an anchor).
+	 * The target to put in JavaScript. This implementation simply refers to the href element, but
+	 * clients may want to override this (e.g. when the HTML element is not an anchor).
 	 */
 	private String target = "href";
 
@@ -89,28 +88,25 @@
 	private int width = -1;
 
 	/**
-	 * The logical name of the window. This can be anything you want, although
-	 * you should use alphanumeric characters only (no spaces or punctuation).
-	 * If you have a window already open and call window.open a second time
-	 * using the same windowName, the first window will be reused rather than
-	 * opening a second window.
+	 * The logical name of the window. This can be anything you want, although you should use
+	 * alphanumeric characters only (no spaces or punctuation). If you have a window already open
+	 * and call window.open a second time using the same windowName, the first window will be reused
+	 * rather than opening a second window.
 	 */
 	private String windowName = null;
 
 	/**
-	 * The pagemap name where the page that will be created by this popuplink
-	 * will be created in.
+	 * The pagemap name where the page that will be created by this popuplink will be created in.
 	 */
 	private String pageMapName;
 
 	/**
-	 * Construct. If you are not using these popup settings with an external
-	 * link - in which case we don't need to know about a page map - you should
-	 * use one of the constructors with a {@link PageMap} argument. Typically,
-	 * you should put any popup in a separate page map as Wicket holds
-	 * references to a limited number of pages/ versions only. If you don't put
-	 * your popup in a separate page map, the user might get page expired
-	 * exceptions when getting back to the main window again.
+	 * Construct. If you are not using these popup settings with an external link - in which case we
+	 * don't need to know about a page map - you should use one of the constructors with a
+	 * {@link PageMap} argument. Typically, you should put any popup in a separate page map as
+	 * Wicket holds references to a limited number of pages/ versions only. If you don't put your
+	 * popup in a separate page map, the user might get page expired exceptions when getting back to
+	 * the main window again.
 	 */
 	public PopupSettings()
 	{
@@ -118,7 +114,7 @@
 
 	/**
 	 * Construct.
-	 *
+	 * 
 	 * @param displayFlags
 	 *            Display flags
 	 */
@@ -129,14 +125,12 @@
 
 	/**
 	 * Construct.
-	 *
+	 * 
 	 * @param pagemap
-	 *            The pagemap where this popup must be in. Typically, you should
-	 *            put any popup in a separate page map as Wicket holds
-	 *            references to a limited number of pages/ versions only. If you
-	 *            don't put your popup in a separate page map, the user might
-	 *            get page expired exceptions when getting back to the main
-	 *            window again.
+	 *            The pagemap where this popup must be in. Typically, you should put any popup in a
+	 *            separate page map as Wicket holds references to a limited number of pages/
+	 *            versions only. If you don't put your popup in a separate page map, the user might
+	 *            get page expired exceptions when getting back to the main window again.
 	 */
 	public PopupSettings(IPageMap pagemap)
 	{
@@ -146,14 +140,12 @@
 
 	/**
 	 * Construct.
-	 *
+	 * 
 	 * @param pagemap
-	 *            The pagemap where this popup must be in. Typically, you should
-	 *            put any popup in a separate page map as Wicket holds
-	 *            references to a limited number of pages/ versions only. If you
-	 *            don't put your popup in a separate page map, the user might
-	 *            get page expired exceptions when getting back to the main
-	 *            window again.
+	 *            The pagemap where this popup must be in. Typically, you should put any popup in a
+	 *            separate page map as Wicket holds references to a limited number of pages/
+	 *            versions only. If you don't put your popup in a separate page map, the user might
+	 *            get page expired exceptions when getting back to the main window again.
 	 * @param displayFlags
 	 *            Display flags
 	 */
@@ -166,7 +158,7 @@
 
 	/**
 	 * Get the onClick javascript event handler.
-	 *
+	 * 
 	 * @return the onClick javascript event handler
 	 */
 	public String getPopupJavaScript()
@@ -220,7 +212,7 @@
 
 	/**
 	 * Sets the popup window height.
-	 *
+	 * 
 	 * @param popupHeight
 	 *            the popup window height.
 	 * @return This
@@ -233,7 +225,7 @@
 
 	/**
 	 * Sets the left position of the popup window.
-	 *
+	 * 
 	 * @param popupPositionLeft
 	 *            the left position of the popup window.
 	 * @return This
@@ -245,10 +237,10 @@
 	}
 
 	/**
-	 * Sets the target of the link. The default implementation simply refers to
-	 * the href element, but clients may want to override this (e.g. when the
-	 * HTML element is not an anchor) by setting the target explicitly.
-	 *
+	 * Sets the target of the link. The default implementation simply refers to the href element,
+	 * but clients may want to override this (e.g. when the HTML element is not an anchor) by
+	 * setting the target explicitly.
+	 * 
 	 * @param target
 	 *            the target of the link
 	 */
@@ -259,7 +251,7 @@
 
 	/**
 	 * Sets the top position of the popup window.
-	 *
+	 * 
 	 * @param popupPositionTop
 	 *            the top position of the popup window.
 	 * @return This
@@ -272,7 +264,7 @@
 
 	/**
 	 * Sets the popup window width.
-	 *
+	 * 
 	 * @param popupWidth
 	 *            the popup window width.
 	 * @return This
@@ -284,12 +276,11 @@
 	}
 
 	/**
-	 * Sets the window name. The logical name of the window. This can be
-	 * anything you want, although you should use alphanumeric characters only
-	 * (no spaces or punctuation). If you have a window already open and call
-	 * window.open a second time using the same windowName, the first window
+	 * Sets the window name. The logical name of the window. This can be anything you want, although
+	 * you should use alphanumeric characters only (no spaces or punctuation). If you have a window
+	 * already open and call window.open a second time using the same windowName, the first window
 	 * will be reused rather than opening a second window
-	 *
+	 * 
 	 * @param popupWindowName
 	 *            window name.
 	 * @return This
@@ -301,9 +292,9 @@
 			windowName = popupWindowName;
 			if (pageMapName != null && (!pageMapName.equals(popupWindowName)))
 			{
-				log.warn("the page map and window name should be the same. The page map was "
-						+ pageMapName + ", and the requested window name is " + popupWindowName
-						+ "; changing the page map to " + popupWindowName);
+				log.warn("the page map and window name should be the same. The page map was " +
+						pageMapName + ", and the requested window name is " + popupWindowName +
+						"; changing the page map to " + popupWindowName);
 			}
 			pageMapName = popupWindowName;
 		}
@@ -322,10 +313,9 @@
 
 	/**
 	 * Gets the pagemap where the popup page must be created in.
-	 *
+	 * 
 	 * @return The pagemap where the popup page must be created in
-	 * @deprecated will be removed in Wicket 2.0; use
-	 *             {@link #getPageMap(Component)} instead
+	 * @deprecated will be removed in Wicket 2.0; use {@link #getPageMap(Component)} instead
 	 */
 	public IPageMap getPageMap()
 	{
@@ -343,7 +333,7 @@
 
 	/**
 	 * Gets the pagemap where the popup page must be created in.
-	 *
+	 * 
 	 * @param callee
 	 *            Calling component
 	 * @return The pagemap where the popup page must be created in
@@ -364,10 +354,10 @@
 			Page page = callee.getPage();
 			if (page == null)
 			{
-				throw new IllegalStateException(callee
-						+ " is not yet set on a page; if you want to use this method "
-						+ "without a page map being set, argument callee must be not null "
-						+ "and added to a page");
+				throw new IllegalStateException(callee +
+						" is not yet set on a page; if you want to use this method " +
+						"without a page map being set, argument callee must be not null " +
+						"and added to a page");
 			}
 			return page.getPageMap();
 		}

Modified: wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/link/ResourceLink.java
URL: http://svn.apache.org/viewvc/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/link/ResourceLink.java?rev=584925&r1=584924&r2=584925&view=diff
==============================================================================
--- wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/link/ResourceLink.java (original)
+++ wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/link/ResourceLink.java Mon Oct 15 14:21:25 2007
@@ -41,8 +41,8 @@
 
 
 	/**
-	 * Constructs an ResourceLink from an resourcereference. That resource
-	 * reference will bind its resource to the current SharedResources.
+	 * Constructs an ResourceLink from an resourcereference. That resource reference will bind its
+	 * resource to the current SharedResources.
 	 * 
 	 * @param id
 	 *            See Component
@@ -55,8 +55,8 @@
 	}
 
 	/**
-	 * Constructs an ResourceLink from an resourcereference. That resource
-	 * reference will bind its resource to the current SharedResources.
+	 * Constructs an ResourceLink from an resourcereference. That resource reference will bind its
+	 * resource to the current SharedResources.
 	 * 
 	 * @param id
 	 *            See Component

Modified: wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/link/StatelessLink.java
URL: http://svn.apache.org/viewvc/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/link/StatelessLink.java?rev=584925&r1=584924&r2=584925&view=diff
==============================================================================
--- wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/link/StatelessLink.java (original)
+++ wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/link/StatelessLink.java Mon Oct 15 14:21:25 2007
@@ -18,9 +18,8 @@
 
 
 /**
- * This link is stateless that means that the url to this link could generate
- * a new page before the link onClick is called. Because of this you can't depend
- * on model data in the onClick method.
+ * This link is stateless that means that the url to this link could generate a new page before the
+ * link onClick is called. Because of this you can't depend on model data in the onClick method.
  * 
  * This Link component is the same as a normal link with the statelesshint to true.
  * 
@@ -32,14 +31,14 @@
 
 	/**
 	 * Construct.
+	 * 
 	 * @param id
 	 */
 	public StatelessLink(String id)
 	{
 		super(id);
 	}
-	
-	
+
 
 	protected boolean getStatelessHint()
 	{

Modified: wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/list/ListItem.java
URL: http://svn.apache.org/viewvc/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/list/ListItem.java?rev=584925&r1=584924&r2=584925&view=diff
==============================================================================
--- wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/list/ListItem.java (original)
+++ wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/list/ListItem.java Mon Oct 15 14:21:25 2007
@@ -27,13 +27,13 @@
 public class ListItem extends WebMarkupContainer
 {
 	private static final long serialVersionUID = 1L;
-	
+
 	/** The index of the ListItem in the parent ListView */
 	private final int index;
 
 	/**
-	 * A constructor which uses the index and the list provided to create a
-	 * ListItem. This constructor is the default one.
+	 * A constructor which uses the index and the list provided to create a ListItem. This
+	 * constructor is the default one.
 	 * 
 	 * @param index
 	 *            The index of the item

Modified: wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/list/ListItemModel.java
URL: http://svn.apache.org/viewvc/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/list/ListItemModel.java?rev=584925&r1=584924&r2=584925&view=diff
==============================================================================
--- wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/list/ListItemModel.java (original)
+++ wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/list/ListItemModel.java Mon Oct 15 14:21:25 2007
@@ -26,7 +26,7 @@
 public class ListItemModel implements IModel
 {
 	private static final long serialVersionUID = 1L;
-	
+
 	/** The ListView itself */
 	private final ListView listView;
 

Modified: wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/list/ListView.java
URL: http://svn.apache.org/viewvc/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/list/ListView.java?rev=584925&r1=584924&r2=584925&view=diff
==============================================================================
--- wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/list/ListView.java (original)
+++ wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/list/ListView.java Mon Oct 15 14:21:25 2007
@@ -35,31 +35,31 @@
  * are situations where it is necessary to work with other collection types, for repeaters that
  * might work better with non-list or database-driven collections see the
  * org.apache.wicket.markup.repeater package.
- *
+ * 
  * Also notice that in a list the item's uniqueness/primary key/id is identified as its index in the
  * list. If this is not the case you should either override {@link #getListItemModel(IModel, int)}
  * to return a model that will work with the item's true primary key, or use a different repeater
  * that does not rely on the list index.
- *
- *
+ * 
+ * 
  * A ListView holds ListItem children. Items can be re-ordered and deleted, either one at a time or
  * many at a time.
  * <p>
  * Example:
- *
+ * 
  * <pre>
  *                      &lt;tbody&gt;
  *                        &lt;tr wicket:id=&quot;rows&quot; class=&quot;even&quot;&gt;
  *                            &lt;td&gt;&lt;span wicket:id=&quot;id&quot;&gt;Test ID&lt;/span&gt;&lt;/td&gt;
  *                        ...
  * </pre>
- *
+ * 
  * <p>
  * Though this example is about a HTML table, ListView is not at all limited to HTML tables. Any
  * kind of list can be rendered using ListView.
  * <p>
  * The related Java code:
- *
+ * 
  * <pre>
  * add(new ListView(&quot;rows&quot;, listData)
  * {
@@ -70,18 +70,18 @@
  * 	}
  * });
  * </pre>
- *
+ * 
  * <p>
  * <strong>NOTE:</strong>
- *
+ * 
  * When you want to change the default generated markup it is important to realize that the ListView
  * instance itself does not correspond to any markup, however, the generated ListItems do.<br/>
- *
+ * 
  * This means that methods like {@link #setRenderBodyOnly(boolean)} and
  * {@link #add(org.apache.wicket.behavior.IBehavior)} should be invoked on the {@link ListItem} that
  * is given in {@link #populateItem(ListItem)} method.
  * </p>
- *
+ * 
  * <p>
  * <strong>WARNING:</strong> though you can nest ListViews within Forms, you HAVE to set the
  * setReuseItems property to true in order to have validation work properly. By default,
@@ -95,7 +95,7 @@
  * components are replaced by new ones, your user will never see the wrong data when setReuseItems
  * is false.
  * </p>
- *
+ * 
  * @author Jonathan Locke
  * @author Juergen Donnerstag
  * @author Johan Compagner
@@ -104,7 +104,7 @@
 public abstract class ListView extends AbstractRepeater
 {
 	/**
-	 *
+	 * 
 	 */
 	private static final long serialVersionUID = 1L;
 
@@ -165,7 +165,7 @@
 	 * Gets the list of items in the listView. This method is final because it is not designed to be
 	 * overridden. If it were allowed to be overridden, the values returned by getModelObject() and
 	 * getList() might not coincide.
-	 *
+	 * 
 	 * @return The list of items in this list view.
 	 */
 	public final List getList()
@@ -184,7 +184,7 @@
 	 * than you must manually call listView.removeAll() in order to rebuild the ListItems. If you
 	 * nest a ListView in a Form, ALLWAYS set this property to true, as otherwise validation will
 	 * not work properly.
-	 *
+	 * 
 	 * @return Whether to reuse items
 	 */
 	public boolean getReuseItems()
@@ -194,7 +194,7 @@
 
 	/**
 	 * Get index of first cell in page. Default is: 0.
-	 *
+	 * 
 	 * @return Index of first cell in page. Default is: 0
 	 */
 	public final int getStartIndex()
@@ -207,7 +207,7 @@
 	 * really will be. E.g. default for viewSize is Integer.MAX_VALUE, if not set via setViewSize().
 	 * If the underlying list has 10 elements, the value returned by getViewSize() will be 10 if
 	 * startIndex = 0.
-	 *
+	 * 
 	 * @return The number of items to be populated and rendered.
 	 */
 	public int getViewSize()
@@ -244,7 +244,7 @@
 
 	/**
 	 * Returns a link that will move the given item "down" (towards the end) in the listView.
-	 *
+	 * 
 	 * @param id
 	 *            Name of move-down link component to create
 	 * @param item
@@ -300,7 +300,7 @@
 
 	/**
 	 * Returns a link that will move the given item "up" (towards the beginning) in the listView.
-	 *
+	 * 
 	 * @param id
 	 *            Name of move-up link component to create
 	 * @param item
@@ -357,7 +357,7 @@
 
 	/**
 	 * Returns a link that will remove this ListItem from the ListView that holds it.
-	 *
+	 * 
 	 * @param id
 	 *            Name of remove link component to create
 	 * @param item
@@ -402,7 +402,7 @@
 	/**
 	 * Sets the model as the provided list and removes all children, so that the next render will be
 	 * using the contents of the model.
-	 *
+	 * 
 	 * @param list
 	 *            The list for the new model. The list must implement {@link Serializable}.
 	 * @return This for chaining
@@ -415,11 +415,11 @@
 	/**
 	 * Sets the model and removes all current children, so that the next render will be using the
 	 * contents of the model.
-	 *
+	 * 
 	 * @param model
 	 *            The new model
 	 * @return This for chaining
-	 *
+	 * 
 	 * @see org.apache.wicket.MarkupContainer#setModel(org.apache.wicket.model.IModel)
 	 */
 	public Component setModel(IModel model)
@@ -433,7 +433,7 @@
 	 * than you must manually call listView.removeAll() in order to rebuild the ListItems. If you
 	 * nest a ListView in a Form, ALLWAYS set this property to true, as otherwise validation will
 	 * not work properly.
-	 *
+	 * 
 	 * @param reuseItems
 	 *            Whether to reuse the child items.
 	 * @return this
@@ -446,7 +446,7 @@
 
 	/**
 	 * Set the index of the first item to render
-	 *
+	 * 
 	 * @param startIndex
 	 *            First index of model object's list to display
 	 * @return This
@@ -469,7 +469,7 @@
 
 	/**
 	 * Define the maximum number of items to render. Default: render all.
-	 *
+	 * 
 	 * @param size
 	 *            Number of items to display
 	 * @return This
@@ -492,7 +492,7 @@
 	 * the underlying list changes a lot (many users using the application), it may not longer be
 	 * appropriate. In that case your own ListItemModel implementation should use an id (e.g. the
 	 * database' record id) to identify and load the list item model object.
-	 *
+	 * 
 	 * @param listViewModel
 	 *            The ListView's model
 	 * @param index
@@ -506,7 +506,7 @@
 
 	/**
 	 * Create a new ListItem for list item at index.
-	 *
+	 * 
 	 * @param index
 	 * @return ListItem
 	 */
@@ -582,7 +582,7 @@
 	 * Comes handy for ready made ListView based components which must implement populateItem() but
 	 * you don't want to lose compile time error checking reminding the user to implement abstract
 	 * populateItem().
-	 *
+	 * 
 	 * @param item
 	 */
 	protected void onBeginPopulateItem(final ListItem item)
@@ -593,19 +593,19 @@
 	 * Populate a given item.
 	 * <p>
 	 * <b>be careful</b> to add any components to the list item. So, don't do:
-	 *
+	 * 
 	 * <pre>
 	 * add(new Label(&quot;foo&quot;, &quot;bar&quot;));
 	 * </pre>
-	 *
+	 * 
 	 * but:
-	 *
+	 * 
 	 * <pre>
 	 * item.add(new Label(&quot;foo&quot;, &quot;bar&quot;));
 	 * </pre>
-	 *
+	 * 
 	 * </p>
-	 *
+	 * 
 	 * @param item
 	 *            The item to populate
 	 */
@@ -622,7 +622,7 @@
 
 	/**
 	 * Render a single item.
-	 *
+	 * 
 	 * @param item
 	 *            The item to be rendered
 	 */

Modified: wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/list/PageableListView.java
URL: http://svn.apache.org/viewvc/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/list/PageableListView.java?rev=584925&r1=584924&r2=584925&view=diff
==============================================================================
--- wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/list/PageableListView.java (original)
+++ wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/list/PageableListView.java Mon Oct 15 14:21:25 2007
@@ -24,16 +24,16 @@
 
 
 /**
- * PageableListView is similar to ListView but provides in addition pageable
- * views. A PageableListView holds pageable rows of information. The rows can be
- * re-ordered and deleted, either one at a time or many at a time.
- *
+ * PageableListView is similar to ListView but provides in addition pageable views. A
+ * PageableListView holds pageable rows of information. The rows can be re-ordered and deleted,
+ * either one at a time or many at a time.
+ * 
  * @author Jonathan Locke
  */
 public abstract class PageableListView extends ListView implements IPageable
 {
 	/**
-	 *
+	 * 
 	 */
 	private static final long serialVersionUID = 1L;
 
@@ -45,7 +45,7 @@
 
 	/**
 	 * Constructor
-	 *
+	 * 
 	 * @param id
 	 *            See Component
 	 * @param model
@@ -60,9 +60,9 @@
 	}
 
 	/**
-	 * Creates a pageable list view having the given number of rows per page that
-	 * uses the provided object as a simple model.
-	 *
+	 * Creates a pageable list view having the given number of rows per page that uses the provided
+	 * object as a simple model.
+	 * 
 	 * @param id
 	 *            See Component
 	 * @param list
@@ -79,7 +79,7 @@
 
 	/**
 	 * Gets the index of the current page being displayed by this list view.
-	 *
+	 * 
 	 * @return Returns the currentPage.
 	 */
 	public final int getCurrentPage()
@@ -95,7 +95,7 @@
 
 	/**
 	 * Gets the number of pages in this list view.
-	 *
+	 * 
 	 * @return The number of pages in this list view
 	 */
 	public final int getPageCount()
@@ -105,7 +105,7 @@
 
 	/**
 	 * Gets the maximum number of rows on each page.
-	 *
+	 * 
 	 * @return the maximum number of rows on each page.
 	 */
 	public final int getRowsPerPage()
@@ -115,8 +115,9 @@
 
 	/**
 	 * Sets the maximum number of rows on each page.
-	 *
-	 * @param rowsPerPage the maximum number of rows on each page.
+	 * 
+	 * @param rowsPerPage
+	 *            the maximum number of rows on each page.
 	 */
 	public final void setRowsPerPage(int rowsPerPage)
 	{
@@ -145,7 +146,7 @@
 
 	/**
 	 * Sets the current page that this list view should show.
-	 *
+	 * 
 	 * @param currentPage
 	 *            The currentPage to set.
 	 */
@@ -169,9 +170,10 @@
 
 	/**
 	 * Prevent users from accidentally using it.
-	 *
+	 * 
 	 * @see org.apache.wicket.markup.html.list.ListView#setStartIndex(int)
-	 * @throws UnsupportedOperationException always
+	 * @throws UnsupportedOperationException
+	 *             always
 	 */
 	public ListView setStartIndex(int startIndex) throws UnsupportedOperationException
 	{
@@ -181,11 +183,12 @@
 
 	/**
 	 * Prevent users from accidentally using it.
-	 *
+	 * 
 	 * @param size
 	 *            the view size
 	 * @return This
-	 * @throws UnsupportedOperationException always
+	 * @throws UnsupportedOperationException
+	 *             always
 	 * @see org.apache.wicket.markup.html.list.ListView#setStartIndex(int)
 	 */
 	public ListView setViewSize(int size) throws UnsupportedOperationException
@@ -206,12 +209,15 @@
 
 		/**
 		 * Construct.
-		 * @param currentPage the former 'current' page
+		 * 
+		 * @param currentPage
+		 *            the former 'current' page
 		 */
 		CurrentPageChange(int currentPage)
 		{
 			this.currentPage = currentPage;
 		}
+
 		/**
 		 * @see org.apache.wicket.version.undo.Change#undo()
 		 */
@@ -241,12 +247,15 @@
 
 		/**
 		 * Construct.
-		 * @param rowsPerPage the former number of rows per page
+		 * 
+		 * @param rowsPerPage
+		 *            the former number of rows per page
 		 */
 		RowsPerPageChange(int rowsPerPage)
 		{
 			this.rowsPerPage = rowsPerPage;
 		}
+
 		/**
 		 * @see org.apache.wicket.version.undo.Change#undo()
 		 */

Modified: wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/list/PropertyListView.java
URL: http://svn.apache.org/viewvc/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/list/PropertyListView.java?rev=584925&r1=584924&r2=584925&view=diff
==============================================================================
--- wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/list/PropertyListView.java (original)
+++ wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/list/PropertyListView.java Mon Oct 15 14:21:25 2007
@@ -23,9 +23,8 @@
 
 
 /**
- * Simple ListVew subclass that wraps its item models in a
- * BoundCompoundPropertyModel. Useful for lists where the item components will
- * be mapped through property expressions.
+ * Simple ListVew subclass that wraps its item models in a BoundCompoundPropertyModel. Useful for
+ * lists where the item components will be mapped through property expressions.
  * 
  * @author Nathan Hamblen
  */
@@ -61,9 +60,8 @@
 	}
 
 	/**
-	 * Construct with a "small," unmodeled List. The object can not be detached
-	 * and will reside in the session, but is convenient for lists of a limited
-	 * size.
+	 * Construct with a "small," unmodeled List. The object can not be detached and will reside in
+	 * the session, but is convenient for lists of a limited size.
 	 * 
 	 * @param id
 	 *            Wicket id

Modified: wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/navigation/paging/IPageable.java
URL: http://svn.apache.org/viewvc/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/navigation/paging/IPageable.java?rev=584925&r1=584924&r2=584925&view=diff
==============================================================================
--- wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/navigation/paging/IPageable.java (original)
+++ wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/navigation/paging/IPageable.java Mon Oct 15 14:21:25 2007
@@ -17,13 +17,11 @@
 package org.apache.wicket.markup.html.navigation.paging;
 
 /**
- * Components that implement this interface will be pageable, they should return
- * the pagecount so that an object/component knows how many pages it can use for
- * the setCurrentPage method.
+ * Components that implement this interface will be pageable, they should return the pagecount so
+ * that an object/component knows how many pages it can use for the setCurrentPage method.
  * 
- * The PageableListView is one example that is Pageable. But also a Form could
- * be pageable so that you can scroll to sets of records that you display in
- * that form with any navigator you want.
+ * The PageableListView is one example that is Pageable. But also a Form could be pageable so that
+ * you can scroll to sets of records that you display in that form with any navigator you want.
  * 
  * @author jcompagner
  */

Modified: wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/navigation/paging/IPagingLabelProvider.java
URL: http://svn.apache.org/viewvc/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/navigation/paging/IPagingLabelProvider.java?rev=584925&r1=584924&r2=584925&view=diff
==============================================================================
--- wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/navigation/paging/IPagingLabelProvider.java (original)
+++ wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/navigation/paging/IPagingLabelProvider.java Mon Oct 15 14:21:25 2007
@@ -19,9 +19,8 @@
 import org.apache.wicket.IClusterable;
 
 /**
- * This interface is used by the PagingNavigator components to get the label of
- * the pages there are for a IPageable component. By default this is only the
- * page number.
+ * This interface is used by the PagingNavigator components to get the label of the pages there are
+ * for a IPageable component. By default this is only the page number.
  * 
  * @author jcompagner
  */

Modified: wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/navigation/paging/PagingNavigationIncrementLink.java
URL: http://svn.apache.org/viewvc/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/navigation/paging/PagingNavigationIncrementLink.java?rev=584925&r1=584924&r2=584925&view=diff
==============================================================================
--- wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/navigation/paging/PagingNavigationIncrementLink.java (original)
+++ wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/navigation/paging/PagingNavigationIncrementLink.java Mon Oct 15 14:21:25 2007
@@ -20,8 +20,8 @@
 import org.apache.wicket.markup.html.link.Link;
 
 /**
- * An incremental link to a page of a PageableListView. Assuming your list view
- * navigation looks like
+ * An incremental link to a page of a PageableListView. Assuming your list view navigation looks
+ * like
  * 
  * <pre>
  *    
@@ -30,8 +30,8 @@
  * </pre>
  * 
  * <p>
- * and "&lt;" meaning the previous and "&lt;&lt;" goto the "current page - 5",
- * than it is this kind of incremental page links which can easily be created.
+ * and "&lt;" meaning the previous and "&lt;&lt;" goto the "current page - 5", than it is this kind
+ * of incremental page links which can easily be created.
  * 
  * @author Juergen Donnerstag
  * @author Martijn Dashorst
@@ -96,8 +96,7 @@
 	}
 
 	/**
-	 * @return True if it is referring to the first page of the underlying
-	 *         PageableListView.
+	 * @return True if it is referring to the first page of the underlying PageableListView.
 	 */
 	public boolean isFirst()
 	{
@@ -105,8 +104,7 @@
 	}
 
 	/**
-	 * @return True if it is referring to the last page of the underlying
-	 *         PageableListView.
+	 * @return True if it is referring to the last page of the underlying PageableListView.
 	 */
 	public boolean isLast()
 	{

Modified: wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/navigation/paging/PagingNavigationLink.java
URL: http://svn.apache.org/viewvc/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/navigation/paging/PagingNavigationLink.java?rev=584925&r1=584924&r2=584925&view=diff
==============================================================================
--- wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/navigation/paging/PagingNavigationLink.java (original)
+++ wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/navigation/paging/PagingNavigationLink.java Mon Oct 15 14:21:25 2007
@@ -29,13 +29,13 @@
 public class PagingNavigationLink extends Link
 {
 	private static final long serialVersionUID = 1L;
-	
+
 	/** The pageable list view. */
 	protected final IPageable pageable;
 
 	/** The page of the PageableListView this link is for. */
 	private final int pageNumber;
-	    
+
 	/**
 	 * Constructor.
 	 * 
@@ -44,11 +44,10 @@
 	 * @param pageable
 	 *            The pageable component for this page link
 	 * @param pageNumber
-	 *            The page number in the PageableListView that this link links
-	 *            to. Negative pageNumbers are relative to the end of the list.
+	 *            The page number in the PageableListView that this link links to. Negative
+	 *            pageNumbers are relative to the end of the list.
 	 */
-	public PagingNavigationLink(final String id,
-			final IPageable pageable, final int pageNumber)
+	public PagingNavigationLink(final String id, final IPageable pageable, final int pageNumber)
 	{
 		super(id);
 		setAutoEnable(true);
@@ -71,12 +70,12 @@
 	 */
 	public final int getPageNumber()
 	{
-	    int idx = pageNumber;
+		int idx = pageNumber;
 		if (idx < 0)
 		{
 			idx = pageable.getPageCount() + idx;
 		}
-		
+
 		if (idx > (pageable.getPageCount() - 1))
 		{
 			idx = pageable.getPageCount() - 1;
@@ -86,13 +85,12 @@
 		{
 			idx = 0;
 		}
-		
+
 		return idx;
 	}
 
 	/**
-	 * @return True if this page is the first page of the containing
-	 *         PageableListView
+	 * @return True if this page is the first page of the containing PageableListView
 	 */
 	public final boolean isFirst()
 	{
@@ -100,8 +98,7 @@
 	}
 
 	/**
-	 * @return True if this page is the last page of the containing
-	 *         PageableListView
+	 * @return True if this page is the last page of the containing PageableListView
 	 */
 	public final boolean isLast()
 	{
@@ -109,8 +106,7 @@
 	}
 
 	/**
-	 * Returns true if this PageableListView navigation link links to the given
-	 * page.
+	 * Returns true if this PageableListView navigation link links to the given page.
 	 * 
 	 * @param page
 	 *            The page

Modified: wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/navigation/paging/PagingNavigator.java
URL: http://svn.apache.org/viewvc/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/navigation/paging/PagingNavigator.java?rev=584925&r1=584924&r2=584925&view=diff
==============================================================================
--- wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/navigation/paging/PagingNavigator.java (original)
+++ wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/navigation/paging/PagingNavigator.java Mon Oct 15 14:21:25 2007
@@ -20,10 +20,9 @@
 import org.apache.wicket.markup.html.panel.Panel;
 
 /**
- * A Wicket panel component to draw and maintain a complete page navigator,
- * meant to be easily added to any PageableListView. A navigation which contains
- * links to the first and last page, the current page +- some increment and
- * which supports paged navigation bars (@see
+ * A Wicket panel component to draw and maintain a complete page navigator, meant to be easily added
+ * to any PageableListView. A navigation which contains links to the first and last page, the
+ * current page +- some increment and which supports paged navigation bars (@see
  * PageableListViewNavigationWithMargin).
  * 
  * @author Juergen Donnerstag
@@ -76,8 +75,8 @@
 	}
 
 	/**
-	 * Create a new increment link. May be subclassed to make use of specialized
-	 * links, e.g. Ajaxian links.
+	 * Create a new increment link. May be subclassed to make use of specialized links, e.g. Ajaxian
+	 * links.
 	 * 
 	 * @param id
 	 *            the link id
@@ -93,8 +92,8 @@
 	}
 
 	/**
-	 * Create a new pagenumber link. May be subclassed to make use of
-	 * specialized links, e.g. Ajaxian links.
+	 * Create a new pagenumber link. May be subclassed to make use of specialized links, e.g.
+	 * Ajaxian links.
 	 * 
 	 * @param id
 	 *            the link id
@@ -110,8 +109,7 @@
 	}
 
 	/**
-	 * Create a new PagingNavigation. May be subclassed to make us of
-	 * specialized PagingNavigation.
+	 * Create a new PagingNavigation. May be subclassed to make us of specialized PagingNavigation.
 	 * 
 	 * @param pageable
 	 *            the pageable component