[ 
https://issues.apache.org/struts/browse/STR-3009?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_40323
 ] 

Michael Jouravlev commented on STR-3009:
----------------------------------------

I think the original decision to use the setting defined in the struts config 
file has ambiguated  the ActionRedirect class name. As Neil points out, nothing 
can be done at this point, Struts is known for its backward compatibility. 

The only thing I can suggest is to disambiguate the class name and to create a 
marker ActionTransfer class, which will do nothing besides not implying a 
specific control transfer type (forward/redirect). ActionTransfer would be 
suggested for future use instead of ActionRedirect.

> ActionRedirect from ForwardConfig not redirecting properly?
> -----------------------------------------------------------
>
>                 Key: STR-3009
>                 URL: https://issues.apache.org/struts/browse/STR-3009
>             Project: Struts 1
>          Issue Type: Bug
>          Components: Core
>    Affects Versions: 1.3.5, 1.3.6
>            Reporter: Jason Millard
>
> While migrating from Struts 1.2.9 to Struts 1.3.5, I noticed ActionRedirect 
> was no longer redirecting as expected. 
> In the Struts 1.2.9, in 
>   public ActionRedirect(ForwardConfig baseConfig) 
> setRedirect was always being set to TRUE, like it is in every other 
> constructor.
> However, as of Struts 1.3.5 it is now being configured as
>  setRedirect(baseConfig.getRedirect());
> This seems to defeat the purpose of a ActionRedirect if you are trying to 
> generate one from a
>   mapping.findForward();

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to