svn commit: r645125 - /roller/branches/roller_4.0/apps/weblogger/web/WEB-INF/jsps/admin/PlanetSubscriptions.jsp

[sn...@apache.org]

Author: snoopdave
Date: Sat Apr  5 08:35:18 2008
New Revision: 645125

URL: http://svn.apache.org/viewvc?rev=645125&view=rev
Log:
Fix for ROL-1700:
    https://issues.apache.org/roller/browse/ROL-1700

Fixes problem be leaving subscription title out of the "are you sure you want to delete? message.

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=645125&r1=645124&r2=645125&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 Sat Apr  5 08:35:18 2008
@@ -18,8 +18,8 @@
 <%@ include file="/WEB-INF/jsps/taglibs-struts2.jsp" %>
 
 <script type="text/javascript">
-function confirmSubDelete(subUrl, title) {
-  if (window.confirm('Are you sure you want to remove subscription: ' + title)) {
+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;
   }
 }
@@ -98,7 +98,9 @@
         
         <td class="rollertable">
             <img src='<s:url value="/images/delete.png"/>' />
-            <a href="javascript: void(0);" onclick="confirmSubDelete('<s:property value="feedURL"/>', '<s:property value="title"/>');"><s:text name="planetSubscriptions.button.delete"/></a>
+            <a href="javascript: void(0);" onclick="confirmSubDelete('<s:property value="feedURL"/>')">
+                <s:text name="planetSubscriptions.button.delete"/>
+            </a>
         </td>       
         
         </tr>