MDL-40056 events: Use correct course module id while fetching context

This commit is contained in:
Ankit Agarwal 2014-01-16 16:31:06 +08:00
parent 0a61619043
commit 5971f6266a
2 changed files with 2 additions and 2 deletions

View File

@ -1316,7 +1316,7 @@ function scorm_delete_attempt($userid, $scorm, $attemptid) {
// Trigger instances list viewed event.
$event = \mod_scorm\event\attempt_deleted::create(array(
'other' => array('attemptid' => $attemptid),
'context' => context_module::instance($cm->instance),
'context' => context_module::instance($cm->id),
'relateduserid' => $userid
));
$event->add_record_snapshot('course_modules', $cm);

View File

@ -74,7 +74,7 @@ class mod_scorm_event_testcase extends advanced_testcase {
$this->assertCount(1, $events);
$this->assertInstanceOf('\mod_scorm\event\attempt_deleted', $event);
$this->assertEquals($USER->id, $event->userid);
$this->assertEquals(context_module::instance($this->eventscorm->id), $event->get_context());
$this->assertEquals(context_module::instance($this->eventcm->id), $event->get_context());
$this->assertEquals(4, $event->other['attemptid']);
$this->assertEquals(2, $event->relateduserid);
$expected = array($this->eventcourse->id, 'scorm', 'delete attempts', 'report.php?id=' . $this->eventcm->id,