You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by jd...@apache.org on 2011/04/23 09:36:40 UTC
svn commit: r1096117 - in /wicket/trunk/wicket-request/src:
main/java/org/apache/wicket/request/
main/java/org/apache/wicket/request/flow/
main/java/org/apache/wicket/request/http/flow/
test/java/org/apache/wicket/request/http/handler/ test/java/org/ap...
Author: jdonnerstag
Date: Sat Apr 23 07:36:40 2011
New Revision: 1096117
URL: http://svn.apache.org/viewvc?rev=1096117&view=rev
Log:
WICKET-3624
fixed: Remove many Eclipse warnings in wicket-request
Modified:
wicket/trunk/wicket-request/src/main/java/org/apache/wicket/request/IRequestCycle.java
wicket/trunk/wicket-request/src/main/java/org/apache/wicket/request/RequestHandlerStack.java
wicket/trunk/wicket-request/src/main/java/org/apache/wicket/request/flow/RedirectToUrlException.java
wicket/trunk/wicket-request/src/main/java/org/apache/wicket/request/http/flow/AbortWithHttpErrorCodeException.java
wicket/trunk/wicket-request/src/test/java/org/apache/wicket/request/http/handler/RedirectRequestHandlerTest.java
wicket/trunk/wicket-request/src/test/java/org/apache/wicket/request/mapper/mount/MountParametersTest.java
Modified: wicket/trunk/wicket-request/src/main/java/org/apache/wicket/request/IRequestCycle.java
URL: http://svn.apache.org/viewvc/wicket/trunk/wicket-request/src/main/java/org/apache/wicket/request/IRequestCycle.java?rev=1096117&r1=1096116&r2=1096117&view=diff
==============================================================================
--- wicket/trunk/wicket-request/src/main/java/org/apache/wicket/request/IRequestCycle.java (original)
+++ wicket/trunk/wicket-request/src/main/java/org/apache/wicket/request/IRequestCycle.java Sat Apr 23 07:36:40 2011
@@ -50,5 +50,10 @@ public interface IRequestCycle
*/
void scheduleRequestHandlerAfterCurrent(IRequestHandler handler);
+ /**
+ * Returns {@link UrlRenderer} for this {@link IRequestCycle}.
+ *
+ * @return UrlRenderer instance.
+ */
UrlRenderer getUrlRenderer();
}
Modified: wicket/trunk/wicket-request/src/main/java/org/apache/wicket/request/RequestHandlerStack.java
URL: http://svn.apache.org/viewvc/wicket/trunk/wicket-request/src/main/java/org/apache/wicket/request/RequestHandlerStack.java?rev=1096117&r1=1096116&r2=1096117&view=diff
==============================================================================
--- wicket/trunk/wicket-request/src/main/java/org/apache/wicket/request/RequestHandlerStack.java (original)
+++ wicket/trunk/wicket-request/src/main/java/org/apache/wicket/request/RequestHandlerStack.java Sat Apr 23 07:36:40 2011
@@ -42,7 +42,7 @@ public abstract class RequestHandlerStac
private IRequestHandler scheduledAfterCurrent = null;
/**
- * {@inheritDoc}
+ * @return active handler
*/
public IRequestHandler getActive()
{
@@ -50,7 +50,7 @@ public abstract class RequestHandlerStac
}
/**
- * {@inheritDoc}
+ * @param handler
*/
public void execute(final IRequestHandler handler)
{
@@ -98,14 +98,16 @@ public abstract class RequestHandlerStac
}
/**
- * Allows the request handler to response to the request
+ * Allows the request handler to respond to the request
*
* @param handler
*/
protected abstract void respond(IRequestHandler handler);
/**
- * {@inheritDoc}
+ * Schedules the handler after the current one
+ *
+ * @param handler
*/
public void schedule(final IRequestHandler handler)
{
@@ -113,7 +115,7 @@ public abstract class RequestHandlerStac
}
/**
- * {@inheritDoc}
+ * @return scheduled request handler after the current one
*/
public IRequestHandler next()
{
@@ -121,7 +123,7 @@ public abstract class RequestHandlerStac
}
/**
- * {@inheritDoc}
+ * @param handler
*/
public void replaceAll(final IRequestHandler handler)
{
@@ -136,7 +138,7 @@ public abstract class RequestHandlerStac
}
/**
- * {@inheritDoc}
+ * Detaches all request handlers
*/
public void detach()
{
Modified: wicket/trunk/wicket-request/src/main/java/org/apache/wicket/request/flow/RedirectToUrlException.java
URL: http://svn.apache.org/viewvc/wicket/trunk/wicket-request/src/main/java/org/apache/wicket/request/flow/RedirectToUrlException.java?rev=1096117&r1=1096116&r2=1096117&view=diff
==============================================================================
--- wicket/trunk/wicket-request/src/main/java/org/apache/wicket/request/flow/RedirectToUrlException.java (original)
+++ wicket/trunk/wicket-request/src/main/java/org/apache/wicket/request/flow/RedirectToUrlException.java Sat Apr 23 07:36:40 2011
@@ -34,11 +34,25 @@ public class RedirectToUrlException exte
{
private static final long serialVersionUID = 1L;
+ /**
+ * Construct.
+ *
+ * @param redirectUrl
+ * URL to redirect to.
+ */
public RedirectToUrlException(final String redirectUrl)
{
this(redirectUrl, HttpServletResponse.SC_MOVED_TEMPORARILY);
}
+ /**
+ * Construct.
+ *
+ * @param redirectUrl
+ * URL to redirect to.
+ * @param statusCode
+ * 301 (Moved permanently) or 302 (Moved temporarily)
+ */
public RedirectToUrlException(final String redirectUrl, final int statusCode)
{
super(new RedirectRequestHandler(redirectUrl, statusCode), true);
Modified: wicket/trunk/wicket-request/src/main/java/org/apache/wicket/request/http/flow/AbortWithHttpErrorCodeException.java
URL: http://svn.apache.org/viewvc/wicket/trunk/wicket-request/src/main/java/org/apache/wicket/request/http/flow/AbortWithHttpErrorCodeException.java?rev=1096117&r1=1096116&r2=1096117&view=diff
==============================================================================
--- wicket/trunk/wicket-request/src/main/java/org/apache/wicket/request/http/flow/AbortWithHttpErrorCodeException.java (original)
+++ wicket/trunk/wicket-request/src/main/java/org/apache/wicket/request/http/flow/AbortWithHttpErrorCodeException.java Sat Apr 23 07:36:40 2011
@@ -41,6 +41,8 @@ public final class AbortWithHttpErrorCod
* @param errorCode
* the servlet error code; use one of the
* {@link javax.servlet.http.HttpServletResponse} constants
+ * @param message
+ * the optional message to send to the client
* @see javax.servlet.http.HttpServletResponse
*/
public AbortWithHttpErrorCodeException(final int errorCode, final String message)
Modified: wicket/trunk/wicket-request/src/test/java/org/apache/wicket/request/http/handler/RedirectRequestHandlerTest.java
URL: http://svn.apache.org/viewvc/wicket/trunk/wicket-request/src/test/java/org/apache/wicket/request/http/handler/RedirectRequestHandlerTest.java?rev=1096117&r1=1096116&r2=1096117&view=diff
==============================================================================
--- wicket/trunk/wicket-request/src/test/java/org/apache/wicket/request/http/handler/RedirectRequestHandlerTest.java (original)
+++ wicket/trunk/wicket-request/src/test/java/org/apache/wicket/request/http/handler/RedirectRequestHandlerTest.java Sat Apr 23 07:36:40 2011
@@ -23,11 +23,17 @@ import org.apache.wicket.request.http.We
import org.junit.Test;
import org.mockito.Mockito;
+/**
+ * RedirectRequestHandlerTest
+ */
public class RedirectRequestHandlerTest
{
private static final String REDIRECT_URL = "redirectUrl";
+ /**
+ * permenanentlyMovedShouldSetLocationHeader()
+ */
@Test
public void permenanentlyMovedShouldSetLocationHeader()
{
@@ -45,6 +51,9 @@ public class RedirectRequestHandlerTest
Mockito.verify(webResponse).setHeader("Location", REDIRECT_URL);
}
+ /**
+ * tempMovedShouldRedirect()
+ */
@Test
public void tempMovedShouldRedirect()
{
Modified: wicket/trunk/wicket-request/src/test/java/org/apache/wicket/request/mapper/mount/MountParametersTest.java
URL: http://svn.apache.org/viewvc/wicket/trunk/wicket-request/src/test/java/org/apache/wicket/request/mapper/mount/MountParametersTest.java?rev=1096117&r1=1096116&r2=1096117&view=diff
==============================================================================
--- wicket/trunk/wicket-request/src/test/java/org/apache/wicket/request/mapper/mount/MountParametersTest.java (original)
+++ wicket/trunk/wicket-request/src/test/java/org/apache/wicket/request/mapper/mount/MountParametersTest.java Sat Apr 23 07:36:40 2011
@@ -22,9 +22,15 @@ import static org.junit.Assert.assertTru
import org.apache.wicket.util.string.StringValue;
import org.junit.Test;
+/**
+ * MountParametersTest
+ */
public class MountParametersTest
{
+ /**
+ * getNonExistingPlaceholder()
+ */
@Test
public void getNonExistingPlaceholder()
{
@@ -34,6 +40,9 @@ public class MountParametersTest
assertTrue(value.isNull());
}
+ /**
+ * getExistingPlaceholder()
+ */
@Test
public void getExistingPlaceholder()
{
@@ -47,6 +56,9 @@ public class MountParametersTest
assertEquals(originalValue.toString(), actualValue.toString());
}
+ /**
+ * cannotModifyByNamesSet()
+ */
@Test(expected = UnsupportedOperationException.class)
public void cannotModifyByNamesSet()
{