cayenne-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Mike Kienenberger <mkien...@gmail.com>
Subject Re: setter for toMany in generated classes
Date Thu, 18 Dec 2014 20:12:52 GMT
As I said before, I think you need to call
removeToManyTarget("${rel.Name}", obj, true); instead of
iterator.remove() and removeAll(), and then you need to call
addToManyTarget() instead of addAll().  Otherwise, you are losing the
reverse relationship changes.

But the rest seems ok.

I'm not saying that adding setList() to the default template is a good
idea, though.   I think having it in your custom template is good
enough.

I do everything with custom templates and rarely use the default
cayenne templates.


On Thu, Dec 18, 2014 at 2:58 PM, Johannes <jotpe.osm@gmail.com> wrote:
> Hello,
>
> following the conversation from
> http://mail-archives.apache.org/mod_mbox/cayenne-user/201412.mbox/%3C548BDF97.1040104%40gmail.com%3E
> there seems to be a common need for having a new method next to
> addXXX, removeXXX a setXXXs in generated classes from CayenneModeler.
>
> I created a first draft
> https://github.com/jotpe/cayenne/commit/6627c09936285c7650d69f44a95ba2f4b118f2b6
>
> Would anybody be so nice to review it? I'm not sure if these lines can
> run in every environment without trouble. What about tests?
>
> Best regards Johannes
>

Mime
View raw message