Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-28166

Quiz tries to send email messages while still in the process_attempt transaction -> exception

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Critical
    • Resolution: Fixed
    • Affects Version/s: 2.1, 2.2
    • Fix Version/s: 2.1.1
    • Component/s: Quiz
    • Labels:
    • Testing Instructions:
      Hide

      1. Create a quiz in a course with students and teachers.
      2. Override the permission so that some combination of confirmation emails for students and notification emails for teachers is enabled.
      3. Complete the quiz as a student. Ensure there are no errors.
      4. Wait for cron to run.
      5. Verify that the expected messages have been sent.

      Show
      1. Create a quiz in a course with students and teachers. 2. Override the permission so that some combination of confirmation emails for students and notification emails for teachers is enabled. 3. Complete the quiz as a student. Ensure there are no errors. 4. Wait for cron to run. 5. Verify that the expected messages have been sent.
    • Workaround:
      Hide

      If you need to keep your quizzes open, turn off notification emails temporarily.

      Show
      If you need to keep your quizzes open, turn off notification emails temporarily.
    • Affected Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE
    • Fixed Branches:
      MOODLE_21_STABLE
    • Pull from Repository:
    • Pull Master Branch:

      Description

      The messaging system does not allow sending message in a transaction. The code needs to be reordered.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                timhunt Tim Hunt
                Reporter:
                timhunt Tim Hunt
                Integrator:
                Eloy Lafuente (stronk7)
                Tester:
                Sam Hemelryk
                Participants:
                Component watchers:
                Tim Hunt, Andrew Nicols, Mathew May, Michael Hawkins, Shamim Rezaie, Simey Lameze
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  1/Aug/11