What subtype of ProcessBean should be fired for an extension?

[ljnelson]

I see that the CDI specification says in section 11.5, where "service provider" means a portable extension:

For each service provider, the container must provide a bean of scope @ApplicationScoped and qualifier @Default, supporting injection of a reference to the service provider instance. The bean types of this bean include the class of the service provider and all superclasses and interfaces.

I can observe that a ProcessBean event is fired when this bean is created.

But the javadocs on ProcessBean say:

The event object type depends upon what kind of bean was discovered:

• For a managed bean with bean class X, the container must raise an event of type ProcessManagedBean.
• For a session bean with bean class X, the container must raise an event of type ProcessSessionBean.
• For a producer method with method return type X of a bean with bean class T, the container must raise an event of type ProcessProducerMethod.
• For a producer field with field type X of a bean with bean class T, the container must raise an event of type ProcessProducerField.
• For a custom implementation of Bean, the container must raise an event of type ProcessSyntheticBean.

A bean representing a portable extension instance, provided by the container, would seem to fit none of these.

Using common sense, it seems like I should get a ProcessSyntheticBean event here, right?

[tremes]

Hi,

It's interesting question. On the other hand ProcessSyntheticBean  javadoc says:

The container fires an event of this type for each custom bean implementation added through
* {@link AfterBeanDiscovery#addBean()} or {@link AfterBeanDiscovery#addBean(Bean)}

ProcessSyntheticBean also has this getSource() method, which doesn't make sense for a bean representing a portable extension.      

[mkouba]

I agree with Tom. It makes sense to fire ProcessBean for all extension and built-in beans. ProcessSyntheticBean is intended for custom beans provided by extensions (hence the getSource() method).

[ljnelson]

Sure; that's fine, but extra specification/javadoc language about this should be added, right?

[tremes]

I am not sure to be honest. ProcessBean chapter quite explicitly states:

The container must fire an event for each bean, interceptor or decorator deployed in a bean archive

which applies also to built-in beans as well. Maybe we can improve javadoc. Feel free to create/suggest an issue at - JBoss Issue Tracker