diff --git a/course/classes/analytics/target/course_enrolments.php b/course/classes/analytics/target/course_enrolments.php index 57b8ee63560..f5321928e5b 100644 --- a/course/classes/analytics/target/course_enrolments.php +++ b/course/classes/analytics/target/course_enrolments.php @@ -74,6 +74,10 @@ abstract class course_enrolments extends \core_analytics\local\target\binary { return get_string('coursenotyetstarted', 'course'); } + if (!$fortraining && !$course->get_course_data()->visible) { + return get_string('hiddenfromstudents'); + } + if (!$this->students = $course->get_students()) { return get_string('nocoursestudents', 'course'); } diff --git a/lib/tests/targets_test.php b/lib/tests/targets_test.php index 24873747085..495076a82de 100644 --- a/lib/tests/targets_test.php +++ b/lib/tests/targets_test.php @@ -132,6 +132,27 @@ class core_analytics_targets_testcase extends advanced_testcase { ], 'isvalid' => get_string('completionnotenabledforcourse', 'completion') ], + 'coursehiddentraining' => [ + 'params' => [ + 'enablecompletion' => 1, + 'startdate' => mktime(0, 0, 0, $month - 1, 24, $year - 1), + 'enddate' => mktime(0, 0, 0, $month - 1, 23, $year), + 'students' => true, + 'visible' => '0', + ], + 'isvalid' => true, + ], + 'coursehiddenprediction' => [ + 'params' => [ + 'enablecompletion' => 1, + 'startdate' => mktime(0, 0, 0, $month - 1, 24, $year), + 'enddate' => mktime(0, 0, 0, $month - 1, 23, $year + 1), + 'students' => true, + 'visible' => '0', + ], + 'isvalid' => get_string('hiddenfromstudents'), + 'fortraining' => false + ], ]; }