MDL-37028 Fixed bug, orphaned modules are moved to the 0-section

diff --git a/course/lib.php b/course/lib.php
index 6025325..50494d5 100644 (file)
--- a/course/lib.php
+++ b/course/lib.php
@@ -949,7 +949,7 @@ function course_integrity_check($courseid, $rawmods = null, $sections = null, $f
                 }
                 $sections[$modsection[$cmid]]->newsequence = trim($sections[$modsection[$cmid]]->newsequence.','.$cmid, ',');
                 $messages[] = $debuggingprefix.'Course module ['.$cmid.'] is missing from sequence of section ['.
-                        $sectionid.']';
+                        $modsection[$cmid].']';
             }
         }
         foreach ($modsection as $cmid => $sectionid) {

diff --git a/course/tests/courselib_test.php b/course/tests/courselib_test.php
index 1c19a15..a890de2 100644 (file)
--- a/course/tests/courselib_test.php
+++ b/course/tests/courselib_test.php
@@ -1866,7 +1866,7 @@ class core_course_courselib_testcase extends advanced_testcase {
         $DB->update_record('course_sections', array('id' => $section1->id, 'sequence' => ''));
         $this->assertEquals(array(
             'Failed integrity check for course ['. $course->id. ']. Course module ['. $page->cmid.
-            '] is missing from sequence of section ['. $section1->id. ']',
+            '] is missing from sequence of section ['. $section0->id. ']',
             'Failed integrity check for course ['. $course->id. ']. Course module ['. $page->cmid.
             '] points to section [8765] instead of ['. $section0->id. ']'),
                 course_integrity_check($course->id, null, null, true));