Details

    • Type: Sub-task
    • Status: Closed
    • Priority: 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

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

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  24/Nov/10