I haven't suggested another way of specifying a forward. Just providing an action that uses the existing, most common way of doing so.
I wouldn't want to add contextRelative attribute to <action> because: - it doesn't relate to the ActionMapping, but to the ActionForward - having the same attribute specified in two places would cause confusion Although, SuccessAction is a trivial action, it does fill a gap that is not fully met by either <action forward=""> or ForwardAction. How would adding it be bad for Struts? Steve > -----Original Message----- > From: David Graham [mailto:[EMAIL PROTECTED] > Sent: August 4, 2003 1:32 PM > To: Struts Developers List > Subject: RE: Addition of two new actions > > > --- Steve Raeburn <[EMAIL PROTECTED]> wrote: > > You're right <action forward=""> is module relative (despite what it > > says in > > the Javadoc). However, I don't see how it can used with a context > > relative > > path. > > Maybe it needs a contextRelative attribute. > > > > > There is no contextRelative attribute on the action config so you don't > > get > > to choose (or change) how your forward works. > > Maybe there should be. Isn't enhancing existing functionality to make it > better simpler than adding yet another way of specifying a forward? > > > > > Of the three ways of defining forwards that you identified only one, > > <forward>, works flexibly with modules and that's the one that we don't > > ship > > an action for with Struts. > > > > SuccessAction works flexibly with modules, it's tool friendly, it's > > simple > > to understand and its configuration is consistent with almost all other > > actions. > > It may be simple to understand when considered alone but in the context of > the variety of other methods of defining forwards it makes things more > complex. Maybe we need to revisit the concept of forwards and/or how we > define them. > > David > > > > > > Steve --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
