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

Replace add_to_log with an event trigger - course reports

    XMLWordPrintable

    Details

    • Type: Task
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.5
    • Fix Version/s: 2.6
    • Component/s: Events API, Logging
    • Labels:
    • Testing Instructions:
      Hide
      1. You must have stats enabled and some data to test this issue.
      2. Visit the pages mentioned in description
      3. For each of them make sure a log entry is created.
      4. Make sure the event entry has all details as before the patch and links to the correct page as before.
      Show
      You must have stats enabled and some data to test this issue. Visit the pages mentioned in description For each of them make sure a log entry is created. Make sure the event entry has all details as before the patch and links to the correct page as before.
    • Affected Branches:
      MOODLE_25_STABLE
    • Fixed Branches:
      MOODLE_26_STABLE
    • Pull Master Branch:
      MDL-40052-master
    • Story Points:
      20
    • Sprint:
      BACKEND Sprint 4

      Description

      Replace the add_to_log calls for the following area, with a call to the get_legacy_logdata function in the event class.

      course / reports
      /report/participation/index.php:66: add_to_log($course->id, "course", "report participation", "report/participation/index.php?id=$course->id", $course->id);
      /report/log/index.php:118: add_to_log($course->id, "course", "report log", "report/log/index.php?id=$course->id", $course->id);
      /report/log/graph.php:70: add_to_log($course->id, 'course', 'report log', "report/log/graph.php?user=$user->id&id=$course->id&type=$type&date=$date", $course->id);
      /report/log/user.php:65: add_to_log($course->id, 'course', 'report log', "report/log/user.php?id=$user->id&course=$course->id&mode=$mode", $course->id);
      /report/loglive/index.php:51: add_to_log($course->id, 'course', 'report live', "report/loglive/index.php?id=$course->id", $course->id);
      /report/stats/index.php:70: add_to_log($course->id, "course", "report stats", "report/stats/index.php?course=$course->id", $course->id);
      /report/stats/graph.php:65: add_to_log($course->id, 'course', 'report stats', "report/stats/graph.php?userid=$userid&id=$course->id&mode=$mode&roleid=$roleid", $course->id);
      /report/stats/user.php:52: add_to_log($course->id, 'course', 'report stats', "report/stats/user.php?id=$user->id&course=$course->id", $course->id);
      /report/outline/index.php:40: add_to_log($course->id, 'course', 'report outline', "report/outline/index.php?id=$course->id", $course->id);
      /report/outline/user.php:56: add_to_log($course->id, 'course', 'report outline', "report/outline/user.php?id=$user->id&course=$course->id&mode=$mode", $course->id);

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              ankit_frenz Ankit Agarwal
              Reporter:
              abgreeve Adrian Greeve
              Peer reviewer:
              Rajesh Taneja
              Integrator:
              Damyon Wiese
              Tester:
              Marina Glancy
              Participants:
              Component watchers:
              Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Sara Arjona (@sarjona), Víctor Déniz Falcón, Andrew Nicols, Jun Pataleta, 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:
                18/Nov/13