Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Won't Fix
    • Labels:
      None

      Description

      If i have a node that i fetch from the repo using $dm->find()

      If i change an attribute in it.
      Then i call $dm->move() on it

      Then i call $dm->flush().

      It will work.

      However if i call $dm->persist() on the node (that i already fetched from $dm->find()) and then call $dm->flush()
      The $dm->move() won't occur, it will be cancelled, but the attribute i changed will still be updated.

        Activity

        Freddy H. created issue -
        Hide
        Lukas Kahwe added a comment -

        i have to think about if this is a bug or a feature.
        its certainly useful to have a way to "cancel" operations ..
        f.e. persist() also cancels remove()

        then again persist() is i guess viewed by many as a non problematic operation since it commonly doesn't break anything if one persisted an object multiple times.

        Show
        Lukas Kahwe added a comment - i have to think about if this is a bug or a feature. its certainly useful to have a way to "cancel" operations .. f.e. persist() also cancels remove() then again persist() is i guess viewed by many as a non problematic operation since it commonly doesn't break anything if one persisted an object multiple times.
        Hide
        Lukas Kahwe added a comment -

        as i expected this is an intentional feature:
        https://github.com/doctrine/phpcr-odm/blob/master/lib/Doctrine/ODM/PHPCR/UnitOfWork.php#L518

        we could think about adding a dedicated method to "reset" the state, but the ORM doesn't have something like that so i rather not introduce it.

        Show
        Lukas Kahwe added a comment - as i expected this is an intentional feature: https://github.com/doctrine/phpcr-odm/blob/master/lib/Doctrine/ODM/PHPCR/UnitOfWork.php#L518 we could think about adding a dedicated method to "reset" the state, but the ORM doesn't have something like that so i rather not introduce it.
        Lukas Kahwe made changes -
        Field Original Value New Value
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Won't Fix [ 2 ]

        This list may be incomplete, as errors occurred whilst retrieving source from linked applications:

        • Request to http://www.doctrine-project.org/fisheye/ failed: Error in remote call to 'FishEye 0 (http://www.doctrine-project.org/fisheye/)' (http://www.doctrine-project.org/fisheye) [AbstractRestCommand{path='/rest-service-fe/search-v1/crossRepositoryQuery', params={query=PHPCR-72, expand=changesets[0:20].revisions[0:29],reviews}, methodType=GET}] : Received status code 503 (Service Temporarily Unavailable)

          People

          • Assignee:
            Lukas Kahwe
            Reporter:
            Freddy H.
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: