Author: snoopdave
Date: Wed Jun 18 05:56:02 2008
New Revision: 669156

URL: http://svn.apache.org/viewvc?rev=669156&view=rev
Log:
Applying patch for ROL-1722
https://issues.apache.org/roller/browse/ROL-1722

Changing display name of frontpage theme to 'Frontpage'Applying patch for 
ROL-1726
https://issues.apache.org/roller/browse/ROL-1726

Fix for NPE when deleting subscription with ampersant in URL

Modified:
    roller/trunk/apps/weblogger/web/WEB-INF/jsps/admin/PlanetSubscriptions.jsp

Modified: 
roller/trunk/apps/weblogger/web/WEB-INF/jsps/admin/PlanetSubscriptions.jsp
URL: 
http://svn.apache.org/viewvc/roller/trunk/apps/weblogger/web/WEB-INF/jsps/admin/PlanetSubscriptions.jsp?rev=669156&r1=669155&r2=669156&view=diff
==============================================================================
--- roller/trunk/apps/weblogger/web/WEB-INF/jsps/admin/PlanetSubscriptions.jsp 
(original)
+++ roller/trunk/apps/weblogger/web/WEB-INF/jsps/admin/PlanetSubscriptions.jsp 
Wed Jun 18 05:56:02 2008
@@ -20,7 +20,7 @@
 <script type="text/javascript">
 function confirmSubDelete(subUrl) {
   if (window.confirm('Are you sure you want to remove this subscription?')) {
-    document.location.href='<s:url action="planetSubscriptions!delete" 
/>?groupHandle=<s:property value="groupHandle"/>&subUrl='+subUrl;
+    document.location.href='<s:url action="planetSubscriptions!delete" 
/>?groupHandle=<s:property 
value="groupHandle"/>&subUrl='+encodeURIComponent(subUrl);
   }
 }
 </script>


Reply via email to