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

svn commit: r590439 - /wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/request/target/coding/IndexedParamUrlCodingStrategy.java

Author: jcompagner
Date: Tue Oct 30 13:35:31 2007
New Revision: 590439

URL: http://svn.apache.org/viewvc?rev=590439&view=rev
Log:
fix for ending with only / that gives an empty param 0
WICKET-1092

Modified:
    wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/request/target/coding/IndexedParamUrlCodingStrategy.java

Modified: wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/request/target/coding/IndexedParamUrlCodingStrategy.java
URL: http://svn.apache.org/viewvc/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/request/target/coding/IndexedParamUrlCodingStrategy.java?rev=590439&r1=590438&r2=590439&view=diff
==============================================================================
--- wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/request/target/coding/IndexedParamUrlCodingStrategy.java (original)
+++ wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/request/target/coding/IndexedParamUrlCodingStrategy.java Tue Oct 30 13:35:31 2007
@@ -131,23 +131,27 @@
 			urlFragment = urlFragment.substring(0, urlFragment.length() - 1);
 		}
 
-		String[] parts = urlFragment.split("/");
-		for (int i = 0; i < parts.length; i++)
+		if (urlFragment.length() > 0)
 		{
-			if (WebRequestCodingStrategy.PAGEMAP.equals(parts[i]))
+			String[] parts = urlFragment.split("/");
+			for (int i = 0; i < parts.length; i++)
 			{
-				i++;
-				params.put(WebRequestCodingStrategy.PAGEMAP, WebRequestCodingStrategy
-						.decodePageMapName(urlDecode(parts[i])));
-			}
-			else if (WebRequestCodingStrategy.INTERFACE_PARAMETER_NAME.equals(parts[i]))
-			{
-				i++;
-				params.put(WebRequestCodingStrategy.INTERFACE_PARAMETER_NAME, urlDecode(parts[i]));
-			}
-			else
-			{
-				params.put(String.valueOf(i), urlDecode(parts[i]));
+				if (WebRequestCodingStrategy.PAGEMAP.equals(parts[i]))
+				{
+					i++;
+					params.put(WebRequestCodingStrategy.PAGEMAP, WebRequestCodingStrategy
+							.decodePageMapName(urlDecode(parts[i])));
+				}
+				else if (WebRequestCodingStrategy.INTERFACE_PARAMETER_NAME.equals(parts[i]))
+				{
+					i++;
+					params.put(WebRequestCodingStrategy.INTERFACE_PARAMETER_NAME,
+							urlDecode(parts[i]));
+				}
+				else
+				{
+					params.put(String.valueOf(i), urlDecode(parts[i]));
+				}
 			}
 		}
 		return params;