I've added some tests and made the code use less reflection. On Tue, Feb 3, 2009 at 1:03 AM, James Gregory <[email protected]>wrote:
> I've added it to my todo list, but it's up for grabs if anyone wants to > tackle it before I do. > > > On Mon, Feb 2, 2009 at 2:02 PM, James Gregory <[email protected]>wrote: > >> Indeed we don't, it concerns me greatly. I should've wrote a test with >> this fix really, but I was more concerned with just getting it out. >> Ideally I'd like to get the automapper to not have to rely on reflection >> at all (possibly some kind of interface might help), but at the very least >> we need to make a test that'll break if the method changes. >> >> >> On Mon, Feb 2, 2009 at 1:36 PM, Paul Batum <[email protected]> wrote: >> >>> Hmm, so I am guessing we have no unit tests for that code? With all that >>> reflection, it seems pretty fragile. >>> >>> >>> On Mon, Feb 2, 2009 at 6:58 AM, James Gregory >>> <[email protected]>wrote: >>> >>>> Ok, I've committed a fix for this. Let me know if anything else crops >>>> up. >>>> >>>> On Sun, Feb 1, 2009 at 7:52 PM, James Gregory >>>> <[email protected]>wrote: >>>> >>>>> Damn! Good catch, I'll look at this ASAP. >>>>> >>>>> >>>>> On Sun, Feb 1, 2009 at 7:42 PM, Duarte Nunes <[email protected]>wrote: >>>>> >>>>>> >>>>>> Hi, >>>>>> >>>>>> I was trying to use the auto mapping feature, but was getting a >>>>>> NullPointerException when the auto mapper was dealing with a many-to- >>>>>> many relationship. Tracking the problem, the exception was being >>>>>> thrown at ManyToManyAutoMapper.Map<T>, on the line: type.GetMethod >>>>>> ("IsInverse").Invoke(manyToManyPart, new object[0]). Since the source >>>>>> contains only methods named "Inverse", I believe this might be a bug. >>>>>> >>>>>> Best regards, >>>>>> Duarte >>>>>> >>>>>> >>>>>> >>>>> >>>> >>>> >>>> >>> >>> >>> >> > > > > --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Fluent NHibernate" group. To post to this group, send email to [email protected] To unsubscribe from this group, send email to [email protected] For more options, visit this group at http://groups.google.com/group/fluent-nhibernate?hl=en -~----------~----~----~----~------~----~------~--~---
