Jason Carreira wrote: <snip>
I would like to undo this, pull the implementation stuff up into
ActionSupport, get rid of the workflow and BaseActionSupport, and create
a DefaultWorkflowInterceptor which will call validate() on the Action if
it implements an Interface, then check for errors like above and return
INPUT without forwarding to the rest of the interceptor stack if it has
errors, or forward to the rest of the interceptor stack if it does....
Probably you'd want to put this as the last interceptor in your stack,
so forwarding would just call the Action.
I know I've talked with Pat about this, and we both hated the default behavior that would return for you without calling your action, but at least this way it would be optional (apply the interceptor or not) and you'd be making a conscious decision, so you'd know what was happening.
I've been bitten by the default behavior way too many times in WW 1.3 - I'd love for it to be optional but still 'default' (meaning perhaps one of a few alternative recommendations for validation). The current way (in WW 1.3) of working around the default behavior of either overriding validate() in your own base class which extends ActionSupport or not subclassing from ActionSupport (or I guess just changing the ActionSupport class outright) is in IHMO less than satisfactory.
Regards,
Bruce Ritchie
smime.p7s
Description: S/MIME Cryptographic Signature
