[ https://issues.apache.org/jira/browse/DELTASPIKE-56?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13190799#comment-13190799 ]
Gerhard Petracek commented on DELTASPIKE-56: -------------------------------------------- +1 >!< for moving this discussion to the mailing list. > Add BeanManagerProvider.getRequiredBeanManager() method > ------------------------------------------------------- > > Key: DELTASPIKE-56 > URL: https://issues.apache.org/jira/browse/DELTASPIKE-56 > Project: DeltaSpike > Issue Type: Improvement > Components: Core > Reporter: Christian Kaltepoth > Assignee: Gerhard Petracek > Priority: Minor > Attachments: DELTASPIKE-56.patch > > > While reviewing the implementation of BeanManagerProvider I saw that > getBeanManager() may return null in some rare circumstances. Unfortunately > this forces everyone calling this method to check the result for null. I > think most code calling the method absolutely requires the BeanManager and > cannot proceed without it. > Therefore it would be nice to have some alternative getBeanManager() method > that doesn't return null if the BeanManager is not available but instead > throws a meaningful runtime exception. That's what Solder does per default. > Calling Solder's BeanManagerLocator.getBeanManager() without a BeanManager > being available will result in a BeanManagerUnavailableException. > The attached patch adds a method getRequiredBeanManager() to the > BeanManagerProvider implementing this behavior. > I'm not sure if this minor thing should be discussed on the mailing list. If > you think this should be discussed, please let me know, then I'll start a > topic on deltaspike-dev. -- 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