sure go on and commit :) txs and LieGrue, strub
----- Ursprüngliche Mail ---- > Von: Matthias Wessendorf <[email protected]> > An: [email protected] > Gesendet: Freitag, den 23. April 2010, 22:36:35 Uhr > Betreff: BeanManagerImpl -> "improvement" on WebBeansConfigurationException ? > > 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: > href="http://matthiaswessendorf.wordpress.com/"; target=_blank > >http://matthiaswessendorf.wordpress.com/ sessions: > href="http://www.slideshare.net/mwessendorf"; target=_blank > >http://www.slideshare.net/mwessendorf twitter: > href="http://twitter.com/mwessendorf"; target=_blank > >http://twitter.com/mwessendorf
