MDL-32508 Add error when trying to access an unknown course section

diff --git a/course/format/renderer.php b/course/format/renderer.php
index e71f2a5..510de03 100644 (file)
--- a/course/format/renderer.php
+++ b/course/format/renderer.php
@@ -419,6 +419,12 @@ abstract class format_section_renderer_base extends plugin_renderer_base {
         $context = context_course::instance($course->id);
         $canviewhidden = has_capability('moodle/course:viewhiddensections', $context);
 
+        if (!isset($sections[$displaysection])) {
+            // This section doesn't exist
+            print_error('unknowncoursesection', 'error', null, $course->fullname);
+            return;
+        }
+
         if (!$sections[$displaysection]->visible && !$canviewhidden) {
             if (!$course->hiddensections) {
                 echo $this->start_section_list();

diff --git a/lang/en/error.php b/lang/en/error.php
index ad5affc..0d45834 100644 (file)
--- a/lang/en/error.php
+++ b/lang/en/error.php
@@ -475,6 +475,7 @@ $string['unknownblockregion'] = 'The block region \'{$a}\' is not recognised on
 $string['unknowncontext'] = 'This is an unknown context.';
 $string['unknowncourse'] = 'Unknown course named "{$a}"';
 $string['unknowncourseidnumber'] = 'Unknown course ID "{$a}"';
+$string['unknowncoursesection'] = 'Unknown course section in course "{$a}"';
 $string['unknowncourserequest'] = 'Unknown course request';
 $string['unknownfiletype'] = 'Error unknown filtertype';
 $string['unknowngroup'] = 'Unknown group "{$a}"';