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

Luca Boncompagni commented on STR-3029:
---------------------------------------

Yes it worked.

Thanks,
Luca



> ActionConfigMatcher does not instantiate the correct ActionForward
> ------------------------------------------------------------------
>
>                 Key: STR-3029
>                 URL: https://issues.apache.org/struts/browse/STR-3029
>             Project: Struts 1
>          Issue Type: Bug
>    Affects Versions: 1.2.9
>         Environment: Solaris WebSphere 5.1.1
>            Reporter: Luca Boncompagni
>             Fix For: 1.3.9, 1.4.0
>
>
> If you create an action config like this:
>       <action    path="/mf/**"
>                  type="it.test.pfwk.actions.MultiFrameAction">
>               <forward name="success" 
> className="it.test.framework.actions.WsActionForward" path="/luca.jsp"/>
>       </action>
> When you do a mapping.findForward() you get an 
> org.apache.struts.action.ActionForward instead of 
> it.test.framework.actions.WsActionForward.
> I think that the problem is in ActionconfigMatcher, convertActionConfig():
>         ForwardConfig[] fConfigs = orig.findForwardConfigs();
>         ForwardConfig cfg;
>         for (int x = 0; x < fConfigs.length; x++) {
>             cfg = new ActionForward();
>             cfg.setContextRelative(fConfigs[x].getContextRelative());
> Here you should instatiate the correct actionForrward, for instance, if you 
> modifiy ActionForwrad in order to have a method like newInstance():
>         ForwardConfig[] fConfigs = orig.findForwardConfigs();
>         ForwardConfig cfg;
>         for (int x = 0; x < fConfigs.length; x++) {
>             cfg = fConfigs[x].newInstnce();
>             cfg.setContextRelative(fConfigs[x].getContextRelative());

-- 
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