Doctrine 2 - ORM
  1. Doctrine 2 - ORM
  2. DDC-2922

Flushing new Entities reachable over several paths that do not all "cascade persist"

    Details

    • Type: Bug Bug
    • Status: Awaiting Feedback
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: 2.3.3
    • Fix Version/s: None
    • Component/s: ORM
    • Labels:
      None

      Description

      Please consider the following associations:

        A <>--> B <-- C  

      A is an "aggregate" type object w/ a OneToMany collection of Bs. This association is set to cascade persist operations.

      C has a unidirectional OneToOne association to B as well.

      Adding a new B to A's collection and flushing the EntityManager works - B is persisted (persistence by reachability).

      However, when also creating a new C and pointing it to the new B, adding C to the EntityManager and then flushing leads to an exception because B is discovered as a new Entity through a relationship not set to cascade persist operations.

      Obviously the problem here is that there are two paths through which we can discover new Entities. The UoW currently starts search from newly (explicitly) added Entities first before it walks through collections of already-managed entities.

      I am unsure if it is just a documentation issue or a deeper problem?

      If the UoW worked the other way round, the same problem would occur if the cascade persist was on the other association.

      A solution would probably require the UoW to first collect all offending (new) Entities and the link they were discovered through, but later on remove Entities from this list if they are found through another association with the cascade persist option. The exception must not be thrown unless the whole reachability graph has been traversed.

        Activity

        Hide
        Marco Pivetta added a comment -

        Can this be factored into a failing test case? I'm not sure if we can support it, but the code would make it much easier to track down the issue into something fixable.

        Show
        Marco Pivetta added a comment - Can this be factored into a failing test case? I'm not sure if we can support it, but the code would make it much easier to track down the issue into something fixable.

          People

          • Assignee:
            Benjamin Eberlei
            Reporter:
            Matthias Pigulla
          • Votes:
            1 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated: