MDL-46710 mod_lti: Removing duplicated generated events
authorDavid Monllao <davidm@moodle.com>
Thu, 1 Oct 2015 03:55:38 +0000 (11:55 +0800)
committerDavid Monllao <davidm@moodle.com>
Thu, 1 Oct 2015 03:55:38 +0000 (11:55 +0800)
Event generation and completion should be generated only once.

mod/lti/view.php

index 7d197fa..e888164 100644 (file)
@@ -94,24 +94,6 @@ if ($launchcontainer == LTI_LAUNCH_CONTAINER_EMBED_NO_BLOCKS) {
     $PAGE->set_pagelayout('incourse');
 }
 
-// Mark viewed by user (if required).
-$completion = new completion_info($course);
-$completion->set_module_viewed($cm);
-
-$params = array(
-    'context' => $context,
-    'objectid' => $lti->id
-);
-
-// Prevent double logging when launching in a new window is configured.
-if ($launchcontainer != LTI_LAUNCH_CONTAINER_WINDOW) {
-    $event = \mod_lti\event\course_module_viewed::create($params);
-    $event->add_record_snapshot('course_modules', $cm);
-    $event->add_record_snapshot('course', $course);
-    $event->add_record_snapshot('lti', $lti);
-    $event->trigger();
-}
-
 $pagetitle = strip_tags($course->shortname.': '.format_string($lti->name));
 $PAGE->set_title($pagetitle);
 $PAGE->set_heading($course->fullname);