Interceptor discovery

2017-07-21 Thread John D. Ament
I have an interceptor defined as: @Interceptor @LoggedIn @Priority(Interceptor.Priority.APPLICATION + 100) public class LoggedInInterceptor { and a beans.xml with This interceptor isn't being discovered. It is discovered when beans.xml is an empty file. This same interceptor is discovered on

Re: ProcessBean being called regardless of type

2017-07-21 Thread John D. Ament
Yes sorry, pointed to the wrong error. This looks much better now, thanks! On Wed, Jul 19, 2017 at 3:41 AM Romain Manni-Bucau wrote: > PS: created https://issues.apache.org/jira/browse/OWB-1203 > > > Romain Manni-Bucau > @rmannibucau | Blog >

OWB 2 not firing ProcessAnnotatedType for all bean classes

2017-07-21 Thread John D. Ament
I do something really lazy, I have an extension that has this method on it: public void findEntities(@Observes @WithAnnotations(Entity.class) ProcessAnnotatedType pat) which just looks for entity classes. They're not going to be CDI beans, but they are annotated types. Per the spec, https://doc

Re: OWB 2 not firing ProcessAnnotatedType for all bean classes

2017-07-21 Thread Romain Manni-Bucau
Hmm, interesting edge case. For me it should be ignored until you make it scanned using @Dependent or so. But fear it is quite undefined or "interpretable" Le 21 juil. 2017 21:10, "John D. Ament" a écrit : > I do something really lazy, I have an extension that has this method on it: > > public v

Re: Interceptor discovery

2017-07-21 Thread Romain Manni-Bucau
Misses @Dependent yes. Le 21 juil. 2017 19:35, "John D. Ament" a écrit : I have an interceptor defined as: @Interceptor @LoggedIn @Priority(Interceptor.Priority.APPLICATION + 100) public class LoggedInInterceptor { and a beans.xml with This interceptor isn't being discovered. It is discove

Re: OWB 2 not firing ProcessAnnotatedType for all bean classes

2017-07-21 Thread John D. Ament
Errr I'm not sure what you mean. The spec states this "before it reads the declared annotations" so I'm not sure why you think it needs to have a bean defining annotation. John On Fri, Jul 21, 2017 at 3:21 PM Romain Manni-Bucau wrote: > Hmm, interesting edge case. For me it should be ignored u

Re: Interceptor discovery

2017-07-21 Thread John D. Ament
The interceptor is intended to be enabled just because of the @Priority annotation. Nothing else should be required. Please review 9.4 of the spec http://docs.jboss.org/cdi/spec/2.0/cdi-spec.html#enabled_interceptors John On Fri, Jul 21, 2017 at 3:22 PM Romain Manni-Bucau wrote: > Misses @Dep

Re: OWB 2 not firing ProcessAnnotatedType for all bean classes

2017-07-21 Thread Romain Manni-Bucau
Cause the scanning by itself is undefined, you can not scan skip not bean types and be spec compliant. Le 21 juil. 2017 21:26, "John D. Ament" a écrit : > Errr I'm not sure what you mean. The spec states this "before it reads the > declared annotations" so I'm not sure why you think it needs t

Re: Interceptor discovery

2017-07-21 Thread Romain Manni-Bucau
Conflicts with annotated part i think. Priority doesnt enable a bean by itself. Le 21 juil. 2017 21:31, "John D. Ament" a écrit : > The interceptor is intended to be enabled just because of the @Priority > annotation. Nothing else should be required. Please review 9.4 of the > spec http://docs