MDL-40918 mod_lti: replaced 'view' and 'view all' add_to_log calls with events

This commit is contained in:
Mark Nelson 2013-12-31 15:09:03 -08:00
parent bbb291b7b7
commit 3fdd73e83d
4 changed files with 82 additions and 2 deletions

View file

@ -0,0 +1,31 @@
<?php
// This file is part of Moodle - http://moodle.org/
//
// Moodle is free software: you can redistribute it and/or modify
// it under the terms of the GNU General Public License as published by
// the Free Software Foundation, either version 3 of the License, or
// (at your option) any later version.
//
// Moodle is distributed in the hope that it will be useful,
// but WITHOUT ANY WARRANTY; without even the implied warranty of
// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
// GNU General Public License for more details.
//
// You should have received a copy of the GNU General Public License
// along with Moodle. If not, see <http://www.gnu.org/licenses/>.
/**
* The mod_lti instance list viewed event.
*
* @package mod_lti
* @copyright 2013 Mark Nelson <markn@moodle.com>
* @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
*/
namespace mod_lti\event;
defined('MOODLE_INTERNAL') || die();
class course_module_instance_list_viewed extends \core\event\course_module_instance_list_viewed {
// No need for any code here as everything is handled by the parent class.
}

View file

@ -0,0 +1,39 @@
<?php
// This file is part of Moodle - http://moodle.org/
//
// Moodle is free software: you can redistribute it and/or modify
// it under the terms of the GNU General Public License as published by
// the Free Software Foundation, either version 3 of the License, or
// (at your option) any later version.
//
// Moodle is distributed in the hope that it will be useful,
// but WITHOUT ANY WARRANTY; without even the implied warranty of
// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
// GNU General Public License for more details.
//
// You should have received a copy of the GNU General Public License
// along with Moodle. If not, see <http://www.gnu.org/licenses/>.
/**
* The mod_lti course module viewed event.
*
* @package mod_lti
* @copyright 2013 Mark Nelson <markn@moodle.com>
* @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
*/
namespace mod_lti\event;
defined('MOODLE_INTERNAL') || die();
class course_module_viewed extends \core\event\course_module_viewed {
/**
* Init method.
*/
protected function init() {
$this->data['objecttable'] = 'lti';
$this->data['crud'] = 'r';
$this->data['level'] = self::LEVEL_PARTICIPATING;
}
}

View file

@ -56,7 +56,11 @@ $course = $DB->get_record('course', array('id'=>$id), '*', MUST_EXIST);
require_login($course); require_login($course);
$PAGE->set_pagelayout('incourse'); $PAGE->set_pagelayout('incourse');
add_to_log($course->id, "lti", "view all", "index.php?id=$course->id", ""); $params = array(
'context' => context_course::instance($course->id)
);
$event = \mod_lti\event\course_module_instance_list_viewed::create($params);
$event->trigger();
$PAGE->set_url('/mod/lti/index.php', array('id' => $course->id)); $PAGE->set_url('/mod/lti/index.php', array('id' => $course->id));
$pagetitle = strip_tags($course->shortname.': '.get_string("modulenamepluralformatted", "lti")); $pagetitle = strip_tags($course->shortname.': '.get_string("modulenamepluralformatted", "lti"));

View file

@ -96,7 +96,13 @@ require_login($course);
$completion = new completion_info($course); $completion = new completion_info($course);
$completion->set_module_viewed($cm); $completion->set_module_viewed($cm);
add_to_log($course->id, "lti", "view", "view.php?id=$cm->id", "$lti->id"); $params = array(
'context' => $context,
'objectid' => $lti->id
);
$event = \mod_lti\event\course_module_viewed::create($params);
$event->add_record_snapshot('lti', $lti);
$event->trigger();
$pagetitle = strip_tags($course->shortname.': '.format_string($lti->name)); $pagetitle = strip_tags($course->shortname.': '.format_string($lti->name));
$PAGE->set_title($pagetitle); $PAGE->set_title($pagetitle);