From 668896e5312b0db2bc3bc03230970c492a28f4e7 Mon Sep 17 00:00:00 2001
From: moodler <moodler>
Date: Mon, 28 Aug 2006 02:10:23 +0000
Subject: [PATCH] It's not required that all modules have the old upgrade
 function and the new upgrade function.

---
 lib/adminlib.php | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

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;
-- 
2.39.5