Doctrine 2 - ORM
  1. Doctrine 2 - ORM
  2. DDC-1392

Merge for not initialized entity proxy object with set ID fails

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.1.1
    • Fix Version/s: 2.1.2
    • Component/s: ORM
    • Security Level: All
    • Labels:
      None

      Description

      Problem is that UnitOfWork::doMerge() calls ClassMetadata::getIdentifierValues() for the entity but that function doesn't have case for not initialized proxy. It just reads the Id properties from the object's reflection.

      It should load the proxy or get the ID stored inside the proxy object or doMerge must initialize it maybe.

        Activity

        Hide
        Aigars Gedroics added a comment -

        I have taken the DDC353 test as the base for this test case.
        Hopefully it works because I have executed it in slightly different version without the Doctrine tests environment.

        Show
        Aigars Gedroics added a comment - I have taken the DDC353 test as the base for this test case. Hopefully it works because I have executed it in slightly different version without the Doctrine tests environment.
        Hide
        Benjamin Eberlei added a comment -

        Fixed

        Show
        Benjamin Eberlei added a comment - Fixed
        Hide
        Aigars Gedroics added a comment -

        I would put the changes inside the method getIdentifierValues not before it's execution.

        Isn't it plausible that it is called with not initialized proxy from somewhere else?

        Show
        Aigars Gedroics added a comment - I would put the changes inside the method getIdentifierValues not before it's execution. Isn't it plausible that it is called with not initialized proxy from somewhere else?

          People

          • Assignee:
            Benjamin Eberlei
            Reporter:
            Aigars Gedroics
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: