You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by eh...@apache.org on 2007/04/17 06:48:55 UTC

svn commit: r529483 - /incubator/wicket/branches/wicket-1.x/jdk-1.4/wicket/src/main/java/org/apache/wicket/request/AbstractRequestCycleProcessor.java

Author: ehillenius
Date: Mon Apr 16 21:48:54 2007
New Revision: 529483

URL: http://svn.apache.org/viewvc?view=rev&rev=529483
Log:
null checks

Modified:
    incubator/wicket/branches/wicket-1.x/jdk-1.4/wicket/src/main/java/org/apache/wicket/request/AbstractRequestCycleProcessor.java

Modified: incubator/wicket/branches/wicket-1.x/jdk-1.4/wicket/src/main/java/org/apache/wicket/request/AbstractRequestCycleProcessor.java
URL: http://svn.apache.org/viewvc/incubator/wicket/branches/wicket-1.x/jdk-1.4/wicket/src/main/java/org/apache/wicket/request/AbstractRequestCycleProcessor.java?view=diff&rev=529483&r1=529482&r2=529483
==============================================================================
--- incubator/wicket/branches/wicket-1.x/jdk-1.4/wicket/src/main/java/org/apache/wicket/request/AbstractRequestCycleProcessor.java (original)
+++ incubator/wicket/branches/wicket-1.x/jdk-1.4/wicket/src/main/java/org/apache/wicket/request/AbstractRequestCycleProcessor.java Mon Apr 16 21:48:54 2007
@@ -133,7 +133,8 @@
 		}
 		else if (e instanceof PageExpiredException)
 		{
-			Class pageExpiredErrorPageClass = application.getApplicationSettings().getPageExpiredErrorPage();
+			Class pageExpiredErrorPageClass = application.getApplicationSettings()
+					.getPageExpiredErrorPage();
 			throw new RestartResponseException(pageExpiredErrorPageClass);
 		}
 		else if (settings.getUnexpectedExceptionDisplay() != IExceptionSettings.SHOW_NO_EXCEPTION_PAGE)
@@ -346,6 +347,15 @@
 			final Page page, final String componentPath, final String interfaceName,
 			final RequestParameters requestParameters)
 	{
+		if (page == null)
+		{
+			throw new IllegalArgumentException("page must not be null");
+		}
+		if (interfaceName == null)
+		{
+			throw new IllegalArgumentException("interfaceName must not be null");
+		}
+
 		if (interfaceName.equals(IRedirectListener.INTERFACE.getName()))
 		{
 			return new RedirectPageRequestTarget(page);