[ 
https://issues.apache.org/cayenne/browse/CAY-436?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12986#action_12986
 ] 

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.

Reply via email to