Browse Source

Security fix changed to use mod/attendance:viewreports capability.

Refactored the code.
MOODLE_26_STABLE
NeillM 11 years ago
parent
commit
9aaf4d69d6
  1. 11
      view.php

11
view.php

@ -63,9 +63,14 @@ $PAGE->navbar->add(get_string('attendancereport', 'attendance'));
$output = $PAGE->get_renderer('mod_attendance'); $output = $PAGE->get_renderer('mod_attendance');
// Only users with proper permissions should be able to see any users individual report. if (isset($pageparams->studentid) && has_capability('mod/attendance:viewreports', $PAGE->context)) {
$userid = (isset($pageparams->studentid) && // Only users with proper permissions should be able to see any user's individual report.
($att->perm->can_manage() || $att->perm->can_take() || $att->perm->can_change())) ? $pageparams->studentid : $USER->id; $userid = $pageparams->studentid;
} else {
// A valid request to see another users report has not been sent, show the user's own.
$userid = $USER->id;
}
$userdata = new attendance_user_data($att, $userid); $userdata = new attendance_user_data($att, $userid);
echo $output->header(); echo $output->header();

Loading…
Cancel
Save