[ http://jira.codehaus.org/browse/CONTINUUM-783?page=all ]

Carlos Sanchez closed CONTINUUM-783.
------------------------------------

      Assignee: Carlos Sanchez  (was: Henry S. Isidro)
    Resolution: Fixed

> Use standard jstl for links or they will break
> ----------------------------------------------
>
>                 Key: CONTINUUM-783
>                 URL: http://jira.codehaus.org/browse/CONTINUUM-783
>             Project: Continuum
>          Issue Type: Bug
>          Components: Web interface
>            Reporter: Carlos Sanchez
>         Assigned To: Carlos Sanchez
>             Fix For: 1.1
>
>         Attachments: CONTINUUM-778-continuum-webapp.patch
>
>
> I see a lot of links constructed like this
>               <a 
> href="${pageContext.request.contextPath}/deleteUser!doDelete.action?accountId=${pageScope.user.accountId}&username=${pageScope.user.username}"><ww:text
>  name="delete"/></a>
> This will break under certain circustancies, like cookies disabled or 
> variables with some chars that need to be encoded
> Also jstl automatically adds the contextPath at the beggining
> See http://www-128.ibm.com/developerworks/java/library/j-jstl0318/#N105D3 for 
> more info
> The right way do do it is
>               <c:url var="deleteUserUrl" value="/deleteUser!doDelete.action">
>                 <c:param name="accountId" value="${user.accountId}"/>
>                 <c:param name="username" value="${user.username}"/>
>               </c:url>
>               <a href="<c:out value='${deleteUserUrl}'/>"><ww:text 
> name="delete"/></a>
> when no parameters need to be added it can just be
>               <a href="<c:url value="/deleteUser!doDelete.action">"><ww:text 
> name="delete"/></a>

-- 
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

        

Reply via email to