From c645a3f04308515b2e54e55b63e6b77e110bfe5d Mon Sep 17 00:00:00 2001 From: Nick Phillips Date: Mon, 29 Jul 2019 13:55:06 +1200 Subject: [PATCH] Add event for viewing of per-student reports, and use in view.php. Version bumped due to new lang string. --- .../student_attendance_sessions_viewed.php | 135 ++++++++++++++++++ lang/en/attendance.php | 1 + version.php | 2 +- view.php | 18 +++ 4 files changed, 155 insertions(+), 1 deletion(-) create mode 100644 classes/event/student_attendance_sessions_viewed.php diff --git a/classes/event/student_attendance_sessions_viewed.php b/classes/event/student_attendance_sessions_viewed.php new file mode 100644 index 0000000..d97bf23 --- /dev/null +++ b/classes/event/student_attendance_sessions_viewed.php @@ -0,0 +1,135 @@ +. + +/** + * This file contains an event for when a student's attendance report is viewed. + * + * @package mod_attendance + * @copyright 2014 onwards Dan Marsden + * @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later + */ + +namespace mod_attendance\event; +defined('MOODLE_INTERNAL') || die(); + +/** + * Event for when a student's attendance report is viewed. + * + * @property-read array $other { + * Extra information about event properties. + * + * string studentid Id of student whose attendances were viewed. + * string mode Mode of the report viewed. + * } + * @package mod_attendance + * @since Moodle 2.7 + * @copyright 2013 onwards Dan Marsden + * @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later + */ +class student_attendance_sessions_viewed extends \core\event\base { + + /** + * Init method. + */ + protected function init() { + $this->data['crud'] = 'r'; + $this->data['edulevel'] = self::LEVEL_TEACHING; + // objecttable and objectid can't be meaningfully specified + } + + /** + * Returns non-localised description of what happened. + * + * @return string + */ + public function get_description() { + return 'User with id ' . $this->userid . ' viewed attendance sessions for student with id ' . + $this->relateduserid; + } + + /** + * Returns localised general event name. + * + * @return string + */ + public static function get_name() { + return get_string('eventstudentattendancesessionsviewed', 'mod_attendance'); + } + + /** + * Get URL related to the action + * + * @return \moodle_url + */ + public function get_url() { + return new \moodle_url('/mod/attendance/view.php', array('id' => $this->contextinstanceid, + 'studentid' => $this->relateduserid, + 'mode' => $this->other['mode'], + 'view' => $this->other['view'], + 'curdate' => $this->other['curdate'])); + } + + /** + * Replace add_to_log() statement. + * + * @return array of parameters to be passed to legacy add_to_log() function. + */ + protected function get_legacy_logdata() { + return array($this->courseid, 'attendance', 'student sessions viewed', $this->get_url(), + 'student id ' . $this->relateduserid, $this->contextinstanceid); + } + + /** + * Get objectid mapping + * + * @return array of parameters for object mapping. + */ + public static function get_objectid_mapping() { + return array(); + } + + /** + * Get other mapping + * + * @return array of parameters for object mapping for objects referenced in 'other' property. + */ + public static function get_other_mapping() { + return array(); + } + + /** + * Custom validation. + * + * @throws \coding_exception + * @return void + */ + protected function validate_data() { + if (!isset($this->relateduserid)) { + throw new \coding_exception('The event mod_attendance\\event\\student_attendance_sessions_viewed must specify relateduserid.'); + } + // view params can be left out as defaults will be the same when log event is viewed as when + // it was stored. + // filter params are important, but stored in session so default effectively unknown, + // hence required here. + if (!isset($this->other['view'])) { + throw new \coding_exception('The event mod_attendance\\event\\student_attendance_sessions_viewed must specify view.'); + } + if (!isset($this->other['curdate'])) { + throw new \coding_exception('The event mod_attendance\\event\\student_attendance_sessions_viewed must specify curdate.'); + } + parent::validate_data(); + } +} diff --git a/lang/en/attendance.php b/lang/en/attendance.php index d299db4..dc8ae3f 100644 --- a/lang/en/attendance.php +++ b/lang/en/attendance.php @@ -216,6 +216,7 @@ $string['eventsessionipshared'] = 'Attendance self-marking IP conflict'; $string['eventsessionupdated'] = 'Session updated'; $string['eventstatusadded'] = 'Status added'; $string['eventstatusupdated'] = 'Status updated'; +$string['eventstudentattendancesessionsviewed'] = 'Student attendance sessions viewed'; $string['eventtaken'] = 'Attendance taken'; $string['eventtakenbystudent'] = 'Attendance taken by student'; $string['export'] = 'Export'; diff --git a/version.php b/version.php index 4e3330f..9a1178c 100755 --- a/version.php +++ b/version.php @@ -23,7 +23,7 @@ */ defined('MOODLE_INTERNAL') || die(); -$plugin->version = 2019072900; +$plugin->version = 2019080800; $plugin->requires = 2019072500; // Requires 3.8. $plugin->release = '3.8.0'; $plugin->maturity = MATURITY_STABLE; diff --git a/view.php b/view.php index 97c05e7..9352322 100644 --- a/view.php +++ b/view.php @@ -78,6 +78,24 @@ if (isset($pageparams->studentid) && $USER->id != $pageparams->studentid) { } $userdata = new attendance_user_data($att, $userid); + +// Trigger viewed event + +// Create url for link in log screen. +$filterparams = array( + 'view' => $userdata->pageparams->view, + 'curdate' => $userdata->pageparams->curdate, + 'startdate' => $userdata->pageparams->startdate, + 'enddate' => $userdata->pageparams->enddate +); +$params = array_merge($userdata->pageparams->get_significant_params(), $filterparams); +$event = \mod_attendance\event\student_attendance_sessions_viewed::create(array( + 'relateduserid' => $userdata->pageparams->studentid, + 'context' => $context, + 'other' => $params)); +$event->add_record_snapshot('course_modules', $cm); +$event->trigger(); + $header = new mod_attendance_header($att); echo $output->header();