You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by jb...@apache.org on 2007/08/03 17:23:28 UTC

svn commit: r562500 - in /wicket/trunk/jdk-1.4/wicket/src: main/java/org/apache/wicket/request/target/coding/MixedParamUrlCodingStrategy.java test/java/org/apache/wicket/request/target/coding/MixedParamUrlCodingStrategyTest.java

Author: jbq
Date: Fri Aug  3 08:23:27 2007
New Revision: 562500

URL: http://svn.apache.org/viewvc?view=rev&rev=562500
Log:
WICKET-739 MixedParamUrlCodingStrategy : no symmetric url encoding/decoding

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

Modified: wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/request/target/coding/MixedParamUrlCodingStrategy.java
URL: http://svn.apache.org/viewvc/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/request/target/coding/MixedParamUrlCodingStrategy.java?view=diff&rev=562500&r1=562499&r2=562500
==============================================================================
--- wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/request/target/coding/MixedParamUrlCodingStrategy.java (original)
+++ wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/request/target/coding/MixedParamUrlCodingStrategy.java Fri Aug  3 08:23:27 2007
@@ -185,7 +185,7 @@
 
 			for (int i = 0; i < pathParts.length; i++)
 			{
-				params.put(parameterNames[i], pathParts[i]);
+				params.put(parameterNames[i], urlDecode(pathParts[i]));
 			}
 		}
 

Modified: wicket/trunk/jdk-1.4/wicket/src/test/java/org/apache/wicket/request/target/coding/MixedParamUrlCodingStrategyTest.java
URL: http://svn.apache.org/viewvc/wicket/trunk/jdk-1.4/wicket/src/test/java/org/apache/wicket/request/target/coding/MixedParamUrlCodingStrategyTest.java?view=diff&rev=562500&r1=562499&r2=562500
==============================================================================
--- wicket/trunk/jdk-1.4/wicket/src/test/java/org/apache/wicket/request/target/coding/MixedParamUrlCodingStrategyTest.java (original)
+++ wicket/trunk/jdk-1.4/wicket/src/test/java/org/apache/wicket/request/target/coding/MixedParamUrlCodingStrategyTest.java Fri Aug  3 08:23:27 2007
@@ -213,9 +213,9 @@
 		urlMap.put("d", "4");
 		urlMap.put("e", "5");
 
-		ValueMap parameterMap = npucs.decodeParameters("/1/2/3", urlMap);
+		ValueMap parameterMap = npucs.decodeParameters("/%C3%A8/2/3", urlMap);
 		assertEquals(5, parameterMap.size());
-		assertContains(parameterMap, "a", "1");
+		assertContains(parameterMap, "a", "\u00e8");
 		assertContains(parameterMap, "b", "2");
 		assertContains(parameterMap, "c", "3");
 		assertContains(parameterMap, "d", "4");