Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implicit bean archive support [PAXCDI-186] #131

Open
ops4j-issues opened this issue May 27, 2015 · 2 comments
Open

Implicit bean archive support [PAXCDI-186] #131

ops4j-issues opened this issue May 27, 2015 · 2 comments
Labels
Milestone

Comments

@ops4j-issues
Copy link

Jozef Hartinger created PAXCDI-186

It seems that Pax CDI still supports the CDI 1.0 explicit bean archives only. Implicit bean archive support was added in CDI 1.1. An implicit bean archive is an archive that either:

  • has beans.xml file with bean-discovery-mode="annotated" attribute. In such archive only classes annotated with a bean defining annotation (scope, stereotype, etc.) are recognized as beans. Other classes are not recognized as CDI beans. During Weld initialization the list of such classes should be sent to Weld as bean classes using http://docs.jboss.org/weld/javadoc/3.0/weld-spi/org/jboss/weld/bootstrap/spi/BeanDeploymentArchive.html#getBeanClasses--
  • or does not have beans.xml file at all - if classes with bean defining annotations are present this archive is treated as if it had beans.xml with bean-discovery-mode="annotated"

http://lists.jboss.org/pipermail/weld-dev/2015-May/003336.html


Fixed in: 1.1.4
Votes: 1, Watches: 4

@ops4j-issues
Copy link
Author

Harald Wellmann commented

This is CDI 1.2 and Weld 2.2.x. The Javadoc for this version says

Gets all classes in the bean deployment archive.

http://docs.jboss.org/weld/javadoc/2.2/weld-spi/org/jboss/weld/bootstrap/spi/BeanDeploymentArchive.html#getBeanClasses

Pax CDI delivers all classes, just filtered by OSGi visibility rules.

I don't quite see the point in the 3.0 version - doesn't Weld have to inspect the scope annotations on (candidate) bean classes anyway, so what's the point of expecting the environment to do it?

@ops4j-issues
Copy link
Author

Antonin Stefanutti commented

Note that the current state of the development seems to cause PAXCDI-212 .

@ops4j-issues ops4j-issues added the type: new feature New Feature label Feb 24, 2021
@ops4j-issues ops4j-issues added this to the 1.1.4 milestone Feb 24, 2021
@jbonofre jbonofre modified the milestones: 1.1.4, 1.1.5 Mar 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants