[ 
https://issues.apache.org/jira/browse/DELTASPIKE-191?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13293423#comment-13293423
 ] 

Mark Struberg commented on DELTASPIKE-191:
------------------------------------------

Yes, we need to make users aware. Actually any @Inject besides injecting the 
BeanManager itself during CDI container bootstrap is not supported by the CDI 
spec and will cause non-portable behaviour. In some cases/containers you will 
even get an Exception if you try to @Inject something else during boot or if 
you call BeanManager#getReference() in that time.

I'll add a flag 'booted' and throw an IllegalStateException if a user tries to 
invoke BeanManagerProvider.getInstance().getBeanManager() before @Observes 
AferDeploymentValidation. 
                
> get rid of BeanManagerProvider#setTestMode()
> --------------------------------------------
>
>                 Key: DELTASPIKE-191
>                 URL: https://issues.apache.org/jira/browse/DELTASPIKE-191
>             Project: DeltaSpike
>          Issue Type: Bug
>          Components: Core
>    Affects Versions: 0.2-incubating
>            Reporter: Mark Struberg
>            Assignee: Mark Struberg
>             Fix For: 0.3-incubating
>
>
> this is an absolute antipattern and we must remove it asap!
> We also need to review all the RootBeanManagerProvider pattern. This seems 
> like huge tons of code just to provide a workaround for a broken container.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to