-
1. Re: ConversationImpl.toString thrwos Exception
andygibson.contact.andygibson.net Jul 16, 2010 11:34 PM (in response to pandora)By default, Weld conversations work out of the box with JSF as it uses the phase listeners but you need some additional config to get conversations working in wicket (basically starting and propagating/ending the conversation over the span of the wicket request). Here it looks like you haven't set up the conversation context at the start of the request.
You can find out more info in the Wicket CDI Integration docs
Cheers,
Andy Gibson
-
2. Re: ConversationImpl.toString thrwos Exception
pandora Jul 17, 2010 4:13 AM (in response to pandora)thanks for that.
Still, don't you consider it a bug that a RuntimeException gets thrown? The action that led to it was actually the weld TRACE level message that said that there was no active conversation context. So weld creates a TRACE log message that leads to a RuntimeException because of toString(). To me that does not seem right. Apart from that, several IDEs use the toString method as well to display object information for debugging, so a toString method throwing Runtime Exceptions just should be avoided IMO.
-
3. Re: ConversationImpl.toString thrwos Exception
andygibson.contact.andygibson.net Jul 17, 2010 8:18 PM (in response to pandora)Oh yeah, I still think its a bug that an exception gets thrown, I was just pointing out that it sounded like your app may not be configured properly. I agree that a toString() shouldn't throw exceptions like that.
Cheers,
Andy Gibson
-
4. Re: ConversationImpl.toString thrwos Exception
swd847 Jul 18, 2010 2:46 AM (in response to pandora)It looks like this is fixed in trunk