Kevin Menard commented on CAY-436:
----------------------------------
All in all, this looks pretty good. A couple questions:
1) In DbLoaderHelper, is there any reason that you need to explicitly update the ObjEntity? Or rather, is there a reason that your observer pattern fails to work here?
2) Is there a reason that the observer pattern fails in ObjRelationshipInfoModel#savePath?
I may be looking at the code wrong, but it strikes me that the EntityMergeListener or something similar could be used in such a way that you don't actually have to hook in explicit updates in so many places.
> In modeler, change default object relationship delete rule
> ----------------------------------------------------------
>
> Key: CAY-436
> URL: https://issues.apache.org/cayenne/browse/CAY-436
> Project: Cayenne
> Issue Type: Improvement
> Components: Cayenne Core Library, CayenneModeler GUI
> Environment: winxp, M7
> Reporter: weidox
> Assignee: Kevin Menard
> Fix For: UNDEFINED FUTURE
>
> Attachments: patch-CAY-436.txt
>
>
> Should be good to change default delete rules for object relationships, one to many - cascade, and vice versa - to nullify. I think it's the rules used mostly for these relationships.
> And the best thing would be to have configurable options for these, but that's more complex.
-- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
This archive was generated by hypermail 2.0.0 : Sun Jun 29 2008 - 18:43:19 EDT