You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@roller.apache.org by gm...@apache.org on 2014/09/09 03:42:43 UTC

svn commit: r1623609 - /roller/trunk/app/src/main/java/org/apache/roller/weblogger/ui/rendering/util/WeblogFeedRequest.java

Author: gmazza
Date: Tue Sep  9 01:42:43 2014
New Revision: 1623609

URL: http://svn.apache.org/r1623609
Log:
fix for ROL-1620 plus signs in categories names result in 404 feeds.

Modified:
    roller/trunk/app/src/main/java/org/apache/roller/weblogger/ui/rendering/util/WeblogFeedRequest.java

Modified: roller/trunk/app/src/main/java/org/apache/roller/weblogger/ui/rendering/util/WeblogFeedRequest.java
URL: http://svn.apache.org/viewvc/roller/trunk/app/src/main/java/org/apache/roller/weblogger/ui/rendering/util/WeblogFeedRequest.java?rev=1623609&r1=1623608&r2=1623609&view=diff
==============================================================================
--- roller/trunk/app/src/main/java/org/apache/roller/weblogger/ui/rendering/util/WeblogFeedRequest.java (original)
+++ roller/trunk/app/src/main/java/org/apache/roller/weblogger/ui/rendering/util/WeblogFeedRequest.java Tue Sep  9 01:42:43 2014
@@ -120,8 +120,9 @@ public class WeblogFeedRequest extends W
          *
          */
         if(request.getParameter("cat") != null) {
+            // replacing plus sign below with its encoded equivalent (http://stackoverflow.com/a/6926987)
             this.weblogCategoryName =
-                    URLUtilities.decode(request.getParameter("cat"));
+                    URLUtilities.decode(request.getParameter("cat").replace("+", "%2B"));
         }
         
         if(request.getParameter("tags") != null) {