diff --git a/course/lib.php b/course/lib.php index b4488d99ac7..dfc33df1c0e 100644 --- a/course/lib.php +++ b/course/lib.php @@ -2929,7 +2929,6 @@ function delete_course_module($id) { // very quick on an empty table) $DB->delete_records('course_modules_completion', array('coursemoduleid' => $cm->id)); $DB->delete_records('course_modules_availability', array('coursemoduleid'=> $cm->id)); - $DB->delete_records('course_completion_criteria', array('moduleinstance'=> $cm->id)); delete_context(CONTEXT_MODULE, $cm->id); return $DB->delete_records('course_modules', array('id'=>$cm->id)); diff --git a/lib/db/upgrade.php b/lib/db/upgrade.php index 28d37ec795d..8d8fffff515 100644 --- a/lib/db/upgrade.php +++ b/lib/db/upgrade.php @@ -317,24 +317,6 @@ function xmldb_main_upgrade($oldversion) { upgrade_main_savepoint(true, 2012031500.04); } - if ($oldversion < 2012032300.01) { - // Delete orphaned criteria which were left when modules were removed - if ($DB->get_dbfamily() === 'mysql') { - $sql = "DELETE cc FROM {course_completion_criteria} cc - LEFT JOIN {course_modules} cm ON cm.id = cc.moduleinstance - WHERE cm.id IS NULL"; - } else { - $sql = "DELETE FROM {course_completion_criteria} - WHERE NOT EXISTS ( - SELECT 'x' FROM {course_modules} - WHERE {course_modules}.id = {course_completion_criteria}.moduleinstance)"; - } - $DB->execute($sql); - - // Main savepoint reached - upgrade_main_savepoint(true, 2012032300.01); - } - if ($oldversion < 2012032300.02) { // Migrate the old admin debug setting. if ($CFG->debug == 38911) {