-------- Forwarded Message -------- From: simon <[EMAIL PROTECTED]> To: MyFaces Development <[email protected]> Subject: Re: [orchestra] ConnectionManagerDataSource.isWrapperFor Date: Sat, 18 Aug 2007 14:08:24 +0200
On Sat, 2007-08-18 at 13:55 +0200, Mario Ivankovits wrote: > Hi! > > In ConnectionManagerDataSource, method isWrapperFor currently returns > > false. Perhaps it should throw UnsupportedOperationException instead? > And what about a solution using reflection? > Maybe the most correct solution could be: Yes, I agree that using reflection is the 100% correct solution. However it's also the most complicated. When we discussed this earlier, I think we agreed we didn't really need to do this. > What do you think? > If you don't like it, simply change the "return false" to the Exception > you proposed. I'll let you decide what's best on this. Cheers, Simon
