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....@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 >