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.
This archive was generated by hypermail 2.0.0 : Tue Jan 22 2008 - 19:04:59 EST