What you are doing will not work - at least, not the way you are doing it.
You are trying to modify the ActionForward instance that is owned by Struts,
and calling setPath() on that instance will result in an
IllegalStateException.
You need to create your own ActionForward instance, instead of trying to
modify Struts' one. You can do that with something like this:
ActionForward goto = mapping.findForward( "success" );
String path = ...; // Put together your new path
ActionForward myGoto = new ActionForward(path, goto.getRedirect());
return myGoto;
--
Martin Cooper
"Glanville, Jay" <[EMAIL PROTECTED]> wrote in message
news:[EMAIL PROTECTED]
I'm trying to solve a problem, but I'm not sure my solution is the best
way. Basically, I want to set a parameter on a forward within the
action's execute.
I'm in my action's execute method. I've just successfully performed
some work, and I now want to forward/redirect to the next page. So,
I've got some code that looks like this:
public ActionForward execute(ActionMapping.....) {
// do some work
ActionForward goto = mapping.findForward( "success" );
return goto;
}
With an action mapping that looks like this:
<action
path="/EntrySave"
type="com.package.EntrySaveAction"
name="EntryForm"
validate="true"
input="/Entry.do">
<forward name="success" redirect="true" path="/Container.do"/>
<forward name="failure" redirect="false" path="/Entry.do" />
</action>
Basically, if I left the EntrySaveAction.execute do what's doing right
now, then upon successful completion, it would redirect to
"/Container.do". However, what I want it to do is redirect to
"/Container.do?id=45", where 45 is the id of the container that I want
to go to. The value of "45" is dynamic, and is know at the time of the
EntrySaveAction.execute command.
The way I'm currently doing it is something like this:
public ActionForward execute(ActionMapping.....) {
// do some work
ActionForward goto = mapping.findForward( "success" );
String path = goto.getPath();
path += "?id=" + container.getId();
goto.setPath( path );
return goto;
}
Is this the correct way to do this? Is there a better way?
Thanks in advance
JDG
--
Jay Glanville
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
