DO NOT REPLY [Bug 37817] New: - TagUtils doesn't create XHTML compliant URLs when called from LinkTag when using forwards defined with redirect="true"

[bu...@apache.org]

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG�
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=37817>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND�
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=37817

           Summary: TagUtils doesn't create XHTML compliant URLs when called
                    from LinkTag when using forwards defined with
                    redirect="true"
           Product: Struts
           Version: 1.2.8
          Platform: Other
        OS/Version: other
            Status: NEW
          Severity: normal
          Priority: P2
         Component: Unknown
        AssignedTo: dev@struts.apache.org
        ReportedBy: jason.lunn@gmail.com


html:link tags using the "forward" attribute to reference an ActionForward where the redirect attribute is 
true do not render XHTML compliant URLs in all cases. The problem manifests when using a Map to 
include multiple parameters because the ampersand in the URL is not properly escaped. This is because 
the separator variable in TagUtils.computeURLWithCharEncoding() is defined after the value of the function 
argument "redirect" is overridden by the value of of forwardConfig.getRedirect()

By simply moving the small block of code that determines the value of "separator" to be before the block 
of code that inspects the state of forwardConfig, XHTML compliant URLs can be rendered by the html:link 
tag without negatively impacting the logic:redirect tags need to force unescaped ampersands.

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@struts.apache.org
For additional commands, e-mail: dev-help@struts.apache.org