I think reset is really only needed for checkboxes. Why would you do database lookups in that method? My proposal is focused on allowing RequestProcessor implementations to report errors when handling ActionForwards.

David



From: "Taylor, Jason" <[EMAIL PROTECTED]>
Reply-To: "Struts Developers List" <[EMAIL PROTECTED]>
To: "'Struts Developers List'" <[EMAIL PROTECTED]>
Subject: RE: [PROPOSAL] Add Exception Handling to ActionForwards
Date: Thu, 20 Feb 2003 14:47:09 -0800

I remember avoiding reset() because any exceptions caused by DB interaction
couldn't be handled gracefully-- i.e., the exceptions would be thrown prior
to the execute() method.

This case seems to be another of the type addressed by this proposal-- am I
right?

> -----Original Message-----
> From: David Graham [mailto:[EMAIL PROTECTED]]
> Sent: Thursday, February 20, 2003 12:30 PM
> To: [EMAIL PROTECTED]
> Subject: [PROPOSAL] Add Exception Handling to ActionForwards
>
>
> I'll start with a specific situation:
> Tiles allows you to configure a Controller class on a tile that gets
> executed whenever that tile gets used.  This is most commonly used to
> prepare data for the tile from some datasource.  However, the
> declarative
> exception handling feature does not apply to this, rendering
> this Tiles
> feature fairly useless.
>
> The general feature idea:
> The problem is that the RequestProcessor only invokes the
> exception handling
> mechanism on an Action's execute() method.  If we extended
> this idea to
> handling exceptions from ActionForwards we could allow
> implementations to
> report errors that occurred while forwarding and allow
> applications to
> recover gracefully.  As it is, I think a ServletException get
> propagated up
> to the servlet.
>
> I believe the changes would include changing
> RequestProcessor.processForwardConfig() throws IOException,
> ServletException
>
> to
>
> RequestProcessor.processForwardConfig() throws Exception
>
> and the 2 places that processForwardConfig() is called in
> RequestProcessor
> would catch the Exception and pass it to processException().
>
> Thoughts?
> David
>
>
>
>
>
>
>
> _________________________________________________________________
> MSN 8 helps eliminate e-mail viruses. Get 2 months FREE*.
> http://join.msn.com/?page=features/virus
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]
>

_________________________________________________________________
Tired of spam? Get advanced junk mail protection with MSN 8. http://join.msn.com/?page=features/junkmail


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

Reply via email to