You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@roller.apache.org by ag...@apache.org on 2006/11/04 01:53:40 UTC

svn commit: r471086 - /incubator/roller/trunk/src/org/apache/roller/util/URLUtilities.java

Author: agilliland
Date: Fri Nov  3 16:53:39 2006
New Revision: 471086

URL: http://svn.apache.org/viewvc?view=rev&rev=471086
Log:
fixing problem with category permalink urls (/weblog/category/foo) which was being caused by the fact that we double encoded that part of the url.  the double encoding was a bad work around for the fact that the WeblogRequestMapper had been improperly using decoded urls for the request mapping.  So now that the WeblogRequestMapper is doing the proper thing we can remove the double encoding.


Modified:
    incubator/roller/trunk/src/org/apache/roller/util/URLUtilities.java

Modified: incubator/roller/trunk/src/org/apache/roller/util/URLUtilities.java
URL: http://svn.apache.org/viewvc/incubator/roller/trunk/src/org/apache/roller/util/URLUtilities.java?view=diff&rev=471086&r1=471085&r2=471086
==============================================================================
--- incubator/roller/trunk/src/org/apache/roller/util/URLUtilities.java (original)
+++ incubator/roller/trunk/src/org/apache/roller/util/URLUtilities.java Fri Nov  3 16:53:39 2006
@@ -175,9 +175,7 @@
         }
         
         if(cat != null && dateString == null) {
-            // yes, for the path based versions we double encode the category
-            // because otherwise some characters cause problems, like "?"
-            pathinfo.append("category/").append(encode(encode(cat)));   
+            pathinfo.append("category/").append(encode(cat));
             
         } else if(dateString != null && cat == null) {
             pathinfo.append("date/").append(dateString);