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

Incorrect event names in grade reports

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.8
    • Fix Version/s: 2.8
    • Component/s: Events API, Gradebook
    • Labels:
    • Testing Instructions:
      Hide
      1. Visit a course as an administrator.
      2. Click on 'Grades' under 'Course administration'.
      3. Use the drop down to visit all the reports.
      4. Click on 'Reports -> logs' under 'Course administration'.
      5. Ensure there is a report viewed event for all reports.
      Show
      Visit a course as an administrator. Click on 'Grades' under 'Course administration'. Use the drop down to visit all the reports. Click on 'Reports -> logs' under 'Course administration'. Ensure there is a report viewed event for all reports.
    • Affected Branches:
      MOODLE_28_STABLE
    • Fixed Branches:
      MOODLE_28_STABLE
    • Pull Master Branch:
      MDL-46565_master
    • Sprint:
      BACKEND Sprint 15

      Description

      It was decided that to make things consistent all abstract class and child classes for events should have same exact name.

      The grade report events break this guideline( Refer MDL-43044)

      The abstract class is named "grade_report_viewed" where as the extending classes are named as "report_viewed"

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              markn Mark Nelson
              Reporter:
              ankit_frenz Ankit Agarwal
              Peer reviewer:
              Ankit Agarwal
              Integrator:
              Marina Glancy
              Tester:
              Michael de Raadt
              Participants:
              Component watchers:
              Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Sara Arjona (@sarjona), Víctor Déniz Falcón, Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                10/Nov/14