Details

    • Type: Sub-task Sub-task
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0
    • Fix Version/s: 2.0
    • Component/s: Database SQL/XMLDB
    • Labels:
      None
    • Affected Branches:
      MOODLE_20_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE

      Description

      This topic was brought forward by Tim in http://moodle.org/mod/forum/discuss.php?d=135847

      I personally do not think we should start using the transactions in moodle core now, but in any case I already considered the nested transactions last year. At that time it could not be done because we were not yet using exceptions.

      Attaching patch that adds nested transactions without API changes, includes unittests.

        Gliffy Diagrams

        1. delegated_transactions_20091103.patch
          66 kB
          Petr Skoda
        2. delegated_transactions_20091104_2.patch
          69 kB
          Petr Skoda
        3. delegated_transactions_20091104_3.patch
          70 kB
          Petr Skoda
        4. delegated_transactions_20091104.patch
          68 kB
          Petr Skoda
        5. nested_transactions_3.patch
          19 kB
          Petr Skoda
        1. TransactionsAndExceptionsFlow.png
          41 kB

          Issue Links

            Activity

            Petr Skoda created issue -
            Petr Skoda made changes -
            Field Original Value New Value
            Attachment nested_exceptions_3.patch [ 18694 ]
            Petr Skoda made changes -
            Attachment nested_exceptions_3.patch [ 18694 ]
            Petr Skoda made changes -
            Attachment nested_transactions_3.patch [ 18695 ]
            Petr Skoda made changes -
            Fix Version/s 2.0 [ 10122 ]
            Martin Dougiamas made changes -
            Assignee Petr Skoda [ skodak ] Eloy Lafuente (stronk7) [ stronk7 ]
            Martin Dougiamas made changes -
            Fix Version/s 2.1 [ 10370 ]
            Petr Skoda made changes -
            Link This issue has a non-specific relationship to MDL-20644 [ MDL-20644 ]
            Eloy Lafuente (stronk7) made changes -
            Attachment TransactionsAndExceptionsFlow.png [ 18742 ]
            Petr Skoda made changes -
            Petr Skoda made changes -
            Petr Skoda made changes -
            Petr Skoda made changes -
            Petr Skoda made changes -
            Fix Version/s 2.0 [ 10122 ]
            Fix Version/s 2.1 [ 10370 ]
            Petr Skoda made changes -
            Status Open [ 1 ] Resolved [ 5 ]
            Resolution Fixed [ 1 ]
            Martin Dougiamas made changes -
            Status Resolved [ 5 ] Closed [ 6 ]
            QA Assignee nobody
            Martin Dougiamas made changes -
            Workflow jira [ 33804 ] MDL Workflow [ 63268 ]
            Martin Dougiamas made changes -
            Workflow MDL Workflow [ 63268 ] MDL Full Workflow [ 92502 ]
            Andrew Nicols made changes -
            Link This issue caused a regression MDL-30029 [ MDL-30029 ]

              People

              • Votes:
                0 Vote for this issue
                Watchers:
                8 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: