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

Jason Millard commented on STR-3009:
------------------------------------

> As Neil points out, nothing can be done at this point, Struts is known for 
> its backward compatibility.

If the ActionRedirect constructors do not default redirect to true, than in 
this case, Struts is not backwards compatible.

I have existing code that no longer works unless I manually set redirect to 
true.

> 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