search

xpoft / spring-vaadin

Spring and Vaadin integration
Other
55 stars 44 forks source link

Resolution of abstract beans fails #4

Closed codeCoder closed 11 years ago

codeCoder commented 11 years ago

Hi,

I use your add-on and found a problem(for me) in DiscoveryNavigator. For example: We have a beanName: "mybean" def. as ...bean id="myBean" abstract="true" class="..." The line Class beanClass = SpringApplicationContext.getApplicationContext().getType(beanName) in method initView() the context returns null as beanClass which lead to a NullPointerException in the following if-Statement.

I think null comes because of abstract nature of the bean. I think You can fix this if you check the beanClass variable against null What do you think ?

mpilone commented 11 years ago

I sent a pull request for this issue with a patch. You can grab my patched version (1.6.6-abstractbeans-patch) to get around the issue in the interm. Hopefully the fixes will be merged into master soon.

xpoft commented 11 years ago

Thx. I've updated the add-on.