Tim Fennell wrote:
I was going to chime in on this, so I may as well now. Stripes, through it's default way of routing events, will allow a submitted form or URL to invoke any method that is public no-arg and returns a Resolution - this seems much safer because Resolution is a Stripes specific class as opposed to String, which obviously isn't. In this case @HandlesEvent is only used to mark methods that have different return types, or need to be aliased.

I personally would like to see XWork enhanced to support action methods returning Result instances directly, however we would have to thoroughly think through the consequences.

Reasons for:
 - Better security
 - Easier, more explicit
- Results will probably be defined as annotations soon, so why not just return the Result directly?
 - Allows non-annotation apps to drastically minimize config
 - Supports more complex results

Reasons against:
 - The Action interface really loses most of its meaning
 - ActionSupport could no longer extend Action
 - XWork internals would need modification since many places expect a result 
code
- Supporting the return of Result, String, and Object might be confusing for new developers - Current Results would need some improvements to make them easier to use (better constructors, default location names/patterns, etc)

Again, all things considered, I'd really like to see us add the ability to return Results directly as I think it just makes things easier. The ability to separate JSP paths from Java code, IMO, is way overrated and years of its practice hasn't shown the additional complexity is really worth it. For those that prefer the old style, they would still have that option.

Don


-t

On Jul 25, 2006, at 4:11 PM, Ted Husted wrote:

On 7/25/06, Jason Carreira <[EMAIL PROTECTED]> wrote:
@Action to mark it as an Action method?

Stripes uses the annontation @DefaultHandler where we would configure
a default action, which might imply the annotation @Handler for what
we are talking about here.

* http://stripes.mc4j.org/confluence/display/stripes/Annotation+Reference

-Ted.

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




---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to