Details
-
Type:
Bug
-
Status:
Closed
-
Priority:
Minor
-
Resolution: Fixed
-
Affects Version/s: 1.1
-
Fix Version/s: 1.2
-
Component/s: None
-
Labels:None
-
Complexity:Intermediate
-
Number of attachments :
Description
The Add and Edit Notifier links on the Project Information tab contain leading whitespace.
This confuses some proxies which need to re-write the urls, which causes errors when the links are clicked.
On the Notifiers tab, the html for the Delete link looks like this:
<a href="/continuum/deleteProjectGroupNotifier!default.action?confirmed=false&notifierType=mail&projectGroupId=7&notifierId=1">
<img src="/continuum/images/delete.gif" alt="Delete" title="Delete" border="0">
</a>
But on the Project Information Tab, Notifiers section, the delete link is:
<a href="%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%0A%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%0A%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%0A%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%0A%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20/continuum/deleteProjectNotifier%21default.action?projectId=8&notifierType=mail&projectGroupId=7&notifierId=2">
<img src="/continuum/images/delete.gif" alt="Delete" title="Delete" border="0"></a>
Patch for continuum/webapp/WEB-INF/jsp/projectView.jsp, based on the 'delete' link on .../jsp/components/projectGroupNotifierSummaryComponent.jsp . The 'edit' link also needs to be fixed.
$ diff -u projectView.jsp.orig projectView.jsp --- projectView.jsp.orig 2008-06-11 15:23:00.000000000 -0700 +++ projectView.jsp 2008-06-11 15:39:57.000000000 -0700 @@ -132,13 +132,15 @@ <redback:ifAuthorized permission="continuum-modify-group" resource="${project.projectGroup.name}"> <c:choose> <c:when test="${!pageScope.notifier.fromProject}"> - <a href='<ww:url value="/deleteProjectNotifier!default.action"> + <ww:url id="removeUrl" action="deleteProjectNotifier" namespace="/"> <ww:param name="projectId" value="project.id"/> <ww:param name="projectGroupId" value="${project.projectGroup.id}"/> <ww:param name="notifierType">${notifier.type}</ww:param> <ww:param name="notifierId" value="${notifier.id}"/> - </ww:url>'> - <img src="<ww:url value='/images/delete.gif' includeParams="none"/>" alt="<ww:text name='delete'/>" title="<ww:text name='delete'/>" border="0"></a> + </ww:url> + <ww:a href="%{removeUrl}"> + <img src="<ww:url value='/images/delete.gif' includeParams="none"/>" alt="<ww:text name="delete"/>" title="<ww:text name="delete"/>" border="0"> + </ww:a> </c:when> <c:otherwise> <img src="<ww:url value='/images/delete_disabled.gif' includeParams="none"/>" alt="<ww:text name='edit'/>" title="<ww:text name='edit'/>" border="0" />$ diff -u projectView.jsp.orig projectView.jsp --- projectView.jsp.orig 2008-06-11 15:23:00.000000000 -0700 +++ projectView.jsp 2008-06-11 15:39:57.000000000 -0700 @@ -132,13 +132,15 @@ <redback:ifAuthorized permission="continuum-modify-group" resource="${project.projectGroup.name}"> <c:choose> <c:when test="${!pageScope.notifier.fromProject}"> - <a href='<ww:url value="/deleteProjectNotifier!default.action"> + <ww:url id="removeUrl" action="deleteProjectNotifier" namespace="/"> <ww:param name="projectId" value="project.id"/> <ww:param name="projectGroupId" value="${project.projectGroup.id}"/> <ww:param name="notifierType">${notifier.type}</ww:param> <ww:param name="notifierId" value="${notifier.id}"/> - </ww:url>'> - <img src="<ww:url value='/images/delete.gif' includeParams="none"/>" alt="<ww:text name='delete'/>" title="<ww:text name='delete'/>" border="0"></a> + </ww:url> + <ww:a href="%{removeUrl}"> + <img src="<ww:url value='/images/delete.gif' includeParams="none"/>" alt="<ww:text name="delete"/>" title="<ww:text name="delete"/>" border="0"> + </ww:a> </c:when> <c:otherwise> <img src="<ww:url value='/images/delete_disabled.gif' includeParams="none"/>" alt="<ww:text name='edit'/>" title="<ww:text name='edit'/>" border="0" />