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

Change event monitor pages to use contextid instead of courseid

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Reopened
    • Priority: Minor
    • Resolution: Unresolved
    • Affects Version/s: 2.8
    • Fix Version/s: BACKEND
    • Component/s: Events API
    • Labels:
    • Affected Branches:
      MOODLE_28_STABLE

      Description

      Part of this will fix the optional_param problems introduced in MDL-47862

      https://github.com/xow/moodle/compare/cf03313...MDL-47862-master#diff-8b70a8b5d0eeb2192d39442c7810e408R85

      Solution could be to refactor the event monitor pages to use contextid as a param rather than courseid

        Attachments

          Issue Links

          Discovered while testing

          Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-47767 Create link between rules page and subscription page

          • Minor - Minor loss of function where workaround is possible
          • Closed

          Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-47862 admin/tool/monitor/index.php link should check at appropriate context

          • Minor - Minor loss of function where workaround is possible
          • Closed
          is duplicated by

          Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-47862 admin/tool/monitor/index.php link should check at appropriate context

          • Minor - Minor loss of function where workaround is possible
          • Closed

            Activity

              People

              • Assignee:
                Unassigned
                Reporter:
                johno John Okely
                Participants:
                Component watchers:
                Jake Dallimore, Jun Pataleta
              • Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated: