--- Steve Raeburn <[EMAIL PROTECTED]> wrote: > 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?
One thing that really bothers me about this action is that it forces the user to use "success" as the forward name. This isn't appropriate in every use case and Struts shouldn't dictate this type of thing. What about specifying the forward name in the "parameter" attribute? Maybe even say that "success" is the default name if parameter isn't provided? That gives you an action that just forwards to some ActionForward without demanding that the forward be named "success". This would also require a rename of SuccessAction to something more suitable (maybe ForwardingAction?). David > > 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] > __________________________________ Do you Yahoo!? Yahoo! SiteBuilder - Free, easy-to-use web site design software http://sitebuilder.yahoo.com --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
