DefaultHref.setFullUrl should not escape parameters using StringEscapeUtils.
----------------------------------------------------------------------------

                 Key: DISPL-404
                 URL: http://jira.codehaus.org/browse/DISPL-404
             Project: DisplayTag
          Issue Type: Bug
          Components: HTML Generation
    Affects Versions: 1.1
            Reporter: george talusan


I have custom cell renderers using displaytag.  The cell renderers make use of 
DefaultHref and PortletHref to generate URLs.  If I have a % in one of my URLs, 
DefaultHref won't escape it to %25 properly.  PortletHref makes use of 
PortletURL and in Pluto/Jetspeed this will encode the URL properly.

DefaultHref should not use StringEscapeUtils.escapeHtml.  It should instead use 
URLEncoder.encode().  The only place that displaytag, at first glance, uses 
these Hrefs is when it creates pagination/sorting links.  The code that 
generates the actual <a> tag should be where StringEscapeUtils.escapeHtml is 
used.

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

        

-------------------------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
displaytag-devel mailing list
displaytag-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/displaytag-devel

Reply via email to