Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Labels:
      None

      Description

      when replacing a mapped child with a new document, we get the cannotMoveByAssignment exception, which imo is a) the wrong message as its not about setting a tracked document to become a child somewhere else but what we should do is delete the existing child document and add the new one.

      this happens at https://github.com/doctrine/phpcr-odm/blob/master/lib/Doctrine/ODM/PHPCR/UnitOfWork.php#L1912

      for multiple children, i am less sure how it should work, if it is possible the same way:
      https://github.com/doctrine/phpcr-odm/blob/master/lib/Doctrine/ODM/PHPCR/UnitOfWork.php#L935

        Activity

        Hide
        David Buchmann added a comment -

        note that as of this time, this is not fully fixed: if the child has children of its own, there is an error about detached documents. see https://github.com/doctrine/phpcr-odm/pull/262

        Show
        David Buchmann added a comment - note that as of this time, this is not fully fixed: if the child has children of its own, there is an error about detached documents. see https://github.com/doctrine/phpcr-odm/pull/262
        Hide
        David Buchmann added a comment -

        the PR got merged https://github.com/doctrine/phpcr-odm/pull/253

        as a side note: what speaks against moving the issues to github and drop this jira project? it would be much more user friendly for reporting bugs, and easier to integrate. and i don't think we use any of the fancy jira features really. (github has tags and now even milestones, if we would want to use them)

        Show
        David Buchmann added a comment - the PR got merged https://github.com/doctrine/phpcr-odm/pull/253 as a side note: what speaks against moving the issues to github and drop this jira project? it would be much more user friendly for reporting bugs, and easier to integrate. and i don't think we use any of the fancy jira features really. (github has tags and now even milestones, if we would want to use them)
        Hide
        David Buchmann added a comment -

        there is a PR to add a form element for an image. when using this, the containing class needs to work around this bug for now https://github.com/doctrine/DoctrinePHPCRBundle/pull/37

        Show
        David Buchmann added a comment - there is a PR to add a form element for an image. when using this, the containing class needs to work around this bug for now https://github.com/doctrine/DoctrinePHPCRBundle/pull/37

          People

          • Assignee:
            Lukas Kahwe
            Reporter:
            David Buchmann
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: