[cdi-dev] [JBoss JIRA] (CDI-456) fix Bean#getBeanClass() definition

Romain Manni-Bucau (JIRA) issues at jboss.org
Thu Sep 4 09:31:02 EDT 2014


    [ https://issues.jboss.org/browse/CDI-456?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12998899#comment-12998899 ] 

Romain Manni-Bucau commented on CDI-456:
----------------------------------------

Well in a war no big deal, in an ear the container does it since it deploys it. If you 100% want to do it from the CDI container "alone" you just can't. Then you are right you can use bean class but it doesn't cover 100% of cases (most of them for sure). It doesn't change the main point: getBeanClass is not designed for it IMHO, no? You think it is here for modules?

> fix Bean#getBeanClass() definition
> ----------------------------------
>
>                 Key: CDI-456
>                 URL: https://issues.jboss.org/browse/CDI-456
>             Project: CDI Specification Issues
>          Issue Type: Bug
>          Components: Beans
>            Reporter: Mark Struberg
>
> currently Bean#getBeanClass() is defined to return the class of the bean it produces but has one important exception: in case of a producer method or field it must return the class of the owner bean of this method or field.
> Imo this only causes troubles and doesn't add any benefit. 
> * At the time when 'using' the Bean (create and destroy) we always ONLY need the type which is to be created.
> * At the time we create interceptors we ONLY need the type which is to be created;
> * At the time we create the normalscoping proxies we ONLY need the type which is to be created;
> In fact the only time we need the ownerBean is when scanning the methods and fields in it. And for creating we really need the owner-Bean and not it's bean-class!
> In OWB we worked around this by having our own method getReturnType() which consistently returns the type which gets created.



--
This message was sent by Atlassian JIRA
(v6.3.1#6329)


More information about the cdi-dev mailing list