From c8ea6c60ab388a8df363102ed079927616fdc32f Mon Sep 17 00:00:00 2001
From: Aaron Barnes <aaronb@catalyst.net.nz>
Date: Wed, 6 Oct 2010 02:23:50 +0000
Subject: [PATCH] MDL-24328 completion Check criteria are set before showing
 links to report

---
 course/report/completion/lib.php | 9 +++++----
 course/report/completion/mod.php | 4 ++--
 2 files changed, 7 insertions(+), 6 deletions(-)

diff --git a/course/report/completion/lib.php b/course/report/completion/lib.php
index 7a933217338..a84f3a25651 100644
--- a/course/report/completion/lib.php
+++ b/course/report/completion/lib.php
@@ -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', ''));
+        }
     }
 }
diff --git a/course/report/completion/mod.php b/course/report/completion/mod.php
index 01c25786d79..90359a0ac34 100644
--- a/course/report/completion/mod.php
+++ b/course/report/completion/mod.php
@@ -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>';
         }
     }