The patch solves an new issue that was introduced by the fix for 2011,
it was so related that I did not feel that I needed to create a new
issue. But I think that was a mistake since the 2011 issue was marked
as resolved and merged to release 2.1.2.

Anyway I have created a new issue for this one 2061, with a patch that
contains a failing test as well as a patch that solves the issue.

Sorry for the confusion, I should have created a new issue to begin
with :)

http://nhjira.koah.net/browse/NH-2061

Regards
Torkel

On Jan 5, 2:53 pm, Fabio Maulo <[email protected]> wrote:
> Torkel...
> If NH-2011 was marked with "resolved" is because it has a test and a patch
> to pass the test.
> What solves the new patch?
>
> A new issue or exactly the same ?
> If is for exactly the same issue why we should have 2 patchs ?
>
> 2010/1/5 Torkel Ödegaard <[email protected]>
>
>
>
>
>
> > Ahh, sorry I linked to the wrong issue :)
>
> > I did attach the patch to the correct issue though :)
>
> >http://nhjira.koah.net/browse/NH-2011
>
> > On Jan 5, 1:20 pm, Fabio Maulo <[email protected]> wrote:
> > > I don't understand.
> > > NH-2009 is open.
> > > NH-2011 is resolved because it pass the test you have sent.
>
> > > The patch you have sent here is for NH-2009, for NH-2011 or is for
> > another
> > > issue ?
> > > If the issue is not exactly the same of NH-2011 please open a new one.
> > > Thanks.
>
> > > 2010/1/5 Torkel Ödegaard <[email protected]>
>
> > > > Yes, will do, sorry :)
>
> > > > The issue was closed so was afraid that a new patch would not be
> > > > noticed.
>
> > > > The issue now contains the new patch.
> > > >http://nhjira.koah.net/browse/NH-2009
>
> > > > Cheers
> > > > Torkel
>
> > > > On Jan 5, 6:27 am, Fabio Maulo <[email protected]> wrote:
> > > > > Please don't use the dev-list for these things; please attach the
> > patch
> > > > > directly to your issue.
> > > > > Thanks.
>
> > > > > 2010/1/4 Torkel Ödegaard <[email protected]>
>
> > > > > > Hi,
>
> > > > > > I discovered a little problem with my patch for issue NH 2011, it
> > > > occurs
> > > > > > when merging a transient entity with a null component.
>
> > > > > > I have attached a patch fix.
>
> > > > > > Cheers
> > > > > > Torkel
>
> > > > > --
> > > > > Fabio Maulo
>
> > > --
> > > Fabio Maulo
>
> --
> Fabio Maulo

Reply via email to