svn commit: r940508 - /wicket/branches/wicket-1.4.x/wicket/src/main/java/org/apache/wicket/request/target/basic/RedirectRequestTarget.java

[jrthomerson]

Author: jrthomerson
Date: Mon May  3 16:44:02 2010
New Revision: 940508

URL: http://svn.apache.org/viewvc?rev=940508&view=rev
Log:
WICKET-2855 - do not accept null or empty redirect URL in RedirectRequestTarget


Modified:
    
wicket/branches/wicket-1.4.x/wicket/src/main/java/org/apache/wicket/request/target/basic/RedirectRequestTarget.java

Modified: 
wicket/branches/wicket-1.4.x/wicket/src/main/java/org/apache/wicket/request/target/basic/RedirectRequestTarget.java
URL: 
http://svn.apache.org/viewvc/wicket/branches/wicket-1.4.x/wicket/src/main/java/org/apache/wicket/request/target/basic/RedirectRequestTarget.java?rev=940508&r1=940507&r2=940508&view=diff
==============================================================================
--- 
wicket/branches/wicket-1.4.x/wicket/src/main/java/org/apache/wicket/request/target/basic/RedirectRequestTarget.java
 (original)
+++ 
wicket/branches/wicket-1.4.x/wicket/src/main/java/org/apache/wicket/request/target/basic/RedirectRequestTarget.java
 Mon May  3 16:44:02 2010
@@ -23,6 +23,7 @@ import org.apache.wicket.Response;
 import org.apache.wicket.markup.html.pages.RedirectPage;
 import org.apache.wicket.protocol.http.portlet.PortletRequestContext;
 import 
org.apache.wicket.request.target.component.listener.RedirectPageRequestTarget;
+import org.apache.wicket.util.string.Strings;
 
 /**
  * A RequestTarget that will send a redirect url to the browser. Use this if 
you want to direct the
@@ -53,6 +54,11 @@ public class RedirectRequestTarget imple
         */
        public RedirectRequestTarget(String redirectUrl)
        {
+               if (Strings.isEmpty(redirectUrl))
+               {
+                       throw new IllegalArgumentException(
+                               "The redirect URL passed to the 
RedirectRequestTarget constructor can not be null or empty.");
+               }
                this.redirectUrl = redirectUrl;
 
        }