mirror_php/moodle
1
0
Fork 0
mirror of https://github.com/moodle/moodle.git synced 2025-04-21 08:22:07 +02:00
Code Issues Projects Releases Wiki Activity

MDL-47527 gradebook: Allow access to overview report

Browse Source
This commit is contained in:
Paula Ardanza Menéndez 2014-11-10 11:26:42 -05:00
parent 683d7de4d1
commit 7852746292
2 changed files with 14 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
grade/report/overview/index.php
View File

@ -43,7 +43,7 @@ $systemcontext = context_system::instance();
require_capability('gradereport/overview:view', $context);
if (empty($userid)) {
require_capability('moodle/grade:viewall', $systemcontext);
require_capability('moodle/grade:viewall', $context);
} else {
if (!$DB->get_record('user', array('id'=>$userid, 'deleted'=>0)) or isguestuser($userid)) {
@ -53,19 +53,19 @@ if (empty($userid)) {
$access = false;
if (has_capability('moodle/grade:viewall', $systemcontext)) {
//ok - can view all course grades
// Ok - can view all course grades.
$access = true;
} else if ($userid == $USER->id and has_capability('moodle/grade:viewall', $context)) {
//ok - can view any own grades
} else if (has_capability('moodle/grade:viewall', $context)) {
// Ok - can view any grades in context.
$access = true;
} else if ($userid == $USER->id and has_capability('moodle/grade:view', $context) and $course->showgrades) {
//ok - can view own course grades
// Ok - can view own course grades.
$access = true;
} else if (has_capability('moodle/grade:viewall', context_user::instance($userid)) and $course->showgrades) {
// ok - can view grades of this user- parent most probably
// Ok - can view grades of this user- parent most probably.
$access = true;
}
@ -86,8 +86,8 @@ $USER->grade_last_report[$course->id] = 'overview';
//first make sure we have proper final grades - this must be done before constructing of the grade tree
grade_regrade_final_grades($courseid);
if (has_capability('moodle/grade:viewall', $systemcontext)) { //Admins will see all student reports
// please note this would be extremely slow if we wanted to implement this properly for all teachers
if (has_capability('moodle/grade:viewall', $context)) {
// Please note this would be extremely slow if we wanted to implement this properly for all teachers.
$groupmode = groups_get_course_groupmode($course); // Groups are being used
$currentgroup = groups_get_course_group($course, true);

7
grade/report/overview/lib.php
View File

@ -135,7 +135,7 @@ class grade_report_overview extends grade_report {
}
public function fill_table() {
global $CFG, $DB, $OUTPUT;
global $CFG, $DB, $OUTPUT, $USER;
// Only show user's courses instead of all courses.
if ($this->courses) {
@ -153,6 +153,11 @@ class grade_report_overview extends grade_report {
continue;
}
if ((!has_capability('moodle/grade:view', $coursecontext) || $this->user->id != $USER->id) &&
!has_capability('moodle/grade:viewall', $coursecontext)) {
continue;
}
$courseshortname = format_string($course->shortname, true, array('context' => $coursecontext));
$courselink = html_writer::link(new moodle_url('/grade/report/user/index.php', array('id' => $course->id, 'userid' => $this->user->id)), $courseshortname);
$canviewhidden = has_capability('moodle/grade:viewhidden', $coursecontext);