MDL-37028 Fixed bug, orphaned modules are moved to the 0-section
authorMarina Glancy <marina@moodle.com>
Fri, 13 Sep 2013 00:28:52 +0000 (10:28 +1000)
committerMarina Glancy <marina@moodle.com>
Fri, 13 Sep 2013 00:28:52 +0000 (10:28 +1000)
course/lib.php
course/tests/courselib_test.php

index 6025325..50494d5 100644 (file)
@@ -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) {
index 1c19a15..a890de2 100644 (file)
@@ -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));