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

Restore delete user/group override event firing

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Blocker
    • Resolution: Fixed
    • Affects Version/s: 2.7
    • Fix Version/s: 2.7
    • Component/s: Quiz
    • Labels:
    • Testing Instructions:
      Hide
      Test 1
      1. Run the unit tests in mod/quiz/tests/events_test.php
      Test 2
      1. Visit <yoursite>/admin/settings.php?section=managelogging and enable 'Standard log'.
      2. Visit a course.
      3. Enrol a user into this course.
      4. Add a quiz.
      5. Click to enter the quiz and then click on 'User overrides' in the menu.
      6. Add a user override.
      7. Click to delete this override.
      8. Check that it was deleted.
      9. Visit <yoursite>/report/log/index.php and select 'Standard log' in the select box next to the submit button.
      10. Check that there is a 'Quiz override deleted' event.
      Show
      Test 1 Run the unit tests in mod/quiz/tests/events_test.php Test 2 Visit <yoursite>/admin/settings.php?section=managelogging and enable 'Standard log'. Visit a course. Enrol a user into this course. Add a quiz. Click to enter the quiz and then click on 'User overrides' in the menu. Add a user override. Click to delete this override. Check that it was deleted. Visit <yoursite>/report/log/index.php and select 'Standard log' in the select box next to the submit button. Check that there is a 'Quiz override deleted' event.
    • Affected Branches:
      MOODLE_27_STABLE
    • Fixed Branches:
      MOODLE_27_STABLE
    • Pull Master Branch:
      MDL-45057_master

      Description

      As discovered by Michael in MDL-40063, the delete user override event doesn't have cmid info available and causes a fatal error.

      In order to release a beta release we commented this out. This should be restored.

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    12/May/14