[
https://issues.apache.org/struts/browse/STR-3009?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_40336
]
Hubert Rabago commented on STR-3009:
------------------------------------
> What do you think about new
> feature of ActionMapping described above? We can even pass
> redirect flag. Basically, if ActionMapping.findForward is
> called with name only it will return a frozen instance of
> ActionForward, otherwise it will create a new instance on the
> fly, with optional parameters and redirect flag. The URL
> parameters can be sent as a separate parameter object, this
> code can pulled out of ActionRedirect class.
I think that's even farther from ActionMapping's purpose than forming a URL is
from ActionRedirect's, so I'm -1 on that.
> 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.
