Doctrine 2 - ORM
  1. Doctrine 2 - ORM
  2. DDC-3197

[GH-1074] [DDC-3160] Alternate fix for DDC-2996 bug

    Details

    • Type: Bug Bug
    • Status: Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: None
    • Security Level: All
    • Labels:
      None

      Description

      This issue is created automatically through a Github pull request on behalf of zimmermanj42:

      Url: https://github.com/doctrine/doctrine2/pull/1074

      Message:

      The fix implemented for DDC-2996 seems to have broken quite a bit of code outside of Doctrine (for instance, the popular DoctrineExtensions project).

      I'm not too well versed in Doctrine internals, but looking at the fix, I found it odd that an entity would live in both the insertion and update tracking in the unit of work.

      This implementation should fix the case described in DDC-2996, while not creating any side effects with objects that are scheduled to be inserted.

      I've ran this test with the Doctrine test suite and have also tested this change with the DoctrineExtensions project; unless I'm doing something wrong, it seems to have good results.

      Again, I'm not too well versed in the inner workings of Doctrine, but I did a little research, poked around some code, and came up with this.

        Activity

        Hide
        Doctrine Bot added a comment -

        A related Github Pull-Request [GH-1057] was closed:
        https://github.com/doctrine/doctrine2/pull/1057

        Show
        Doctrine Bot added a comment - A related Github Pull-Request [GH-1057] was closed: https://github.com/doctrine/doctrine2/pull/1057
        Hide
        Doctrine Bot added a comment -

        A related Github Pull-Request [GH-1074] was closed:
        https://github.com/doctrine/doctrine2/pull/1074

        Show
        Doctrine Bot added a comment - A related Github Pull-Request [GH-1074] was closed: https://github.com/doctrine/doctrine2/pull/1074

          People

          • Assignee:
            Benjamin Eberlei
            Reporter:
            Doctrine Bot
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated: