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

When quiz previews are deleted automatically, that should not be logged as attempt deleted

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.8.8, 2.9.2, 3.0
    • Fix Version/s: 2.8.9, 2.9.3
    • Component/s: Events API, Quiz
    • Labels:
    • Affected Branches:
      MOODLE_28_STABLE, MOODLE_29_STABLE, MOODLE_30_STABLE
    • Fixed Branches:
      MOODLE_28_STABLE, MOODLE_29_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-52014-master
    • Pull Master Diff URL:
    • Testing Instructions:
      Hide
      1. At teacher
      2. Create a new quiz with one essay question
      3. Preview the quiz, and submit your preview attempt
      4. Edit the quiz and add a new multiple choice question
      5. View the course logs
      6. VERIFY there is no 'Quiz attempt deleted' log entry against the teacher
      Show
      At teacher Create a new quiz with one essay question Preview the quiz, and submit your preview attempt Edit the quiz and add a new multiple choice question View the course logs VERIFY there is no 'Quiz attempt deleted' log entry against the teacher

      Description

      The quiz_delete_attempt method in mod/quiz/locallib.php should only log the attempt_deleted event if the attempt is not a preview (Only log is $attempt->preview is 0).

        Attachments

          Issue Links

          is a regression caused by

          Improvement - An enhancement to an existing Moodle feature. MDL-40063 Replace add_to_log with an event trigger - mod_quiz

          • Major - Major loss of function, incorrect output
          • Closed

            Activity

              People

              Assignee:
              poltawski Dan Poltawski
              Reporter:
              timhunt Tim Hunt
              Peer reviewer:
              Tim Hunt Tim Hunt
              Integrator:
              Eloy Lafuente (stronk7) Eloy Lafuente (stronk7)
              Tester:
              Simey Lameze Simey Lameze
              Participants:
              Component watchers:
              Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Ilya Tregubov, Sara Arjona (@sarjona), Tim Hunt, Andrew Lyons, Dongsheng Cai, Huong Nguyen, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                9/Nov/15