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

Romain Manni-Bucau (JIRA) issues at jboss.org
Thu Sep 4 08:53:01 EDT 2014


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

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

Classloading isolation can be restricted but it doesn't mean you don't inherit from other beans (typically in OSGi you can get proxies to solve it).

In EE that's the same, nothing prevents it if you *custom* beans handle it correctly. In EE that's worse since the deployment is pretty clear (=hierarchic) so no need to test Class to know what should be done.



> 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