Upload observing instead of intercepting #256
Conversation
ghost
assigned 
|
|
||
| @Override | ||
| public void intercept(InterceptorStack stack, ControllerMethod method, Object controllerInstance) { | ||
| public void upload(@Observes ControllerMethod controllerMethod) { |
There was a problem hiding this comment.
Is this the correct event? wasn't something like ReadyToExecute?
There was a problem hiding this comment.
we need run it before ParameterInstantiatorInterceptor, ReadyToExecute is fired after
There was a problem hiding this comment.
Ok.. where is it being fired? what else is observing this event?
ControllerMethod is not an event, it's a product of one. Maybe we should wrap it in another class...
Just maybe ;)
There was a problem hiding this comment.
Ok.. where is it being fired? what else is observing this event?
fired by ControllerLookupInterceptor,
observed by ControllerMethodFactory, NullMultipart and CommonsUploadMultipart
Maybe we should wrap it in another class...
I think about that before using ControllerMethod directly, do you have a good name for this new event?
There was a problem hiding this comment.
something like ControllerMethodDiscovered
Upload observing instead of intercepting
#247