wyona / yanel

http://www.yanel.org
Apache License 2.0
10 stars 5 forks source link

Yanel.getInstance().init() is still broken #6

Closed baszero closed 12 years ago

baszero commented 12 years ago

Hi Michael,

on March 22nd 2011 I sent an email to the Yanel distribution mail about this issue, you replied by saying that it should actually work and that this will get fixed, it seems this is still pending so I created this issue.

If you try to use Yanel standalone (outside of a servlet container), e.g. a Reindexer or alike, then the following code does not work:

Yanel yanel = Yanel.getInstance();
yanel.init();

Exception:


org.springframework.beans.factory.NoSuchBeanDefinitionException: No bean named 'map' is defined
    at org.springframework.beans.factory.support.DefaultListableBeanFactory.getBeanDefinition(DefaultListableBeanFactory.java:521)
    at org.springframework.beans.factory.support.AbstractBeanFactory.getMergedLocalBeanDefinition(AbstractBeanFactory.java:1068)
    at org.springframework.beans.factory.support.AbstractBeanFactory.doGetBean(AbstractBeanFactory.java:274)
    at org.springframework.beans.factory.support.AbstractBeanFactory.getBean(AbstractBeanFactory.java:190)
    at org.springframework.context.support.AbstractApplicationContext.getBean(AbstractApplicationContext.java:1075)
    at org.wyona.yanel.core.Yanel.init(Yanel.java:79)

If I discover some more infos on this, I'll post them in here.

michaelwechner commented 12 years ago

IIRC I actually fixed it, but you are right it does not work as expected. It seems that it does not find the library, which is set inside

src/build/targets/cmdl.xml

but I just realize that I have updated the version number and inside cmdl.xml it was hard-coded. I have it dynamic and now it should be fixed for good.

baszero commented 12 years ago

Great! Maybe you can add closes #6 into the appropriate git commit message, then you can directly jump to the code where it has been fixed... it's really cool if you do that :)

baszero commented 12 years ago

Sorry! Found it myself: 6824d3c5979b909f7339140d4794c0aa441f484f

baszero commented 12 years ago

reopen at #7