mirror_php/moodle
1
0
Fork 0
mirror of https://github.com/moodle/moodle.git synced 2025-01-18 05:58:34 +01:00
Code Issues Projects Releases Wiki Activity

MDL-54030 enrol_lti: confirm tool exists before deleting users

Browse Source
This commit is contained in:
Mark Nelson 2016-05-05 12:26:11 +08:00 committed by Andrew Nicols
parent 8668796317
commit a8abd53a32
1 changed files with 6 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
enrol/lti/lib.php
View File

@ -174,11 +174,12 @@ class enrol_lti_plugin extends enrol_plugin {
public function unenrol_user(stdClass $instance, $userid) {
global $DB;
// Get the tool associated with this instance.
$tool = $DB->get_record('enrol_lti_tools', array('enrolid' => $instance->id), 'id', MUST_EXIST);
// Need to remove the user from the users table.
$DB->delete_records('enrol_lti_users', array('userid' => $userid, 'toolid' => $tool->id));
// Get the tool associated with this instance. Note - it may not exist if we have deleted
// the tool. This is fine because we have already cleaned the 'enrol_lti_users' table.
if ($tool = $DB->get_record('enrol_lti_tools', array('enrolid' => $instance->id), 'id')) {
// Need to remove the user from the users table.
$DB->delete_records('enrol_lti_users', array('userid' => $userid, 'toolid' => $tool->id));
}
parent::unenrol_user($instance, $userid);
}