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
    • Rank:
      24322

      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.

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

        Issue Links

          Activity

          Petr Škoda created issue -
          Petr Škoda made changes -
          Field Original Value New Value
          Attachment nested_exceptions_3.patch [ 18694 ]
          Petr Škoda made changes -
          Attachment nested_exceptions_3.patch [ 18694 ]
          Petr Škoda made changes -
          Attachment nested_transactions_3.patch [ 18695 ]
          Petr Škoda 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 Škoda 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 Škoda made changes -
          Petr Škoda made changes -
          Petr Škoda made changes -
          Petr Škoda made changes -
          Petr Škoda made changes -
          Fix Version/s 2.0 [ 10122 ]
          Fix Version/s 2.1 [ 10370 ]
          Petr Škoda 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: