:polemico: Simplifying Events

[lucascs]

• NewRequest now holds the corresponding RequestInfo
• ControllerMethod turned into an event, so we can remove ControllerMethodDiscovered. @Observes ControllerMethod reads just fine, IMHO.
• MethodInfo now observes changes in ControllerMethod and @Produces it, so we can remove ControllerMethodFactory.

I tried to remove NewRequest also, but there is some weird dependency.
WANT: remove CDIProvider.

What do you think?

[Turini]

I would change the field to requestInfo (and getRequestInfo, sure =)
Just to be clear that its not the real request. What do you think?

[Turini]

besides the minor on field name, everthink seems fair enough to me. 👍

[Turini]

I tried to remove NewRequest also, but there is some weird dependency.

which dependency?

[garcia-jj]

:thanks_god: I love it

[lucascs]

@Turini I don't know which dependency, I get a StackOverflow if RequestObserverHandler receives a @Observes RequestInfo

[Turini]

Turini I don't know which dependency, I get a StackOverflow if RequestObserverHandler
receives a @observes RequestInfo

I'm having a different problem changing it, because CDIRequestInfoFactory observes
RequestInfo too, so the execution order breaks the code. Did you face this problem?

[lucascs]

I'll try again tonight and see if I come up with some good solution.