adding capabilities check
This commit is contained in:
@@ -76,8 +76,8 @@ class grade_export {
|
||||
if (! $course = get_record("course", "id", $id)) {
|
||||
error("Course ID was incorrect");
|
||||
}
|
||||
|
||||
require_capability('moodle/course:viewcoursegrades', get_context_instance(CONTEXT_COURSE, $id));
|
||||
$context = get_context_instance(CONTEXT_COURSE, $id);
|
||||
require_capability('moodle/course:viewcoursegrades', $context);
|
||||
|
||||
$this->id = $id;
|
||||
$this->course = $course;
|
||||
@@ -104,7 +104,7 @@ class grade_export {
|
||||
if ($currentgroup) {
|
||||
$this->students = get_group_students($currentgroup, "u.lastname ASC");
|
||||
} else {
|
||||
$this->students = grade_get_course_students($course->id);
|
||||
$this->students = get_role_users(@implode(',', $CFG->gradebookroles), $context);
|
||||
}
|
||||
|
||||
if (!empty($this->students)) {
|
||||
|
||||
@@ -26,6 +26,10 @@ require_once($CFG->dirroot.'/grade/export/lib.php');
|
||||
require_once('grade_export_ods.php');
|
||||
|
||||
$id = required_param('id', PARAM_INT); // course id
|
||||
|
||||
require_login($id);
|
||||
require_capability('moodle/course:downloadallgrades', get_context_instance(CONTEXT_COURSE, $id));
|
||||
|
||||
$course = get_record('course', 'id', $id);
|
||||
$feedback = optional_param('feedback', '', PARAM_ALPHA);
|
||||
$strgrades = get_string('grades', 'grades');
|
||||
|
||||
@@ -26,6 +26,10 @@ require_once($CFG->dirroot.'/grade/export/lib.php');
|
||||
require_once('grade_export_txt.php');
|
||||
|
||||
$id = required_param('id', PARAM_INT); // course id
|
||||
|
||||
require_login($id);
|
||||
require_capability('moodle/course:downloadallgrades', get_context_instance(CONTEXT_COURSE, $id));
|
||||
|
||||
$feedback = optional_param('feedback', '', PARAM_ALPHA);
|
||||
$course = get_record('course', 'id', $id);
|
||||
$strgrades = get_string('grades', 'grades');
|
||||
|
||||
@@ -26,6 +26,10 @@ require_once($CFG->dirroot.'/grade/export/lib.php');
|
||||
require_once('grade_export_xls.php');
|
||||
|
||||
$id = required_param('id', PARAM_INT); // course id
|
||||
|
||||
require_login($id);
|
||||
require_capability('moodle/course:downloadallgrades', get_context_instance(CONTEXT_COURSE, $id));
|
||||
|
||||
$course = get_record('course', 'id', $id);
|
||||
$feedback = optional_param('feedback', '', PARAM_ALPHA);
|
||||
|
||||
|
||||
@@ -25,7 +25,11 @@ require_once("../../../config.php");
|
||||
require_once($CFG->dirroot.'/grade/export/lib.php');
|
||||
require_once('grade_export_xml.php');
|
||||
|
||||
$id = required_param('id', PARAM_INT); // course id
|
||||
$id = required_param('id', PARAM_INT); // course id
|
||||
|
||||
require_login($id);
|
||||
require_capability('moodle/course:downloadallgrades', get_context_instance(CONTEXT_COURSE, $id));
|
||||
|
||||
$course = get_record('course', 'id', $id);
|
||||
$feedback = optional_param('feedback', '', PARAM_ALPHA);
|
||||
|
||||
|
||||
@@ -50,7 +50,7 @@ class grade_import_mapping_form extends moodleform {
|
||||
|
||||
if ($header) {
|
||||
foreach ($header as $i=>$h) {
|
||||
$mapfromoptions[$i] = $h;
|
||||
$mapfromoptions[$i] = s($h);
|
||||
}
|
||||
}
|
||||
$mform->addElement('select', 'mapfrom', get_string('mapfrom', 'grades'), $mapfromoptions);
|
||||
|
||||
Reference in New Issue
Block a user