Hi,

I think that on the BeanManagerImpl class there is (some) room for improvements.
Below is a patch to have a slightly "better" exception text.

Does one mind if I commit that ?

Thanks!
Matthias



Index: 
webbeans-impl/src/main/java/org/apache/webbeans/container/BeanManagerImpl.java
===================================================================
--- 
webbeans-impl/src/main/java/org/apache/webbeans/container/BeanManagerImpl.java  
    (revision
937508)
+++ 
webbeans-impl/src/main/java/org/apache/webbeans/container/BeanManagerImpl.java  
    (working
copy)
@@ -1008,7 +1008,7 @@
             {
                 if (!bean.getScope().equals(Dependent.class))
                 {
-                    throw new WebBeansConfigurationException("Bean "
+ bean + "scope can not define other scope except @Dependent to inject
InjectionPoint");
+                    throw new WebBeansConfigurationException("Bean "
+ bean.getBeanClass() + " scope can not define other scope except
@Dependent to inject InjectionPoint");
                 }
             }
         }


-- 
Matthias Wessendorf

blog: http://matthiaswessendorf.wordpress.com/
sessions: http://www.slideshare.net/mwessendorf
twitter: http://twitter.com/mwessendorf

Reply via email to