On Mon, February 14, 2005 12:09 pm, Joe Germuska said: > I don't really care about deprecating Action, but if the new > interface specifies "execute(ActionContext)", then Action's > "execute(ActionMapping,ActionForm,HttpServletRequest,HttpServletResponse)" > method should be deprecated, no? I suppose it's not as simple as > that, but it seems weird to me not to make the direction clear.
As a matter of direction, yes, it should be deprecated. But clearly you'd have the old-style execute() rolling it's own ActionContext and calling the new-style execute(), to maintain backwards compatibility. >>FWIW, I think Frank's idea of supporting POJO's as actions is the >>best. The JSF model of "action" is, IMO, the way to go: >> 1. It is a POJO that doesn't have to extend or implement anything >> 2. It can have properties for simple pages that don't need separate >> forms >> 3. It can be scoped so this silly "threadsafe" thing can go away, >>and scoping at a session level even opens up new interesting >>possibilities. > > I've been thinking about this too, and I have no objection -- but > would someone care to propose a config syntax for it? Would it be > yet another "overload" of the "type" attribute? (Fine with me; how > to specify which method gets called?) And how would the properties > be populated? (item 2) Perhaps the approach taken with my framework way back when... simply add an optional "method" attribute to the <action> element that specifies the method to call for that mapping. If it isn't present, you call the usual execute() (whichever style we're talking about) You mention how to determine what properties get populated... are you talking about ActionForms with that comment? I don't see how it applies to Actions... My bigger question is in my last post... if we make Actions POJOs, does that mean they can't be Commands? Are we looking at two mutually-exclusive possibilities, or do I just not understand something well enough here? > Joe Frank > -- > Joe Germuska > [EMAIL PROTECTED] > http://blog.germuska.com > "Narrow minds are weapons made for mass destruction" -The Ex > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > > --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
