From: moodler <moodler>
Date: Mon, 28 Aug 2006 02:10:23 +0000 (+0000)
Subject: It's not required that all modules have the old upgrade function and
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=668896e5312b0db2bc3bc03230970c492a28f4e7;p=moodle.git

It's not required that all modules have the old upgrade function and
the new upgrade function.
---

diff --git a/lib/adminlib.php b/lib/adminlib.php
index 13a8edae0b..3bc935827d 100644
--- a/lib/adminlib.php
+++ b/lib/adminlib.php
@@ -243,7 +243,6 @@ function upgrade_activity_modules($return) {
                 } else {
                     notify ('Upgrade function ' . $oldupgrade_function . ' was not available in ' .
                              $mod . ': ' . $fullmod . '/db/' . $CFG->dbtype . '.php');
-                    continue;
                 }
 
             /// Then, the new function if exists and the old one was ok
@@ -254,11 +253,10 @@ function upgrade_activity_modules($return) {
                 } else {
                     notify ('Upgrade function ' . $newupgrade_function . ' was not available in ' .
                              $mod . ': ' . $fullmod . '/db/upgrade.php');
-                    continue;
                 }
 
             /// Now analyze upgrade results
-                if ($oldupgrade_status && $newupgrade_status) {
+                if ($oldupgrade_status && $newupgrade_status) {    // No upgrading failed
                     $db->debug=false;
                     // OK so far, now update the modules record
                     $module->id = $currmodule->id;