Re: Nested ROP contexts and relationships

From: Andrus Adamchik (andru..bjectstyle.org)
Date: Sun Apr 05 2009 - 04:31:15 EDT

  • Next message: Andrus Adamchik: "Incorrectly mapped jira name: andrey"

    Cool. I got a further along in my investigation. I will put the
    details in a Jira and work on fixing it.

    Andrus

    On Apr 4, 2009, at 8:31 AM, Andrey Razumovsky wrote:

    > Hi Andrus,
    >
    > I'm afraid I don't remember if it was done intentionally. I only
    > know that
    > the code is different from normal contexts' diff processing (that's
    > why I
    > feel a large refactoring is needed). Feel free to change the code, of
    > course. Alternatively, if you open a JIRA and post you JUnit there,
    > I'll
    > have a look.
    >
    > Andrey
    >
    > 2009/4/3 Andrus Adamchik <andru..bjectstyle.org>
    >
    >> I am in the middle of debugging a problem with nested ROP contexts
    >> losing
    >> arc changes when committing to a parent context. Since I was not
    >> involved in
    >> the ROP nested context work, I figured I'd post my thoughts here
    >> before I
    >> start changing the code.
    >>
    >> I noticed that per CAY-1119, there is a special subclass of
    >> ChildDiffLoader
    >> called CayenneContextChildDiffLoader that calls 'propertyChanged'
    >> on the
    >> parent context after syncing a simple property change. It seems
    >> like we need
    >> to do the same for relationships as well, to record arc changes in
    >> the
    >> parent diff list.
    >>
    >> Andrey, do you have any comments on that? I wonder if it was omitted
    >> intentionally. I will open a Jira (I think we don't have one for
    >> this), and
    >> add some tests with various relationships, but before I dig any
    >> deeper
    >> figured I'll need a sanity check.
    >>
    >> Thanks,
    >> Andrus
    >>



    This archive was generated by hypermail 2.0.0 : Sun Apr 05 2009 - 04:31:51 EDT