[
https://issues.apache.org/cayenne/browse/CAY-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12708#action_12708
]
Tore Halset commented on CAY-966:
---------------------------------
Thanks for the patch. I have committed a similar fix inspired by yours. Here
are a list of the changes from your patch.
* Remove reverse ObjRelationship as well
* Remove DbRelationship (still in the model) as well. Including reverse.
* junit tests
I hope this works as expected by you
> ObjRelationsship's are not removed processing DropColumnToModel
> ---------------------------------------------------------------
>
> Key: CAY-966
> URL: https://issues.apache.org/cayenne/browse/CAY-966
> Project: Cayenne
> Issue Type: Bug
> Components: Cayenne Core Library
> Affects Versions: 3.0
> Environment: all
> Reporter: Øyvind Harboe
> Assignee: Tore Halset
> Attachments: DbEntity.java, DropColumnToModel.java,
> surefire-reports.zip
>
>
> I'll attatch a modified DbEntity.java that fixes the problem here....
> Basically the code below won't work right because the ObjRelationship's
> aren't removed.
> MergerContext mergerContext = new
> ExecutingMergerContext(
> payBackDataMap, dataNode);
> for (MergerToken token : tokens)
> {
> if (token.getDirection() ==
> MergeDirection.TO_DB)
> {
> token =
> token.createReverse(dataNode.getAdapter()
> .mergerFactory());
> }
> token.execute(mergerContext);
> }
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
