MDL-24328 completion Check criteria are set before showing links to report
This commit is contained in:
@@ -37,9 +37,10 @@ function completion_report_extend_navigation($navigation, $course, $context) {
|
||||
global $CFG, $OUTPUT;
|
||||
|
||||
if (has_capability('coursereport/completion:view', $context)) {
|
||||
require_once($CFG->libdir.'/completionlib.php');
|
||||
|
||||
$url = new moodle_url('/course/report/completion/index.php', array('course'=>$course->id));
|
||||
$navigation->add(get_string('pluginname','coursereport_completion'), $url, navigation_node::TYPE_SETTING, null, null, new pix_icon('i/report', ''));
|
||||
$completion = new completion_info($course);
|
||||
if ($completion->is_enabled() && $completion->has_criteria()) {
|
||||
$url = new moodle_url('/course/report/completion/index.php', array('course'=>$course->id));
|
||||
$navigation->add(get_string('pluginname','coursereport_completion'), $url, navigation_node::TYPE_SETTING, null, null, new pix_icon('i/report', ''));
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
@@ -6,9 +6,9 @@
|
||||
|
||||
if (has_capability('coursereport/completion:view', $context)) {
|
||||
$completion = new completion_info($course);
|
||||
if ($completion->is_enabled()) {
|
||||
if ($completion->is_enabled() && $completion->has_criteria()) {
|
||||
echo '<p>';
|
||||
echo '<a href="'.$CFG->wwwroot.'/course/report/completion/index.php?course='.$course->id.'">'.get_string('coursecompletionreport','completion').'</a>';
|
||||
echo '<a href="'.$CFG->wwwroot.'/course/report/completion/index.php?coursetest='.$course->id.'">'.get_string('coursecompletionreport','completion').'</a>';
|
||||
echo '</p>';
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user