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>
