Skip to content

Commit f0775d8

Browse files
authored
Merge pull request #228 from catalyst/issue227-MOODLE_403_STABLE-manager-enrol-fix
Issue #227: Enable managers and admins to be enrolled into course
2 parents 7700fcc + 6df2fca commit f0775d8

File tree

1 file changed

+10
-13
lines changed

1 file changed

+10
-13
lines changed

editattendees.php

Lines changed: 10 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -91,20 +91,17 @@
9191
continue; // Invalid userid.
9292
}
9393

94+
$user = $DB->get_record('user', ['id' => $adduser]);
9495
// Make sure that the user is enroled in the course.
95-
if (!has_capability('moodle/course:view', $context, $adduser)) {
96-
$user = $DB->get_record('user', ['id' => $adduser]);
97-
// Make sure that the user is enroled in the course.
98-
if (!is_enrolled($context, $user)) {
99-
$defaultroleid = null;
100-
// Get default role ID for manual enrollment.
101-
$conditions = ['courseid' => $course->id, 'enrol' => 'manual'];
102-
$defaultroleid = $DB->get_field('enrol', 'roleid', $conditions, IGNORE_MISSING);
103-
if (!enrol_try_internal_enrol($course->id, $user->id, $defaultroleid)) {
104-
$errors[] = get_string('error:enrolmentfailed', 'facetoface', fullname($user));
105-
$errors[] = get_string('error:addattendee', 'facetoface', fullname($user));
106-
continue; // Don't sign the user up.
107-
}
96+
if (!is_enrolled($context, $user)) {
97+
$defaultroleid = null;
98+
// Get default role ID for manual enrollment.
99+
$conditions = ['courseid' => $course->id, 'enrol' => 'manual'];
100+
$defaultroleid = $DB->get_field('enrol', 'roleid', $conditions, IGNORE_MISSING);
101+
if (!enrol_try_internal_enrol($course->id, $user->id, $defaultroleid)) {
102+
$errors[] = get_string('error:enrolmentfailed', 'facetoface', fullname($user));
103+
$errors[] = get_string('error:addattendee', 'facetoface', fullname($user));
104+
continue; // Don't sign the user up.
108105
}
109106
}
110107

0 commit comments

Comments
 (0)