Mailing-List: contact users-help@isis.apache.org; run by ezmlm
Precedence: bulk
Reply-To: users@isis.apache.org
Received-SPF: error (athena.apache.org: local policy)
MIME-Version: 1.0
In-Reply-To: 
 <C86DA4A0A6AAD1409A222F3A507B21F82BA7921A@e2k1002.infusionsoft.phx>
References: 
 <C86DA4A0A6AAD1409A222F3A507B21F82BA790E6@e2k1002.infusionsoft.phx>
 <CALJOYLExT1_BuHziUVLWfGM1GBmxvfmvR0zKA_uVkpvsh4NbzQ@mail.gmail.com>
 <C86DA4A0A6AAD1409A222F3A507B21F82BA7921A@e2k1002.infusionsoft.phx>
From: Dan Haywood <dan@haywood-associates.co.uk>
Date: Thu, 12 Sep 2013 13:55:09 +0100
Message-ID: 
 <CALJOYLFYqpvAuBMneWVOZcGZjziX+q4LxhCbm+17Bx4Txq+96Q@mail.gmail.com>
Subject: Re: NullPointerException when trying to persist a domain entity
To: users <users@isis.apache.org>
Cc: dev <dev@isis.apache.org>
Content-Type: multipart/alternative; boundary=047d7bb04734ae069c04e62f40c6

--047d7bb04734ae069c04e62f40c6
Content-Type: text/plain; charset=ISO-8859-1

Further to this, I've now changed the framework [1] so that simply newing
up the object will work (rather than requiring newTransientInstance())

There are some (relatively minor) implications to this, in the ticket.

I've also added Javadoc so that this is all more easily discoverable.

Any questions/issues, let me know

Cheers
Dan


[1] https://issues.apache.org/jira/browse/ISIS-528


On 10 September 2013 20:06, Jeremy Gurr <jeremyg@infusionsoft.com> wrote:

> Yeah, using newTransientInstance to obtain the entity vs. newing one up
> manually worked. Thanks.
>
> Automatic recovery would be nice to make it easier for newbies to get up
> to speed with the framework. At a minimum, the javadoc for the persist
> method should indicate that you can't just persist any data entity, but
> that the entity must be obtained from the newTransientInstance method.
>
>
> ________________________________________
> From: Dan Haywood [dan@haywood-associates.co.uk]
> Sent: Tuesday, September 10, 2013 11:51 AM
> To: users
> Subject: Re: NullPointerException when trying to persist a domain entity
>
> How did you instantiate the transient object?  Did you just new it up, or
> did you use container.newTransientInstance()?
>
> I'm *hoping* the former.  The right thing to do is the latter.
>
> If the above doesn't sort it out, then could you restate the problem but
> using some different class names.  I got a bit confused with your
> description, which class is the persistable entity (cf Customer), which
> class is the domain service (cf CustomerRepo).
>
> I agree that we should improve the error messages here.  We might even try
> to automatically recover from the situation (by creating the adapter lazily
> if not present) if we are passed a pojo that was created "outside" of the
> container.  Off the top of my head, I can't see why that won't work (but I
> might be forgetting something).
>
> Cheers
> Dan
>

--047d7bb04734ae069c04e62f40c6--