[jira] [Commented] (WW-3965) Double bang on form action

["Lukasz Lenart (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/WW-3965?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13555928#comment-13555928 ] 

Lukasz Lenart commented on WW-3965:
-----------------------------------

[~gregh99] I don't understand how this issue is related to WW-1967?
                
> Double bang on form action
> --------------------------
>
>                 Key: WW-3965
>                 URL: https://issues.apache.org/jira/browse/WW-3965
>             Project: Struts 2
>          Issue Type: Bug
>          Components: Core Actions
>    Affects Versions: 2.3.8
>         Environment: Centos Apache/tomcat with mod_jk
>            Reporter: Greg Huber
>            Priority: Minor
>
> Hello,
> I sometimes use the dynamic method invocation on the form action for submitting and have noticed for a while now, that there is a double bang on the compiled page: 
> eg. If I use action="license!member" on the compiled page I get
> action="/events/license!member!member.action"
> on the jsp:
> <s:form action="license!member" >
> ..
> </s:form>
> gives:
> <form method="post" action="/events/license!member!member.action" id="license!member" name="license!member">
> ''
> </form>
> It only seems to be on the form tag though.
> Have traced it to DefaultActionMapper, line 539.  It needs to check to make sure there is no bang already before adding another one, ie follow existing logic in other methods.
> {code:java}
> if (null != mapping.getMethod() && !"".equals(mapping.getMethod())) {
>   uri.append("!").append(mapping.getMethod());
> }
> {code}
> This seems to have fixed it:
> {code:java}
> //if (null != mapping.getMethod() && !"".equals(mapping.getMethod())) {
> //uri.append("!").append(mapping.getMethod());
> //}
> if (allowDynamicMethodCalls) {
> if (null != mapping.getMethod() && !"".equals(mapping.getMethod())) {
> // handle "name!method" convention.
> name = mapping.getName();
> if (name.indexOf("!") == -1) {
> // Append the method as no bang found
> uri.append("!").append(mapping.getMethod());
> }
> }
> } else {
> if (null != mapping.getMethod() && !"".equals(mapping.getMethod())) {
> uri.append("!").append(mapping.getMethod());
> }
> }
> {code}
> I would do a patch but I have already modified the DefaultActionMappe for WW-1967 which has still hopefully outstanding.
> Cheers Greg.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira