roller-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From agillil...@apache.org
Subject svn commit: r471086 - /incubator/roller/trunk/src/org/apache/roller/util/URLUtilities.java
Date Sat, 04 Nov 2006 00:53:40 GMT
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);  



Mime
View raw message