[JIRA] Commented: (CAY-966) ObjRelationsship's are not removed processing DropColumnToModel

From: Tore Halset (JIRA) ("Tore)
Date: Tue Jan 22 2008 - 19:04:33 EST

  • Next message: JIRA: "[JIRA] Commented: (CAY-966) ObjRelationsship's are not removed processing DropColumnToModel"

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



    This archive was generated by hypermail 2.0.0 : Tue Jan 22 2008 - 19:04:59 EST