Doctrine 2 - ORM
  1. Doctrine 2 - ORM
  2. DDC-1370

preInsert, postInsert, prePersist, postPersist, preUpdate, postUpdate code and documentation of events

    Details

    • Type: Improvement Improvement
    • Status: Awaiting Feedback
    • Priority: Minor Minor
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: 2.x
    • Component/s: ORM
    • Security Level: All
    • Labels:
      None

      Description

      Currently we have a set of Lifecycle events, but they seem to be misleading both in actual implementation and documentation.

      One good example is prePersist and postPersist, which is only fired when you're creating new entities. It should be renamed to preInsert and postInsert.
      As of preUpdate and postUpdate, they seem quite valid.

      But if we rename prePersist and postPersist to (pre|post)Insert, we may have a situation where you wanna cover both insert and update.
      For this, (pre|post)Persist should be reinstated, but acting differently from what it does currently.

        Activity

        Hide
        Matt McNeill added a comment -

        Commenting here to say that this caused a lot of headaches for our project until we got it sorted out what these events really do.

        The problem is that the term 'persist' is ambiguous - it means both persist and update in the context of the persist() function call, but only means INSERT in the context of the event system.

        Show
        Matt McNeill added a comment - Commenting here to say that this caused a lot of headaches for our project until we got it sorted out what these events really do. The problem is that the term 'persist' is ambiguous - it means both persist and update in the context of the persist() function call, but only means INSERT in the context of the event system.
        Hide
        Guilherme Blanco added a comment -

        Updating fix version

        Show
        Guilherme Blanco added a comment - Updating fix version
        Hide
        Guilherme Blanco added a comment -

        RDohms, this paragraph was already sorted out.

        The actual ticket is still valid here.

        Show
        Guilherme Blanco added a comment - RDohms, this paragraph was already sorted out. The actual ticket is still valid here.
        Hide
        Rafael Dohms added a comment -

        Also, documentation for post* methods is broken at the website:

        "Changes in here are not relevant to the persistence in the database, but you can use this events to"

        It cuts off in mid-sentence.

        Show
        Rafael Dohms added a comment - Also, documentation for post* methods is broken at the website: "Changes in here are not relevant to the persistence in the database, but you can use this events to" It cuts off in mid-sentence.

          People

          • Assignee:
            Benjamin Eberlei
            Reporter:
            Guilherme Blanco
          • Votes:
            1 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated: