From: defacer <defacer>
Date: Thu, 27 May 2004 14:16:27 +0000 (+0000)
Subject: Fixed a bug: blocks without db functions caused the upgrade process to become stuck.
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=b88f7004929792b314668c18b3df7a07d511faaf;p=moodle.git

Fixed a bug: blocks without db functions caused the upgrade process to become stuck.
---

diff --git a/lib/blocklib.php b/lib/blocklib.php
index 637e7f51b3..6610c9c9d8 100644
--- a/lib/blocklib.php
+++ b/lib/blocklib.php
@@ -496,23 +496,32 @@ function upgrade_blocks_plugins($continueto) {
                     $strblocksetup    = get_string("blocksetup");
                     print_header($strblocksetup, $strblocksetup, $strblocksetup, "", "", false, "&nbsp;", "&nbsp;");
                 }
-                print_heading("$block->name block needs upgrading");
-                $upgrade_function = $block->name."_upgrade";
+                print_heading('New version of '.$blocktitle.' ('.$block->name.') exists');
+                $upgrade_function = $block->name.'_upgrade';
                 if (function_exists($upgrade_function)) {
                     $db->debug=true;
                     if ($upgrade_function($currblock->version, $block)) {
-                        $db->debug=false;
-                        // OK so far, now update the blocks record
-                        $block->id = $currblock->id;
-                        if (! update_record("blocks", $block)) {
-                            error("Could not update block $block->name record in blocks table!");
-                        }
-                        notify(get_string('blocksuccess', '', $blocktitle), 'green');
-                        echo "<HR>";
+
+                        $upgradesuccess = true;
                     } else {
-                        $db->debug=false;
-                        notify("Upgrading block $block->name from $currblock->version to $block->version FAILED!");
+                        $upgradesuccess = false;
                     }
+                    $db->debug=false;
+                }
+                else {
+                    $upgradesuccess = true;
+                }
+                if(!$upgradesuccess) {
+                    notify("Upgrading block $block->name from $currblock->version to $block->version FAILED!");
+                }
+                else {
+                    // OK so far, now update the blocks record
+                    $block->id = $currblock->id;
+                    if (! update_record('blocks', $block)) {
+                        error("Could not update block $block->name record in blocks table!");
+                    }
+                    notify(get_string('blocksuccess', '', $blocktitle), 'green');
+                    echo '<hr />';
                 }
                 $updated_blocks = true;
             } else {