You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@roller.apache.org by sn...@apache.org on 2008/06/18 03:39:28 UTC
svn commit: r669046 -
/roller/branches/roller_4.0/apps/weblogger/web/WEB-INF/jsps/admin/PlanetSubscriptions.jsp
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>