[jira] Commented: (CONTINUUM-1796) Project Information tab's Add/Edit Notifiers links contain leading whitespace

[Wendy Smoak (JIRA)]

    [ 
http://jira.codehaus.org/browse/CONTINUUM-1796?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=138289#action_138289
 ] 

Wendy Smoak commented on CONTINUUM-1796:
----------------------------------------

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.

{noformat}
$ 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" />
{noformat}

> Project Information tab's Add/Edit Notifiers links contain leading whitespace
> -----------------------------------------------------------------------------
>
>                 Key: CONTINUUM-1796
>                 URL: http://jira.codehaus.org/browse/CONTINUUM-1796
>             Project: Continuum
>          Issue Type: Bug
>    Affects Versions: 1.1
>            Reporter: Wendy Smoak
>            Priority: Minor
>
> 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:
> {noformat}
>         <a 
> href="/continuum/deleteProjectGroupNotifier!default.action?confirmed=false&amp;notifierType=mail&amp;projectGroupId=7&amp;notifierId=1">
>           <img src="/continuum/images/delete.gif" alt="Delete" title="Delete" 
> border="0">
>         </a>
> {noformat}
> But on the Project Information Tab, Notifiers section, the delete link is:
> {noformat}
>                       <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&amp;notifierType=mail&amp;projectGroupId=7&amp;notifierId=2">
>                         <img src="/continuum/images/delete.gif" alt="Delete" 
> title="Delete" border="0"></a>
> {noformat}

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira