roller-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From snoopd...@apache.org
Subject svn commit: r669046 - /roller/branches/roller_4.0/apps/weblogger/web/WEB-INF/jsps/admin/PlanetSubscriptions.jsp
Date Wed, 18 Jun 2008 01:39:28 GMT
Author: snoopdave
Date: Tue Jun 17 18:39:28 2008
New Revision: 669046

URL: http://svn.apache.org/viewvc?rev=669046&view=rev
Log:
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/branches/roller_4.0/apps/weblogger/web/WEB-INF/jsps/admin/PlanetSubscriptions.jsp

Modified: roller/branches/roller_4.0/apps/weblogger/web/WEB-INF/jsps/admin/PlanetSubscriptions.jsp
URL: http://svn.apache.org/viewvc/roller/branches/roller_4.0/apps/weblogger/web/WEB-INF/jsps/admin/PlanetSubscriptions.jsp?rev=669046&r1=669045&r2=669046&view=diff
==============================================================================
--- roller/branches/roller_4.0/apps/weblogger/web/WEB-INF/jsps/admin/PlanetSubscriptions.jsp
(original)
+++ roller/branches/roller_4.0/apps/weblogger/web/WEB-INF/jsps/admin/PlanetSubscriptions.jsp
Tue Jun 17 18:39:28 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>



Mime
View raw message