Update course cache when modules are moved up or down

diff --git a/course/lib.php b/course/lib.php
index a3ed29db8ad5bd7939a97178aec6e69299874bc2..7d85456909df4f85a64258a73d5c3cd846e7abd7 100644 (file)
--- a/course/lib.php
+++ b/course/lib.php
@@ -722,17 +722,13 @@ function delete_mod_from_section($mod, $section) {
 }
 
 
-function move_module($id, $move) {
+function move_module($cm, $move) {
     GLOBAL $db;
 
     if (!$move) {
         return true;
     }
 
-    if (! $cm = get_record("course_modules", "id", $id)) {
-        error("This course module doesn't exist");
-    }
-    
     if (! $thissection = get_record("course_sections", "id", $cm->section)) {
         error("This course section doesn't exist");
     }

diff --git a/course/mod.php b/course/mod.php
index 77908b7ac80c878930bc860d39bf48bbd17fc8f6..638fa168688742fae9d9401d1651858b3f88ca21 100644 (file)
--- a/course/mod.php
+++ b/course/mod.php
@@ -87,7 +87,6 @@
         }
 
         $modinfo = serialize(get_array_of_activities($mod->course));
-
         if (!set_field("course", "modinfo", $modinfo, "id", $mod->course)) {
             error("Could not cache module information!");
         }
@@ -108,7 +107,16 @@
 
         require_variable($id);   
 
-        move_module($id, $move);
+        if (! $cm = get_record("course_modules", "id", $id)) {
+            error("This course module doesn't exist");
+        }
+    
+        move_module($cm, $move);
+
+        $modinfo = serialize(get_array_of_activities($cm->course));
+        if (!set_field("course", "modinfo", $modinfo, "id", $cm->course)) {
+            error("Could not cache module information!");
+        }
 
         redirect($HTTP_REFERER);
         exit;