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

Replace add_to_log with an event trigger - mod_imscp

    XMLWordPrintable

    Details

    • Testing Instructions:
      Hide
      1. Create a course.
      2. Create an imscp resource.
      3. Click on this imscp resource.
      4. Change the view.php to index.php and the id to the id of the course in the URL.
      5. Visit <yoursite>/report/log/index.php?id=1 and click 'Get these logs'.
      6. Ensure there are two logs; viewing all imscp instances and viewing an individual imscp.
      Show
      Create a course. Create an imscp resource. Click on this imscp resource. Change the view.php to index.php and the id to the id of the course in the URL. Visit <yoursite>/report/log/index.php?id=1 and click 'Get these logs'. Ensure there are two logs; viewing all imscp instances and viewing an individual imscp.
    • Affected Branches:
      MOODLE_26_STABLE
    • Fixed Branches:
      MOODLE_27_STABLE
    • Pull Master Branch:
      MDL-40916_master
    • Story Points:
      8
    • Sprint:
      BACKEND Sprint 11

      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_imscp
      /mod/imscp/index.php:36:add_to_log($course->id, 'imscp', 'view all', "index.php?id=$course->id", '');
      /mod/imscp/view.php:49:add_to_log($course->id, 'imscp', 'view', 'view.php?id='.$cm->id, $imscp->id, $cm->id);

        Attachments

          Activity

            People

            Assignee:
            markn Mark Nelson
            Reporter:
            abgreeve Adrian Greeve
            Peer reviewer:
            Petr Skoda
            Integrator:
            Dan Poltawski
            Tester:
            Michael de Raadt
            Participants:
            Component watchers:
            Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Dates

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