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

Replace add_to_log with an event trigger - mod_lti

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.6
    • Fix Version/s: 2.7
    • Component/s: Logging
    • Labels:
    • Testing Instructions:
      Hide
      1. Create a course.
      2. Create an External tool resource.
      3. Click on the External tool resource.
      4. Change the view.php to index.php and the id to the id of the course.
      5. Visit <yoursite>/report/log/index.php?id=1 and click 'Get these logs'.
      6. Ensure there are two events; viewing all LTI instances and viewing an individual LTI.
      Show
      Create a course. Create an External tool resource. Click on the External tool resource. Change the view.php to index.php and the id to the id of the course. Visit <yoursite>/report/log/index.php?id=1 and click 'Get these logs'. Ensure there are two events; viewing all LTI instances and viewing an individual LTI.
    • Affected Branches:
      MOODLE_26_STABLE
    • Fixed Branches:
      MOODLE_27_STABLE
    • Pull Master Branch:
      MDL-40918_master
    • Story Points:
      8
    • Sprint:
      BACKEND Sprint 8

      Description

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

      mod_lti
      /mod/lti/index.php:59:add_to_log($course->id, "lti", "view all", "index.php?id=$course->id", "");
      /mod/lti/launch.php:62:add_to_log($course->id, "lti", "launch", "launch.php?id=$cm->id", "$lti->id");
      /mod/lti/view.php:95:add_to_log($course->id, "lti", "view", "view.php?id=$cm->id", "$lti->id");

        Attachments

          Activity

            People

            Assignee:
            markn Mark Nelson
            Reporter:
            abgreeve Adrian Greeve
            Peer reviewer:
            Petr Skoda
            Integrator:
            Dan Poltawski
            Tester:
            Rajesh Taneja
            Participants:
            Component watchers:
            Andrew Nicols, 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:
              12/May/14