P.S., Paul, I'd suggest going ahead and opening a ticket for this and reference this thread... say that a couple of different solutions were suggested and patches can be easily created once a consensus on the right answer is reached (I know you said you'd create a patch, and I would too if necessary once we can all come together to decide which way to go). This will put it more officially in front of the committers for their consideration, since ultimately they are going to have to decide if they agree this is an issue or not, and then agree with how to resolve it. The first step in all that is generally a ticket being opened. This is your baby, so go for it! :)

Frank

Paul Benedict wrote:
If everyone used dispatch-type Actions, I would disagree because then it would 
just be a matter
of providing a cancel() method and making sure that got called.
There's actually a funny bug here. cancel() method ALWAYS gets called in a 
dispatch action BUT
it's default behavior is to return null; and if null is returned then it 
considers it
non-implemented and goes to the intended method.

That's actually a bug too because NULL is a legitmate return value from a 
struts action. If you
handled the request directly, say by writing directly to the servlet 
OutputStream or
HttpServletResponse.sendRedirect(), you're supposed to retun null. That's what 
the docs say =o)
But in this case NULL means "not implemented".

Anyway, it must be the weekend because no one from the Struts team have chimed 
in -- but they
deserve days off too. :) .... maybe ;) I'd like to actually have a solution to 
submit; I am going
to implement my suggestion privately in my applications but I would like an 
"official" one get
into Struts too.

Paul


__________________________________________________
Do You Yahoo!?
Tired of spam? Yahoo! Mail has the best spam protection around http://mail.yahoo.com
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



.


--
Frank W. Zammetti
Founder and Chief Software Architect
Omnytex Technologies
http://www.omnytex.com
AIM: fzammetti
Yahoo: fzammetti
MSN: [EMAIL PROTECTED]

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

Reply via email to