diff --git a/enrol/renderer.php b/enrol/renderer.php index 9f7d8fad35af7..2a2ee1f8caa1b 100644 --- a/enrol/renderer.php +++ b/enrol/renderer.php @@ -193,7 +193,7 @@ public function user_roles_and_actions($userid, $roles, $assignableroles, $canas if ($canassign and (is_siteadmin() or isset($assignableroles[$roleid])) and !$role['unchangeable']) { $strunassign = get_string('unassignarole', 'role', $role['text']); $icon = html_writer::empty_tag('img', array('alt'=>$strunassign, 'src'=>$iconenrolremove)); - $url = new moodle_url($pageurl, array('action'=>'unassign', 'role'=>$roleid, 'user'=>$userid)); + $url = new moodle_url($pageurl, array('action'=>'unassign', 'roleid'=>$roleid, 'user'=>$userid)); $rolesoutput .= html_writer::tag('div', $role['text'] . html_writer::link($url, $icon, array('class'=>'unassignrolelink', 'rel'=>$roleid, 'title'=>$strunassign)), array('class'=>'role role_'.$roleid)); } else { $rolesoutput .= html_writer::tag('div', $role['text'], array('class'=>'role unchangeable', 'rel'=>$roleid)); diff --git a/enrol/users.php b/enrol/users.php index d8453fb1f80e6..1101a9a28a9b1 100644 --- a/enrol/users.php +++ b/enrol/users.php @@ -73,7 +73,7 @@ */ case 'unassign': if (has_capability('moodle/role:assign', $manager->get_context())) { - $role = required_param('role', PARAM_INT); + $role = required_param('roleid', PARAM_INT); $user = required_param('user', PARAM_INT); if ($confirm && $manager->unassign_role_from_user($user, $role)) { redirect($PAGE->url); @@ -81,7 +81,7 @@ $user = $DB->get_record('user', array('id'=>$user), '*', MUST_EXIST); $allroles = $manager->get_all_roles(); $role = $allroles[$role]; - $yesurl = new moodle_url($PAGE->url, array('action'=>'unassign', 'role'=>$role->id, 'user'=>$user->id, 'confirm'=>1, 'sesskey'=>sesskey())); + $yesurl = new moodle_url($PAGE->url, array('action'=>'unassign', 'roleid'=>$role->id, 'user'=>$user->id, 'confirm'=>1, 'sesskey'=>sesskey())); $message = get_string('unassignconfirm', 'role', array('user'=>fullname($user, true), 'role'=>$role->localname)); $pagetitle = get_string('unassignarole', 'role', $role->localname); $pagecontent = $OUTPUT->confirm($message, $yesurl, $PAGE->url);