MDL-23068 removing obsolete $result and removing empty upgrade files that would be...
authorPetr Skoda <skodak@moodle.org>
Sun, 4 Jul 2010 10:27:56 +0000 (10:27 +0000)
committerPetr Skoda <skodak@moodle.org>
Sun, 4 Jul 2010 10:27:56 +0000 (10:27 +0000)
73 files changed:
admin/xmldb/actions/XMLDBAction.class.php
blocks/activity_modules/db/upgrade.php [deleted file]
blocks/calendar_month/db/upgrade.php [deleted file]
blocks/calendar_upcoming/db/upgrade.php [deleted file]
blocks/community/db/upgrade.php
blocks/course_list/db/upgrade.php [deleted file]
blocks/course_summary/db/upgrade.php [deleted file]
blocks/news_items/db/upgrade.php [deleted file]
blocks/online_users/db/upgrade.php [deleted file]
blocks/participants/db/upgrade.php [deleted file]
blocks/recent_activity/db/upgrade.php [deleted file]
blocks/rss_client/db/upgrade.php [deleted file]
blocks/search/db/upgrade.php [deleted file]
blocks/search_forums/db/upgrade.php [deleted file]
blocks/section_links/db/upgrade.php [deleted file]
blocks/social_activities/db/upgrade.php [deleted file]
enrol/authorize/db/upgrade.php
enrol/paypal/db/upgrade.php [deleted file]
lib/db/upgrade.php
lib/xmldb/xmldb_constants.php
message/output/email/db/upgrade.php
message/output/jabber/db/upgrade.php
message/output/popup/db/upgrade.php
mod/assignment/db/upgrade.php
mod/chat/db/upgrade.php
mod/choice/db/upgrade.php
mod/data/db/upgrade.php
mod/feedback/db/upgrade.php
mod/folder/db/upgrade.php
mod/forum/db/upgrade.php
mod/glossary/db/upgrade.php
mod/hotpot/db/upgrade.php
mod/imscp/db/upgrade.php
mod/label/db/upgrade.php
mod/lesson/db/upgrade.php
mod/page/db/upgrade.php
mod/quiz/db/upgrade.php
mod/quiz/report/overview/db/upgrade.php
mod/quiz/report/statistics/db/upgrade.php
mod/resource/db/upgrade.php
mod/scorm/db/upgrade.php
mod/survey/db/upgrade.php
mod/url/db/upgrade.php
mod/wiki/db/upgrade.php
mod/workshop/db/upgrade.php
portfolio/mahara/db/upgrade.php [deleted file]
question/type/calculated/db/upgrade.php
question/type/essay/db/upgrade.php [deleted file]
question/type/match/db/upgrade.php [deleted file]
question/type/multianswer/db/upgrade.php
question/type/multichoice/db/upgrade.php
question/type/numerical/db/upgrade.php
question/type/randomsamatch/db/upgrade.php [deleted file]
question/type/shortanswer/db/upgrade.php [deleted file]
question/type/truefalse/db/upgrade.php [deleted file]
repository/alfresco/db/upgrade.php [deleted file]
repository/boxnet/db/upgrade.php [deleted file]
repository/dropbox/db/upgrade.php [deleted file]
repository/filesystem/db/upgrade.php [deleted file]
repository/flickr/db/upgrade.php [deleted file]
repository/flickr_public/db/upgrade.php [deleted file]
repository/googledocs/db/upgrade.php [deleted file]
repository/local/db/upgrade.php [deleted file]
repository/merlot/db/upgrade.php [deleted file]
repository/picasa/db/upgrade.php [deleted file]
repository/recent/db/upgrade.php [deleted file]
repository/s3/db/upgrade.php [deleted file]
repository/upload/db/upgrade.php [deleted file]
repository/url/db/upgrade.php [deleted file]
repository/user/db/upgrade.php [deleted file]
repository/webdav/db/upgrade.php [deleted file]
repository/wikimedia/db/upgrade.php [deleted file]
repository/youtube/db/upgrade.php [deleted file]

index c388ced..2ee6509 100644 (file)
@@ -239,22 +239,22 @@ class XMLDBAction {
             case 'lib': /// has own savepoint function
                 $result = XMLDB_LINEFEED .
                           '    /// Main savepoint reached' . XMLDB_LINEFEED .
-                          '        upgrade_main_savepoint($result, XXXXXXXXXX);' . XMLDB_LINEFEED;
+                          '        upgrade_main_savepoint(true, XXXXXXXXXX);' . XMLDB_LINEFEED;
                 break;
             case 'mod': /// has own savepoint function
                 $result = XMLDB_LINEFEED .
                           '    /// ' . $pluginname . ' savepoint reached' . XMLDB_LINEFEED .
-                          '        upgrade_mod_savepoint($result, XXXXXXXXXX, ' . "'$pluginname'" . ');' . XMLDB_LINEFEED;
+                          '        upgrade_mod_savepoint(true, XXXXXXXXXX, ' . "'$pluginname'" . ');' . XMLDB_LINEFEED;
                 break;
             case 'block': /// has own savepoint function
                 $result = XMLDB_LINEFEED .
                           '    /// ' . $pluginname . ' savepoint reached' . XMLDB_LINEFEED .
-                          '        upgrade_block_savepoint($result, XXXXXXXXXX, ' . "'$pluginname'" . ');' . XMLDB_LINEFEED;
+                          '        upgrade_block_savepoint(true, XXXXXXXXXX, ' . "'$pluginname'" . ');' . XMLDB_LINEFEED;
                 break;
             default: /// rest of plugins
                 $result = XMLDB_LINEFEED .
                           '    /// ' . $pluginname . ' savepoint reached' . XMLDB_LINEFEED .
-                          '        upgrade_plugin_savepoint($result, XXXXXXXXXX, ' . "'$plugintype'" . ', ' . "'$pluginname'" . ');' . XMLDB_LINEFEED;
+                          '        upgrade_plugin_savepoint(true, XXXXXXXXXX, ' . "'$plugintype'" . ', ' . "'$pluginname'" . ');' . XMLDB_LINEFEED;
         }
         return $result;
     }
diff --git a/blocks/activity_modules/db/upgrade.php b/blocks/activity_modules/db/upgrade.php
deleted file mode 100644 (file)
index adc675a..0000000
+++ /dev/null
@@ -1,41 +0,0 @@
-<?php
-
-// This file keeps track of upgrades to
-// the activity_modules block
-//
-// Sometimes, changes between versions involve
-// alterations to database structures and other
-// major things that may break installations.
-//
-// The upgrade function in this file will attempt
-// to perform all the necessary actions to upgrade
-// your older installtion to the current version.
-//
-// If there's something it cannot do itself, it
-// will tell you what you need to do.
-//
-// The commands in here will all be database-neutral,
-// using the methods of database_manager class
-//
-// Please do not forget to use upgrade_set_timeout()
-// before any action that may take longer time to finish.
-
-function xmldb_block_activity_modules_upgrade($oldversion) {
-    global $CFG, $DB;
-
-    $dbman = $DB->get_manager();
-    $result = true;
-
-/// And upgrade begins here. For each one, you'll need one
-/// block of code similar to the next one. Please, delete
-/// this comment lines once this file start handling proper
-/// upgrade code.
-
-/// if ($result && $oldversion < YYYYMMDD00) { //New version in version.php
-///     $result = result of database_manager methods
-/// }
-
-    return $result;
-}
-
-
diff --git a/blocks/calendar_month/db/upgrade.php b/blocks/calendar_month/db/upgrade.php
deleted file mode 100644 (file)
index e727494..0000000
+++ /dev/null
@@ -1,41 +0,0 @@
-<?php
-
-// This file keeps track of upgrades to
-// the calendar_month block
-//
-// Sometimes, changes between versions involve
-// alterations to database structures and other
-// major things that may break installations.
-//
-// The upgrade function in this file will attempt
-// to perform all the necessary actions to upgrade
-// your older installtion to the current version.
-//
-// If there's something it cannot do itself, it
-// will tell you what you need to do.
-//
-// The commands in here will all be database-neutral,
-// using the methods of database_manager class
-//
-// Please do not forget to use upgrade_set_timeout()
-// before any action that may take longer time to finish.
-
-function xmldb_block_calendar_month_upgrade($oldversion) {
-    global $CFG, $DB;
-
-    $dbman = $DB->get_manager();
-    $result = true;
-
-/// And upgrade begins here. For each one, you'll need one
-/// block of code similar to the next one. Please, delete
-/// this comment lines once this file start handling proper
-/// upgrade code.
-
-/// if ($result && $oldversion < YYYYMMDD00) { //New version in version.php
-///     $result = result of database_manager methods
-/// }
-
-    return $result;
-}
-
-
diff --git a/blocks/calendar_upcoming/db/upgrade.php b/blocks/calendar_upcoming/db/upgrade.php
deleted file mode 100644 (file)
index 604b08c..0000000
+++ /dev/null
@@ -1,41 +0,0 @@
-<?php
-
-// This file keeps track of upgrades to
-// the calendar_upcoming block
-//
-// Sometimes, changes between versions involve
-// alterations to database structures and other
-// major things that may break installations.
-//
-// The upgrade function in this file will attempt
-// to perform all the necessary actions to upgrade
-// your older installtion to the current version.
-//
-// If there's something it cannot do itself, it
-// will tell you what you need to do.
-//
-// The commands in here will all be database-neutral,
-// using the methods of database_manager class
-//
-// Please do not forget to use upgrade_set_timeout()
-// before any action that may take longer time to finish.
-
-function xmldb_block_calendar_upcoming_upgrade($oldversion) {
-    global $CFG, $DB;
-
-    $dbman = $DB->get_manager();
-    $result = true;
-
-/// And upgrade begins here. For each one, you'll need one
-/// block of code similar to the next one. Please, delete
-/// this comment lines once this file start handling proper
-/// upgrade code.
-
-/// if ($result && $oldversion < YYYYMMDD00) { //New version in version.php
-///     $result = result of database_manager methods
-/// }
-
-    return $result;
-}
-
-
index 942099e..7ea6fd8 100644 (file)
@@ -47,9 +47,8 @@ function xmldb_block_community_upgrade($oldversion) {
     global $CFG, $DB;
 
     $dbman = $DB->get_manager();
-    $result = true;
 
-    if ($result && $oldversion < 2010042701) {
+    if ($oldversion < 2010042701) {
 
         /// Define table block_community to be created
         $table = new xmldb_table('block_community');
@@ -71,9 +70,9 @@ function xmldb_block_community_upgrade($oldversion) {
         }
 
         /// community savepoint reached
-        upgrade_block_savepoint($result, 2010042701, 'community');
+        upgrade_block_savepoint(true, 2010042701, 'community');
     }
 
 
-    return $result;
+    return true;
 }
\ No newline at end of file
diff --git a/blocks/course_list/db/upgrade.php b/blocks/course_list/db/upgrade.php
deleted file mode 100644 (file)
index bc4255c..0000000
+++ /dev/null
@@ -1,41 +0,0 @@
-<?php
-
-// This file keeps track of upgrades to
-// the course_list block
-//
-// Sometimes, changes between versions involve
-// alterations to database structures and other
-// major things that may break installations.
-//
-// The upgrade function in this file will attempt
-// to perform all the necessary actions to upgrade
-// your older installtion to the current version.
-//
-// If there's something it cannot do itself, it
-// will tell you what you need to do.
-//
-// The commands in here will all be database-neutral,
-// using the methods of database_manager class
-//
-// Please do not forget to use upgrade_set_timeout()
-// before any action that may take longer time to finish.
-
-function xmldb_block_course_list_upgrade($oldversion) {
-    global $CFG, $DB;
-
-    $dbman = $DB->get_manager();
-    $result = true;
-
-/// And upgrade begins here. For each one, you'll need one
-/// block of code similar to the next one. Please, delete
-/// this comment lines once this file start handling proper
-/// upgrade code.
-
-/// if ($result && $oldversion < YYYYMMDD00) { //New version in version.php
-///     $result = result of database_manager methods
-/// }
-
-    return $result;
-}
-
-
diff --git a/blocks/course_summary/db/upgrade.php b/blocks/course_summary/db/upgrade.php
deleted file mode 100644 (file)
index d408e84..0000000
+++ /dev/null
@@ -1,41 +0,0 @@
-<?php
-
-// This file keeps track of upgrades to
-// the course_summary block
-//
-// Sometimes, changes between versions involve
-// alterations to database structures and other
-// major things that may break installations.
-//
-// The upgrade function in this file will attempt
-// to perform all the necessary actions to upgrade
-// your older installation to the current version.
-//
-// If there's something it cannot do itself, it
-// will tell you what you need to do.
-//
-// The commands in here will all be database-neutral,
-// using the methods of database_manager class
-//
-// Please do not forget to use upgrade_set_timeout()
-// before any action that may take longer time to finish.
-
-function xmldb_block_course_summary_upgrade($oldversion) {
-    global $CFG, $DB;
-
-    $dbman = $DB->get_manager();
-    $result = true;
-
-/// And upgrade begins here. For each one, you'll need one
-/// block of code similar to the next one. Please, delete
-/// this comment lines once this file start handling proper
-/// upgrade code.
-
-/// if ($result && $oldversion < YYYYMMDD00) { //New version in version.php
-///     $result = result of database_manager methods
-/// }
-
-    return $result;
-}
-
-
diff --git a/blocks/news_items/db/upgrade.php b/blocks/news_items/db/upgrade.php
deleted file mode 100644 (file)
index 81fc020..0000000
+++ /dev/null
@@ -1,41 +0,0 @@
-<?php
-
-// This file keeps track of upgrades to
-// the news_items block
-//
-// Sometimes, changes between versions involve
-// alterations to database structures and other
-// major things that may break installations.
-//
-// The upgrade function in this file will attempt
-// to perform all the necessary actions to upgrade
-// your older installtion to the current version.
-//
-// If there's something it cannot do itself, it
-// will tell you what you need to do.
-//
-// The commands in here will all be database-neutral,
-// using the methods of database_manager class
-//
-// Please do not forget to use upgrade_set_timeout()
-// before any action that may take longer time to finish.
-
-function xmldb_block_news_items_upgrade($oldversion) {
-    global $CFG, $DB;
-
-    $dbman = $DB->get_manager();
-    $result = true;
-
-/// And upgrade begins here. For each one, you'll need one
-/// block of code similar to the next one. Please, delete
-/// this comment lines once this file start handling proper
-/// upgrade code.
-
-/// if ($result && $oldversion < YYYYMMDD00) { //New version in version.php
-///     $result = result of database_manager methods
-/// }
-
-    return $result;
-}
-
-
diff --git a/blocks/online_users/db/upgrade.php b/blocks/online_users/db/upgrade.php
deleted file mode 100644 (file)
index 0773b0d..0000000
+++ /dev/null
@@ -1,41 +0,0 @@
-<?php
-
-// This file keeps track of upgrades to
-// the online_users block
-//
-// Sometimes, changes between versions involve
-// alterations to database structures and other
-// major things that may break installations.
-//
-// The upgrade function in this file will attempt
-// to perform all the necessary actions to upgrade
-// your older installtion to the current version.
-//
-// If there's something it cannot do itself, it
-// will tell you what you need to do.
-//
-// The commands in here will all be database-neutral,
-// using the methods of database_manager class
-//
-// Please do not forget to use upgrade_set_timeout()
-// before any action that may take longer time to finish.
-
-function xmldb_block_online_users_upgrade($oldversion) {
-    global $CFG, $DB;
-
-    $dbman = $DB->get_manager();
-    $result = true;
-
-/// And upgrade begins here. For each one, you'll need one
-/// block of code similar to the next one. Please, delete
-/// this comment lines once this file start handling proper
-/// upgrade code.
-
-/// if ($result && $oldversion < YYYYMMDD00) { //New version in version.php
-///     $result = result of database_manager methods
-/// }
-
-    return $result;
-}
-
-
diff --git a/blocks/participants/db/upgrade.php b/blocks/participants/db/upgrade.php
deleted file mode 100644 (file)
index 9c161dd..0000000
+++ /dev/null
@@ -1,41 +0,0 @@
-<?php
-
-// This file keeps track of upgrades to
-// the participants block
-//
-// Sometimes, changes between versions involve
-// alterations to database structures and other
-// major things that may break installations.
-//
-// The upgrade function in this file will attempt
-// to perform all the necessary actions to upgrade
-// your older installtion to the current version.
-//
-// If there's something it cannot do itself, it
-// will tell you what you need to do.
-//
-// The commands in here will all be database-neutral,
-// using the methods of database_manager class
-//
-// Please do not forget to use upgrade_set_timeout()
-// before any action that may take longer time to finish.
-
-function xmldb_block_participants_upgrade($oldversion) {
-    global $CFG, $DB;
-
-    $dbman = $DB->get_manager();
-    $result = true;
-
-/// And upgrade begins here. For each one, you'll need one
-/// block of code similar to the next one. Please, delete
-/// this comment lines once this file start handling proper
-/// upgrade code.
-
-/// if ($result && $oldversion < YYYYMMDD00) { //New version in version.php
-///     $result = result of database_manager methods
-/// }
-
-    return $result;
-}
-
-
diff --git a/blocks/recent_activity/db/upgrade.php b/blocks/recent_activity/db/upgrade.php
deleted file mode 100644 (file)
index 817cee6..0000000
+++ /dev/null
@@ -1,41 +0,0 @@
-<?php
-
-// This file keeps track of upgrades to
-// the recent_activity block
-//
-// Sometimes, changes between versions involve
-// alterations to database structures and other
-// major things that may break installations.
-//
-// The upgrade function in this file will attempt
-// to perform all the necessary actions to upgrade
-// your older installtion to the current version.
-//
-// If there's something it cannot do itself, it
-// will tell you what you need to do.
-//
-// The commands in here will all be database-neutral,
-// using the methods of database_manager class
-//
-// Please do not forget to use upgrade_set_timeout()
-// before any action that may take longer time to finish.
-
-function xmldb_block_recent_activity_upgrade($oldversion) {
-    global $CFG, $DB;
-
-    $dbman = $DB->get_manager();
-    $result = true;
-
-/// And upgrade begins here. For each one, you'll need one
-/// block of code similar to the next one. Please, delete
-/// this comment lines once this file start handling proper
-/// upgrade code.
-
-/// if ($result && $oldversion < YYYYMMDD00) { //New version in version.php
-///     $result = result of database_manager methods
-/// }
-
-    return $result;
-}
-
-
diff --git a/blocks/rss_client/db/upgrade.php b/blocks/rss_client/db/upgrade.php
deleted file mode 100644 (file)
index 8946f54..0000000
+++ /dev/null
@@ -1,52 +0,0 @@
-<?php
-
-// This file keeps track of upgrades to
-// the rss_client block
-//
-// Sometimes, changes between versions involve
-// alterations to database structures and other
-// major things that may break installations.
-//
-// The upgrade function in this file will attempt
-// to perform all the necessary actions to upgrade
-// your older installtion to the current version.
-//
-// If there's something it cannot do itself, it
-// will tell you what you need to do.
-//
-// The commands in here will all be database-neutral,
-// using the methods of database_manager class
-//
-// Please do not forget to use upgrade_set_timeout()
-// before any action that may take longer time to finish.
-
-function xmldb_block_rss_client_upgrade($oldversion) {
-    global $CFG, $DB;
-
-    $dbman = $DB->get_manager();
-    $result = true;
-
-/// And upgrade begins here. For each one, you'll need one
-/// block of code similar to the next one. Please, delete
-/// this comment lines once this file start handling proper
-/// upgrade code.
-
-    if ($result && $oldversion < 2007080100) { //New version in version.php
-    /// block_rss_timeout config setting must be block_rss_client_timeout
-        $DB->set_field('config', 'name', 'block_rss_client_timeout', array('name'=>'block_rss_timeout'));
-
-    /// rss_client savepoint reached
-        upgrade_block_savepoint($result, 2007080100, 'rss_client');
-    }
-
-    if ($result && $oldversion < 2009072901) {
-        // Remove config variable which is no longer used..
-        $result = $result && $DB->delete_records('config', array('name' =>'block_rss_client_submitters'));
-        upgrade_block_savepoint($result, 2009072901, 'rss_client');
-    }
-
-
-    return $result;
-}
-
-
diff --git a/blocks/search/db/upgrade.php b/blocks/search/db/upgrade.php
deleted file mode 100644 (file)
index 1c6da1d..0000000
+++ /dev/null
@@ -1,43 +0,0 @@
-<?php
-
-// This file keeps track of upgrades to
-// the search block
-//
-// Sometimes, changes between versions involve
-// alterations to database structures and other
-// major things that may break installations.
-//
-// The upgrade function in this file will attempt
-// to perform all the necessary actions to upgrade
-// your older installtion to the current version.
-//
-// If there's something it cannot do itself, it
-// will tell you what you need to do.
-//
-// The commands in here will all be database-neutral,
-// using the methods of database_manager class
-//
-// Please do not forget to use upgrade_set_timeout()
-// before any action that may take longer time to finish.
-
-function xmldb_block_search_upgrade($oldversion) {
-    global $CFG, $DB;
-
-    $dbman = $DB->get_manager();
-    $result = true;
-
-/// And upgrade begins here. For each one, you'll need one
-/// block of code similar to the next one. Please, delete
-/// this comment lines once this file start handling proper
-/// upgrade code.
-
-/// if ($result && $oldversion < YYYYMMDD00) { //New version in version.php
-///     $result = result of database_manager methods
-/// }
-
-//===== 1.9.0 upgrade line ======//
-
-    return $result;
-}
-
-
diff --git a/blocks/search_forums/db/upgrade.php b/blocks/search_forums/db/upgrade.php
deleted file mode 100644 (file)
index 5c78d21..0000000
+++ /dev/null
@@ -1,41 +0,0 @@
-<?php
-
-// This file keeps track of upgrades to
-// the search_forums block
-//
-// Sometimes, changes between versions involve
-// alterations to database structures and other
-// major things that may break installations.
-//
-// The upgrade function in this file will attempt
-// to perform all the necessary actions to upgrade
-// your older installtion to the current version.
-//
-// If there's something it cannot do itself, it
-// will tell you what you need to do.
-//
-// The commands in here will all be database-neutral,
-// using the methods of database_manager class
-//
-// Please do not forget to use upgrade_set_timeout()
-// before any action that may take longer time to finish.
-
-function xmldb_block_search_forums_upgrade($oldversion) {
-    global $CFG, $DB;
-
-    $dbman = $DB->get_manager();
-    $result = true;
-
-/// And upgrade begins here. For each one, you'll need one
-/// block of code similar to the next one. Please, delete
-/// this comment lines once this file start handling proper
-/// upgrade code.
-
-/// if ($result && $oldversion < YYYYMMDD00) { //New version in version.php
-///     $result = result of database_manager methods
-/// }
-
-    return $result;
-}
-
-
diff --git a/blocks/section_links/db/upgrade.php b/blocks/section_links/db/upgrade.php
deleted file mode 100644 (file)
index 143824d..0000000
+++ /dev/null
@@ -1,41 +0,0 @@
-<?php
-
-// This file keeps track of upgrades to
-// the section_links block
-//
-// Sometimes, changes between versions involve
-// alterations to database structures and other
-// major things that may break installations.
-//
-// The upgrade function in this file will attempt
-// to perform all the necessary actions to upgrade
-// your older installtion to the current version.
-//
-// If there's something it cannot do itself, it
-// will tell you what you need to do.
-//
-// The commands in here will all be database-neutral,
-// using the methods of database_manager class
-//
-// Please do not forget to use upgrade_set_timeout()
-// before any action that may take longer time to finish.
-
-function xmldb_block_section_links_upgrade($oldversion) {
-    global $CFG, $DB;
-
-    $dbman = $DB->get_manager();
-    $result = true;
-
-/// And upgrade begins here. For each one, you'll need one
-/// block of code similar to the next one. Please, delete
-/// this comment lines once this file start handling proper
-/// upgrade code.
-
-/// if ($result && $oldversion < YYYYMMDD00) { //New version in version.php
-///     $result = result of database_manager methods
-/// }
-
-    return $result;
-}
-
-
diff --git a/blocks/social_activities/db/upgrade.php b/blocks/social_activities/db/upgrade.php
deleted file mode 100644 (file)
index 59a1b8e..0000000
+++ /dev/null
@@ -1,41 +0,0 @@
-<?php
-
-// This file keeps track of upgrades to
-// the social_activities block
-//
-// Sometimes, changes between versions involve
-// alterations to database structures and other
-// major things that may break installations.
-//
-// The upgrade function in this file will attempt
-// to perform all the necessary actions to upgrade
-// your older installtion to the current version.
-//
-// If there's something it cannot do itself, it
-// will tell you what you need to do.
-//
-// The commands in here will all be database-neutral,
-// using the methods of database_manager class
-//
-// Please do not forget to use upgrade_set_timeout()
-// before any action that may take longer time to finish.
-
-function xmldb_block_social_activities_upgrade($oldversion) {
-    global $CFG, $DB;
-
-    $dbman = $DB->get_manager();
-    $result = true;
-
-/// And upgrade begins here. For each one, you'll need one
-/// block of code similar to the next one. Please, delete
-/// this comment lines once this file start handling proper
-/// upgrade code.
-
-/// if ($result && $oldversion < YYYYMMDD00) { //New version in version.php
-///     $result = result of database_manager methods
-/// }
-
-    return $result;
-}
-
-
index d63a5d7..cc8f2f5 100644 (file)
@@ -24,11 +24,10 @@ function xmldb_enrol_authorize_upgrade($oldversion) {
     global $CFG, $DB, $OUTPUT;
 
     $dbman = $DB->get_manager();
-    $result = true;
 
     //===== 1.9.0 upgrade line ======//
 
-    if ($result && $oldversion < 2008020500 && enrol_is_enabled('authorize')) {
+    if ($oldversion < 2008020500 && enrol_is_enabled('authorize')) {
         require_once($CFG->dirroot.'/enrol/authorize/localfuncs.php');
         if (!check_curl_available()) {
             echo $OUTPUT->notification("You are using the authorize.net enrolment plugin for payment handling but cUrl is not available.
@@ -36,10 +35,10 @@ function xmldb_enrol_authorize_upgrade($oldversion) {
         }
 
         /// authorize savepoint reached
-        upgrade_plugin_savepoint($result, 2008020500, 'enrol', 'authorize');
+        upgrade_plugin_savepoint(true, 2008020500, 'enrol', 'authorize');
     }
 
-    if ($result && $oldversion < 2008092700) {
+    if ($oldversion < 2008092700) {
         /// enrol_authorize.transid
         /// Define index transid (not unique) to be dropped form enrol_authorize
         $table = new xmldb_table('enrol_authorize');
@@ -77,11 +76,11 @@ function xmldb_enrol_authorize_upgrade($oldversion) {
         $dbman->add_index($table, $index);
 
         /// authorize savepoint reached
-        upgrade_plugin_savepoint($result, 2008092700, 'enrol', 'authorize');
+        upgrade_plugin_savepoint(true, 2008092700, 'enrol', 'authorize');
     }
 
     /// Dropping all enums/check contraints from core. MDL-18577
-    if ($result && $oldversion < 2009042700) {
+    if ($oldversion < 2009042700) {
 
     /// Changing list of values (enum) of field paymentmethod on table enrol_authorize to none
         $table = new xmldb_table('enrol_authorize');
@@ -91,10 +90,10 @@ function xmldb_enrol_authorize_upgrade($oldversion) {
         $dbman->drop_enum_from_field($table, $field);
 
         /// authorize savepoint reached
-        upgrade_plugin_savepoint($result, 2009042700, 'enrol', 'authorize');
+        upgrade_plugin_savepoint(true, 2009042700, 'enrol', 'authorize');
     }
 
-    return $result;
+    return true;
 }
 
 
diff --git a/enrol/paypal/db/upgrade.php b/enrol/paypal/db/upgrade.php
deleted file mode 100644 (file)
index fa6c3c4..0000000
+++ /dev/null
@@ -1,41 +0,0 @@
-<?php
-
-// This file keeps track of upgrades to
-// the paypal enrol plugin
-//
-// Sometimes, changes between versions involve
-// alterations to database structures and other
-// major things that may break installations.
-//
-// The upgrade function in this file will attempt
-// to perform all the necessary actions to upgrade
-// your older installtion to the current version.
-//
-// If there's something it cannot do itself, it
-// will tell you what you need to do.
-//
-// The commands in here will all be database-neutral,
-// using the methods of database_manager class
-//
-// Please do not forget to use upgrade_set_timeout()
-// before any action that may take longer time to finish.
-
-function xmldb_enrol_paypal_upgrade($oldversion) {
-    global $CFG, $DB;
-
-    $dbman = $DB->get_manager();
-    $result = true;
-
-/// And upgrade begins here. For each one, you'll need one
-/// block of code similar to the next one. Please, delete
-/// this comment lines once this file start handling proper
-/// upgrade code.
-
-/// if ($result && $oldversion < YYYYMMDD00) { //New version in version.php
-///     $result = result of database_manager methods
-/// }
-
-    return $result;
-}
-
-
index ab21857..1c9ed55 100644 (file)
@@ -33,15 +33,13 @@ function xmldb_main_upgrade($oldversion) {
 
     require_once($CFG->libdir.'/db/upgradelib.php'); // Core Upgrade-related functions
 
-    $result = true;
-
     $dbman = $DB->get_manager(); // loads ddl manager and xmldb classes
 
     ////////////////////////////////////////
     ///upgrade supported only from 1.9.x ///
     ////////////////////////////////////////
 
-    if ($result && $oldversion < 2008030600) {
+    if ($oldversion < 2008030600) {
         //NOTE: this table was added much later, that is why this step is repeated later in this file
 
     /// Define table upgrade_log to be created
@@ -70,10 +68,10 @@ function xmldb_main_upgrade($oldversion) {
         $dbman->create_table($table);
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2008030600);
+        upgrade_main_savepoint(true, 2008030600);
     }
 
-    if ($result && $oldversion < 2008030601) {
+    if ($oldversion < 2008030601) {
         @unlink($CFG->dataroot.'/cache/languages');
 
         // rename old lang directory so that the new and old langs do not mix
@@ -90,10 +88,10 @@ function xmldb_main_upgrade($oldversion) {
 
 
         // Main savepoint reached
-        upgrade_main_savepoint($result, 2008030601);
+        upgrade_main_savepoint(true, 2008030601);
     }
 
-    if ($result && $oldversion < 2008030700) {
+    if ($oldversion < 2008030700) {
         upgrade_set_timeout(60*20); // this may take a while
 
     /// Define index contextid-lowerboundary (not unique) to be dropped form grade_letters
@@ -113,31 +111,31 @@ function xmldb_main_upgrade($oldversion) {
         $dbman->add_index($table, $index);
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2008030700);
+        upgrade_main_savepoint(true, 2008030700);
     }
 
-    if ($result && $oldversion < 2008050100) {
+    if ($oldversion < 2008050100) {
         // Update courses that used weekscss to weeks
         $DB->set_field('course', 'format', 'weeks', array('format' => 'weekscss'));
-        upgrade_main_savepoint($result, 2008050100);
+        upgrade_main_savepoint(true, 2008050100);
     }
 
-    if ($result && $oldversion < 2008050200) {
+    if ($oldversion < 2008050200) {
         // remove unused config options
         unset_config('statsrolesupgraded');
-        upgrade_main_savepoint($result, 2008050200);
+        upgrade_main_savepoint(true, 2008050200);
     }
 
-    if ($result && $oldversion < 2008050700) {
+    if ($oldversion < 2008050700) {
         upgrade_set_timeout(60*20); // this may take a while
 
     /// Fix minor problem caused by MDL-5482.
         require_once($CFG->dirroot . '/question/upgrade.php');
         question_fix_random_question_parents();
-        upgrade_main_savepoint($result, 2008050700);
+        upgrade_main_savepoint(true, 2008050700);
     }
 
-    if ($result && $oldversion < 2008051201) {
+    if ($oldversion < 2008051201) {
         echo $OUTPUT->notification('Increasing size of user idnumber field, this may take a while...', 'notifysuccess');
         upgrade_set_timeout(60*20); // this may take a while
 
@@ -168,10 +166,10 @@ function xmldb_main_upgrade($oldversion) {
         $dbman->add_index($table, $index);
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2008051201);
+        upgrade_main_savepoint(true, 2008051201);
     }
 
-    if ($result && $oldversion < 2008051202) {
+    if ($oldversion < 2008051202) {
         $log_action = new object();
         $log_action->module = 'course';
         $log_action->action = 'unenrol';
@@ -180,17 +178,17 @@ function xmldb_main_upgrade($oldversion) {
         if (!$DB->record_exists('log_display', array('action'=>'unenrol', 'module'=>'course'))) {
             $DB->insert_record('log_display', $log_action);
         }
-        upgrade_main_savepoint($result, 2008051202);
+        upgrade_main_savepoint(true, 2008051202);
     }
 
-    if ($result && $oldversion < 2008051203) {
+    if ($oldversion < 2008051203) {
         $table = new xmldb_table('mnet_enrol_course');
         $field = new xmldb_field('sortorder', XMLDB_TYPE_INTEGER, '10', XMLDB_UNSIGNED, XMLDB_NOTNULL, null, 0);
         $dbman->change_field_precision($table, $field);
-        upgrade_main_savepoint($result, 2008051203);
+        upgrade_main_savepoint(true, 2008051203);
     }
 
-    if ($result && $oldversion < 2008063001) {
+    if ($oldversion < 2008063001) {
         upgrade_set_timeout(60*20); // this may take a while
 
         // table to be modified
@@ -214,15 +212,15 @@ function xmldb_main_upgrade($oldversion) {
         }
 
         /// Main savepoint reached
-        upgrade_main_savepoint($result, 2008063001);
+        upgrade_main_savepoint(true, 2008063001);
     }
 
-    if ($result && $oldversion < 2008070300) {
+    if ($oldversion < 2008070300) {
         $DB->delete_records_select('role_names', $DB->sql_isempty('role_names', 'name', false, false));
-        upgrade_main_savepoint($result, 2008070300);
+        upgrade_main_savepoint(true, 2008070300);
     }
 
-    if ($result && $oldversion < 2008070701) {
+    if ($oldversion < 2008070701) {
 
     /// Define table portfolio_instance to be created
         $table = new xmldb_table('portfolio_instance');
@@ -282,10 +280,10 @@ function xmldb_main_upgrade($oldversion) {
         }
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2008070701);
+        upgrade_main_savepoint(true, 2008070701);
     }
 
-    if ($result && $oldversion < 2008072400) {
+    if ($oldversion < 2008072400) {
     /// Create the database tables for message_processors
         $table = new xmldb_table('message_processors');
         $table->add_field('id', XMLDB_TYPE_INTEGER, '10', null, XMLDB_NOTNULL, XMLDB_SEQUENCE, null);
@@ -353,10 +351,10 @@ function xmldb_main_upgrade($oldversion) {
         $dbman->create_table($table);
 
 
-        upgrade_main_savepoint($result, 2008072400);
+        upgrade_main_savepoint(true, 2008072400);
     }
 
-    if ($result && $oldversion < 2008072800) {
+    if ($oldversion < 2008072800) {
 
     /// Define field enablecompletion to be added to course
         $table = new xmldb_table('course');
@@ -429,10 +427,10 @@ function xmldb_main_upgrade($oldversion) {
         }
 
         /// Main savepoint reached
-        upgrade_main_savepoint($result, 2008072800);
+        upgrade_main_savepoint(true, 2008072800);
     }
 
-    if ($result && $oldversion < 2008073000) {
+    if ($oldversion < 2008073000) {
 
     /// Define table portfolio_log to be created
         $table = new xmldb_table('portfolio_log');
@@ -457,10 +455,10 @@ function xmldb_main_upgrade($oldversion) {
         }
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2008073000);
+        upgrade_main_savepoint(true, 2008073000);
     }
 
-    if ($result && $oldversion < 2008073104) {
+    if ($oldversion < 2008073104) {
     /// Drop old table that might exist for some people
         $table = new xmldb_table('message_providers');
         if ($dbman->table_exists($table)) {
@@ -485,10 +483,10 @@ function xmldb_main_upgrade($oldversion) {
     /// Create table for message_providers
         $dbman->create_table($table);
 
-        upgrade_main_savepoint($result, 2008073104);
+        upgrade_main_savepoint(true, 2008073104);
     }
 
-    if ($result && $oldversion < 2008073111) {
+    if ($oldversion < 2008073111) {
     /// Define table files to be created
         $table = new xmldb_table('files');
 
@@ -526,10 +524,10 @@ function xmldb_main_upgrade($oldversion) {
         $dbman->create_table($table);
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2008073111);
+        upgrade_main_savepoint(true, 2008073111);
     }
 
-    if ($result && $oldversion < 2008073112) {
+    if ($oldversion < 2008073112) {
         // Define field legacyfiles to be added to course
         $table = new xmldb_table('course');
         $field = new xmldb_field('legacyfiles', XMLDB_TYPE_INTEGER, '4', XMLDB_UNSIGNED, XMLDB_NOTNULL, null, '0', 'maxbytes');
@@ -540,24 +538,24 @@ function xmldb_main_upgrade($oldversion) {
         $DB->execute("UPDATE {course} SET legacyfiles = 2");
 
         // Main savepoint reached
-        upgrade_main_savepoint($result, 2008073112);
+        upgrade_main_savepoint(true, 2008073112);
     }
 
-    if ($result && $oldversion < 2008073113) {
+    if ($oldversion < 2008073113) {
     /// move all course, backup and other files to new filepool based storage
         upgrade_migrate_files_courses();
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2008073113);
+        upgrade_main_savepoint(true, 2008073113);
     }
 
-    if ($result && $oldversion < 2008073114) {
+    if ($oldversion < 2008073114) {
     /// move all course, backup and other files to new filepool based storage
         upgrade_migrate_files_blog();
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2008073114);
+        upgrade_main_savepoint(true, 2008073114);
     }
 
-    if ($result && $oldversion < 2008080400) {
+    if ($oldversion < 2008080400) {
         // Add field ssl_jump_url to mnet application, and populate existing default applications
         $table = new xmldb_table('mnet_application');
         $field = new xmldb_field('sso_jump_url');
@@ -569,10 +567,10 @@ function xmldb_main_upgrade($oldversion) {
         }
 
         /// Main savepoint reached
-        upgrade_main_savepoint($result, 2008080400);
+        upgrade_main_savepoint(true, 2008080400);
     }
 
-    if ($result && $oldversion < 2008080500) {
+    if ($oldversion < 2008080500) {
 
    /// Define table portfolio_tempdata to be created
         $table = new xmldb_table('portfolio_tempdata');
@@ -590,10 +588,10 @@ function xmldb_main_upgrade($oldversion) {
         }
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2008080500);
+        upgrade_main_savepoint(true, 2008080500);
     }
 
-    if ($result && $oldversion < 2008080600) {
+    if ($oldversion < 2008080600) {
 
         $DB->delete_records('portfolio_tempdata'); // there shouldnt' be any, and it will cause problems with this upgrade.
     /// Define field expirytime to be added to portfolio_tempdata
@@ -606,69 +604,69 @@ function xmldb_main_upgrade($oldversion) {
         }
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2008080600);
+        upgrade_main_savepoint(true, 2008080600);
     }
 
-    if ($result && $oldversion < 2008081500) {
+    if ($oldversion < 2008081500) {
     /// Changing the type of all the columns that the question bank uses to store grades to be NUMBER(12, 7).
         $table = new xmldb_table('question');
         $field = new xmldb_field('defaultgrade', XMLDB_TYPE_NUMBER, '12, 7', null, null, null, null, 'generalfeedback');
         $dbman->change_field_type($table, $field);
-        upgrade_main_savepoint($result, 2008081500);
+        upgrade_main_savepoint(true, 2008081500);
     }
 
-    if ($result && $oldversion < 2008081501) {
+    if ($oldversion < 2008081501) {
         $table = new xmldb_table('question');
         $field = new xmldb_field('penalty', XMLDB_TYPE_NUMBER, '12, 7', null, null, null, null, 'defaultgrade');
         $dbman->change_field_type($table, $field);
-        upgrade_main_savepoint($result, 2008081501);
+        upgrade_main_savepoint(true, 2008081501);
     }
 
-    if ($result && $oldversion < 2008081502) {
+    if ($oldversion < 2008081502) {
         $table = new xmldb_table('question_answers');
         $field = new xmldb_field('fraction', XMLDB_TYPE_NUMBER, '12, 7', null, null, null, null, 'answer');
         $dbman->change_field_type($table, $field);
-        upgrade_main_savepoint($result, 2008081502);
+        upgrade_main_savepoint(true, 2008081502);
     }
 
-    if ($result && $oldversion < 2008081503) {
+    if ($oldversion < 2008081503) {
         $table = new xmldb_table('question_sessions');
         $field = new xmldb_field('sumpenalty', XMLDB_TYPE_NUMBER, '12, 7', null, null, null, null, 'newgraded');
         $dbman->change_field_type($table, $field);
-        upgrade_main_savepoint($result, 2008081503);
+        upgrade_main_savepoint(true, 2008081503);
     }
 
-    if ($result && $oldversion < 2008081504) {
+    if ($oldversion < 2008081504) {
         $table = new xmldb_table('question_states');
         $field = new xmldb_field('grade', XMLDB_TYPE_NUMBER, '12, 7', null, null, null, null, 'event');
         $dbman->change_field_type($table, $field);
-        upgrade_main_savepoint($result, 2008081504);
+        upgrade_main_savepoint(true, 2008081504);
     }
 
-    if ($result && $oldversion < 2008081505) {
+    if ($oldversion < 2008081505) {
         $table = new xmldb_table('question_states');
         $field = new xmldb_field('raw_grade', XMLDB_TYPE_NUMBER, '12, 7', null, null, null, null, 'grade');
         $dbman->change_field_type($table, $field);
-        upgrade_main_savepoint($result, 2008081505);
+        upgrade_main_savepoint(true, 2008081505);
     }
 
-    if ($result && $oldversion < 2008081506) {
+    if ($oldversion < 2008081506) {
         $table = new xmldb_table('question_states');
         $field = new xmldb_field('penalty', XMLDB_TYPE_NUMBER, '12, 7', null, null, null, null, 'raw_grade');
         $dbman->change_field_type($table, $field);
-        upgrade_main_savepoint($result, 2008081506);
+        upgrade_main_savepoint(true, 2008081506);
     }
 
-    if ($result && $oldversion < 2008081600) {
+    if ($oldversion < 2008081600) {
 
     /// all 1.9 sites and fresh installs must already be unicode, not needed anymore
         unset_config('unicodedb');
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2008081600);
+        upgrade_main_savepoint(true, 2008081600);
     }
 
-    if ($result && $oldversion < 2008081900) {
+    if ($oldversion < 2008081900) {
     /// Define field userid to be added to portfolio_tempdata
         $table = new xmldb_table('portfolio_tempdata');
         $field = new xmldb_field('userid', XMLDB_TYPE_INTEGER, '10', XMLDB_UNSIGNED, null, null, null, 'expirytime');
@@ -693,9 +691,9 @@ function xmldb_main_upgrade($oldversion) {
     /// Launch add key userfk
         $dbman->add_key($table, $key);
 
-        upgrade_main_savepoint($result, 2008081900);
+        upgrade_main_savepoint(true, 2008081900);
     }
-    if ($result && $oldversion < 2008082602) {
+    if ($oldversion < 2008082602) {
 
     /// Define table repository to be dropped
         $table = new xmldb_table('repository');
@@ -761,10 +759,10 @@ function xmldb_main_upgrade($oldversion) {
         }
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2008082602);
+        upgrade_main_savepoint(true, 2008082602);
     }
 
-    if ($result && $oldversion < 2008082700) {
+    if ($oldversion < 2008082700) {
     /// Add a new column to the question sessions table to record whether a
     /// question has been flagged.
 
@@ -778,10 +776,10 @@ function xmldb_main_upgrade($oldversion) {
         }
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2008082700);
+        upgrade_main_savepoint(true, 2008082700);
     }
 
-    if ($result && $oldversion < 2008082900) {
+    if ($oldversion < 2008082900) {
 
     /// Changing precision of field parent_type on table mnet_rpc to (20)
         $table = new xmldb_table('mnet_rpc');
@@ -791,10 +789,10 @@ function xmldb_main_upgrade($oldversion) {
         $dbman->change_field_precision($table, $field);
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2008082900);
+        upgrade_main_savepoint(true, 2008082900);
     }
 
-    if ($result && $oldversion < 2008090108) {
+    if ($oldversion < 2008090108) {
         $repo = new object();
         $repo->type      = 'upload';
         $repo->visible   = 1;
@@ -835,11 +833,11 @@ function xmldb_main_upgrade($oldversion) {
             $DB->insert_record('repository_instances', $instance);
         }
 
-        upgrade_main_savepoint($result, 2008090108);
+        upgrade_main_savepoint(true, 2008090108);
     }
 
     // MDL-16411 Move all plugintype_pluginname_version values from config to config_plugins.
-    if ($result && $oldversion < 2008091000) {
+    if ($oldversion < 2008091000) {
         foreach (get_object_vars($CFG) as $name => $value) {
             if (substr($name, strlen($name) - 8) !== '_version') {
                 continue;
@@ -861,12 +859,12 @@ function xmldb_main_upgrade($oldversion) {
             set_config('version', $value, $pluginname);
             unset_config($name);
         }
-        upgrade_main_savepoint($result, 2008091000);
+        upgrade_main_savepoint(true, 2008091000);
     }
 
     //Add a readonly field to the repository_instances table
     //in order to support instance created automatically by a repository plugin
-     if ($result && $oldversion < 2008091611) {
+     if ($oldversion < 2008091611) {
 
     /// Define field readonly to be added to repository_instances
         $table = new xmldb_table('repository_instances');
@@ -878,28 +876,28 @@ function xmldb_main_upgrade($oldversion) {
         }
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2008091611);
+        upgrade_main_savepoint(true, 2008091611);
     }
 
-    if ($result && $oldversion < 2008092300) {
+    if ($oldversion < 2008092300) {
         unset_config('editorspelling');
         unset_config('editordictionary');
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2008092300);
+        upgrade_main_savepoint(true, 2008092300);
     }
 
-    if ($result && $oldversion < 2008101300) {
+    if ($oldversion < 2008101300) {
 
         if (!get_config(NULL, 'statsruntimedays')) {
             set_config('statsruntimedays', '31');
         }
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2008101300);
+        upgrade_main_savepoint(true, 2008101300);
     }
 
     /// Drop the deprecated teacher, teachers, student and students columns from the course table.
-    if ($result && $oldversion < 2008111200) {
+    if ($oldversion < 2008111200) {
         $table = new xmldb_table('course');
 
     /// Conditionally launch drop field teacher
@@ -927,11 +925,11 @@ function xmldb_main_upgrade($oldversion) {
         }
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2008111200);
+        upgrade_main_savepoint(true, 2008111200);
     }
 
 /// Add a unique index to the role.name column.
-    if ($result && $oldversion < 2008111800) {
+    if ($oldversion < 2008111800) {
 
     /// Define index name (unique) to be added to role
         $table = new xmldb_table('role');
@@ -943,11 +941,11 @@ function xmldb_main_upgrade($oldversion) {
         }
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2008111800);
+        upgrade_main_savepoint(true, 2008111800);
     }
 
 /// Add a unique index to the role.shortname column.
-    if ($result && $oldversion < 2008111801) {
+    if ($oldversion < 2008111801) {
 
     /// Define index shortname (unique) to be added to role
         $table = new xmldb_table('role');
@@ -959,10 +957,10 @@ function xmldb_main_upgrade($oldversion) {
         }
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2008111801);
+        upgrade_main_savepoint(true, 2008111801);
     }
 
-    if ($result && $oldversion < 2008120700) {
+    if ($oldversion < 2008120700) {
 
     /// Changing precision of field shortname on table course_request to (100)
         $table = new xmldb_table('course_request');
@@ -988,10 +986,10 @@ function xmldb_main_upgrade($oldversion) {
         }
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2008120700);
+        upgrade_main_savepoint(true, 2008120700);
     }
 
-    if ($result && $oldversion < 2008120801) {
+    if ($oldversion < 2008120801) {
 
     /// Changing precision of field shortname on table mnet_enrol_course to (100)
         $table = new xmldb_table('mnet_enrol_course');
@@ -1001,10 +999,10 @@ function xmldb_main_upgrade($oldversion) {
         $dbman->change_field_precision($table, $field);
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2008120801);
+        upgrade_main_savepoint(true, 2008120801);
     }
 
-    if ($result && $oldversion < 2008121701) {
+    if ($oldversion < 2008121701) {
 
     /// Define field availablefrom to be added to course_modules
         $table = new xmldb_table('course_modules');
@@ -1059,18 +1057,18 @@ function xmldb_main_upgrade($oldversion) {
         rebuild_course_cache(0, true);
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2008121701);
+        upgrade_main_savepoint(true, 2008121701);
     }
 
-    if ($result && $oldversion < 2009010500) {
+    if ($oldversion < 2009010500) {
     /// clean up config table a bit
         unset_config('session_error_counter');
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009010500);
+        upgrade_main_savepoint(true, 2009010500);
     }
 
-    if ($result && $oldversion < 2009010600) {
+    if ($oldversion < 2009010600) {
 
     /// Define field originalquestion to be dropped from question_states
         $table = new xmldb_table('question_states');
@@ -1082,10 +1080,10 @@ function xmldb_main_upgrade($oldversion) {
         }
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009010600);
+        upgrade_main_savepoint(true, 2009010600);
     }
 
-    if ($result && $oldversion < 2009010601) {
+    if ($oldversion < 2009010601) {
 
     /// Changing precision of field ip on table log to (45)
         $table = new xmldb_table('log');
@@ -1095,10 +1093,10 @@ function xmldb_main_upgrade($oldversion) {
         $dbman->change_field_precision($table, $field);
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009010601);
+        upgrade_main_savepoint(true, 2009010601);
     }
 
-    if ($result && $oldversion < 2009010602) {
+    if ($oldversion < 2009010602) {
 
     /// Changing precision of field lastip on table user to (45)
         $table = new xmldb_table('user');
@@ -1108,10 +1106,10 @@ function xmldb_main_upgrade($oldversion) {
         $dbman->change_field_precision($table, $field);
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009010602);
+        upgrade_main_savepoint(true, 2009010602);
     }
 
-    if ($result && $oldversion < 2009010603) {
+    if ($oldversion < 2009010603) {
 
     /// Changing precision of field ip_address on table mnet_host to (45)
         $table = new xmldb_table('mnet_host');
@@ -1121,10 +1119,10 @@ function xmldb_main_upgrade($oldversion) {
         $dbman->change_field_precision($table, $field);
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009010603);
+        upgrade_main_savepoint(true, 2009010603);
     }
 
-    if ($result && $oldversion < 2009010604) {
+    if ($oldversion < 2009010604) {
 
     /// Changing precision of field ip on table mnet_log to (45)
         $table = new xmldb_table('mnet_log');
@@ -1134,10 +1132,10 @@ function xmldb_main_upgrade($oldversion) {
         $dbman->change_field_precision($table, $field);
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009010604);
+        upgrade_main_savepoint(true, 2009010604);
     }
 
-    if ($result && $oldversion < 2009011000) {
+    if ($oldversion < 2009011000) {
 
     /// Changing nullability of field configdata on table block_instance to null
         $table = new xmldb_table('block_instance');
@@ -1148,20 +1146,20 @@ function xmldb_main_upgrade($oldversion) {
         $dbman->change_field_notnull($table, $field);
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009011000);
+        upgrade_main_savepoint(true, 2009011000);
     }
 
-    if ($result && $oldversion < 2009011100) {
+    if ($oldversion < 2009011100) {
     /// Remove unused settings
         unset_config('zip');
         unset_config('unzip');
         unset_config('adminblocks_initialised');
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009011100);
+        upgrade_main_savepoint(true, 2009011100);
     }
 
-    if ($result && $oldversion < 2009011101) {
+    if ($oldversion < 2009011101) {
     /// Migrate backup settings to core plugin config table
         $configs = $DB->get_records('backup_config');
         foreach ($configs as $config) {
@@ -1175,10 +1173,10 @@ function xmldb_main_upgrade($oldversion) {
         $dbman->drop_table($table);
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009011101);
+        upgrade_main_savepoint(true, 2009011101);
     }
 
-    if ($result && $oldversion < 2009011303) {
+    if ($oldversion < 2009011303) {
 
     /// Define table config_log to be created
         $table = new xmldb_table('config_log');
@@ -1203,10 +1201,10 @@ function xmldb_main_upgrade($oldversion) {
         $dbman->create_table($table);
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009011303);
+        upgrade_main_savepoint(true, 2009011303);
     }
 
-    if ($result && $oldversion < 2009011900) {
+    if ($oldversion < 2009011900) {
 
     /// Define table sessions2 to be dropped
         $table = new xmldb_table('sessions2');
@@ -1252,10 +1250,10 @@ function xmldb_main_upgrade($oldversion) {
         $dbman->create_table($table);
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009011900);
+        upgrade_main_savepoint(true, 2009011900);
     }
 
-    if ($result && $oldversion < 2009012901) {
+    if ($oldversion < 2009012901) {
         // NOTE: this table may already exist, see beginning of this file ;-)
 
     /// Define table upgrade_log to be created
@@ -1286,10 +1284,10 @@ function xmldb_main_upgrade($oldversion) {
         }
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009012901);
+        upgrade_main_savepoint(true, 2009012901);
     }
 
-    if ($result && $oldversion < 2009021800) {
+    if ($oldversion < 2009021800) {
         // Converting format of grade conditions, if any exist, to percentages.
         $DB->execute("
 UPDATE {course_modules_availability} SET grademin=(
@@ -1307,9 +1305,9 @@ UPDATE {course_modules_availability} SET grademax=(
 WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009021800);
+        upgrade_main_savepoint(true, 2009021800);
     }
-    if ($result && $oldversion < 2009021801) {
+    if ($oldversion < 2009021801) {
     /// Define field backuptype to be added to backup_log
         $table = new xmldb_table('backup_log');
         $field = new xmldb_field('backuptype', XMLDB_TYPE_CHAR, '50', null, XMLDB_NOTNULL, null, null, 'info');
@@ -1320,10 +1318,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         }
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009021801);
+        upgrade_main_savepoint(true, 2009021801);
     }
     /// Add default sort order for question types.
-    if ($result && $oldversion < 2009030300) {
+    if ($oldversion < 2009030300) {
         set_config('multichoice_sortorder', 1, 'question');
         set_config('truefalse_sortorder', 2, 'question');
         set_config('shortanswer_sortorder', 3, 'question');
@@ -1337,19 +1335,19 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         set_config('random_sortorder', 11, 'question');
         set_config('missingtype_sortorder', 12, 'question');
 
-        upgrade_main_savepoint($result, 2009030300);
+        upgrade_main_savepoint(true, 2009030300);
     }
-    if ($result && $oldversion < 2009030501) {
+    if ($oldversion < 2009030501) {
     /// setup default repository plugins
         require_once($CFG->dirroot . '/repository/lib.php');
         repository_setup_default_plugins();
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009030501);
+        upgrade_main_savepoint(true, 2009030501);
     }
 
     /// MDL-18132 replace the use a new Role allow switch settings page, instead of
     /// $CFG->allowuserswitchrolestheycantassign
-    if ($result && $oldversion < 2009032000) {
+    if ($oldversion < 2009032000) {
     /// First create the new table.
             $table = new xmldb_table('role_allow_switch');
 
@@ -1372,10 +1370,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         }
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009032000);
+        upgrade_main_savepoint(true, 2009032000);
     }
 
-    if ($result && $oldversion < 2009032001) {
+    if ($oldversion < 2009032001) {
     /// Copy from role_allow_assign into the new table.
         $DB->execute('INSERT INTO {role_allow_switch} (roleid, allowswitch)
                 SELECT roleid, allowassign FROM {role_allow_assign}');
@@ -1384,17 +1382,17 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         unset_config('allowuserswitchrolestheycantassign');
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009032001);
+        upgrade_main_savepoint(true, 2009032001);
     }
 
-    if ($result && $oldversion < 2009033100) {
+    if ($oldversion < 2009033100) {
         require_once("$CFG->dirroot/filter/tex/lib.php");
         filter_tex_updatedcallback(null);
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009033100);
+        upgrade_main_savepoint(true, 2009033100);
     }
 
-    if ($result && $oldversion < 2009040300) {
+    if ($oldversion < 2009040300) {
 
     /// Define table filter_active to be created
         $table = new xmldb_table('filter_active');
@@ -1419,10 +1417,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         }
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009040300);
+        upgrade_main_savepoint(true, 2009040300);
     }
 
-    if ($result && $oldversion < 2009040301) {
+    if ($oldversion < 2009040301) {
 
     /// Define table filter_config to be created
         $table = new xmldb_table('filter_config');
@@ -1447,10 +1445,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         }
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009040301);
+        upgrade_main_savepoint(true, 2009040301);
     }
 
-    if ($result && $oldversion < 2009040302) {
+    if ($oldversion < 2009040302) {
     /// Transfer current settings from $CFG->textfilters
         $disabledfilters = filter_get_all_installed();
         if (empty($CFG->textfilters)) {
@@ -1471,10 +1469,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         }
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009040302);
+        upgrade_main_savepoint(true, 2009040302);
     }
 
-    if ($result && $oldversion < 2009040600) {
+    if ($oldversion < 2009040600) {
     /// Ensure that $CFG->stringfilters is set.
         if (empty($CFG->stringfilters)) {
             if (!empty($CFG->filterall)) {
@@ -1488,10 +1486,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         unset_config('textfilters');
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009040600);
+        upgrade_main_savepoint(true, 2009040600);
     }
 
-    if ($result && $oldversion < 2009041700) {
+    if ($oldversion < 2009041700) {
     /// To ensure the UI remains consistent with no behaviour change, any
     /// 'until' date in an activity condition should have 1 second subtracted
     /// (to go from 0:00 on the following day to 23:59 on the previous one).
@@ -1500,10 +1498,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         rebuild_course_cache(0, true);
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009041700);
+        upgrade_main_savepoint(true, 2009041700);
     }
 
-    if ($result && $oldversion < 2009042600) {
+    if ($oldversion < 2009042600) {
     /// Deleting orphaned messages from deleted users.
         require_once($CFG->dirroot.'/message/lib.php');
     /// Detect deleted users with messages sent(useridfrom) and not read
@@ -1516,11 +1514,11 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
             }
         }
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009042600);
+        upgrade_main_savepoint(true, 2009042600);
     }
 
     /// Dropping all enums/check contraints from core. MDL-18577
-    if ($result && $oldversion < 2009042700) {
+    if ($oldversion < 2009042700) {
 
     /// Changing list of values (enum) of field stattype on table stats_daily to none
         $table = new xmldb_table('stats_daily');
@@ -1551,23 +1549,23 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         $dbman->drop_enum_from_field($table, $field);
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009042700);
+        upgrade_main_savepoint(true, 2009042700);
     }
 
-    if ($result && $oldversion < 2009043000) {
+    if ($oldversion < 2009043000) {
         unset_config('grade_report_showgroups');
-        upgrade_main_savepoint($result, 2009043000);
+        upgrade_main_savepoint(true, 2009043000);
     }
 
-    if ($result && $oldversion < 2009050600) {
+    if ($oldversion < 2009050600) {
     /// Site front page blocks need to be moved due to page name change.
         $DB->set_field('block_instance', 'pagetype', 'site-index', array('pagetype' => 'course-view', 'pageid' => SITEID));
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009050600);
+        upgrade_main_savepoint(true, 2009050600);
     }
 
-    if ($result && $oldversion < 2009050601) {
+    if ($oldversion < 2009050601) {
 
     /// Define table block_instance to be renamed to block_instances
         $table = new xmldb_table('block_instance');
@@ -1576,10 +1574,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         $dbman->rename_table($table, 'block_instances');
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009050601);
+        upgrade_main_savepoint(true, 2009050601);
     }
 
-    if ($result && $oldversion < 2009050602) {
+    if ($oldversion < 2009050602) {
 
     /// Define table block_instance to be renamed to block_instance_old
         $table = new xmldb_table('block_pinned');
@@ -1588,10 +1586,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         $dbman->rename_table($table, 'block_pinned_old');
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009050602);
+        upgrade_main_savepoint(true, 2009050602);
     }
 
-    if ($result && $oldversion < 2009050603) {
+    if ($oldversion < 2009050603) {
 
     /// Define table block_instance_old to be created
         $table = new xmldb_table('block_instance_old');
@@ -1621,18 +1619,18 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         }
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009050603);
+        upgrade_main_savepoint(true, 2009050603);
     }
 
-    if ($result && $oldversion < 2009050604) {
+    if ($oldversion < 2009050604) {
     /// Copy current blocks data from block_instances to block_instance_old
         $DB->execute('INSERT INTO {block_instance_old} (oldid, blockid, pageid, pagetype, position, weight, visible, configdata)
             SELECT id, blockid, pageid, pagetype, position, weight, visible, configdata FROM {block_instances} ORDER BY id');
 
-        upgrade_main_savepoint($result, 2009050604);
+        upgrade_main_savepoint(true, 2009050604);
     }
 
-    if ($result && $oldversion < 2009050605) {
+    if ($oldversion < 2009050605) {
 
     /// Define field multiple to be dropped from block
         $table = new xmldb_table('block');
@@ -1644,10 +1642,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         }
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009050605);
+        upgrade_main_savepoint(true, 2009050605);
     }
 
-    if ($result && $oldversion < 2009050606) {
+    if ($oldversion < 2009050606) {
         $table = new xmldb_table('block_instances');
 
     /// Rename field weight on table block_instances to defaultweight
@@ -1659,10 +1657,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         $dbman->rename_field($table, $field, 'defaultregion');
 
         /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009050606);
+        upgrade_main_savepoint(true, 2009050606);
     }
 
-    if ($result && $oldversion < 2009050607) {
+    if ($oldversion < 2009050607) {
     /// Changing precision of field defaultregion on table block_instances to (16)
         $table = new xmldb_table('block_instances');
         $field = new xmldb_field('defaultregion', XMLDB_TYPE_CHAR, '16', null, XMLDB_NOTNULL, null, null, 'pagetype');
@@ -1671,10 +1669,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         $dbman->change_field_precision($table, $field);
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009050607);
+        upgrade_main_savepoint(true, 2009050607);
     }
 
-    if ($result && $oldversion < 2009050608) {
+    if ($oldversion < 2009050608) {
     /// Change regions to the new notation
         $DB->set_field('block_instances', 'defaultregion', 'side-pre', array('defaultregion' => 'l'));
         $DB->set_field('block_instances', 'defaultregion', 'side-post', array('defaultregion' => 'r'));
@@ -1683,10 +1681,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         // flex page course format. Hopefully this new value is an adequate alternative.
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009050608);
+        upgrade_main_savepoint(true, 2009050608);
     }
 
-    if ($result && $oldversion < 2009050609) {
+    if ($oldversion < 2009050609) {
 
     /// Define key blockname (unique) to be added to block
         $table = new xmldb_table('block');
@@ -1696,10 +1694,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         $dbman->add_key($table, $key);
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009050609);
+        upgrade_main_savepoint(true, 2009050609);
     }
 
-    if ($result && $oldversion < 2009050610) {
+    if ($oldversion < 2009050610) {
         $table = new xmldb_table('block_instances');
 
     /// Define field blockname to be added to block_instances
@@ -1727,10 +1725,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         }
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009050610);
+        upgrade_main_savepoint(true, 2009050610);
     }
 
-    if ($result && $oldversion < 2009050611) {
+    if ($oldversion < 2009050611) {
         $table = new xmldb_table('block_instances');
 
     /// Fill in blockname from blockid
@@ -1740,10 +1738,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         $DB->execute("UPDATE {block_instances} SET showinsubcontexts = 0");
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009050611);
+        upgrade_main_savepoint(true, 2009050611);
     }
 
-    if ($result && $oldversion < 2009050612) {
+    if ($oldversion < 2009050612) {
 
     /// Rename field pagetype on table block_instances to pagetypepattern
         $table = new xmldb_table('block_instances');
@@ -1753,10 +1751,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         $dbman->rename_field($table, $field, 'pagetypepattern');
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009050612);
+        upgrade_main_savepoint(true, 2009050612);
     }
 
-    if ($result && $oldversion < 2009050613) {
+    if ($oldversion < 2009050613) {
     /// fill in contextid and subpage, and update pagetypepattern from pagetype and pageid
 
     /// site-index
@@ -1838,18 +1836,18 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         }
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009050613);
+        upgrade_main_savepoint(true, 2009050613);
     }
 
-    if ($result && $oldversion < 2009050614) {
+    if ($oldversion < 2009050614) {
     /// fill in any missing contextids with a dummy value, so we can add the not-null constraint.
         $DB->execute("UPDATE {block_instances} SET contextid = 0 WHERE contextid IS NULL");
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009050614);
+        upgrade_main_savepoint(true, 2009050614);
     }
 
-    if ($result && $oldversion < 2009050615) {
+    if ($oldversion < 2009050615) {
         $table = new xmldb_table('block_instances');
 
     /// Arrived here, any block_instances record without blockname is one
@@ -1869,10 +1867,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         $dbman->change_field_notnull($table, $field);
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009050615);
+        upgrade_main_savepoint(true, 2009050615);
     }
 
-    if ($result && $oldversion < 2009050616) {
+    if ($oldversion < 2009050616) {
     /// Add exiting sticky blocks.
         $blocks = $DB->get_records('block');
         $syscontext = get_context_instance(CONTEXT_SYSTEM);
@@ -1906,10 +1904,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         }
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009050616);
+        upgrade_main_savepoint(true, 2009050616);
     }
 
-    if ($result && $oldversion < 2009050617) {
+    if ($oldversion < 2009050617) {
 
     /// Define table block_positions to be created
         $table = new xmldb_table('block_positions');
@@ -1938,10 +1936,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         }
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009050617);
+        upgrade_main_savepoint(true, 2009050617);
     }
 
-    if ($result && $oldversion < 2009050618) {
+    if ($oldversion < 2009050618) {
     /// And block instances with visible = 0, copy that information to block_positions
         $DB->execute("INSERT INTO {block_positions} (blockinstanceid, contextid, pagetype, subpage, visible, region, weight)
                 SELECT id, contextid,
@@ -1957,10 +1955,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
                 FROM {block_instances} WHERE visible = 0 AND pagetypepattern <> 'admin-*'");
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009050618);
+        upgrade_main_savepoint(true, 2009050618);
     }
 
-    if ($result && $oldversion < 2009050619) {
+    if ($oldversion < 2009050619) {
         $table = new xmldb_table('block_instances');
 
     /// Define field blockid to be dropped from block_instances
@@ -1994,21 +1992,21 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         }
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009050619);
+        upgrade_main_savepoint(true, 2009050619);
     }
 
-    if ($result && $oldversion < 2009051200) {
+    if ($oldversion < 2009051200) {
     /// Let's check the status of mandatory mnet_host records, fixing them
     /// and moving "orphan" users to default localhost record. MDL-16879
         echo $OUTPUT->notification('Fixing mnet records, this may take a while...', 'notifysuccess');
         upgrade_fix_incorrect_mnethostids();
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009051200);
+        upgrade_main_savepoint(true, 2009051200);
     }
 
 
-    if ($result && $oldversion < 2009051700) {
+    if ($oldversion < 2009051700) {
     /// migrate editor settings
         if (empty($CFG->htmleditor)) {
             set_config('texteditors', 'textarea');
@@ -2020,10 +2018,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         unset_config('defaulthtmleditor');
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009051700);
+        upgrade_main_savepoint(true, 2009051700);
     }
 
-    if ($result && $oldversion < 2009060200) {
+    if ($oldversion < 2009060200) {
     /// Define table files_cleanup to be dropped - not needed
         $table = new xmldb_table('files_cleanup');
 
@@ -2033,10 +2031,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         }
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009060200);
+        upgrade_main_savepoint(true, 2009060200);
     }
 
-    if ($result && $oldversion < 2009061300) {
+    if ($oldversion < 2009061300) {
         //TODO: copy this to the very beginning of this upgrade script so that we may log upgrade queries
 
     /// Define table log_queries to be created
@@ -2062,12 +2060,12 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         }
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009061300);
+        upgrade_main_savepoint(true, 2009061300);
     }
 
     /// Repeat 2009050607 upgrade step, which Petr commented out becuase of XMLDB
     /// stupidity, so lots of peopel will have missed.
-    if ($result && $oldversion < 2009061600) {
+    if ($oldversion < 2009061600) {
     /// Changing precision of field defaultregion on table block_instances to (16)
         $table = new xmldb_table('block_instances');
         $field = new xmldb_field('defaultregion', XMLDB_TYPE_CHAR, '16', null, XMLDB_NOTNULL, null, null, 'configdata');
@@ -2076,10 +2074,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         $dbman->change_field_precision($table, $field);
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009061600);
+        upgrade_main_savepoint(true, 2009061600);
     }
 
-    if ($result && $oldversion < 2009061702) {
+    if ($oldversion < 2009061702) {
         // standardizing plugin names
         if ($configs = $DB->get_records_select('config_plugins', "plugin LIKE 'quizreport_%'")) {
             foreach ($configs as $config) {
@@ -2089,10 +2087,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
             }
         }
         unset($configs);
-        upgrade_main_savepoint($result, 2009061702);
+        upgrade_main_savepoint(true, 2009061702);
     }
 
-    if ($result && $oldversion < 2009061703) {
+    if ($oldversion < 2009061703) {
         // standardizing plugin names
         if ($configs = $DB->get_records_select('config_plugins', "plugin LIKE 'assignment_type_%'")) {
             foreach ($configs as $config) {
@@ -2102,10 +2100,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
             }
         }
         unset($configs);
-        upgrade_main_savepoint($result, 2009061703);
+        upgrade_main_savepoint(true, 2009061703);
     }
 
-    if ($result && $oldversion < 2009061704) {
+    if ($oldversion < 2009061704) {
         // change component string in capability records to new "_" format
         if ($caps = $DB->get_records('capabilities')) {
             foreach ($caps as $cap) {
@@ -2114,10 +2112,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
             }
         }
         unset($caps);
-        upgrade_main_savepoint($result, 2009061704);
+        upgrade_main_savepoint(true, 2009061704);
     }
 
-    if ($result && $oldversion < 2009061705) {
+    if ($oldversion < 2009061705) {
         // change component string in events_handlers records to new "_" format
         if ($handlers = $DB->get_records('events_handlers')) {
             foreach ($handlers as $handler) {
@@ -2126,10 +2124,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
             }
         }
         unset($handlers);
-        upgrade_main_savepoint($result, 2009061705);
+        upgrade_main_savepoint(true, 2009061705);
     }
 
-    if ($result && $oldversion < 2009061706) {
+    if ($oldversion < 2009061706) {
         // change component string in message_providers records to new "_" format
         if ($mps = $DB->get_records('message_providers')) {
             foreach ($mps as $mp) {
@@ -2138,10 +2136,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
             }
         }
         unset($caps);
-        upgrade_main_savepoint($result, 2009061706);
+        upgrade_main_savepoint(true, 2009061706);
     }
 
-    if ($result && $oldversion < 2009063000) {
+    if ($oldversion < 2009063000) {
         // upgrade format of _with_advanced settings - quiz only
         // note: this can be removed later, not needed for upgrades from 1.9.x
         if ($quiz = get_config('quiz')) {
@@ -2154,10 +2152,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
                 unset_config($name, 'quiz');
             }
         }
-        upgrade_main_savepoint($result, 2009063000);
+        upgrade_main_savepoint(true, 2009063000);
     }
 
-    if ($result && $oldversion < 2009071000) {
+    if ($oldversion < 2009071000) {
 
     /// Rename field contextid on table block_instances to parentcontextid
         $table = new xmldb_table('block_instances');
@@ -2167,10 +2165,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         $dbman->rename_field($table, $field, 'parentcontextid');
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009071000);
+        upgrade_main_savepoint(true, 2009071000);
     }
 
-    if ($result && $oldversion < 2009071300) {
+    if ($oldversion < 2009071300) {
 
     /// Create contexts for every block. In the past, only non-sticky course block had contexts.
     /// This is a copy of the code in create_contexts.
@@ -2187,10 +2185,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         build_context_path();
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009071300);
+        upgrade_main_savepoint(true, 2009071300);
     }
 
-    if ($result && $oldversion < 2009071600) {
+    if ($oldversion < 2009071600) {
 
     /// Define field summaryformat to be added to post
         $table = new xmldb_table('post');
@@ -2202,10 +2200,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         }
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009071600);
+        upgrade_main_savepoint(true, 2009071600);
     }
 
-    if ($result && $oldversion < 2009072400) {
+    if ($oldversion < 2009072400) {
 
     /// Define table comments to be created
         $table = new xmldb_table('comments');
@@ -2229,7 +2227,7 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         }
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009072400);
+        upgrade_main_savepoint(true, 2009072400);
     }
 
     /**
@@ -2246,7 +2244,7 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
      * This leaves one hairy end in that we will create block_instances within the
      * DB before the blocks themselves are created within the DB
      */
-    if ($result && $oldversion < 2009082800) {
+    if ($oldversion < 2009082800) {
 
         echo $OUTPUT->notification(get_string('navigationupgrade', 'admin'));
 
@@ -2312,7 +2310,7 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         upgrade_cleanup_unwanted_block_contexts($contextids);
 
         $instanceidstring = join(',',$instanceids);
-        $outcome1 = $result && $DB->delete_records_select('block_positions', 'blockinstanceid IN ('.$instanceidstring.')');
+        $DB->delete_records_select('block_positions', 'blockinstanceid IN ('.$instanceidstring.')');
 
         unset($allblockinstances);
         unset($contextids);
@@ -2332,11 +2330,11 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         }
         unset($newblockinstances);
 
-        upgrade_main_savepoint($result, 2009082800);
+        upgrade_main_savepoint(true, 2009082800);
         // The end of the navigation upgrade
     }
 
-    if ($result && $oldversion < 2009090800){
+    if ($oldversion < 2009090800){
         //insert new record for log_display table
         //used to record tag update.
         if (!$DB->record_exists('log_display', array('action'=>'update', 'module'=>'tag'))) {
@@ -2346,12 +2344,12 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
             $log_action->mtable = 'tag';
             $log_action->field  = 'name';
 
-            $result  = $result && $DB->insert_record('log_display', $log_action);
+            $DB->insert_record('log_display', $log_action);
         }
-        upgrade_main_savepoint($result, 2009090800);
+        upgrade_main_savepoint(true, 2009090800);
     }
 
-    if ($result && $oldversion < 2009100601) {
+    if ($oldversion < 2009100601) {
         // drop all previous tables defined during the dev phase
         $dropold = array('external_services_users', 'external_services_functions', 'external_services', 'external_functions');
         foreach ($dropold as $tablename) {
@@ -2360,10 +2358,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
                 $dbman->drop_table($table);
             }
         }
-        upgrade_main_savepoint($result, 2009100601);
+        upgrade_main_savepoint(true, 2009100601);
     }
 
-    if ($result && $oldversion < 2009100602) {
+    if ($oldversion < 2009100602) {
     /// Define table external_functions to be created
         $table = new xmldb_table('external_functions');
 
@@ -2385,10 +2383,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         $dbman->create_table($table);
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009100602);
+        upgrade_main_savepoint(true, 2009100602);
     }
 
-    if ($result && $oldversion < 2009100603) {
+    if ($oldversion < 2009100603) {
     /// Define table external_services to be created
         $table = new xmldb_table('external_services');
 
@@ -2410,10 +2408,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         $dbman->create_table($table);
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009100603);
+        upgrade_main_savepoint(true, 2009100603);
     }
 
-    if ($result && $oldversion < 2009100604) {
+    if ($oldversion < 2009100604) {
     /// Define table external_services_functions to be created
         $table = new xmldb_table('external_services_functions');
 
@@ -2430,10 +2428,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         $dbman->create_table($table);
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009100604);
+        upgrade_main_savepoint(true, 2009100604);
     }
 
-    if ($result && $oldversion < 2009100605) {
+    if ($oldversion < 2009100605) {
     /// Define table external_services_users to be created
         $table = new xmldb_table('external_services_users');
 
@@ -2454,10 +2452,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         $dbman->create_table($table);
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009100605);
+        upgrade_main_savepoint(true, 2009100605);
     }
 
-    if ($result && $oldversion < 2009102600) {
+    if ($oldversion < 2009102600) {
 
     /// Define table external_tokens to be created
         $table = new xmldb_table('external_tokens');
@@ -2485,10 +2483,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         $dbman->create_table($table);
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009102600);
+        upgrade_main_savepoint(true, 2009102600);
     }
 
-   if ($result && $oldversion < 2009103000) {
+   if ($oldversion < 2009103000) {
 
     /// Define table blog_association to be created
         $table = new xmldb_table('blog_association');
@@ -2550,10 +2548,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
             upgrade_log(UPGRADE_LOG_NOTICE, null, $subject, $description);
         }
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009103000);
+        upgrade_main_savepoint(true, 2009103000);
     }
 
-    if ($result && $oldversion < 2009110400) {
+    if ($oldversion < 2009110400) {
 
         // An array used to store the table name and keys of summary and trust fields
         // to be added
@@ -2590,10 +2588,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
 
         unset($extendtables);
 
-        upgrade_main_savepoint($result, 2009110400);
+        upgrade_main_savepoint(true, 2009110400);
     }
 
-    if ($result && $oldversion < 2009110605) {
+    if ($oldversion < 2009110605) {
 
     /// Define field timecreated to be added to external_services
         $table = new xmldb_table('external_services');
@@ -2614,10 +2612,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         }
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009110605);
+        upgrade_main_savepoint(true, 2009110605);
     }
 
-    if ($result && $oldversion < 2009111600) {
+    if ($oldversion < 2009111600) {
 
     /// Define field instance to be added to portfolio_tempdata
         $table = new xmldb_table('portfolio_tempdata');
@@ -2634,10 +2632,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         $dbman->add_key($table, $key);
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009111600);
+        upgrade_main_savepoint(true, 2009111600);
     }
 
-    if ($result && $oldversion < 2009111700) {
+    if ($oldversion < 2009111700) {
 
     /// Define field tempdataid to be added to portfolio_log
         $table = new xmldb_table('portfolio_log');
@@ -2649,10 +2647,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         }
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009111700);
+        upgrade_main_savepoint(true, 2009111700);
     }
 
-    if ($result && $oldversion < 2009111701) {
+    if ($oldversion < 2009111701) {
 
     /// Define field returnurl to be added to portfolio_log
         $table = new xmldb_table('portfolio_log');
@@ -2664,10 +2662,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         }
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009111701);
+        upgrade_main_savepoint(true, 2009111701);
     }
 
-    if ($result && $oldversion < 2009111702) {
+    if ($oldversion < 2009111702) {
 
     /// Define field continueurl to be added to portfolio_log
         $table = new xmldb_table('portfolio_log');
@@ -2679,19 +2677,19 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         }
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2009111702);
+        upgrade_main_savepoint(true, 2009111702);
     }
 
-    if ($result && $oldversion < 2009112400) {
+    if ($oldversion < 2009112400) {
         if (empty($CFG->passwordsaltmain)) {
             $subject = get_string('check_passwordsaltmain_name', 'report_security');
             $description = get_string('check_passwordsaltmain_warning', 'report_security');;
             upgrade_log(UPGRADE_LOG_NOTICE, null, $subject, $description);
         }
-        upgrade_main_savepoint($result, 2009112400);
+        upgrade_main_savepoint(true, 2009112400);
     }
 
-    if ($result && $oldversion < 2010010601) {
+    if ($oldversion < 2010010601) {
 
     /// Define field creatorid to be added to external_tokens
         $table = new xmldb_table('external_tokens');
@@ -2710,10 +2708,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         $dbman->add_key($table, $key);
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2010010601);
+        upgrade_main_savepoint(true, 2010010601);
     }
 
-    if ($result && $oldversion < 2010011200) {
+    if ($oldversion < 2010011200) {
         $table = new xmldb_table('grade_categories');
         $field = new xmldb_field('hidden', XMLDB_TYPE_INTEGER, '10', XMLDB_UNSIGNED, XMLDB_NOTNULL, null, 0);
 
@@ -2721,16 +2719,16 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
             $dbman->add_field($table, $field);
         }
 
-        upgrade_main_savepoint($result, 2010011200);
+        upgrade_main_savepoint(true, 2010011200);
     }
 
 
-    if ($result && $oldversion < 2010012500) {
+    if ($oldversion < 2010012500) {
         upgrade_fix_incorrect_mnethostids();
-        upgrade_main_savepoint($result, 2010012500);
+        upgrade_main_savepoint(true, 2010012500);
     }
 
-    if ($result && $oldversion < 2010012600) {
+    if ($oldversion < 2010012600) {
         // do stuff to the mnet table
         $table = new xmldb_table('mnet_rpc');
 
@@ -2756,10 +2754,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         }
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2010012600);
+        upgrade_main_savepoint(true, 2010012600);
     }
 
-    if ($result && $oldversion < 2010012900) {
+    if ($oldversion < 2010012900) {
 
     /// Define table mnet_remote_rpc to be created
         $table = new xmldb_table('mnet_remote_rpc');
@@ -2815,10 +2813,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
 
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2010012900);
+        upgrade_main_savepoint(true, 2010012900);
     }
 
-    if ($result && $oldversion < 2010012901) {
+    if ($oldversion < 2010012901) {
 
         /// Define field plugintype to be added to mnet_remote_rpc
         $table = new xmldb_table('mnet_remote_rpc');
@@ -2838,10 +2836,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         }
 
         /// Main savepoint reached
-        upgrade_main_savepoint($result, 2010012901);
+        upgrade_main_savepoint(true, 2010012901);
     }
 
-    if ($result && $oldversion < 2010012902) {
+    if ($oldversion < 2010012902) {
 
     /// Define field enabled to be added to mnet_remote_rpc
         $table = new xmldb_table('mnet_remote_rpc');
@@ -2853,11 +2851,11 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         }
 
         /// Main savepoint reached
-        upgrade_main_savepoint($result, 2010012902);
+        upgrade_main_savepoint(true, 2010012902);
     }
 
     /// MDL-17863. Increase the portno column length on mnet_host to handle any port number
-    if ($result && $oldversion < 2010020100) {
+    if ($oldversion < 2010020100) {
     /// Changing precision of field portno on table mnet_host to (5)
         $table = new xmldb_table('mnet_host');
         $field = new xmldb_field('portno', XMLDB_TYPE_INTEGER, '5', XMLDB_UNSIGNED, XMLDB_NOTNULL, null, '0', 'transport');
@@ -2865,10 +2863,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
     /// Launch change of precision for field portno
         $dbman->change_field_precision($table, $field);
 
-        upgrade_main_savepoint($result, 2010020100);
+        upgrade_main_savepoint(true, 2010020100);
     }
 
-    if ($result && $oldversion < 2010020300) {
+    if ($oldversion < 2010020300) {
 
     /// Define field timecreated to be added to user
         $table = new xmldb_table('user');
@@ -2883,38 +2881,38 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
             $sql = "UPDATE {user} SET timecreated = " . time() ." where timecreated = 0";
             $DB->execute($sql);
         }
-        upgrade_main_savepoint($result, 2010020300);
+        upgrade_main_savepoint(true, 2010020300);
     }
 
     // MDL-21407. Trim leading spaces from default tex latexpreamble causing problems under some confs
-    if ($result && $oldversion < 2010020301) {
+    if ($oldversion < 2010020301) {
         if ($preamble = $CFG->filter_tex_latexpreamble) {
             $preamble = preg_replace('/^ +/m', '', $preamble);
             set_config('filter_tex_latexpreamble', $preamble);
         }
-        upgrade_main_savepoint($result, 2010020301);
+        upgrade_main_savepoint(true, 2010020301);
     }
 
-    if ($result && $oldversion < 2010021400) {
+    if ($oldversion < 2010021400) {
     /// Changes to modinfo mean we need to rebuild course cache
         require_once($CFG->dirroot . '/course/lib.php');
         rebuild_course_cache(0, true);
-        upgrade_main_savepoint($result, 2010021400);
+        upgrade_main_savepoint(true, 2010021400);
     }
 
-    if ($result && $oldversion < 2010021800) {
+    if ($oldversion < 2010021800) {
         $DB->set_field('mnet_application', 'sso_jump_url', '/auth/mnet/jump.php', array('name' => 'moodle'));
-        upgrade_main_savepoint($result, 2010021800);
+        upgrade_main_savepoint(true, 2010021800);
     }
 
-    if ($result && $oldversion < 2010031900) {
+    if ($oldversion < 2010031900) {
         // regeneration of sessions is always enabled, no need for this setting any more
         unset_config('regenloginsession');
-        upgrade_main_savepoint($result, 2010031900);
+        upgrade_main_savepoint(true, 2010031900);
     }
 
 
-    if ($result && $oldversion < 2010032400) {
+    if ($oldversion < 2010032400) {
         // Upgrade all of those using the standardold theme to the use the standard
         // theme instead
         if ($CFG->theme == 'standardold') {
@@ -2927,10 +2925,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         $DB->execute('UPDATE {course} SET theme=? WHERE theme=?', array('standard', 'standardold'));
         // User
         $DB->execute('UPDATE {user} SET theme=? WHERE theme=?', array('standard', 'standardold'));
-        upgrade_main_savepoint($result, 2010032400);
+        upgrade_main_savepoint(true, 2010032400);
     }
 
-    if ($result && $oldversion < 2010033101.01) {
+    if ($oldversion < 2010033101.01) {
 
     /// Define field source to be added to files
         $table = new xmldb_table('files');
@@ -2956,10 +2954,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
             $dbman->add_field($table, $field);
         }
 
-        upgrade_main_savepoint($result, 2010033101.01);
+        upgrade_main_savepoint(true, 2010033101.01);
     }
 
-    if ($result && $oldversion < 2010033101.02) {
+    if ($oldversion < 2010033101.02) {
 
     /// Define table license to be created
         $table = new xmldb_table('license');
@@ -3151,10 +3149,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         set_config('sitedefaultlicense', 'allrightsreserved');
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2010033101.02);
+        upgrade_main_savepoint(true, 2010033101.02);
     }
 
-    if ($result && $oldversion < 2010033102.00) {
+    if ($oldversion < 2010033102.00) {
         // rename course view capability to participate
         $params = array('view'=>'moodle/course:view', 'participate'=>'moodle/course:participate');
         $sql = "UPDATE {role_capabilities} SET capability = :participate WHERE capability = :view";
@@ -3162,18 +3160,18 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         $sql = "UPDATE {capabilities} SET name = :participate WHERE name = :view";
         $DB->execute($sql, $params);
         // note: the view capability is readded again at the end of upgrade, but with different meaning
-        upgrade_main_savepoint($result, 2010033102.00);
+        upgrade_main_savepoint(true, 2010033102.00);
     }
 
-    if ($result && $oldversion < 2010033102.01) {
+    if ($oldversion < 2010033102.01) {
         // Define field archetype to be added to role table
         $table = new xmldb_table('role');
         $field = new xmldb_field('archetype', XMLDB_TYPE_CHAR, '30', null, XMLDB_NOTNULL, null, null, 'sortorder');
         $dbman->add_field($table, $field);
-        upgrade_main_savepoint($result, 2010033102.01);
+        upgrade_main_savepoint(true, 2010033102.01);
     }
 
-    if ($result && $oldversion < 2010033102.02) {
+    if ($oldversion < 2010033102.02) {
         // Set archetype for existing roles and change admin role to manager role
         $sql = "SELECT r.*, rc.capability
                   FROM {role} r
@@ -3198,10 +3196,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         }
         $roles->close();
 
-        upgrade_main_savepoint($result, 2010033102.02);
+        upgrade_main_savepoint(true, 2010033102.02);
     }
 
-    if ($result && $oldversion < 2010033102.03) {
+    if ($oldversion < 2010033102.03) {
         // Now pick site admins (===have manager role assigned at the system context)
         // and store them in the new $CFG->siteadmins setting as comma separated list
         $sql = "SELECT ra.id, ra.userid
@@ -3218,10 +3216,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
             $DB->delete_records('role_assignments', array('id'=>$ra->id));
         }
 
-        upgrade_main_savepoint($result, 2010033102.03);
+        upgrade_main_savepoint(true, 2010033102.03);
     }
 
-    if ($result && $oldversion < 2010033102.04) {
+    if ($oldversion < 2010033102.04) {
         // clean up the manager roles
         $managers = $DB->get_records('role', array('archetype'=>'manager'));
         foreach ($managers as $manager) {
@@ -3258,19 +3256,19 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
             }
         }
 
-        upgrade_main_savepoint($result, 2010033102.04);
+        upgrade_main_savepoint(true, 2010033102.04);
     }
 
-    if ($result && $oldversion < 2010033102.05) {
+    if ($oldversion < 2010033102.05) {
         // remove course:view from all roles that are not used for enrolment, it does NOT belong there because it really means user is enrolled!
         $noenrolroles = $DB->get_records_select('role', "archetype IN ('guest', 'user', 'manager', 'coursecreator', 'frontpage')");
         foreach ($noenrolroles as $role) {
             $DB->delete_records('role_capabilities', array('roleid'=>$role->id, 'capability'=>'moodle/course:participate'));
         }
-        upgrade_main_savepoint($result, 2010033102.05);
+        upgrade_main_savepoint(true, 2010033102.05);
     }
 
-    if ($result && $oldversion < 2010033102.06) {
+    if ($oldversion < 2010033102.06) {
         // make sure there is nothing weird in default user role
         if (!empty($CFG->defaultuserroleid)) {
             if ($role = $DB->get_record('role', array('id'=>$CFG->defaultuserroleid))) {
@@ -3282,18 +3280,18 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
                 unset_config('defaultuserroleid');
             }
         }
-        upgrade_main_savepoint($result, 2010033102.06);
+        upgrade_main_savepoint(true, 2010033102.06);
     }
 
-    if ($result && $oldversion < 2010033102.07) {
+    if ($oldversion < 2010033102.07) {
         if (!empty($CFG->displayloginfailures) and $CFG->displayloginfailures === 'teacher') {
             upgrade_log(UPGRADE_LOG_NOTICE, null, 'Displaying of login failuters to teachers is not supported any more.');
             unset_config('displayloginfailures');
         }
-        upgrade_main_savepoint($result, 2010033102.07);
+        upgrade_main_savepoint(true, 2010033102.07);
     }
 
-    if ($result && $oldversion < 2010033102.08) {
+    if ($oldversion < 2010033102.08) {
         // make sure there are no problems in default guest role settings
         if (!empty($CFG->guestroleid)) {
             if ($role = $DB->get_record('role', array('id'=>$CFG->guestroleid))) {
@@ -3310,11 +3308,11 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         // the guest account gets all the role assignments on the fly which works fine in has_capability(),
         $DB->delete_records_select('role_assignments', "userid IN (SELECT id FROM {user} WHERE username = 'guest')");
 
-        upgrade_main_savepoint($result, 2010033102.08);
+        upgrade_main_savepoint(true, 2010033102.08);
     }
 
     /// New table for storing which roles can be assigned in which contexts.
-    if ($result && $oldversion < 2010033102.09) {
+    if ($oldversion < 2010033102.09) {
 
     /// Define table role_context_levels to be created
         $table = new xmldb_table('role_context_levels');
@@ -3335,10 +3333,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         }
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2010033102.09);
+        upgrade_main_savepoint(true, 2010033102.09);
     }
 
-    if ($result && $oldversion < 2010033102.10) {
+    if ($oldversion < 2010033102.10) {
         // Now populate the role_context_levels table with the default values
         // NOTE: do not use accesslib methods here
 
@@ -3394,10 +3392,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         unset($rolecontextlevels);
 
         // Main savepoint reached
-        upgrade_main_savepoint($result, 2010033102.10);
+        upgrade_main_savepoint(true, 2010033102.10);
     }
 
-    if ($result && $oldversion < 2010040700) {
+    if ($oldversion < 2010040700) {
         // migrate old groupings --> groupmembersonly setting
         if (isset($CFG->enablegroupings)) {
             set_config('enablegroupmembersonly', $CFG->enablegroupings);
@@ -3405,10 +3403,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         }
 
         // Main savepoint reached
-        upgrade_main_savepoint($result, 2010040700);
+        upgrade_main_savepoint(true, 2010040700);
     }
 
-    if ($result && $oldversion < 2010040900) {
+    if ($oldversion < 2010040900) {
 
         // Changing the default of field lang on table user to good old "en"
         $table = new xmldb_table('user');
@@ -3434,10 +3432,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         }
 
         // Main savepoint reached
-        upgrade_main_savepoint($result, 2010040900);
+        upgrade_main_savepoint(true, 2010040900);
     }
 
-    if ($result && $oldversion < 2010040901) {
+    if ($oldversion < 2010040901) {
 
         // Remove "_utf8" suffix from all langs in user table
         $langs = $DB->get_records_sql("SELECT DISTINCT lang FROM {user} WHERE lang LIKE ?", array('%_utf8'));
@@ -3449,10 +3447,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         }
 
         // Main savepoint reached
-        upgrade_main_savepoint($result, 2010040901);
+        upgrade_main_savepoint(true, 2010040901);
     }
 
-    if ($result && $oldversion < 2010041301) {
+    if ($oldversion < 2010041301) {
         $sql = "UPDATE {block} SET name=? WHERE name=?";
         $DB->execute($sql, array('navigation', 'global_navigation_tree'));
         $DB->execute($sql, array('settings', 'settings_navigation_tree'));
@@ -3460,10 +3458,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         $sql = "UPDATE {block_instances} SET blockname=? WHERE blockname=?";
         $DB->execute($sql, array('navigation', 'global_navigation_tree'));
         $DB->execute($sql, array('settings', 'settings_navigation_tree'));
-        upgrade_main_savepoint($result, 2010041301);
+        upgrade_main_savepoint(true, 2010041301);
     }
 
-    if ($result && $oldversion < 2010042100) {
+    if ($oldversion < 2010042100) {
 
     /// Define table backup_controllers to be created
         $table = new xmldb_table('backup_controllers');
@@ -3521,10 +3519,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         }
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2010042100);
+        upgrade_main_savepoint(true, 2010042100);
     }
 
-    if ($result && $oldversion < 2010042301) {
+    if ($oldversion < 2010042301) {
 
         $table = new xmldb_table('course_sections');
         $field = new xmldb_field('name', XMLDB_TYPE_CHAR, '255', null, null, null, null, 'section');
@@ -3533,10 +3531,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
             $dbman->add_field($table, $field);
         }
 
-        upgrade_main_savepoint($result, 2010042301);
+        upgrade_main_savepoint(true, 2010042301);
     }
 
-    if ($result && $oldversion < 2010042302) {
+    if ($oldversion < 2010042302) {
         // Define table cohort to be created
         $table = new xmldb_table('cohort');
 
@@ -3560,10 +3558,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
             $dbman->create_table($table);
         }
 
-        upgrade_main_savepoint($result, 2010042302);
+        upgrade_main_savepoint(true, 2010042302);
     }
 
-    if ($result && $oldversion < 2010042303) {
+    if ($oldversion < 2010042303) {
         // Define table cohort_members to be created
         $table = new xmldb_table('cohort_members');
 
@@ -3587,10 +3585,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         }
 
         // Main savepoint reached
-        upgrade_main_savepoint($result, 2010042303);
+        upgrade_main_savepoint(true, 2010042303);
     }
 
-    if ($result && $oldversion < 2010042800) {
+    if ($oldversion < 2010042800) {
         //drop the previously created ratings table
         $table = new xmldb_table('ratings');
         if ($dbman->table_exists($table)) {
@@ -3628,10 +3626,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
             $dbman->create_table($table);
         }
 
-        upgrade_main_savepoint($result, 2010042800);
+        upgrade_main_savepoint(true, 2010042800);
     }
 
-    if ($result && $oldversion < 2010042801) {
+    if ($oldversion < 2010042801) {
         // migrating old comments block content
         $DB->execute("UPDATE {comments}
                          SET contextid = (SELECT parentcontextid
@@ -3648,10 +3646,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
 
         // remove all orphaned record
         $DB->delete_records('comments', array('commentarea'=>'block_comments'));
-        upgrade_main_savepoint($result, 2010042801);
+        upgrade_main_savepoint(true, 2010042801);
     }
 
-    if ($result && $oldversion < 2010042802) { // Change backup_controllers->type to varchar10 (recreate dep. index)
+    if ($oldversion < 2010042802) { // Change backup_controllers->type to varchar10 (recreate dep. index)
 
     /// Define index typeitem_ix (not unique) to be dropped form backup_controllers
         $table = new xmldb_table('backup_controllers');
@@ -3679,10 +3677,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         }
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2010042802);
+        upgrade_main_savepoint(true, 2010042802);
     }
 
-    if ($result && $oldversion < 2010043000) {  // Adding new course completion feature
+    if ($oldversion < 2010043000) {  // Adding new course completion feature
 
     /// Add course completion tables
     /// Define table course_completion_aggr_methd to be created
@@ -3838,10 +3836,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
             $dbman->add_field($table, $field);
         }
 
-        upgrade_main_savepoint($result, 2010043000);
+        upgrade_main_savepoint(true, 2010043000);
     }
 
-    if ($result && $oldversion < 2010043001) {
+    if ($oldversion < 2010043001) {
 
     /// Define table registration_hubs to be created
         $table = new xmldb_table('registration_hubs');
@@ -3862,10 +3860,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         }
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2010043001);
+        upgrade_main_savepoint(true, 2010043001);
     }
 
-    if ($result && $oldversion < 2010050200) {
+    if ($oldversion < 2010050200) {
 
     /// Define table backup_logs to be created
         $table = new xmldb_table('backup_logs');
@@ -3908,10 +3906,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         }
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2010050200);
+        upgrade_main_savepoint(true, 2010050200);
     }
 
-    if ($result && $oldversion < 2010050403) {  // my_pages for My Moodle and Public Profile pages
+    if ($oldversion < 2010050403) {  // my_pages for My Moodle and Public Profile pages
 
     /// Define table my_pages to be created
         $table = new xmldb_table('my_pages');
@@ -3986,20 +3984,20 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         }
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2010050403);
+        upgrade_main_savepoint(true, 2010050403);
     }
 
     // fix repository_alfresco version number
-    if ($result && $oldversion < 2010050411) {
+    if ($oldversion < 2010050411) {
         // force to change repository_alfresco version number to upgrade smoothly (from preview 1 -> preview 2)
         if ($repository_plugin = $DB->get_record('config_plugins', array('plugin'=>'repository_alfresco', 'name'=>'version'))) {
             $repository_plugin->value = '2008000000';
             $DB->update_record('config_plugins', $repository_plugin);
         }
-        upgrade_main_savepoint($result, 2010050411);
+        upgrade_main_savepoint(true, 2010050411);
     }
 
-    if ($result && $oldversion < 2010051500) {
+    if ($oldversion < 2010051500) {
 
     /// Fix a bad table name that existed for a few days in HEAD
         $table = new xmldb_table('published_courses');
@@ -4027,20 +4025,20 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         }
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2010051500);
+        upgrade_main_savepoint(true, 2010051500);
     }
 
-    if ($result && $oldversion < 2010051600) {
+    if ($oldversion < 2010051600) {
 
     /// Delete the blocks completely.  All the contexts, instances etc were cleaned up above in 2009082800
         $DB->delete_records('block', array('name'=>'admin'));
         $DB->delete_records('block', array('name'=>'admin_tree'));
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2010051600);
+        upgrade_main_savepoint(true, 2010051600);
     }
 
-    if ($result && $oldversion < 2010051800) {
+    if ($oldversion < 2010051800) {
         // switching to userid in config settings because user names are not unique and reliable enough
         if (!empty($CFG->courserequestnotify) and $CFG->courserequestnotify !== '$@NONE@$' and $CFG->courserequestnotify !== '$@ALL@$') {
             list($where, $params) = $DB->get_in_or_equal(explode(',', $CFG->courserequestnotify));
@@ -4052,10 +4050,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
                 set_config('courserequestnotify', '$@NONE@$');
             }
         }
-        upgrade_main_savepoint($result, 2010051800);
+        upgrade_main_savepoint(true, 2010051800);
     }
 
-    if ($result && $oldversion < 2010051801) {
+    if ($oldversion < 2010051801) {
         // Update the notifyloginfailures setting.
         if ($CFG->notifyloginfailures == 'mainadmin') {
             if ($admins = explode(',', $CFG->siteadmins)) {
@@ -4073,18 +4071,18 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
             set_config('notifyloginfailures', '$@NONE@$');
         }
 
-        upgrade_main_savepoint($result, 2010051801);
+        upgrade_main_savepoint(true, 2010051801);
     }
 
-    if ($result && $oldversion < 2010052100) {
+    if ($oldversion < 2010052100) {
         // Switch to html purifier as default cleaning engine - KSES is really very bad
         if (empty($CFG->enablehtmlpurifier)) {
             unset_config('enablehtmlpurifier');
         }
-        upgrade_main_savepoint($result, 2010052100);
+        upgrade_main_savepoint(true, 2010052100);
     }
 
-    if ($result && $oldversion < 2010052200) {
+    if ($oldversion < 2010052200) {
         // Define field legacyfiles to be added to course - just in case we are upgrading from PR1
         $table = new xmldb_table('course');
         $field = new xmldb_field('legacyfiles', XMLDB_TYPE_INTEGER, '4', XMLDB_UNSIGNED, XMLDB_NOTNULL, null, '0', 'maxbytes');
@@ -4097,10 +4095,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         }
 
         // Main savepoint reached
-        upgrade_main_savepoint($result, 2010052200);
+        upgrade_main_savepoint(true, 2010052200);
     }
 
-    if ($result && $oldversion < 2010052401) {
+    if ($oldversion < 2010052401) {
 
     /// Define field status to be added to course_published
         $table = new xmldb_table('course_published');
@@ -4121,10 +4119,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         }
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2010052401);
+        upgrade_main_savepoint(true, 2010052401);
     }
 
-    if ($result && $oldversion < 2010052700) {
+    if ($oldversion < 2010052700) {
 
     /// Define field summaryformat to be added to course sections table
         $table = new xmldb_table('course_sections');
@@ -4136,17 +4134,17 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         }
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2010052700);
+        upgrade_main_savepoint(true, 2010052700);
     }
 
-    if ($result && $oldversion < 2010052800) {
+    if ($oldversion < 2010052800) {
     /// Changes to modinfo mean we need to rebuild course cache
         require_once($CFG->dirroot . '/course/lib.php');
         rebuild_course_cache(0, true);
-        upgrade_main_savepoint($result, 2010052800);
+        upgrade_main_savepoint(true, 2010052800);
     }
 
-    if ($result && $oldversion < 2010052801) {
+    if ($oldversion < 2010052801) {
 
     /// Define field sortorder to be added to files
         $table = new xmldb_table('files');
@@ -4158,10 +4156,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         }
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2010052801);
+        upgrade_main_savepoint(true, 2010052801);
     }
 
-    if ($result && $oldversion < 2010061900.01) {
+    if ($oldversion < 2010061900.01) {
         // Define table enrol to be created
         $table = new xmldb_table('enrol');
 
@@ -4206,10 +4204,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         $dbman->create_table($table);
 
         // Main savepoint reached
-        upgrade_main_savepoint($result, 2010061900.01);
+        upgrade_main_savepoint(true, 2010061900.01);
     }
 
-    if ($result && $oldversion < 2010061900.02) {
+    if ($oldversion < 2010061900.02) {
         // Define table course_participant to be created
         $table = new xmldb_table('user_enrolments');
 
@@ -4237,10 +4235,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         $dbman->create_table($table);
 
         // Main savepoint reached
-        upgrade_main_savepoint($result, 2010061900.02);
+        upgrade_main_savepoint(true, 2010061900.02);
     }
 
-    if ($result && $oldversion < 2010061900.03) {
+    if ($oldversion < 2010061900.03) {
         // Define field itemid to be added to role_assignments
         $table = new xmldb_table('role_assignments');
         $field = new xmldb_field('itemid', XMLDB_TYPE_INTEGER, '10', XMLDB_UNSIGNED, XMLDB_NOTNULL, null, 0, 'enrol');
@@ -4259,10 +4257,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         }
 
         // Main savepoint reached
-        upgrade_main_savepoint($result, 2010061900.03);
+        upgrade_main_savepoint(true, 2010061900.03);
     }
 
-    if ($result && $oldversion < 2010061900.04) {
+    if ($oldversion < 2010061900.04) {
         // there is no default course role any more, each enrol plugin has to handle it separately
         if (!empty($CFG->defaultcourseroleid)) {
             $sql = "UPDATE {course} SET defaultrole = :default WHERE defaultrole = 0";
@@ -4272,10 +4270,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         unset_config('defaultcourseroleid');
 
         // Main savepoint reached
-        upgrade_main_savepoint($result, 2010061900.04);
+        upgrade_main_savepoint(true, 2010061900.04);
     }
 
-    if ($result && $oldversion < 2010061900.05) {
+    if ($oldversion < 2010061900.05) {
         // make sure enrol settings make actually sense and tweak defaults a bit
 
         $sqlempty = $DB->sql_empty();
@@ -4295,10 +4293,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         }
 
         // Main savepoint reached
-        upgrade_main_savepoint($result, 2010061900.05);
+        upgrade_main_savepoint(true, 2010061900.05);
     }
 
-    if ($result && $oldversion < 2010061900.06) {
+    if ($oldversion < 2010061900.06) {
         $sqlempty = $DB->sql_empty();
         $params = array('siteid'=>SITEID);
 
@@ -4330,10 +4328,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
                  WHERE guest = 2 and password <> '$sqlempty' AND id <> :siteid";
         $DB->execute($sql, $params);
 
-        upgrade_main_savepoint($result, 2010061900.06);
+        upgrade_main_savepoint(true, 2010061900.06);
     }
 
-    if ($result && $oldversion < 2010061900.07) {
+    if ($oldversion < 2010061900.07) {
         // now migrate old style "interactive" enrol plugins - we know them by looking into course.enrol
         $params = array('siteid'=>SITEID);
         $enabledplugins = explode(',', $CFG->enrol_plugins_enabled);
@@ -4352,10 +4350,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
             $params['plugin'] = $plugin;
             $DB->execute($sql, $params);
         }
-        upgrade_main_savepoint($result, 2010061900.07);
+        upgrade_main_savepoint(true, 2010061900.07);
     }
 
-    if ($result && $oldversion < 2010061900.08) {
+    if ($oldversion < 2010061900.08) {
         // now migrate the rest - these plugins are not in course.enrol, instead we just look for suspicious role assignments,
         // unfortunately old enrol plugins were doing sometimes weird role assignments :-(
 
@@ -4397,10 +4395,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         }
         $DB->execute($sql, $params);
 
-        upgrade_main_savepoint($result, 2010061900.08);
+        upgrade_main_savepoint(true, 2010061900.08);
     }
 
-    if ($result && $oldversion < 2010061900.09) {
+    if ($oldversion < 2010061900.09) {
         // unfortunately there may be still some leftovers
         // after reconfigured, uninstalled or borked enrol plugins,
         // unfortunately this may be a bit slow - but there should not be many of these
@@ -4418,20 +4416,20 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
             $DB->insert_record('enrol', $enrol);
         }
         $rs->close();
-        upgrade_main_savepoint($result, 2010061900.09);
+        upgrade_main_savepoint(true, 2010061900.09);
     }
 
-    if ($result && $oldversion < 2010061900.10) {
+    if ($oldversion < 2010061900.10) {
         // migrate existing setup of meta courses
         $sql = "INSERT INTO {enrol} (enrol, status, courseid, sortorder, customint1)
                 SELECT 'meta', 0, parent_course, 5, child_course
                   FROM {course_meta}";
         $DB->execute($sql);
 
-        upgrade_main_savepoint($result, 2010061900.10);
+        upgrade_main_savepoint(true, 2010061900.10);
     }
 
-    if ($result && $oldversion < 2010061900.11) {
+    if ($oldversion < 2010061900.11) {
         // nuke any old role assignments+enrolments in previous meta courses, we have to start from scratch
         $select = "SELECT ctx.id
                      FROM {context} ctx
@@ -4445,10 +4443,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         $dbman->drop_table($table);
 
         // Main savepoint reached
-        upgrade_main_savepoint($result, 2010061900.11);
+        upgrade_main_savepoint(true, 2010061900.11);
     }
 
-    if ($result && $oldversion < 2010061900.12) {
+    if ($oldversion < 2010061900.12) {
         // finally remove all obsolete fields from the course table - yay!
         // all the information was migrated to the enrol table
 
@@ -4578,10 +4576,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
             $dbman->drop_field($table, $field);
         }
 
-        upgrade_main_savepoint($result, 2010061900.12);
+        upgrade_main_savepoint(true, 2010061900.12);
     }
 
-    if ($result && $oldversion < 2010061900.13) {
+    if ($oldversion < 2010061900.13) {
         // Define field visibleold to be added to course_categories
         $table = new xmldb_table('course_categories');
         $field = new xmldb_field('visibleold', XMLDB_TYPE_INTEGER, '1', null, XMLDB_NOTNULL, null, '1', 'visible');
@@ -4590,10 +4588,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         $dbman->add_field($table, $field);
 
         // Main savepoint reached
-        upgrade_main_savepoint($result, 2010061900.13);
+        upgrade_main_savepoint(true, 2010061900.13);
     }
 
-    if ($result && $oldversion < 2010061900.14) {
+    if ($oldversion < 2010061900.14) {
         // keep previous visible state
         $DB->execute("UPDATE {course_categories} SET visibleold = visible");
 
@@ -4607,10 +4605,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
                 $DB->set_field('course_categories', 'visible', 0, array('id'=>$cat->id));
             }
         }
-        upgrade_main_savepoint($result, 2010061900.14);
+        upgrade_main_savepoint(true, 2010061900.14);
     }
 
-    if ($result && $oldversion < 2010061900.15) {
+    if ($oldversion < 2010061900.15) {
         // Define field visibleold to be added to course
         $table = new xmldb_table('course');
         $field = new xmldb_field('visibleold', XMLDB_TYPE_INTEGER, '1', XMLDB_UNSIGNED, XMLDB_NOTNULL, null, '1', 'visible');
@@ -4619,20 +4617,20 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         $dbman->add_field($table, $field);
 
         // Main savepoint reached
-        upgrade_main_savepoint($result, 2010061900.15);
+        upgrade_main_savepoint(true, 2010061900.15);
     }
 
-    if ($result && $oldversion < 2010061900.16) {
+    if ($oldversion < 2010061900.16) {
         // keep previous visible state
         $DB->execute("UPDATE {course} SET visibleold = visible");
 
         // make sure all courses in hidden categories are hidden
         $DB->execute("UPDATE {course} SET visible = 0 WHERE category IN (SELECT id FROM {course_categories} WHERE visible = 0)");
 
-        upgrade_main_savepoint($result, 2010061900.16);
+        upgrade_main_savepoint(true, 2010061900.16);
     }
 
-    if ($result && $oldversion < 2010061900.20) {
+    if ($oldversion < 2010061900.20) {
         // now set up the enrolments - look for roles with course:participate only at course context - the category enrolments are synchronised later by archetype and new capability
 
         $syscontext = get_context_instance(CONTEXT_SYSTEM);
@@ -4651,10 +4649,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
               GROUP BY e.id, ra.userid";
         $DB->execute($sql, $params);
 
-        upgrade_main_savepoint($result, 2010061900.20);
+        upgrade_main_savepoint(true, 2010061900.20);
     }
 
-    if ($result && $oldversion < 2010061900.21) {
+    if ($oldversion < 2010061900.21) {
         // hidden is completely removed, timestart+timeend are now in the user_enrolments table
 
         // Define field hidden to be dropped from role_assignments
@@ -4685,10 +4683,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         }
 
         // Main savepoint reached
-        upgrade_main_savepoint($result, 2010061900.21);
+        upgrade_main_savepoint(true, 2010061900.21);
     }
 
-    if ($result && $oldversion < 2010061900.22) {
+    if ($oldversion < 2010061900.22) {
         // Rename field enrol on table role_assignments to component and update content
 
         $table = new xmldb_table('role_assignments');
@@ -4736,10 +4734,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         $DB->execute($sql, $params);
 
         // Main savepoint reached
-        upgrade_main_savepoint($result, 2010061900.22);
+        upgrade_main_savepoint(true, 2010061900.22);
     }
 
-    if ($result && $oldversion < 2010061900.23) {
+    if ($oldversion < 2010061900.23) {
         // add proper itemid to role assignments that were added by enrolment plugins
         $sql = "UPDATE {role_assignments}
                    SET itemid = (SELECT MIN({enrol}.id)
@@ -4749,10 +4747,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
                  WHERE component <> 'enrol_manual' AND component LIKE 'enrol_%'";
         $DB->execute($sql);
         // Main savepoint reached
-        upgrade_main_savepoint($result, 2010061900.23);
+        upgrade_main_savepoint(true, 2010061900.23);
     }
 
-    if ($result && $oldversion < 2010061900.30) {
+    if ($oldversion < 2010061900.30) {
         // make new list of active enrol plugins - order is important, meta should be always last, manual first
         $enabledplugins = explode(',', $CFG->enrol_plugins_enabled);
         $enabledplugins = array_merge(array('manual', 'guest', 'self', 'cohort'), $enabledplugins);
@@ -4763,10 +4761,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         set_config('enrol_plugins_enabled', implode(',', $enabledplugins));
 
         // Main savepoint reached
-        upgrade_main_savepoint($result, 2010061900.30);
+        upgrade_main_savepoint(true, 2010061900.30);
     }
 
-    if ($result && $oldversion < 2010061900.31) {
+    if ($oldversion < 2010061900.31) {
         // finalise all new enrol settings and cleanup old settings
 
         // legacy allowunenrol was deprecated in 1.9 already
@@ -4813,17 +4811,17 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
             unset_config('enrol_manual_showhint');
         }
 
-        upgrade_main_savepoint($result, 2010061900.31);
+        upgrade_main_savepoint(true, 2010061900.31);
     }
 
-    if ($result && $oldversion < 2010061900.32) {
+    if ($oldversion < 2010061900.32) {
         // MDL-22797 course completion has to be updated to use new enrol framework, it will not be enabled in final 2.0
         set_config('enableavailability', 0);
         set_config('enablecompletion', 0);
-        upgrade_main_savepoint($result, 2010061900.32);
+        upgrade_main_savepoint(true, 2010061900.32);
     }
 
-    if ($result && $oldversion < 2010062101) {
+    if ($oldversion < 2010062101) {
 
     /// Define field huburl to be dropped from course_published
         $table = new xmldb_table('course_published');
@@ -4844,10 +4842,10 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         }
 
     /// Main savepoint reached
-        upgrade_main_savepoint($result, 2010062101);
+        upgrade_main_savepoint(true, 2010062101);
     }
 
-    if ($result && $oldversion < 2010070300) {
+    if ($oldversion < 2010070300) {
         //TODO: this is a temporary hack for upgrade from PR3, to be removed later
 
         // Define field component to be added to files
@@ -4903,11 +4901,11 @@ WHERE gradeitemid IS NOT NULL AND grademax IS NOT NULL");
         }
 
         // Main savepoint reached
-        upgrade_main_savepoint($result, 2010070300);
+        upgrade_main_savepoint(true, 2010070300);
     }
 
 
-    return $result;
+    return true;
 }
 
 //TODO: Cleanup before the 2.0 release - we do not want to drag along these dev machine fixes forever
index 39139f4..1454ffc 100644 (file)
@@ -68,5 +68,5 @@
 
 /// Some strings used widely
     define ('XMLDB_LINEFEED', "\n");
-    define ('XMLDB_PHP_HEADER', '    if ($result && $oldversion < XXXXXXXXXX) {' . XMLDB_LINEFEED);
+    define ('XMLDB_PHP_HEADER', '    if ($oldversion < XXXXXXXXXX) {' . XMLDB_LINEFEED);
     define ('XMLDB_PHP_FOOTER', '    }' . XMLDB_LINEFEED);
index b6f1d3a..d2d8889 100644 (file)
@@ -35,20 +35,19 @@ function xmldb_message_email_upgrade($oldversion) {
     global $CFG, $DB;
 
     $dbman = $DB->get_manager();
-    $result = true;
 
-    if ($result && $oldversion < 2008072401) {
+    if ($oldversion < 2008072401) {
         $processor = new object();
         $processor->name  = 'email';
-        if (! $DB->record_exists('message_processors', array('name' => $processor->name)) ){
-            $result = $result && $DB->insert_record('message_processors', $processor);
+        if (!$DB->record_exists('message_processors', array('name' => $processor->name)) ){
+            $DB->insert_record('message_processors', $processor);
         }
 
     /// email savepoint reached
-        upgrade_plugin_savepoint($result, 2008072401, 'message', 'email');
+        upgrade_plugin_savepoint(true, 2008072401, 'message', 'email');
     }
 
-    return $result;
+    return true;
 }
 
 
index 33e9648..49d92c5 100644 (file)
@@ -35,20 +35,19 @@ function xmldb_message_jabber_upgrade($oldversion) {
     global $CFG, $DB;
 
     $dbman = $DB->get_manager();
-    $result = true;
 
-    if ($result && $oldversion < 2008072401) {
+    if ($oldversion < 2008072401) {
         $processor = new object();
         $processor->name  = 'jabber';
-        if (! $DB->record_exists('message_processors', array('name' => $processor->name))){
-            $result = $result && $DB->insert_record('message_processors', $processor);
+        if (!$DB->record_exists('message_processors', array('name' => $processor->name))){
+            $DB->insert_record('message_processors', $processor);
         }
 
     /// jabber savepoint reached
-        upgrade_plugin_savepoint($result, 2008072401, 'message', 'jabber');
+        upgrade_plugin_savepoint(true, 2008072401, 'message', 'jabber');
     }
 
-    return $result;
+    return true;
 }
 
 
index 1895c44..34301fe 100644 (file)
@@ -35,20 +35,19 @@ function xmldb_message_popup_upgrade($oldversion) {
     global $CFG, $DB;
 
     $dbman = $DB->get_manager();
-    $result = true;
 
-    if ($result && $oldversion < 2008072401) {
+    if ($oldversion < 2008072401) {
         $processor = new object();
         $processor->name  = 'popup';
         if (! $DB->record_exists('message_processors', array('name' => $processor->name))){
-            $result = $result && $DB->insert_record('message_processors', $processor);
+            $DB->insert_record('message_processors', $processor);
         }
 
     /// popup savepoint reached
-        upgrade_plugin_savepoint($result, 2008072401, 'message', 'popup');
+        upgrade_plugin_savepoint(true, 2008072401, 'message', 'popup');
     }
 
-    return $result;
+    return true;
 }
 
 
index f8f6da1..bbd060d 100644 (file)
@@ -24,18 +24,17 @@ function xmldb_assignment_upgrade($oldversion) {
     global $CFG, $DB, $OUTPUT;
 
     $dbman = $DB->get_manager();
-    $result = true;
 
 //===== 1.9.0 upgrade line ======//
 
-    if ($result && $oldversion < 2007101511) {
+    if ($oldversion < 2007101511) {
         // change grade typo to text if no grades MDL-13920
         require_once $CFG->dirroot.'/mod/assignment/lib.php';
         assignment_upgrade_grades();
-        upgrade_mod_savepoint($result, 2007101511, 'assignment');
+        upgrade_mod_savepoint(true, 2007101511, 'assignment');
     }
 
-    if ($result && $oldversion < 2008081900) {
+    if ($oldversion < 2008081900) {
 
         /////////////////////////////////////
         /// new file storage upgrade code ///
@@ -127,10 +126,10 @@ function xmldb_assignment_upgrade($oldversion) {
 
         }
 
-        upgrade_mod_savepoint($result, 2008081900, 'assignment');
+        upgrade_mod_savepoint(true, 2008081900, 'assignment');
     }
 
-    if ($result && $oldversion < 2009042000) {
+    if ($oldversion < 2009042000) {
 
     /// Rename field description on table assignment to intro
         $table = new xmldb_table('assignment');
@@ -140,10 +139,10 @@ function xmldb_assignment_upgrade($oldversion) {
         $dbman->rename_field($table, $field, 'intro');
 
     /// assignment savepoint reached
-        upgrade_mod_savepoint($result, 2009042000, 'assignment');
+        upgrade_mod_savepoint(true, 2009042000, 'assignment');
     }
 
-    if ($result && $oldversion < 2009042001) {
+    if ($oldversion < 2009042001) {
 
     /// Rename field format on table assignment to introformat
         $table = new xmldb_table('assignment');
@@ -153,10 +152,10 @@ function xmldb_assignment_upgrade($oldversion) {
         $dbman->rename_field($table, $field, 'introformat');
 
     /// assignment savepoint reached
-        upgrade_mod_savepoint($result, 2009042001, 'assignment');
+        upgrade_mod_savepoint(true, 2009042001, 'assignment');
     }
 
-    return $result;
+    return true;
 }
 
 
index b83e78c..9682f7e 100644 (file)
@@ -24,11 +24,10 @@ function xmldb_chat_upgrade($oldversion) {
     global $CFG, $DB;
 
     $dbman = $DB->get_manager();
-    $result = true;
 
     //===== 1.9.0 upgrade line ======//
 
-    if ($result && $oldversion < 2010050100) {
+    if ($oldversion < 2010050100) {
 
     /// Changing precision of field ip on table chat_users to (45)
         $table = new xmldb_table('chat_users');
@@ -38,10 +37,10 @@ function xmldb_chat_upgrade($oldversion) {
         $dbman->change_field_precision($table, $field);
 
     /// chat savepoint reached
-        upgrade_mod_savepoint($result, 2010050100, 'chat');
+        upgrade_mod_savepoint(true, 2010050100, 'chat');
     }
 
-    if ($result && $oldversion < 2010050101) {
+    if ($oldversion < 2010050101) {
 
     /// Define field introformat to be added to chat
         $table = new xmldb_table('chat');
@@ -53,11 +52,11 @@ function xmldb_chat_upgrade($oldversion) {
         }
 
     /// chat savepoint reached
-        upgrade_mod_savepoint($result, 2010050101, 'chat');
+        upgrade_mod_savepoint(true, 2010050101, 'chat');
     }
 
     /// Creating chat_messages_current to hold latest chat messages
-    if ($result && $oldversion < 2010050102) {
+    if ($oldversion < 2010050102) {
 
     /// Define table chat_messages_current to be created
         $table = new xmldb_table('chat_messages_current');
@@ -86,9 +85,9 @@ function xmldb_chat_upgrade($oldversion) {
         }
 
     /// chat savepoint reached
-        upgrade_mod_savepoint($result, 2010050102, 'chat');
+        upgrade_mod_savepoint(true, 2010050102, 'chat');
     }
-    return $result;
+    return true;
 }
 
 
index 0ae71dd..9ca8057 100644 (file)
@@ -24,11 +24,10 @@ function xmldb_choice_upgrade($oldversion) {
     global $CFG, $DB;
 
     $dbman = $DB->get_manager();
-    $result = true;
 
 //===== 1.9.0 upgrade line ======//
 
-    if ($result && $oldversion < 2009042000) {
+    if ($oldversion < 2009042000) {
 
     /// Rename field text on table choice to NEWNAMEGOESHERE
         $table = new xmldb_table('choice');
@@ -38,10 +37,10 @@ function xmldb_choice_upgrade($oldversion) {
         $dbman->rename_field($table, $field, 'intro');
 
     /// choice savepoint reached
-        upgrade_mod_savepoint($result, 2009042000, 'choice');
+        upgrade_mod_savepoint(true, 2009042000, 'choice');
     }
 
-    if ($result && $oldversion < 2009042001) {
+    if ($oldversion < 2009042001) {
 
     /// Rename field format on table choice to NEWNAMEGOESHERE
         $table = new xmldb_table('choice');
@@ -51,10 +50,10 @@ function xmldb_choice_upgrade($oldversion) {
         $dbman->rename_field($table, $field, 'introformat');
 
     /// choice savepoint reached
-        upgrade_mod_savepoint($result, 2009042001, 'choice');
+        upgrade_mod_savepoint(true, 2009042001, 'choice');
     }
 
-    return $result;
+    return true;
 }
 
 
index c2421de..6318b93 100644 (file)
@@ -24,11 +24,10 @@ function xmldb_data_upgrade($oldversion) {
     global $CFG, $DB, $OUTPUT;
 
     $dbman = $DB->get_manager();
-    $result = true;
 
 //===== 1.9.0 upgrade line ======//
 
-    if ($result && $oldversion < 2007101512) {
+    if ($oldversion < 2007101512) {
     /// Launch add field asearchtemplate again if does not exists yet - reported on several sites
 
         $table = new xmldb_table('data');
@@ -37,14 +36,14 @@ function xmldb_data_upgrade($oldversion) {
         if (!$dbman->field_exists($table, $field)) {
             $dbman->add_field($table, $field);
         }
-        upgrade_mod_savepoint($result, 2007101512, 'data');
+        upgrade_mod_savepoint(true, 2007101512, 'data');
     }
 
-    if ($result && $oldversion < 2007101513) {
+    if ($oldversion < 2007101513) {
         // Upgrade all the data->notification currently being
         // NULL to 0
         $sql = "UPDATE {data} SET notification=0 WHERE notification IS NULL";
-        $result = $DB->execute($sql);
+        $DB->execute($sql);
 
         $table = new xmldb_table('data');
         $field = new xmldb_field('notification', XMLDB_TYPE_INTEGER, '10', XMLDB_UNSIGNED, XMLDB_NOTNULL, null, '0', 'editany');
@@ -52,10 +51,10 @@ function xmldb_data_upgrade($oldversion) {
         $dbman->change_field_notnull($table, $field);
         // Second step, Set default to 0
         $dbman->change_field_default($table, $field);
-        upgrade_mod_savepoint($result, 2007101513, 'data');
+        upgrade_mod_savepoint(true, 2007101513, 'data');
     }
 
-    if ($result && $oldversion < 2008081400) {
+    if ($oldversion < 2008081400) {
         if ($rs = $DB->get_recordset('data')) {
             $pattern = '/\#\#delete\#\#(\s+)\#\#approve\#\#/';
             $replacement = '##delete##$1##approve##$1##export##';
@@ -66,10 +65,10 @@ function xmldb_data_upgrade($oldversion) {
             }
             $rs->close();
         }
-        upgrade_mod_savepoint($result, 2008081400, 'data');
+        upgrade_mod_savepoint(true, 2008081400, 'data');
     }
 
-    if ($result && $oldversion < 2008091400) {
+    if ($oldversion < 2008091400) {
 
         /////////////////////////////////////
         /// new file storage upgrade code ///
@@ -143,10 +142,10 @@ function xmldb_data_upgrade($oldversion) {
             }
             $rs->close();
         }
-        upgrade_mod_savepoint($result, 2008091400, 'data');
+        upgrade_mod_savepoint(true, 2008091400, 'data');
     }
 
-    if ($result && $oldversion < 2008112700) {
+    if ($oldversion < 2008112700) {
         if (!get_config('data', 'requiredentriesfixflag')) {
             $databases = $DB->get_records_sql("SELECT d.*, c.fullname
                                                  FROM {data} d, {course} c
@@ -164,10 +163,10 @@ function xmldb_data_upgrade($oldversion) {
             }
         }
         unset_config('requiredentriesfixflag', 'data'); // remove old flag
-        upgrade_mod_savepoint($result, 2008112700, 'data');
+        upgrade_mod_savepoint(true, 2008112700, 'data');
     }
 
-    if ($result && $oldversion < 2009042000) {
+    if ($oldversion < 2009042000) {
 
     /// Define field introformat to be added to data
         $table = new xmldb_table('data');
@@ -177,10 +176,10 @@ function xmldb_data_upgrade($oldversion) {
         $dbman->add_field($table, $field);
 
     /// data savepoint reached
-        upgrade_mod_savepoint($result, 2009042000, 'data');
+        upgrade_mod_savepoint(true, 2009042000, 'data');
     }
 
-    if ($result && $oldversion < 2009111701) {
+    if ($oldversion < 2009111701) {
         upgrade_set_timeout(60*20);
 
     /// Define table data_comments to be dropped
@@ -233,10 +232,10 @@ function xmldb_data_upgrade($oldversion) {
         }
 
     /// data savepoint reached
-        upgrade_mod_savepoint($result, 2009111701, 'data');
+        upgrade_mod_savepoint(true, 2009111701, 'data');
     }
 
-    if ($result && $oldversion < 2010031602) {
+    if ($oldversion < 2010031602) {
         //add assesstimestart and assesstimefinish columns to data
         $table = new xmldb_table('data');
 
@@ -252,10 +251,10 @@ function xmldb_data_upgrade($oldversion) {
             $dbman->add_field($table, $field);
         }
 
-        upgrade_mod_savepoint($result, 2010031602, 'data');
+        upgrade_mod_savepoint(true, 2010031602, 'data');
     }
 
-    if ($result && $oldversion < 2010042800) {
+    if ($oldversion < 2010042800) {
         //migrate data ratings to the central rating table
         require_once($CFG->dirroot . '/lib/db/upgradelib.php');
         $table = new xmldb_table('data_ratings');
@@ -280,10 +279,10 @@ function xmldb_data_upgrade($oldversion) {
             $dbman->drop_table($table);
         }
 
-        upgrade_mod_savepoint($result, 2010042800, 'data');
+        upgrade_mod_savepoint(true, 2010042800, 'data');
     }
 
-    return $result;
+    return true;
 }
 
 
index cc18d19..827997e 100644 (file)
@@ -24,9 +24,8 @@ function xmldb_feedback_upgrade($oldversion) {
     global $CFG, $DB;
 
     $dbman = $DB->get_manager();
-    $result = true;
 
-    if ($result && $oldversion < 2007012310) {
+    if ($oldversion < 2007012310) {
 
         //create a new table feedback_completedtmp and the field-definition
         $table = new xmldb_table('feedback_completedtmp');
@@ -99,10 +98,10 @@ function xmldb_feedback_upgrade($oldversion) {
 
         $dbman->create_table($table);
         ////////////////////////////////////////////////////////////
-        upgrade_mod_savepoint($result, 2007012310, 'feedback');
+        upgrade_mod_savepoint(true, 2007012310, 'feedback');
     }
 
-    if ($result && $oldversion < 2007050504) {
+    if ($oldversion < 2007050504) {
 
         /// Define field random_response to be added to feedback_completed
         $table = new xmldb_table('feedback_completed');
@@ -129,10 +128,10 @@ function xmldb_feedback_upgrade($oldversion) {
         $dbman->add_field($table, $field);
 
         ////////////////////////////////////////////////////////////
-        upgrade_mod_savepoint($result, 2007050504, 'feedback');
+        upgrade_mod_savepoint(true, 2007050504, 'feedback');
     }
 
-    if ($result && $oldversion < 2007102600) {
+    if ($oldversion < 2007102600) {
         // public is a reserved word on Oracle
 
         $table = new xmldb_table('feedback_template');
@@ -140,118 +139,116 @@ function xmldb_feedback_upgrade($oldversion) {
         if (!$dbman->field_exists($table, $field)) {
             $dbman->add_field($table, $field);
         }
-        upgrade_mod_savepoint($result, 2007102600, 'feedback');
+        upgrade_mod_savepoint(true, 2007102600, 'feedback');
     }
 
-    if ($result && $oldversion < 2008042400) { //New version in version.php
+    if ($oldversion < 2008042400) { //New version in version.php
         if ($all_nonanonymous_feedbacks = $DB->get_records('feedback', 'anonymous', 2)) {
             $update_sql = 'UPDATE {feedback_completed} SET anonymous_response = 2 WHERE feedback = ';
             foreach ($all_nonanonymous_feedbacks as $fb) {
-                $result = $result && $DB->execute($update_sql.$fb->id);
+                $DB->execute($update_sql.$fb->id);
             }
         }
-        upgrade_mod_savepoint($result, 2008042400, 'feedback');
+        upgrade_mod_savepoint(true, 2008042400, 'feedback');
     }
 
-    if ($result && $oldversion < 2008042401) { //New version in version.php
-        if ($result) {
-            $concat_radio    = $DB->sql_concat("'r>>>>>'",'presentation');
-            $concat_check    = $DB->sql_concat("'c>>>>>'",'presentation');
-            $concat_dropdown = $DB->sql_concat("'d>>>>>'",'presentation');
+    if ($oldversion < 2008042401) { //New version in version.php
+        $concat_radio    = $DB->sql_concat("'r>>>>>'",'presentation');
+        $concat_check    = $DB->sql_concat("'c>>>>>'",'presentation');
+        $concat_dropdown = $DB->sql_concat("'d>>>>>'",'presentation');
 
-            $update_sql1 = "UPDATE {feedback_item} SET presentation = ".$concat_radio." WHERE typ IN('radio','radiorated')";
-            $update_sql2 = "UPDATE {feedback_item} SET presentation = ".$concat_dropdown." WHERE typ IN('dropdown','dropdownrated')";
-            $update_sql3 = "UPDATE {feedback_item} SET presentation = ".$concat_check." WHERE typ = 'check'";
+        $update_sql1 = "UPDATE {feedback_item} SET presentation = ".$concat_radio." WHERE typ IN('radio','radiorated')";
+        $update_sql2 = "UPDATE {feedback_item} SET presentation = ".$concat_dropdown." WHERE typ IN('dropdown','dropdownrated')";
+        $update_sql3 = "UPDATE {feedback_item} SET presentation = ".$concat_check." WHERE typ = 'check'";
 
-            $result = $result && $DB->execute($update_sql1);
-            $result = $result && $DB->execute($update_sql2);
-            $result = $result && $DB->execute($update_sql3);
-        }
-        if ($result) {
-            $update_sql1 = "UPDATE {feedback_item} SET typ = 'multichoice' WHERE typ IN('radio','check','dropdown')";
-            $update_sql2 = "UPDATE {feedback_item} SET typ = 'multichoicerated' WHERE typ IN('radiorated','dropdownrated')";
-            $result = $result && $DB->execute($update_sql1);
-            $result = $result && $DB->execute($update_sql2);
-        }
-        upgrade_mod_savepoint($result, 2008042401, 'feedback');
+        $DB->execute($update_sql1);
+        $DB->execute($update_sql2);
+        $DB->execute($update_sql3);
+
+        $update_sql1 = "UPDATE {feedback_item} SET typ = 'multichoice' WHERE typ IN('radio','check','dropdown')";
+        $update_sql2 = "UPDATE {feedback_item} SET typ = 'multichoicerated' WHERE typ IN('radiorated','dropdownrated')";
+        $DB->execute($update_sql1);
+        $DB->execute($update_sql2);
+
+        upgrade_mod_savepoint(true, 2008042401, 'feedback');
     }
 
-    if ($result && $oldversion < 2008042801) {
+    if ($oldversion < 2008042801) {
         $new_log_display = new object();
         $new_log_display->module = 'feedback';
         $new_log_display->action = 'startcomplete';
         $new_log_display->mtable = 'feedback';
         $new_log_display->field = 'name';
-        $result = $result && $DB->insert_record('log_display', $new_log_display);
+        $DB->insert_record('log_display', $new_log_display);
 
         $new_log_display = clone($new_log_display);
         $new_log_display->action = 'submit';
-        $result = $result && $DB->insert_record('log_display', $new_log_display);
+        $DB->insert_record('log_display', $new_log_display);
 
         $new_log_display = clone($new_log_display);
         $new_log_display->action = 'delete';
-        $result = $result && $DB->insert_record('log_display', $new_log_display);
+        $DB->insert_record('log_display', $new_log_display);
 
         $new_log_display = clone($new_log_display);
         $new_log_display->action = 'view';
-        $result = $result && $DB->insert_record('log_display', $new_log_display);
+        $DB->insert_record('log_display', $new_log_display);
 
         $new_log_display = clone($new_log_display);
         $new_log_display->action = 'view all';
         $new_log_display->mtable = 'course';
         $new_log_display->field = 'shortname';
-        $result = $result && $DB->insert_record('log_display', $new_log_display);
+        $DB->insert_record('log_display', $new_log_display);
 
-        upgrade_mod_savepoint($result, 2008042801, 'feedback');
+        upgrade_mod_savepoint(true, 2008042801, 'feedback');
     }
 
-    if ($result && $oldversion < 2008042900) {
+    if ($oldversion < 2008042900) {
         /// Define field autonumbering to be added to feedback
         $table = new xmldb_table('feedback');
         $field = new xmldb_field('autonumbering', XMLDB_TYPE_INTEGER, '1', XMLDB_UNSIGNED, XMLDB_NOTNULL, null, '1', 'multiple_submit');
         /// Launch add field2
         $dbman->add_field($table, $field);
 
-        upgrade_mod_savepoint($result, 2008042900, 'feedback');
+        upgrade_mod_savepoint(true, 2008042900, 'feedback');
     }
 
-    if ($result && $oldversion < 2008050104) {
+    if ($oldversion < 2008050104) {
         /// Define field site_after_submit to be added to feedback
         $table = new xmldb_table('feedback');
         $field = new xmldb_field('site_after_submit', XMLDB_TYPE_CHAR, '255', null, null, false, '', 'autonumbering');
         /// Launch add field2
         $dbman->add_field($table, $field);
 
-        upgrade_mod_savepoint($result, 2008050104, 'feedback');
+        upgrade_mod_savepoint(true, 2008050104, 'feedback');
     }
 
-    if ($result && $oldversion < 2008050105) {
+    if ($oldversion < 2008050105) {
         //field count is not more needed
         $table = new xmldb_table('feedback_tracking');
         $field = new xmldb_field('count');
         $dbman->drop_field($table, $field);
 
-        upgrade_mod_savepoint($result, 2008050105, 'feedback');
+        upgrade_mod_savepoint(true, 2008050105, 'feedback');
     }
 
-    if ($result && $oldversion < 2008073002) {
+    if ($oldversion < 2008073002) {
         $update_sql = "UPDATE {feedback_item} SET presentation = '-|-' WHERE " . $DB->sql_compare_text('presentation') . " = '0|0' AND typ = 'numeric'";
-        $result = $result && $DB->execute($update_sql);
+        $DB->execute($update_sql);
 
-        upgrade_mod_savepoint($result, 2008073002, 'feedback');
+        upgrade_mod_savepoint(true, 2008073002, 'feedback');
     }
 
-    if ($result && $oldversion < 2009031301) {
+    if ($oldversion < 2009031301) {
         /// Define field label to be added to feedback_item
         $table = new xmldb_table('feedback_item');
         $field = new xmldb_field('label', XMLDB_TYPE_CHAR, '255', null, null, false, '', 'name');
         /// Launch add field2
         $dbman->add_field($table, $field);
 
-        upgrade_mod_savepoint($result, 2009031301, 'feedback');
+        upgrade_mod_savepoint(true, 2009031301, 'feedback');
     }
 
-    if ($result && $oldversion < 2009042000) {
+    if ($oldversion < 2009042000) {
 
     /// Rename field summary on table feedback to intro
         $table = new xmldb_table('feedback');
@@ -261,10 +258,10 @@ function xmldb_feedback_upgrade($oldversion) {
         $dbman->rename_field($table, $field, 'intro');
 
     /// feedback savepoint reached
-        upgrade_mod_savepoint($result, 2009042000, 'feedback');
+        upgrade_mod_savepoint(true, 2009042000, 'feedback');
     }
 
-    if ($result && $oldversion < 2009042001) {
+    if ($oldversion < 2009042001) {
 
     /// Define field introformat to be added to feedback
         $table = new xmldb_table('feedback');
@@ -274,10 +271,10 @@ function xmldb_feedback_upgrade($oldversion) {
         $dbman->add_field($table, $field);
 
     /// feedback savepoint reached
-        upgrade_mod_savepoint($result, 2009042001, 'feedback');
+        upgrade_mod_savepoint(true, 2009042001, 'feedback');
     }
 
-    if ($result && $oldversion < 2009112000) {
+    if ($oldversion < 2009112000) {
         /// Define field page_after_submitformat to be added to feedback
         $table = new xmldb_table('feedback');
         $field = new xmldb_field('page_after_submitformat', XMLDB_TYPE_INTEGER, '2', XMLDB_UNSIGNED, XMLDB_NOTNULL, null, '0', 'page_after_submit');
@@ -288,40 +285,40 @@ function xmldb_feedback_upgrade($oldversion) {
         }
 
         // feedback savepoint reached
-        upgrade_mod_savepoint($result, 2009112000, 'feedback');
+        upgrade_mod_savepoint(true, 2009112000, 'feedback');
     }
 
-    if ($result && $oldversion < 2010051101) {
+    if ($oldversion < 2010051101) {
         /// Define field options to be added to feedback_item
         $table = new xmldb_table('feedback_item');
         $field = new xmldb_field('options', XMLDB_TYPE_CHAR, '255', null, null, false, '', 'required');
         /// Launch add field
         $dbman->add_field($table, $field);
 
-        upgrade_mod_savepoint($result, 2010051101, 'feedback');
+        upgrade_mod_savepoint(true, 2010051101, 'feedback');
     }
 
-    if ($result && $oldversion < 2010051600) {
+    if ($oldversion < 2010051600) {
         /// Define field options to be added to feedback_item
         $table = new xmldb_table('feedback_item');
         $field = new xmldb_field('dependitem', XMLDB_TYPE_INTEGER, '10', XMLDB_UNSIGNED, XMLDB_NOTNULL, null, '0', 'required');
         /// Launch add field
         $dbman->add_field($table, $field);
 
-        upgrade_mod_savepoint($result, 2010051600, 'feedback');
+        upgrade_mod_savepoint(true, 2010051600, 'feedback');
     }
 
-    if ($result && $oldversion < 2010051601) {
+    if ($oldversion < 2010051601) {
         /// Define field options to be added to feedback_item
         $table = new xmldb_table('feedback_item');
         $field = new xmldb_field('dependvalue', XMLDB_TYPE_CHAR, '255', null, null, false, '', 'dependitem');
         /// Launch add field
         $dbman->add_field($table, $field);
 
-        upgrade_mod_savepoint($result, 2010051601, 'feedback');
+        upgrade_mod_savepoint(true, 2010051601, 'feedback');
     }
 
-    return $result;
+    return true;
 }
 
 
index 27833ec..cf5d4ca 100644 (file)
@@ -47,8 +47,7 @@ function xmldb_folder_upgrade($oldversion) {
     global $CFG, $DB;
 
     $dbman = $DB->get_manager();
-    $result = true;
 
 
-    return $result;
+    return true;
 }
index c1a838c..8f4236d 100644 (file)
@@ -45,18 +45,17 @@ function xmldb_forum_upgrade($oldversion) {
     global $CFG, $DB, $OUTPUT;
 
     $dbman = $DB->get_manager(); // loads ddl manager and xmldb classes
-    $result = true;
 
 //===== 1.9.0 upgrade line ======//
 
-    if ($result && $oldversion < 2007101511) {
+    if ($oldversion < 2007101511) {
         //MDL-13866 - send forum ratins to gradebook again
         require_once($CFG->dirroot.'/mod/forum/lib.php');
         forum_upgrade_grades();
-        upgrade_mod_savepoint($result, 2007101511, 'forum');
+        upgrade_mod_savepoint(true, 2007101511, 'forum');
     }
 
-    if ($result && $oldversion < 2008072800) {
+    if ($oldversion < 2008072800) {
     /// Define field completiondiscussions to be added to forum
         $table = new xmldb_table('forum');
         $field = new xmldb_field('completiondiscussions');
@@ -83,10 +82,10 @@ function xmldb_forum_upgrade($oldversion) {
         if(!$dbman->field_exists($table,$field)) {
             $dbman->add_field($table, $field);
         }
-        upgrade_mod_savepoint($result, 2008072800, 'forum');
+        upgrade_mod_savepoint(true, 2008072800, 'forum');
     }
 
-    if ($result && $oldversion < 2008081900) {
+    if ($oldversion < 2008081900) {
 
         /////////////////////////////////////
         /// new file storage upgrade code ///
@@ -151,10 +150,10 @@ function xmldb_forum_upgrade($oldversion) {
             $rs->close();
         }
 
-        upgrade_mod_savepoint($result, 2008081900, 'forum');
+        upgrade_mod_savepoint(true, 2008081900, 'forum');
     }
 
-    if ($result && $oldversion < 2008090800) {
+    if ($oldversion < 2008090800) {
 
     /// Define field maxattachments to be added to forum
         $table = new xmldb_table('forum');
@@ -166,10 +165,10 @@ function xmldb_forum_upgrade($oldversion) {
         }
 
     /// forum savepoint reached
-        upgrade_mod_savepoint($result, 2008090800, 'forum');
+        upgrade_mod_savepoint(true, 2008090800, 'forum');
     }
 
-    if ($result && $oldversion < 2009042000) {
+    if ($oldversion < 2009042000) {
 
     /// Rename field format on table forum_posts to messageformat
         $table = new xmldb_table('forum_posts');
@@ -179,10 +178,10 @@ function xmldb_forum_upgrade($oldversion) {
         $dbman->rename_field($table, $field, 'messageformat');
 
     /// forum savepoint reached
-        upgrade_mod_savepoint($result, 2009042000, 'forum');
+        upgrade_mod_savepoint(true, 2009042000, 'forum');
     }
 
-    if ($result && $oldversion < 2009042001) {
+    if ($oldversion < 2009042001) {
 
     /// Define field messagetrust to be added to forum_posts
         $table = new xmldb_table('forum_posts');
@@ -192,10 +191,10 @@ function xmldb_forum_upgrade($oldversion) {
         $dbman->add_field($table, $field);
 
     /// forum savepoint reached
-        upgrade_mod_savepoint($result, 2009042001, 'forum');
+        upgrade_mod_savepoint(true, 2009042001, 'forum');
     }
 
-    if ($result && $oldversion < 2009042002) {
+    if ($oldversion < 2009042002) {
         $trustmark = '#####TRUSTTEXT#####';
         $rs = $DB->get_recordset_sql("SELECT * FROM {forum_posts} WHERE message LIKE '$trustmark%'");
         foreach ($rs as $post) {
@@ -210,10 +209,10 @@ function xmldb_forum_upgrade($oldversion) {
         $rs->close();
 
     /// forum savepoint reached
-        upgrade_mod_savepoint($result, 2009042002, 'forum');
+        upgrade_mod_savepoint(true, 2009042002, 'forum');
     }
 
-    if ($result && $oldversion < 2009042003) {
+    if ($oldversion < 2009042003) {
 
     /// Define field introformat to be added to forum
         $table = new xmldb_table('forum');
@@ -223,19 +222,19 @@ function xmldb_forum_upgrade($oldversion) {
         $dbman->add_field($table, $field);
 
     /// forum savepoint reached
-        upgrade_mod_savepoint($result, 2009042003, 'forum');
+        upgrade_mod_savepoint(true, 2009042003, 'forum');
     }
 
-    if ($result && $oldversion < 2009042004) {
+    if ($oldversion < 2009042004) {
     /// set format to current
         $DB->set_field('forum', 'introformat', FORMAT_MOODLE, array());
 
     /// quiz savepoint reached
-        upgrade_mod_savepoint($result, 2009042004, 'forum');
+        upgrade_mod_savepoint(true, 2009042004, 'forum');
     }
 
     /// Dropping all enums/check contraints from core. MDL-18577
-    if ($result && $oldversion < 2009042700) {
+    if ($oldversion < 2009042700) {
 
     /// Changing list of values (enum) of field type on table forum to none
         $table = new xmldb_table('forum');
@@ -245,19 +244,19 @@ function xmldb_forum_upgrade($oldversion) {
         $dbman->drop_enum_from_field($table, $field);
 
     /// forum savepoint reached
-        upgrade_mod_savepoint($result, 2009042700, 'forum');
+        upgrade_mod_savepoint(true, 2009042700, 'forum');
     }
 
-    if ($result && $oldversion < 2009050400) {
+    if ($oldversion < 2009050400) {
 
     /// Clean existing wrong rates. MDL-18227
         $DB->delete_records('forum_ratings', array('post' => 0));
 
     /// forum savepoint reached
-        upgrade_mod_savepoint($result, 2009050400, 'forum');
+        upgrade_mod_savepoint(true, 2009050400, 'forum');
     }
 
-    if ($result && $oldversion < 2010042800) {
+    if ($oldversion < 2010042800) {
         //migrate forumratings to the central rating table
         require_once($CFG->dirroot . '/lib/db/upgradelib.php');
         $table = new xmldb_table('forum_ratings');
@@ -282,10 +281,10 @@ function xmldb_forum_upgrade($oldversion) {
             $dbman->drop_table($table);
         }
 
-        upgrade_mod_savepoint($result, 2010042800, 'forum');
+        upgrade_mod_savepoint(true, 2010042800, 'forum');
     }
 
-    return $result;
+    return true;
 }
 
 
index d155b5a..516b893 100644 (file)
@@ -24,11 +24,10 @@ function xmldb_glossary_upgrade($oldversion) {
     global $CFG, $DB, $OUTPUT;
 
     $dbman = $DB->get_manager();
-    $result = true;
 
 //===== 1.9.0 upgrade line ======//
 
-    if ($result && $oldversion < 2008081900) {
+    if ($oldversion < 2008081900) {
 
         /////////////////////////////////////
         /// new file storage upgrade code ///
@@ -97,10 +96,10 @@ function xmldb_glossary_upgrade($oldversion) {
             $rs->close();
         }
 
-        upgrade_mod_savepoint($result, 2008081900, 'glossary');
+        upgrade_mod_savepoint(true, 2008081900, 'glossary');
     }
 
-    if ($result && $oldversion < 2009042000) {
+    if ($oldversion < 2009042000) {
 
     /// Rename field definitionformat on table glossary_entries to definitionformat
         $table = new xmldb_table('glossary_entries');
@@ -110,10 +109,10 @@ function xmldb_glossary_upgrade($oldversion) {
         $dbman->rename_field($table, $field, 'definitionformat');
 
     /// glossary savepoint reached
-        upgrade_mod_savepoint($result, 2009042000, 'glossary');
+        upgrade_mod_savepoint(true, 2009042000, 'glossary');
     }
 
-    if ($result && $oldversion < 2009042001) {
+    if ($oldversion < 2009042001) {
 
     /// Define field definitiontrust to be added to glossary_entries
         $table = new xmldb_table('glossary_entries');
@@ -123,10 +122,10 @@ function xmldb_glossary_upgrade($oldversion) {
         $dbman->add_field($table, $field);
 
     /// glossary savepoint reached
-        upgrade_mod_savepoint($result, 2009042001, 'glossary');
+        upgrade_mod_savepoint(true, 2009042001, 'glossary');
     }
 
-    if ($result && $oldversion < 2009042002) {
+    if ($oldversion < 2009042002) {
 
     /// Rename field format on table glossary_comments to NEWNAMEGOESHERE
         $table = new xmldb_table('glossary_comments');
@@ -136,10 +135,10 @@ function xmldb_glossary_upgrade($oldversion) {
         $dbman->rename_field($table, $field, 'entrycommentformat');
 
     /// glossary savepoint reached
-        upgrade_mod_savepoint($result, 2009042002, 'glossary');
+        upgrade_mod_savepoint(true, 2009042002, 'glossary');
     }
 
-    if ($result && $oldversion < 2009042003) {
+    if ($oldversion < 2009042003) {
 
     /// Define field entrycommenttrust to be added to glossary_comments
         $table = new xmldb_table('glossary_comments');
@@ -151,10 +150,10 @@ function xmldb_glossary_upgrade($oldversion) {
         }
 
     /// glossary savepoint reached
-        upgrade_mod_savepoint($result, 2009042003, 'glossary');
+        upgrade_mod_savepoint(true, 2009042003, 'glossary');
     }
 
-    if ($result && $oldversion < 2009042004) {
+    if ($oldversion < 2009042004) {
         $trustmark = '#####TRUSTTEXT#####';
         $rs = $DB->get_recordset_sql("SELECT * FROM {glossary_entries} WHERE definition LIKE '$trustmark%'");
         foreach ($rs as $entry) {
@@ -169,10 +168,10 @@ function xmldb_glossary_upgrade($oldversion) {
         $rs->close();
 
     /// glossary savepoint reached
-        upgrade_mod_savepoint($result, 2009042004, 'glossary');
+        upgrade_mod_savepoint(true, 2009042004, 'glossary');
     }
 
-    if ($result && $oldversion < 2009042005) {
+    if ($oldversion < 2009042005) {
         $trustmark = '#####TRUSTTEXT#####';
         $rs = $DB->get_recordset_sql("SELECT * FROM {glossary_comments} WHERE entrycomment LIKE '$trustmark%'");
         foreach ($rs as $comment) {
@@ -187,10 +186,10 @@ function xmldb_glossary_upgrade($oldversion) {
         $rs->close();
 
     /// glossary savepoint reached
-        upgrade_mod_savepoint($result, 2009042005, 'glossary');
+        upgrade_mod_savepoint(true, 2009042005, 'glossary');
     }
 
-    if ($result && $oldversion < 2009042006) {
+    if ($oldversion < 2009042006) {
 
     /// Define field introformat to be added to glossary
         $table = new xmldb_table('glossary');
@@ -202,9 +201,9 @@ function xmldb_glossary_upgrade($oldversion) {
         }
 
     /// glossary savepoint reached
-        upgrade_mod_savepoint($result, 2009042006, 'glossary');
+        upgrade_mod_savepoint(true, 2009042006, 'glossary');
     }
-    if ($result && $oldversion < 2009110800) {
+    if ($oldversion < 2009110800) {
         require_once($CFG->dirroot . '/comment/lib.php');
         upgrade_set_timeout(60*20);
 
@@ -258,10 +257,10 @@ function xmldb_glossary_upgrade($oldversion) {
         }
 
     /// glossary savepoint reached
-        upgrade_mod_savepoint($result, 2009110800, 'glossary');
+        upgrade_mod_savepoint(true, 2009110800, 'glossary');
     }
 
-    if ($result && $oldversion < 2010042800) {
+    if ($oldversion < 2010042800) {
         //migrate glossary_ratings to the central rating table
         require_once($CFG->dirroot . '/lib/db/upgradelib.php');
 
@@ -288,10 +287,10 @@ function xmldb_glossary_upgrade($oldversion) {
             $dbman->drop_table($table);
         }
 
-        upgrade_mod_savepoint($result, 2010042800, 'glossary');
+        upgrade_mod_savepoint(true, 2010042800, 'glossary');
     }
 
-    return $result;
+    return true;
 }
 
 
index 857c60c..defc6d9 100644 (file)
@@ -9,23 +9,22 @@ function xmldb_hotpot_upgrade($oldversion) {
     global $CFG, $DB;
 
     $dbman = $DB->get_manager();
-    $result = true;
 
 //===== 1.9.0 upgrade line ======//
 
     // update hotpot grades from sites earlier than Moodle 1.9, 27th March 2008
-    if ($result && $oldversion < 2007101511) {
+    if ($oldversion < 2007101511) {
         // ensure "hotpot_upgrade_grades" function is available
         require_once $CFG->dirroot.'/mod/hotpot/lib.php';
         hotpot_upgrade_grades();
-        upgrade_mod_savepoint($result, 2007101511, 'hotpot');
+        upgrade_mod_savepoint(true, 2007101511, 'hotpot');
     }
 
-    if ($result && $oldversion < 2008011200) {
+    if ($oldversion < 2008011200) {
         // remove not used setting
         unset_config('hotpot_initialdisable');
-        upgrade_mod_savepoint($result, 2008011200, 'hotpot');
+        upgrade_mod_savepoint(true, 2008011200, 'hotpot');
     }
 
-    return $result;
+    return true;
 }
index e542ca4..2401801 100644 (file)
@@ -27,8 +27,7 @@ function xmldb_imscp_upgrade($oldversion) {
     global $CFG, $DB;
 
     $dbman = $DB->get_manager();
-    $result = true;
 
 
-    return $result;
+    return true;
 }
index 7138045..20507d7 100644 (file)
@@ -24,17 +24,16 @@ function xmldb_label_upgrade($oldversion) {
     global $CFG, $DB;
 
     $dbman = $DB->get_manager();
-    $result = true;
 
 //===== 1.9.0 upgrade line ======//
 
-    if ($result && $oldversion < 2007101510) {
+    if ($oldversion < 2007101510) {
         $sql = "UPDATE {log_display} SET mtable = 'label' WHERE module = 'label'";
-        $result = $DB->execute($sql);
-        upgrade_mod_savepoint($result, 2007101510, 'label');
+        $DB->execute($sql);
+        upgrade_mod_savepoint(true, 2007101510, 'label');
     }
 
-    if ($result && $oldversion < 2009042200) {
+    if ($oldversion < 2009042200) {
 
     /// Rename field content on table label to intro
         $table = new xmldb_table('label');
@@ -44,10 +43,10 @@ function xmldb_label_upgrade($oldversion) {
         $dbman->rename_field($table, $field, 'intro');
 
     /// label savepoint reached
-        upgrade_mod_savepoint($result, 2009042200, 'label');
+        upgrade_mod_savepoint(true, 2009042200, 'label');
     }
 
-    if ($result && $oldversion < 2009042201) {
+    if ($oldversion < 2009042201) {
 
     /// Define field introformat to be added to label
         $table = new xmldb_table('label');
@@ -57,10 +56,10 @@ function xmldb_label_upgrade($oldversion) {
         $dbman->add_field($table, $field);
 
     /// label savepoint reached
-        upgrade_mod_savepoint($result, 2009042201, 'label');
+        upgrade_mod_savepoint(true, 2009042201, 'label');
     }
 
-    if ($result && $oldversion < 2009080400) {
+    if ($oldversion < 2009080400) {
 
     /// Define field introformat to be added to label
         $table = new xmldb_table('label');
@@ -73,10 +72,10 @@ function xmldb_label_upgrade($oldversion) {
         $DB->set_field('label', 'introformat', 0, array('introformat' => 4));
 
     /// label savepoint reached
-        upgrade_mod_savepoint($result, 2009080400, 'label');
+        upgrade_mod_savepoint(true, 2009080400, 'label');
     }
 
-    return $result;
+    return true;
 }
 
 
index 63a0235..3beeb74 100644 (file)
@@ -45,11 +45,10 @@ function xmldb_lesson_upgrade($oldversion) {
     global $CFG, $DB, $OUTPUT;
 
     $dbman = $DB->get_manager();
-    $result = true;
 
 //===== 1.9.0 upgrade line ======//
 
-    if ($result && $oldversion < 2007072201) {
+    if ($oldversion < 2007072201) {
 
         $table = new xmldb_table('lesson');
         $field = new xmldb_field('usegrademax');
@@ -63,18 +62,18 @@ function xmldb_lesson_upgrade($oldversion) {
             $dbman->rename_field($table, $field, 'usemaxgrade');
         }
 
-        upgrade_mod_savepoint($result, 2007072201, 'lesson');
+        upgrade_mod_savepoint(true, 2007072201, 'lesson');
     }
 
-    if ($result && $oldversion < 2008112601) {
+    if ($oldversion < 2008112601) {
         require_once($CFG->dirroot.'/mod/lesson/lib.php');
 
         lesson_upgrade_grades();
 
-        upgrade_mod_savepoint($result, 2008112601, 'lesson');
+        upgrade_mod_savepoint(true, 2008112601, 'lesson');
     }
 
-    if ($result && $oldversion < 2009111600) {
+    if ($oldversion < 2009111600) {
         /**
          * Change the grade field within lesson_answers to an unsigned int and increment
          * the length by one to ensure that no values are changed (reduced)
@@ -83,10 +82,10 @@ function xmldb_lesson_upgrade($oldversion) {
         $field = new xmldb_field('grade');
         $field->set_attributes(XMLDB_TYPE_INTEGER, '4', false, XMLDB_NOTNULL, null, '0', 'jumpto');
         $dbman->change_field_type($table, $field);
-        upgrade_mod_savepoint($result, 2009111600, 'lesson');
+        upgrade_mod_savepoint(true, 2009111600, 'lesson');
     }
 
-    if ($result && $oldversion < 2009120400) {
+    if ($oldversion < 2009120400) {
 
         /**
          * Move any media files associated with the lesson to use the new file
@@ -144,19 +143,19 @@ function xmldb_lesson_upgrade($oldversion) {
             }
         }
 
-        upgrade_mod_savepoint($result, 2009120400, 'lesson');
+        upgrade_mod_savepoint(true, 2009120400, 'lesson');
     }
 
-    if ($result && $oldversion < 2009120800) {
+    if ($oldversion < 2009120800) {
         /**
          * Drop the lesson_default table, as of Moodle 2.0 it is no longer used
          * the module now has a settings.php instead
          */
         $table = new xmldb_table('lesson_default');
         $dbman->drop_table($table);
-        upgrade_mod_savepoint($result, 2009120800, 'lesson');
+        upgrade_mod_savepoint(true, 2009120800, 'lesson');
     }
-    if ($result && $oldversion < 2009120801) {
+    if ($oldversion < 2009120801) {
 
     /// Define field contentsformat to be added to lesson_pages
         $table = new xmldb_table('lesson_pages');
@@ -168,11 +167,11 @@ function xmldb_lesson_upgrade($oldversion) {
         }
 
     /// lesson savepoint reached
-        upgrade_mod_savepoint($result, 2009120801, 'lesson');
+        upgrade_mod_savepoint(true, 2009120801, 'lesson');
     }
 
 
-    return $result;
+    return true;
 }
 
 
index 2235e05..02a381d 100644 (file)
@@ -48,8 +48,8 @@ function xmldb_page_upgrade($oldversion) {
     require_once("$CFG->dirroot/mod/page/db/upgradelib.php");
 
     $dbman = $DB->get_manager();
-    $result = true;
 
 
-    return $result;
+
+    return true;
 }
index e99b9e9..5dd3e97 100644 (file)
@@ -24,11 +24,10 @@ function xmldb_quiz_upgrade($oldversion) {
     global $CFG, $DB;
 
     $dbman = $DB->get_manager();
-    $result = true;
 
 //===== 1.9.0 upgrade line ======//
 
-    if ($result && $oldversion < 2008062000) {
+    if ($oldversion < 2008062000) {
 
     /// Define table quiz_report to be created
         $table = new xmldb_table('quiz_report');
@@ -46,34 +45,34 @@ function xmldb_quiz_upgrade($oldversion) {
             $dbman->create_table($table);
         }
 
-        upgrade_mod_savepoint($result, 2008062000, 'quiz');
+        upgrade_mod_savepoint(true, 2008062000, 'quiz');
     }
 
-    if ($result && $oldversion < 2008062001) {
+    if ($oldversion < 2008062001) {
         $reporttoinsert = new object();
         $reporttoinsert->name = 'overview';
         $reporttoinsert->displayorder = 10000;
-        $result = $result && $DB->insert_record('quiz_report', $reporttoinsert);
+        $DB->insert_record('quiz_report', $reporttoinsert);
 
         $reporttoinsert = new object();
         $reporttoinsert->name = 'responses';
         $reporttoinsert->displayorder = 9000;
-        $result = $result && $DB->insert_record('quiz_report', $reporttoinsert);
+        $DB->insert_record('quiz_report', $reporttoinsert);
 
         $reporttoinsert = new object();
         $reporttoinsert->name = 'regrade';
         $reporttoinsert->displayorder = 7000;
-        $result = $result && $DB->insert_record('quiz_report', $reporttoinsert);
+        $DB->insert_record('quiz_report', $reporttoinsert);
 
         $reporttoinsert = new object();
         $reporttoinsert->name = 'grading';
         $reporttoinsert->displayorder = 6000;
-        $result = $result && $DB->insert_record('quiz_report', $reporttoinsert);
+        $DB->insert_record('quiz_report', $reporttoinsert);
 
-        upgrade_mod_savepoint($result, 2008062001, 'quiz');
+        upgrade_mod_savepoint(true, 2008062001, 'quiz');
     }
 
-    if ($result && $oldversion < 2008072402) {
+    if ($oldversion < 2008072402) {
 
     /// Define field lastcron to be added to quiz_report
         $table = new xmldb_table('quiz_report');
@@ -93,18 +92,18 @@ function xmldb_quiz_upgrade($oldversion) {
         }
 
     /// quiz savepoint reached
-        upgrade_mod_savepoint($result, 2008072402, 'quiz');
+        upgrade_mod_savepoint(true, 2008072402, 'quiz');
     }
 
-    if ($result && $oldversion < 2008072900) {
+    if ($oldversion < 2008072900) {
     /// Delete the regrade report - it is now part of the overview report.
-        $result = $result && $DB->delete_records('quiz_report', array('name' => 'regrade'));
+        $DB->delete_records('quiz_report', array('name' => 'regrade'));
 
     /// quiz savepoint reached
-        upgrade_mod_savepoint($result, 2008072900, 'quiz');
+        upgrade_mod_savepoint(true, 2008072900, 'quiz');
     }
 
-    if ($result && $oldversion < 2008081500) {
+    if ($oldversion < 2008081500) {
     /// Define table quiz_question_versions to be dropped
         $table = new xmldb_table('quiz_question_versions');
 
@@ -112,61 +111,61 @@ function xmldb_quiz_upgrade($oldversion) {
         $dbman->drop_table($table);
 
     /// quiz savepoint reached
-        upgrade_mod_savepoint($result, 2008081500, 'quiz');
+        upgrade_mod_savepoint(true, 2008081500, 'quiz');
     }
 
     /// Changing the type of all the columns that store grades to be NUMBER(10, 5) or similar.
-    if ($result && $oldversion < 2008081501) {
+    if ($oldversion < 2008081501) {
         $table = new xmldb_table('quiz');
         $field = new xmldb_field('sumgrades', XMLDB_TYPE_NUMBER, '10, 5', null, null, null, null, 'questions');
         $dbman->change_field_type($table, $field);
-        upgrade_mod_savepoint($result, 2008081501, 'quiz');
+        upgrade_mod_savepoint(true, 2008081501, 'quiz');
     }
 
-    if ($result && $oldversion < 2008081502) {
+    if ($oldversion < 2008081502) {
         $table = new xmldb_table('quiz');
         $field = new xmldb_field('grade', XMLDB_TYPE_NUMBER, '10, 5', null, null, null, null, 'sumgrades');
         $dbman->change_field_type($table, $field);
-        upgrade_mod_savepoint($result, 2008081502, 'quiz');
+        upgrade_mod_savepoint(true, 2008081502, 'quiz');
     }
 
-    if ($result && $oldversion < 2008081503) {
+    if ($oldversion < 2008081503) {
         $table = new xmldb_table('quiz_attempts');
         $field = new xmldb_field('sumgrades', XMLDB_TYPE_NUMBER, '10, 5', null, null, null, null, 'attempt');
         $dbman->change_field_type($table, $field);
-        upgrade_mod_savepoint($result, 2008081503, 'quiz');
+        upgrade_mod_savepoint(true, 2008081503, 'quiz');
     }
 
-    if ($result && $oldversion < 2008081504) {
+    if ($oldversion < 2008081504) {
         $table = new xmldb_table('quiz_feedback');
         $field = new xmldb_field('mingrade', XMLDB_TYPE_NUMBER, '10, 5', null, null, null, null, 'feedbacktext');
         $dbman->change_field_type($table, $field);
-        upgrade_mod_savepoint($result, 2008081504, 'quiz');
+        upgrade_mod_savepoint(true, 2008081504, 'quiz');
     }
 
-    if ($result && $oldversion < 2008081505) {
+    if ($oldversion < 2008081505) {
         $table = new xmldb_table('quiz_feedback');
         $field = new xmldb_field('maxgrade', XMLDB_TYPE_NUMBER, '10, 5', null, null, null, null, 'mingrade');
         $dbman->change_field_type($table, $field);
-        upgrade_mod_savepoint($result, 2008081505, 'quiz');
+        upgrade_mod_savepoint(true, 2008081505, 'quiz');
     }
 
-    if ($result && $oldversion < 2008081506) {
+    if ($oldversion < 2008081506) {
         $table = new xmldb_table('quiz_grades');
         $field = new xmldb_field('grade', XMLDB_TYPE_NUMBER, '10, 5', null, null, null, null, 'userid');
         $dbman->change_field_type($table, $field);
-        upgrade_mod_savepoint($result, 2008081506, 'quiz');
+        upgrade_mod_savepoint(true, 2008081506, 'quiz');
     }
 
-    if ($result && $oldversion < 2008081507) {
+    if ($oldversion < 2008081507) {
         $table = new xmldb_table('quiz_question_instances');
         $field = new xmldb_field('grade', XMLDB_TYPE_NUMBER, '12, 7', null, null, null, null, 'question');
         $dbman->change_field_type($table, $field);
-        upgrade_mod_savepoint($result, 2008081507, 'quiz');
+        upgrade_mod_savepoint(true, 2008081507, 'quiz');
     }
 
     /// Move all of the quiz config settings from $CFG to the config_plugins table.
-    if ($result && $oldversion < 2008082200) {
+    if ($oldversion < 2008082200) {
         foreach (get_object_vars($CFG) as $name => $value) {
             if (strpos($name, 'quiz_') === 0) {
                 $shortname = substr($name, 5);
@@ -174,11 +173,11 @@ function xmldb_quiz_upgrade($oldversion) {
                     // Special case - remove old inconsistency.
                     $shortname == 'fix_optionflags';
                 }
-                $result = $result && set_config($shortname, $value, 'quiz');
-                $result = $result && unset_config($name);
+                set_config($shortname, $value, 'quiz');
+                unset_config($name);
             }
         }
-        upgrade_mod_savepoint($result, 2008082200, 'quiz');
+        upgrade_mod_savepoint(true, 2008082200, 'quiz');
     }
 
     /// Now that the quiz is no longer responsible for creating all the question
@@ -186,13 +185,13 @@ function xmldb_quiz_upgrade($oldversion) {
     /// datasetdependent question type, which did not have a version.php file before,
     /// we need to say that these tables are already installed, otherwise XMLDB
     /// will try to create them again and give an error.
-    if ($result && $oldversion < 2008082600) {
+    if ($oldversion < 2008082600) {
         // Since MDL-16505 was fixed, and we eliminated the datasetdependent
         // question type, this is now a no-op.
-        upgrade_mod_savepoint($result, 2008082600, 'quiz');
+        upgrade_mod_savepoint(true, 2008082600, 'quiz');
     }
 
-    if ($result && $oldversion < 2008112101) {
+    if ($oldversion < 2008112101) {
 
     /// Define field lastcron to be added to quiz_report
         $table = new xmldb_table('quiz_report');
@@ -204,10 +203,10 @@ function xmldb_quiz_upgrade($oldversion) {
         }
 
     /// quiz savepoint reached
-        upgrade_mod_savepoint($result, 2008112101, 'quiz');
+        upgrade_mod_savepoint(true, 2008112101, 'quiz');
     }
 
-    if ($result && $oldversion < 2009010700) {
+    if ($oldversion < 2009010700) {
 
     /// Define field showuserpicture to be added to quiz
         $table = new xmldb_table('quiz');
@@ -219,10 +218,10 @@ function xmldb_quiz_upgrade($oldversion) {
         }
 
     /// quiz savepoint reached
-        upgrade_mod_savepoint($result, 2009010700, 'quiz');
+        upgrade_mod_savepoint(true, 2009010700, 'quiz');
     }
 
-    if ($result && $oldversion < 2009030900) {
+    if ($oldversion < 2009030900) {
     /// If there are no quiz settings set to advanced yet, the set up the default
     /// advanced fields from Moodle 2.0.
         $quizconfig = get_config('quiz');
@@ -246,10 +245,10 @@ function xmldb_quiz_upgrade($oldversion) {
         }
 
     /// quiz savepoint reached
-        upgrade_mod_savepoint($result, 2009030900, 'quiz');
+        upgrade_mod_savepoint(true, 2009030900, 'quiz');
     }
 
-    if ($result && $oldversion < 2009031000) {
+    if ($oldversion < 2009031000) {
     /// Add new questiondecimaldigits setting, separate form the overall decimaldigits one.
         $table = new xmldb_table('quiz');
         $field = new xmldb_field('questiondecimalpoints', XMLDB_TYPE_INTEGER, '4', null, XMLDB_NOTNULL, null, '2', 'decimalpoints');
@@ -258,20 +257,20 @@ function xmldb_quiz_upgrade($oldversion) {
         }
 
     /// quiz savepoint reached
-        upgrade_mod_savepoint($result, 2009031000, 'quiz');
+        upgrade_mod_savepoint(true, 2009031000, 'quiz');
     }
 
-    if ($result && $oldversion < 2009031001) {
+    if ($oldversion < 2009031001) {
     /// Convert quiz.timelimit from minutes to seconds.
         $DB->execute('UPDATE {quiz} SET timelimit = timelimit * 60');
         $default = get_config('quiz', 'timelimit');
         set_config('timelimit', 60 * $default, 'quiz');
 
     /// quiz savepoint reached
-        upgrade_mod_savepoint($result, 2009031001, 'quiz');
+        upgrade_mod_savepoint(true, 2009031001, 'quiz');
     }
 
-    if ($result && $oldversion < 2009042000) {
+    if ($oldversion < 2009042000) {
 
     /// Define field introformat to be added to quiz
         $table = new xmldb_table('quiz');
@@ -284,10 +283,10 @@ function xmldb_quiz_upgrade($oldversion) {
         $DB->set_field('quiz', 'introformat', FORMAT_MOODLE, array());
 
     /// quiz savepoint reached
-        upgrade_mod_savepoint($result, 2009042000, 'quiz');
+        upgrade_mod_savepoint(true, 2009042000, 'quiz');
     }
 
-    if ($result && $oldversion < 2010030501) {
+    if ($oldversion < 2010030501) {
     /// fix log actions
         update_log_display_entry('quiz', 'edit override', 'quiz', 'name');
         update_log_display_entry('quiz', 'delete override', 'quiz', 'name');
@@ -318,10 +317,10 @@ function xmldb_quiz_upgrade($oldversion) {
         }
 
     /// quiz savepoint reached
-        upgrade_mod_savepoint($result, 2010030501, 'quiz');
+        upgrade_mod_savepoint(true, 2010030501, 'quiz');
     }
 
-    if ($result && $oldversion < 2010051800) {
+    if ($oldversion < 2010051800) {
 
     /// Define field showblocks to be added to quiz
         $table = new xmldb_table('quiz');
@@ -333,9 +332,9 @@ function xmldb_quiz_upgrade($oldversion) {
         }
 
     /// quiz savepoint reached
-        upgrade_mod_savepoint($result, 2010051800, 'quiz');
+        upgrade_mod_savepoint(true, 2010051800, 'quiz');
     }
 
-    return $result;
+    return true;
 }
 
index 45c6c8b..9b7bd6c 100644 (file)
@@ -4,11 +4,10 @@ function xmldb_quiz_overview_upgrade($oldversion) {
     global $CFG, $DB;
 
     $dbman = $DB->get_manager();
-    $result = true;
 
 //===== 1.9.0 upgrade line ======//
 
-    if ($result && $oldversion < 2009091400) {
+    if ($oldversion < 2009091400) {
 
     /// Define table quiz_question_regrade to be created
         $table = new xmldb_table('quiz_question_regrade');
@@ -31,10 +30,10 @@ function xmldb_quiz_overview_upgrade($oldversion) {
         }
 
     /// overview savepoint reached
-        upgrade_plugin_savepoint($result, 2009091400, 'quizreport', 'overview');
+        upgrade_plugin_savepoint(true, 2009091400, 'quizreport', 'overview');
     }
 
-    return $result;
+    return true;
 }
 
 
index 80030d9..0d6795d 100644 (file)
@@ -6,19 +6,17 @@ function xmldb_quiz_statistics_upgrade($oldversion) {
 
     $dbman = $DB->get_manager();
 
-    $result = true;
-
 //===== 1.9.0 upgrade line ======//
 
-    if ($result && $oldversion < 2008072401) {
+    if ($oldversion < 2008072401) {
         //register cron to run every 5 hours.
-        $result = $result && $DB->set_field('quiz_report', 'cron', HOURSECS*5, array('name'=>'statistics'));
+        $DB->set_field('quiz_report', 'cron', HOURSECS*5, array('name'=>'statistics'));
 
     /// statistics savepoint reached
-        upgrade_plugin_savepoint($result, 2008072401, 'quizreport', 'statistics');
+        upgrade_plugin_savepoint(true, 2008072401, 'quizreport', 'statistics');
     }
 
-    if ($result && $oldversion < 2008072500) {
+    if ($oldversion < 2008072500) {
 
     /// Define field s to be added to quiz_question_statistics
         $table = new xmldb_table('quiz_question_statistics');
@@ -30,10 +28,10 @@ function xmldb_quiz_statistics_upgrade($oldversion) {
         }
 
     /// statistics savepoint reached
-        upgrade_plugin_savepoint($result, 2008072500, 'quizreport', 'statistics');
+        upgrade_plugin_savepoint(true, 2008072500, 'quizreport', 'statistics');
     }
 
-    if ($result && $oldversion < 2008072800) {
+    if ($oldversion < 2008072800) {
 
     /// Define field maxgrade to be added to quiz_question_statistics
         $table = new xmldb_table('quiz_question_statistics');
@@ -45,10 +43,10 @@ function xmldb_quiz_statistics_upgrade($oldversion) {
         }
 
     /// statistics savepoint reached
-        upgrade_plugin_savepoint($result, 2008072800, 'quizreport', 'statistics');
+        upgrade_plugin_savepoint(true, 2008072800, 'quizreport', 'statistics');
     }
 
-    if ($result && $oldversion < 2008072801) {
+    if ($oldversion < 2008072801) {
 
     /// Define field positions to be added to quiz_question_statistics
         $table = new xmldb_table('quiz_question_statistics');
@@ -60,10 +58,10 @@ function xmldb_quiz_statistics_upgrade($oldversion) {
         }
 
     /// statistics savepoint reached
-        upgrade_plugin_savepoint($result, 2008072801, 'quizreport', 'statistics');
+        upgrade_plugin_savepoint(true, 2008072801, 'quizreport', 'statistics');
     }
 
-    if ($result && $oldversion < 2008081500) {
+    if ($oldversion < 2008081500) {
     /// Changing type of field maxgrade on table quiz_question_statistics to number
         $table = new xmldb_table('quiz_question_statistics');
         $field = new xmldb_field('maxgrade', XMLDB_TYPE_NUMBER, '12, 7', null, null, null, null, 'subquestions');
@@ -72,10 +70,10 @@ function xmldb_quiz_statistics_upgrade($oldversion) {
         $dbman->change_field_type($table, $field);
 
     /// statistics savepoint reached
-        upgrade_plugin_savepoint($result, 2008081500, 'quizreport', 'statistics');
+        upgrade_plugin_savepoint(true, 2008081500, 'quizreport', 'statistics');
     }
 
-    if ($result && $oldversion < 2008082600) {
+    if ($oldversion < 2008082600) {
 
     /// Define table quiz_question_response_stats to be created
         $table = new xmldb_table('quiz_question_response_stats');
@@ -98,73 +96,70 @@ function xmldb_quiz_statistics_upgrade($oldversion) {
         }
 
     /// statistics savepoint reached
-        upgrade_plugin_savepoint($result, 2008082600, 'quizreport', 'statistics');
+        upgrade_plugin_savepoint(true, 2008082600, 'quizreport', 'statistics');
     }
 
-    if ($result && $oldversion < 2008090500) {
+    if ($oldversion < 2008090500) {
         //delete all cached results first
-        $result = $result && $DB->delete_records('quiz_statistics');
-        $result = $result && $DB->delete_records('quiz_question_statistics');
-        $result = $result && $DB->delete_records('quiz_question_response_stats');
-        if ($result){
+        $DB->delete_records('quiz_statistics');
+        $DB->delete_records('quiz_question_statistics');
+        $DB->delete_records('quiz_question_response_stats');
         /// Define field anssubqid to be dropped from quiz_question_response_stats
-            $table = new xmldb_table('quiz_question_response_stats');
-            $field = new xmldb_field('anssubqid');
+        $table = new xmldb_table('quiz_question_response_stats');
+        $field = new xmldb_field('anssubqid');
 
         /// Conditionally launch drop field subqid
-            if ($dbman->field_exists($table, $field)) {
-                $dbman->drop_field($table, $field);
-            }
+        if ($dbman->field_exists($table, $field)) {
+            $dbman->drop_field($table, $field);
+        }
 
         /// Define field subqid to be added to quiz_question_response_stats
-            $field = new xmldb_field('subqid', XMLDB_TYPE_INTEGER, '10', XMLDB_UNSIGNED, XMLDB_NOTNULL, null, null, 'questionid');
+        $field = new xmldb_field('subqid', XMLDB_TYPE_INTEGER, '10', XMLDB_UNSIGNED, XMLDB_NOTNULL, null, null, 'questionid');
 
         /// Conditionally launch add field subqid
-            if (!$dbman->field_exists($table, $field)) {
-                $dbman->add_field($table, $field);
-            }
+        if (!$dbman->field_exists($table, $field)) {
+            $dbman->add_field($table, $field);
+        }
 
         /// Define field aid to be added to quiz_question_response_stats
-            $field = new xmldb_field('aid', XMLDB_TYPE_INTEGER, '10', XMLDB_UNSIGNED, XMLDB_NOTNULL, null, null, 'subqid');
+        $field = new xmldb_field('aid', XMLDB_TYPE_INTEGER, '10', XMLDB_UNSIGNED, XMLDB_NOTNULL, null, null, 'subqid');
 
         /// Conditionally launch add field aid
-            if (!$dbman->field_exists($table, $field)) {
-                $dbman->add_field($table, $field);
-            }
+        if (!$dbman->field_exists($table, $field)) {
+            $dbman->add_field($table, $field);
         }
 
     /// statistics savepoint reached
-        upgrade_plugin_savepoint($result, 2008090500, 'quizreport', 'statistics');
+        upgrade_plugin_savepoint(true, 2008090500, 'quizreport', 'statistics');
     }
 
-    if ($result && $oldversion < 2008111000) {
+    if ($oldversion < 2008111000) {
         //delete all cached results first
-        $result = $result && $DB->delete_records('quiz_statistics');
-        $result = $result && $DB->delete_records('quiz_question_statistics');
-        $result = $result && $DB->delete_records('quiz_question_response_stats');
-        if ($result){
+        $DB->delete_records('quiz_statistics');
+        $DB->delete_records('quiz_question_statistics');
+        $DB->delete_records('quiz_question_response_stats');
+
         /// Define field anssubqid to be dropped from quiz_question_response_stats
-            $table = new xmldb_table('quiz_question_statistics');
+        $table = new xmldb_table('quiz_question_statistics');
         /// Define field subqid to be added to quiz_question_response_stats
-            $field = new xmldb_field('negcovar', XMLDB_TYPE_INTEGER, '2', XMLDB_UNSIGNED, XMLDB_NOTNULL, null, '0', 'effectiveweight');
+        $field = new xmldb_field('negcovar', XMLDB_TYPE_INTEGER, '2', XMLDB_UNSIGNED, XMLDB_NOTNULL, null, '0', 'effectiveweight');
 
         /// Conditionally launch add field subqid
-            if (!$dbman->field_exists($table, $field)) {
-                $dbman->add_field($table, $field);
-            }
+        if (!$dbman->field_exists($table, $field)) {
+            $dbman->add_field($table, $field);
         }
 
     /// statistics savepoint reached
-        upgrade_plugin_savepoint($result, 2008111000, 'quizreport', 'statistics');
+        upgrade_plugin_savepoint(true, 2008111000, 'quizreport', 'statistics');
     }
 
-    if ($result && $oldversion < 2008112100) {
-        $result = $result && $DB->set_field('quiz_report', 'capability', 'quizreport/statistics:view', array('name'=>'statistics'));
+    if ($oldversion < 2008112100) {
+        $DB->set_field('quiz_report', 'capability', 'quizreport/statistics:view', array('name'=>'statistics'));
 
     /// statistics savepoint reached
-        upgrade_plugin_savepoint($result, 2008112100, 'quizreport', 'statistics');
+        upgrade_plugin_savepoint(true, 2008112100, 'quizreport', 'statistics');
     }
 
-    return $result;
+    return true;
 }
 
index fde3388..522fe84 100644 (file)
@@ -49,11 +49,10 @@ function xmldb_resource_upgrade($oldversion) {
     require_once("$CFG->dirroot/mod/resource/db/upgradelib.php");
 
     $dbman = $DB->get_manager();
-    $result = true;
 
 //===== 1.9.0 upgrade line ======//
 
-    if ($result && $oldversion < 2009042000) {
+    if ($oldversion < 2009042000) {
        // Rename field summary on table resource to intro
         $table = new xmldb_table('resource');
         $field = new xmldb_field('summary', XMLDB_TYPE_TEXT, 'small', null, null, null, null, 'reference');
@@ -62,10 +61,10 @@ function xmldb_resource_upgrade($oldversion) {
         $dbman->rename_field($table, $field, 'intro');
 
         // resource savepoint reached
-        upgrade_mod_savepoint($result, 2009042000, 'resource');
+        upgrade_mod_savepoint(true, 2009042000, 'resource');
     }
 
-    if ($result && $oldversion < 2009042001) {
+    if ($oldversion < 2009042001) {
         // Define field introformat to be added to resource
         $table = new xmldb_table('resource');
         $field = new xmldb_field('introformat', XMLDB_TYPE_INTEGER, '4', XMLDB_UNSIGNED, XMLDB_NOTNULL, null, '0', 'intro');
@@ -77,23 +76,23 @@ function xmldb_resource_upgrade($oldversion) {
         $DB->set_field('resource', 'introformat', FORMAT_MOODLE, array());
 
         // resource savepoint reached
-        upgrade_mod_savepoint($result, 2009042001, 'resource');
+        upgrade_mod_savepoint(true, 2009042001, 'resource');
     }
 
-    if ($result && $oldversion < 2009062500) {
+    if ($oldversion < 2009062500) {
         // fix log actions
         update_log_display_entry('resource', 'view all', 'resource', 'name');
         // resource savepoint reached
-        upgrade_mod_savepoint($result, 2009062500, 'resource');
+        upgrade_mod_savepoint(true, 2009062500, 'resource');
     }
 
-    if ($result && $oldversion < 2009062501) {
+    if ($oldversion < 2009062501) {
         resource_20_prepare_migration();
         // resource savepoint reached
-        upgrade_mod_savepoint($result, 2009062501, 'resource');
+        upgrade_mod_savepoint(true, 2009062501, 'resource');
     }
 
-    if ($result && $oldversion < 2009062600) {
+    if ($oldversion < 2009062600) {
         $res_count = $DB->count_records('resource');
         $old_count = $DB->count_records('resource_old', array('migrated'=>0));
         if ($res_count != $old_count) {
@@ -111,10 +110,10 @@ function xmldb_resource_upgrade($oldversion) {
         }
 
         // resource savepoint reached
-        upgrade_mod_savepoint($result, 2009062600, 'resource');
+        upgrade_mod_savepoint(true, 2009062600, 'resource');
     }
 
-    if ($result && $oldversion < 2009062601) {
+    if ($oldversion < 2009062601) {
         $table = new xmldb_table('resource');
         // Define field tobemigrated to be added to resource
         $field = new xmldb_field('tobemigrated', XMLDB_TYPE_INTEGER, '4', XMLDB_UNSIGNED, XMLDB_NOTNULL, null, '0', 'introformat');
@@ -169,15 +168,15 @@ function xmldb_resource_upgrade($oldversion) {
         $DB->set_field('resource', 'tobemigrated', 1, array());
 
         // resource savepoint reached
-        upgrade_mod_savepoint($result, 2009062601, 'resource');
+        upgrade_mod_savepoint(true, 2009062601, 'resource');
     }
 
-    if ($result && $oldversion < 2009062603) {
+    if ($oldversion < 2009062603) {
         resource_20_migrate();
-        upgrade_mod_savepoint($result, 2009062603, 'resource');
+        upgrade_mod_savepoint(true, 2009062603, 'resource');
     }
 
-    if ($result && $oldversion < 2009063000) {
+    if ($oldversion < 2009063000) {
         //migrate and prune old settings - admins need to review and set up all module settings anyway
         if (!empty($CFG->resource_framesize)) {
             set_config('framesize', $CFG->resource_framesize, 'resource');
@@ -208,10 +207,10 @@ function xmldb_resource_upgrade($oldversion) {
             unset_config($setting);
         }
 
-        upgrade_mod_savepoint($result, 2009063000, 'resource');
+        upgrade_mod_savepoint(true, 2009063000, 'resource');
     }
 
-    if ($result && $oldversion < 2009080501) {
+    if ($oldversion < 2009080501) {
         require_once("$CFG->libdir/filelib.php");
 
         $sql = "SELECT r.id,
@@ -243,9 +242,9 @@ function xmldb_resource_upgrade($oldversion) {
         }
 
     /// resource savepoint reached
-        upgrade_mod_savepoint($result, 2009080501, 'resource');
+        upgrade_mod_savepoint(true, 2009080501, 'resource');
     }
 
 
-    return $result;
+    return true;
 }
index 0d6238c..6ef0381 100644 (file)
@@ -24,12 +24,11 @@ function xmldb_scorm_upgrade($oldversion) {
     global $CFG, $DB;
 
     $dbman = $DB->get_manager();
-    $result = true;
 
 //===== 1.9.0 upgrade line ======//
 
     // Adding missing 'whatgrade' field to table scorm
-    if ($result && $oldversion < 2008073000) {
+    if ($oldversion < 2008073000) {
         $table = new xmldb_table('scorm');
         $field = new xmldb_field('whatgrade');
         $field->set_attributes(XMLDB_TYPE_INTEGER, '10', XMLDB_UNSIGNED, XMLDB_NOTNULL, null, '0', 'grademethod');
@@ -39,10 +38,10 @@ function xmldb_scorm_upgrade($oldversion) {
             $dbman->add_field($table, $field);
         }
 
-        upgrade_mod_savepoint($result, 2008073000, 'scorm');
+        upgrade_mod_savepoint(true, 2008073000, 'scorm');
     }
 
-     if ($result && $oldversion < 2008082500) {
+     if ($oldversion < 2008082500) {
 
     /// Define field scormtype to be added to scorm
         $table = new xmldb_table('scorm');
@@ -52,10 +51,10 @@ function xmldb_scorm_upgrade($oldversion) {
         $dbman->add_field($table, $field);
 
     /// scorm savepoint reached
-        upgrade_mod_savepoint($result, 2008082500, 'scorm');
+        upgrade_mod_savepoint(true, 2008082500, 'scorm');
     }
 
-    if ($result && $oldversion < 2008090300) {
+    if ($oldversion < 2008090300) {
 
     /// Define field sha1hash to be added to scorm
         $table = new xmldb_table('scorm');
@@ -65,10 +64,10 @@ function xmldb_scorm_upgrade($oldversion) {
         $dbman->add_field($table, $field);
 
     /// scorm savepoint reached
-        upgrade_mod_savepoint($result, 2008090300, 'scorm');
+        upgrade_mod_savepoint(true, 2008090300, 'scorm');
     }
 
-    if ($result && $oldversion < 2008090301) {
+    if ($oldversion < 2008090301) {
 
     /// Define field revision to be added to scorm
         $table = new xmldb_table('scorm');
@@ -78,10 +77,10 @@ function xmldb_scorm_upgrade($oldversion) {
         $dbman->add_field($table, $field);
 
     /// scorm savepoint reached
-        upgrade_mod_savepoint($result, 2008090301, 'scorm');
+        upgrade_mod_savepoint(true, 2008090301, 'scorm');
     }
 
-    if ($result && $oldversion < 2008090302) {
+    if ($oldversion < 2008090302) {
         $sql = "UPDATE {scorm}
                    SET scormtype = 'external'
                  WHERE reference LIKE ? OR reference LIKE ? OR reference LIKE ?";
@@ -97,19 +96,19 @@ function xmldb_scorm_upgrade($oldversion) {
         $DB->execute($sql, array('#%'));
 
     /// scorm savepoint reached
-        upgrade_mod_savepoint($result, 2008090302, 'scorm');
+        upgrade_mod_savepoint(true, 2008090302, 'scorm');
     }
 
-    if ($result && $oldversion < 2008090303) {
+    if ($oldversion < 2008090303) {
         //remove obsoleted config settings
         unset_config('scorm_advancedsettings');
         unset_config('scorm_windowsettings');
 
     /// scorm savepoint reached
-        upgrade_mod_savepoint($result, 2008090303, 'scorm');
+        upgrade_mod_savepoint(true, 2008090303, 'scorm');
     }
 
-    if ($result && $oldversion < 2008090304) {
+    if ($oldversion < 2008090304) {
 
         /////////////////////////////////////
         /// new file storage upgrade code ///
@@ -232,11 +231,11 @@ function xmldb_scorm_upgrade($oldversion) {
         }
 
     /// scorm savepoint reached
-        upgrade_mod_savepoint($result, 2008090304, 'scorm');
+        upgrade_mod_savepoint(true, 2008090304, 'scorm');
     }
 
 
-    if ($result && $oldversion < 2008090305) {
+    if ($oldversion < 2008090305) {
 
     /// Define new fields forcecompleted, forcenewattempt, displayattemptstatus, and displaycoursestructure to be added to scorm
         $table = new xmldb_table('scorm');
@@ -262,12 +261,12 @@ function xmldb_scorm_upgrade($oldversion) {
         }
 
     /// scorm savepoint reached
-        upgrade_mod_savepoint($result, 2008090305, 'scorm');
+        upgrade_mod_savepoint(true, 2008090305, 'scorm');
     }
 
 
     // remove redundant config values
-    if ($result && $oldversion < 2008090306) {
+    if ($oldversion < 2008090306) {
         /*
          * comment this out as it is handled by the update mark 2008090310 below
          * left for historical documentation as some early adopters may have done
@@ -289,13 +288,13 @@ function xmldb_scorm_upgrade($oldversion) {
         }
          */
         /// scorm savepoint reached
-        upgrade_mod_savepoint($result, 2008090306, 'scorm');
+        upgrade_mod_savepoint(true, 2008090306, 'scorm');
     }
 
 
 
     // remove redundant config values
-    if ($result && $oldversion < 2008090307) {
+    if ($oldversion < 2008090307) {
         /*
          * comment this out as it is handled by the update mark 2008090310 below
          * left for historical documentation as some early adopters may have done
@@ -336,10 +335,10 @@ function xmldb_scorm_upgrade($oldversion) {
          */
 
         /// scorm savepoint reached
-        upgrade_mod_savepoint($result, 2008090307, 'scorm');
+        upgrade_mod_savepoint(true, 2008090307, 'scorm');
     }
 
-    if ($result && $oldversion < 2008090308) {
+    if ($oldversion < 2008090308) {
         $table = new xmldb_table('scorm');
         $field = new xmldb_field('timeopen', XMLDB_TYPE_INTEGER, '10', XMLDB_UNSIGNED, XMLDB_NOTNULL, null, '0', 'height');
         if (!$dbman->field_exists($table,$field)) {
@@ -351,11 +350,11 @@ function xmldb_scorm_upgrade($oldversion) {
         }
 
         /// scorm savepoint reached
-        upgrade_mod_savepoint($result, 2008090308, 'scorm');
+        upgrade_mod_savepoint(true, 2008090308, 'scorm');
     }
 
 
-    if ($result && $oldversion < 2008090310) {
+    if ($oldversion < 2008090310) {
         // take above blocks that delete config and move the values in to config_plugins
 
         $redundant_config = array(
@@ -395,16 +394,16 @@ function xmldb_scorm_upgrade($oldversion) {
         foreach ($redundant_config as $rcfg) {
             if (isset($CFG->$rcfg)) {
                 $shortname = substr($rcfg, 6);
-                $result = $result && set_config($shortname, $CFG->$rcfg, 'scorm');
-                $result = $result && unset_config($rcfg);
+                set_config($shortname, $CFG->$rcfg, 'scorm');
+                unset_config($rcfg);
             }
         }
 
         /// scorm savepoint reached
-        upgrade_mod_savepoint($result, 2008090310, 'scorm');
+        upgrade_mod_savepoint(true, 2008090310, 'scorm');
     }
 
-    if ($result && $oldversion < 2009042000) {
+    if ($oldversion < 2009042000) {
 
     /// Rename field summary on table scorm to intro
         $table = new xmldb_table('scorm');
@@ -414,10 +413,10 @@ function xmldb_scorm_upgrade($oldversion) {
         $dbman->rename_field($table, $field, 'intro');
 
     /// scorm savepoint reached
-        upgrade_mod_savepoint($result, 2009042000, 'scorm');
+        upgrade_mod_savepoint(true, 2009042000, 'scorm');
     }
 
-    if ($result && $oldversion < 2009042001) {
+    if ($oldversion < 2009042001) {
 
     /// Define field introformat to be added to scorm
         $table = new xmldb_table('scorm');
@@ -427,10 +426,10 @@ function xmldb_scorm_upgrade($oldversion) {
         $dbman->add_field($table, $field);
 
     /// scorm savepoint reached
-        upgrade_mod_savepoint($result, 2009042001, 'scorm');
+        upgrade_mod_savepoint(true, 2009042001, 'scorm');
     }
 
-    if ($result && $oldversion < 2009042002) {
+    if ($oldversion < 2009042002) {
 
     /// Define field introformat to be added to scorm
         $table = new xmldb_table('scorm_scoes');
@@ -440,10 +439,10 @@ function xmldb_scorm_upgrade($oldversion) {
         $dbman->change_field_type($table, $field);
 
     /// scorm savepoint reached
-        upgrade_mod_savepoint($result, 2009042002, 'scorm');
+        upgrade_mod_savepoint(true, 2009042002, 'scorm');
     }
 
-    return $result;
+    return true;
 }
 
 
index 837e801..fe0a78f 100644 (file)
@@ -24,11 +24,10 @@ function xmldb_survey_upgrade($oldversion) {
     global $CFG, $DB;
 
     $dbman = $DB->get_manager();
-    $result = true;
 
 //===== 1.9.0 upgrade line ======//
 
-    if ($result && $oldversion < 2009042002) {
+    if ($oldversion < 2009042002) {
 
     /// Define field introformat to be added to survey
         $table = new xmldb_table('survey');
@@ -38,10 +37,10 @@ function xmldb_survey_upgrade($oldversion) {
         $dbman->add_field($table, $field);
 
     /// survey savepoint reached
-        upgrade_mod_savepoint($result, 2009042002, 'survey');
+        upgrade_mod_savepoint(true, 2009042002, 'survey');
     }
 
-    return $result;
+    return true;
 }
 
 
index e4c0f72..c698ccd 100644 (file)
@@ -47,8 +47,7 @@ function xmldb_url_upgrade($oldversion) {
     global $CFG, $DB;
 
     $dbman = $DB->get_manager();
-    $result = true;
 
 
-    return $result;
+    return true;
 }
index 27697f0..4cc5b11 100644 (file)
@@ -70,19 +70,18 @@ function xmldb_wiki_upgrade($oldversion) {
     global $CFG, $DB, $OUTPUT;
 
     $dbman = $DB->get_manager();
-    $result = true;
 
     // Step 0: Add new fields to main wiki table
-    if ($result && $oldversion < 2010040100) {
+    if ($oldversion < 2010040100) {
         require_once(dirname(__FILE__) . '/upgradelib.php');
         echo $OUTPUT->notification('Adding new fields to wiki table', 'notifysuccess');
         wiki_add_wiki_fields();
 
-        upgrade_mod_savepoint($result, 2010040100, 'wiki');
+        upgrade_mod_savepoint(true, 2010040100, 'wiki');
     }
 
     // Step 1: Rename old tables
-    if ($result && $oldversion < 2010040101) {
+    if ($oldversion < 2010040101) {
         $tables = array('wiki_pages', 'wiki_locks', 'wiki_entries');
 
         echo $OUTPUT->notification('Renaming old wiki module tables', 'notifysuccess');
@@ -94,19 +93,19 @@ function xmldb_wiki_upgrade($oldversion) {
                 }
             }
         }
-        upgrade_mod_savepoint($result, 2010040101, 'wiki');
+        upgrade_mod_savepoint(true, 2010040101, 'wiki');
     }
 
     // Step 2: Creating new tables
-    if ($result && $oldversion < 2010040102) {
+    if ($oldversion < 2010040102) {
         require_once(dirname(__FILE__) . '/upgradelib.php');
         echo $OUTPUT->notification('Installing new wiki module tables', 'notifysuccess');
         wiki_upgrade_install_20_tables();
-        upgrade_mod_savepoint($result, 2010040102, 'wiki');
+        upgrade_mod_savepoint(true, 2010040102, 'wiki');
     }
 
     // Step 3: migrating wiki instances
-    if ($result && $oldversion < 2010040103) {
+    if ($oldversion < 2010040103) {
         upgrade_set_timeout();
 
         // Setting up wiki configuration
@@ -142,11 +141,11 @@ function xmldb_wiki_upgrade($oldversion) {
         }
 
         echo $OUTPUT->notification('Migrating old wikis to new wikis', 'notifysuccess');
-        upgrade_mod_savepoint($result, 2010040103, 'wiki');
+        upgrade_mod_savepoint(true, 2010040103, 'wiki');
     }
 
     // Step 4: migrating wiki entries to new subwikis
-    if ($result && $oldversion < 2010040104) {
+    if ($oldversion < 2010040104) {
         /**
          * Migrating wiki entries to new subwikis
          */
@@ -157,11 +156,11 @@ function xmldb_wiki_upgrade($oldversion) {
 
         $DB->execute($sql, array());
 
-        upgrade_mod_savepoint($result, 2010040104, 'wiki');
+        upgrade_mod_savepoint(true, 2010040104, 'wiki');
     }
 
     // Step 5: Migrating pages
-    if ($result && $oldversion < 2010040105) {
+    if ($oldversion < 2010040105) {
         /**
          * Filling pages table with latest versions of every page.
          *
@@ -187,19 +186,19 @@ function xmldb_wiki_upgrade($oldversion) {
 
         $DB->execute($sql, array('**reparse needed**'));
 
-        upgrade_mod_savepoint($result, 2010040105, 'wiki');
+        upgrade_mod_savepoint(true, 2010040105, 'wiki');
     }
 
     // Step 6: Migrating versions
-    if ($result && $oldversion < 2010040106) {
+    if ($oldversion < 2010040106) {
         require_once(dirname(__FILE__) . '/upgradelib.php');
         echo $OUTPUT->notification('Migrating old history to new history', 'notifysuccess');
         wiki_upgrade_migrate_versions();
-        upgrade_mod_savepoint($result, 2010040106, 'wiki');
+        upgrade_mod_savepoint(true, 2010040106, 'wiki');
     }
 
     // Step 7: refresh cachedcontent and fill wiki links table
-    if ($result && $oldversion < 2010040107) {
+    if ($oldversion < 2010040107) {
         require_once($CFG->dirroot. '/mod/wiki/locallib.php');
         upgrade_set_timeout();
 
@@ -214,10 +213,10 @@ function xmldb_wiki_upgrade($oldversion) {
         $pages->close();
 
         echo $OUTPUT->notification('Caching content', 'notifysuccess');
-        upgrade_mod_savepoint($result, 2010040107, 'wiki');
+        upgrade_mod_savepoint(true, 2010040107, 'wiki');
     }
     // Step 8, migrating files
-    if ($result && $oldversion < 2010040108) {
+    if ($oldversion < 2010040108) {
         $fs = get_file_storage();
         $sql = "SELECT DISTINCT po.pagename, w.id AS wikiid, po.userid,
             po.meta AS filemeta, eo.id AS entryid, eo.groupid, s.id AS subwiki,
@@ -281,11 +280,11 @@ function xmldb_wiki_upgrade($oldversion) {
             }
         }
         $rs->close();
-        upgrade_mod_savepoint($result, 2010040108, 'wiki');
+        upgrade_mod_savepoint(true, 2010040108, 'wiki');
     }
 
     // Step 9: clean wiki table
-    if ($result && $oldversion < 2010040109) {
+    if ($oldversion < 2010040109) {
         $fields = array('summary', 'pagename', 'wtype', 'ewikiprinttitle', 'htmlmode', 'ewikiacceptbinary', 'disablecamelcase', 'setpageflags', 'strippages', 'removepages', 'revertchanges', 'initialcontent');
         $table = new xmldb_table('wiki');
         foreach ($fields as $fieldname) {
@@ -296,13 +295,13 @@ function xmldb_wiki_upgrade($oldversion) {
 
         }
         echo $OUTPUT->notification('Cleaning wiki table', 'notifysuccess');
-        upgrade_mod_savepoint($result, 2010040109, 'wiki');
+        upgrade_mod_savepoint(true, 2010040109, 'wiki');
     }
 
     // TODO: Will hold the old tables so we will have chance to fix problems
     // Will remove old tables once migrating 100% stable
     // Step 10: delete old tables
-    if ($result && $oldversion < 2010040120) {
+    if ($oldversion < 2010040120) {
         //$tables = array('wiki_pages', 'wiki_locks', 'wiki_entries');
 
         //foreach ($tables as $tablename) {
@@ -312,8 +311,8 @@ function xmldb_wiki_upgrade($oldversion) {
             //}
         //}
         //echo $OUTPUT->notification('Droping old tables', 'notifysuccess');
-        //upgrade_mod_savepoint($result, 2010040120, 'wiki');
+        //upgrade_mod_savepoint(true, 2010040120, 'wiki');
     }
 
-    return $result;
+    return true;
 }
index c2bcc2a..8a81be4 100644 (file)
@@ -36,7 +36,6 @@ function xmldb_workshop_upgrade($oldversion) {
     global $CFG, $DB, $OUTPUT;
 
     $dbman = $DB->get_manager();
-    $result = true;
 
     //===== 1.9.0 upgrade line ======//
 
@@ -45,7 +44,7 @@ function xmldb_workshop_upgrade($oldversion) {
      * so that it does not matter if we are upgrading from 1.9.0, 1.9.3 or 1.9.whatever
      */
 
-    if ($result && $oldversion < 2007101510) {
+    if ($oldversion < 2007101510) {
         $orphans = $DB->get_records_sql('SELECT wa.id
                                            FROM {workshop_assessments} wa
                                       LEFT JOIN {workshop_submissions} ws ON wa.submissionid = ws.id
@@ -56,7 +55,7 @@ function xmldb_workshop_upgrade($oldversion) {
                 $DB->delete_records('workshop_assessments', 'id', $waid);
             }
         }
-        upgrade_mod_savepoint($result, 2007101510, 'workshop');
+        upgrade_mod_savepoint(true, 2007101510, 'workshop');
     }
 
     //===== end of 1.9.0 upgrade line ======//
@@ -71,7 +70,7 @@ function xmldb_workshop_upgrade($oldversion) {
     /**
      * Migration from 1.9 - step 1 - rename old tables
      */
-    if ($result && $oldversion < 2009102901) {
+    if ($oldversion < 2009102901) {
         echo $OUTPUT->notification('Renaming old workshop module tables', 'notifysuccess');
         foreach (array('workshop', 'workshop_elements', 'workshop_rubrics', 'workshop_submissions', 'workshop_assessments',
                 'workshop_grades', 'workshop_comments', 'workshop_stockcomments') as $tableorig) {
@@ -94,43 +93,43 @@ function xmldb_workshop_upgrade($oldversion) {
                 $dbman->add_field($table, $field);
             }
         }
-        upgrade_mod_savepoint($result, 2009102901, 'workshop');
+        upgrade_mod_savepoint(true, 2009102901, 'workshop');
     }
 
     /**
      * Migration from 1.9 - step 2 - create new workshop core tables
      */
-    if ($result && $oldversion < 2009102902) {
+    if ($oldversion < 2009102902) {
         require_once(dirname(__FILE__) . '/upgradelib.php');
         echo $OUTPUT->notification('Preparing new workshop module tables', 'notifysuccess');
         workshop_upgrade_prepare_20_tables();
-        upgrade_mod_savepoint($result, 2009102902, 'workshop');
+        upgrade_mod_savepoint(true, 2009102902, 'workshop');
     }
 
     /**
      * Migration from 1.9 - step 3 - migrate workshop instances
      */
-    if ($result && $oldversion < 2009102903) {
+    if ($oldversion < 2009102903) {
         require_once(dirname(__FILE__) . '/upgradelib.php');
         echo $OUTPUT->notification('Copying workshop core data', 'notifysuccess');
         workshop_upgrade_module_instances();
-        upgrade_mod_savepoint($result, 2009102903, 'workshop');
+        upgrade_mod_savepoint(true, 2009102903, 'workshop');
     }
 
     /**
      * Migration from 1.9 - step 4 - migrate submissions
      */
-    if ($result && $oldversion < 2009102904) {
+    if ($oldversion < 2009102904) {
         require_once(dirname(__FILE__) . '/upgradelib.php');
         echo $OUTPUT->notification('Copying submissions', 'notifysuccess');
         workshop_upgrade_submissions();
-        upgrade_mod_savepoint($result, 2009102904, 'workshop');
+        upgrade_mod_savepoint(true, 2009102904, 'workshop');
     }
 
     /**
      * Migration from 1.9 - step 5 - migrate submission attachment to new file storage
      */
-    if ($result && $oldversion < 2009102905) {
+    if ($oldversion < 2009102905) {
         // $filearea = "$workshop->course/$CFG->moddata/workshop/$submission->id";
         $fs     = get_file_storage();
         $from   = 'FROM {workshop_submissions} s
@@ -187,17 +186,17 @@ function xmldb_workshop_upgrade($oldversion) {
             @rmdir("$CFG->dataroot/$submission->course");
         }
         $rs->close();
-        upgrade_mod_savepoint($result, 2009102905, 'workshop');
+        upgrade_mod_savepoint(true, 2009102905, 'workshop');
     }
 
     /**
      * Migration from 1.9 - step 6 - migrate assessments
      */
-    if ($result && $oldversion < 2009102906) {
+    if ($oldversion < 2009102906) {
         require_once(dirname(__FILE__) . '/upgradelib.php');
         echo $OUTPUT->notification('Copying assessments', 'notifysuccess');
         workshop_upgrade_assessments();
-        upgrade_mod_savepoint($result, 2009102906, 'workshop');
+        upgrade_mod_savepoint(true, 2009102906, 'workshop');
     }
 
     /**
@@ -207,14 +206,14 @@ function xmldb_workshop_upgrade($oldversion) {
     /**
      * Add 'published' field into workshop_submissions
      */
-    if ($result && $oldversion < 2009121800) {
+    if ($oldversion < 2009121800) {
         $table = new xmldb_table('workshop_submissions');
         $field = new xmldb_field('published', XMLDB_TYPE_INTEGER, '2', XMLDB_UNSIGNED, null, null, '0', 'timegraded');
         if (!$dbman->field_exists($table, $field)) {
             $dbman->add_field($table, $field);
         }
-        upgrade_mod_savepoint($result, 2009121800, 'workshop');
+        upgrade_mod_savepoint(true, 2009121800, 'workshop');
     }
 
-    return $result;
+    return true;
 }
diff --git a/portfolio/mahara/db/upgrade.php b/portfolio/mahara/db/upgrade.php
deleted file mode 100644 (file)
index ea11f8e..0000000
+++ /dev/null
@@ -1,32 +0,0 @@
-<?php
-
-// This file keeps track of upgrades to
-// the portfolio/mahara plugin
-//
-// Sometimes, changes between versions involve
-// alterations to database structures and other
-// major things that may break installations.
-//
-// The upgrade function in this file will attempt
-// to perform all the necessary actions to upgrade
-// your older installtion to the current version.
-//
-// If there's something it cannot do itself, it
-// will tell you what you need to do.
-//
-// The commands in here will all be database-neutral,
-// using the methods of database_manager class
-//
-// Please do not forget to use upgrade_set_timeout()
-// before any action that may take longer time to finish.
-
-function xmldb_portfolio_mahara_upgrade($oldversion) {
-
-    global $CFG, $DB;
-
-    $dbman = $DB->get_manager(); // loads ddl manager and xmldb classes
-    $result = true;
-
-    return $result;
-
-}
index 2e3159b..d8f13e1 100644 (file)
@@ -24,16 +24,16 @@ function xmldb_qtype_calculated_upgrade($oldversion) {
     global $CFG, $DB;
 
     $dbman = $DB->get_manager();
-    $result = true;
 
     // MDL-16505.
-    if ($result && $oldversion < 2008091700 ) { //New version in version.php
+    if ($oldversion < 2008091700 ) { //New version in version.php
         if (get_config('qtype_datasetdependent', 'version')) {
-            $result = $result && unset_config('version', 'qtype_datasetdependent');
+            unset_config('version', 'qtype_datasetdependent');
         }
-        upgrade_plugin_savepoint($result, 2008091700, 'qtype', 'calculated');
+        upgrade_plugin_savepoint(true, 2008091700, 'qtype', 'calculated');
     }
-    if ($result && $oldversion < 2009082000 ) { //New version in version.php
+
+    if ($oldversion < 2009082000) { //New version in version.php
 
 // this should be changed if merged to 1.9
 //    let if ($dbman->table_exists()) replace the normal $oldversion test
@@ -56,9 +56,10 @@ function xmldb_qtype_calculated_upgrade($oldversion) {
             // $dbman->create_table doesnt return a result, we just have to trust it
             $dbman->create_table($table);
         }
-        upgrade_plugin_savepoint($result, 2009082000 , 'qtype', 'calculated');
+        upgrade_plugin_savepoint(true, 2009082000 , 'qtype', 'calculated');
     }
-    if ( $result && $oldversion < 2009092000 ) { //New version in version.php
+
+    if ( $oldversion < 2009092000) { //New version in version.php
 
     /// Define field multichoice to be added to question_calculated_options
     ///ALTER TABLE `moodle`.`mdl_question_calculated_options` DROP COLUMN `multichoice`;
@@ -119,30 +120,25 @@ function xmldb_qtype_calculated_upgrade($oldversion) {
         if (!$dbman->field_exists($table, $field)) {
             $dbman->add_field($table, $field);
         }
-        upgrade_plugin_savepoint($result, 2009092000, 'qtype', 'calculated');
+        upgrade_plugin_savepoint(true, 2009092000, 'qtype', 'calculated');
     }
 
-   if ($result && $oldversion < 2010020800) {
+    if ($oldversion < 2010020800) {
 
     /// Define field multiplechoice to be dropped from question_calculated_options
         $table = new xmldb_table('question_calculated_options');
         $field = new xmldb_field('multichoice');
-        
+
     /// Conditionally launch drop field multiplechoice
         if ($dbman->field_exists($table, $field)) {
             $dbman->drop_field($table, $field);
         }
 
     /// calculated savepoint reached
-        upgrade_plugin_savepoint($result, 2010020800, 'qtype', 'calculated');
+        upgrade_plugin_savepoint(true, 2010020800, 'qtype', 'calculated');
     }
-/// calculated savepoint reached
-/// if ($result && $oldversion < YYYYMMDD00) { //New version in version.php
-///     $result = result of database_manager methods
-///     upgrade_plugin_savepoint($result, YYYYMMDD00, 'qtype', 'calculated');
-/// }
 
-    return $result;
+    return true;
 }
 
 
diff --git a/question/type/essay/db/upgrade.php b/question/type/essay/db/upgrade.php
deleted file mode 100644 (file)
index eee7859..0000000
+++ /dev/null
@@ -1,41 +0,0 @@
-<?php
-
-// This file keeps track of upgrades to
-// the essay qtype plugin
-//
-// Sometimes, changes between versions involve
-// alterations to database structures and other
-// major things that may break installations.
-//
-// The upgrade function in this file will attempt
-// to perform all the necessary actions to upgrade
-// your older installtion to the current version.
-//
-// If there's something it cannot do itself, it
-// will tell you what you need to do.
-//
-// The commands in here will all be database-neutral,
-// using the methods of database_manager class
-//
-// Please do not forget to use upgrade_set_timeout()
-// before any action that may take longer time to finish.
-
-function xmldb_qtype_essay_upgrade($oldversion) {
-    global $CFG, $DB;
-
-    $dbman = $DB->get_manager();
-    $result = true;
-
-/// And upgrade begins here. For each one, you'll need one
-/// block of code similar to the next one. Please, delete
-/// this comment lines once this file start handling proper
-/// upgrade code.
-
-/// if ($result && $oldversion < YYYYMMDD00) { //New version in version.php
-///     $result = result of database_manager methods
-/// }
-
-    return $result;
-}
-
-
diff --git a/question/type/match/db/upgrade.php b/question/type/match/db/upgrade.php
deleted file mode 100644 (file)
index 2708e11..0000000
+++ /dev/null
@@ -1,41 +0,0 @@
-<?php
-
-// This file keeps track of upgrades to
-// the match qtype plugin
-//
-// Sometimes, changes between versions involve
-// alterations to database structures and other
-// major things that may break installations.
-//
-// The upgrade function in this file will attempt
-// to perform all the necessary actions to upgrade
-// your older installtion to the current version.
-//
-// If there's something it cannot do itself, it
-// will tell you what you need to do.
-//
-// The commands in here will all be database-neutral,
-// using the methods of database_manager class
-//
-// Please do not forget to use upgrade_set_timeout()
-// before any action that may take longer time to finish.
-
-function xmldb_qtype_match_upgrade($oldversion) {
-    global $CFG, $DB;
-
-    $dbman = $DB->get_manager();
-    $result = true;
-
-/// And upgrade begins here. For each one, you'll need one
-/// block of code similar to the next one. Please, delete
-/// this comment lines once this file start handling proper
-/// upgrade code.
-
-/// if ($result && $oldversion < YYYYMMDD00) { //New version in version.php
-///     $result = result of database_manager methods
-/// }
-
-    return $result;
-}
-
-
index c4a70b8..891607c 100644 (file)
@@ -24,41 +24,26 @@ function xmldb_qtype_multianswer_upgrade($oldversion) {
     global $CFG, $DB;
 
     $dbman = $DB->get_manager();
-    $result = true;
 
-    if ($result && $oldversion < 2008050800) {
-        question_multianswer_fix_subquestion_parents_and_categories();
-    /// multianswer savepoint reached
-        upgrade_plugin_savepoint($result, 2008050800, 'qtype', 'multianswer');
-    }
-
-    return $result;
-}
-
-/**
- * Due to MDL-14750, subquestions of multianswer questions restored from backup will
- * have the wrong parent, and due to MDL-10899 subquestions of multianswer questions
- * that have been moved between categories will be in the wrong category, This code fixes these up.
- */
-function question_multianswer_fix_subquestion_parents_and_categories() {
-    global $CFG, $DB;
+    if ($oldversion < 2008050800) {
+        //hey - no functions here in this file !!!!!!!
 
-    $result = true;
-    $rs = $DB->get_recordset_sql('SELECT q.id, q.category, qma.sequence
-                                    FROM {question} q JOIN {question_multianswer} qma ON q.id = qma.question');
-    if ($rs) {
+        $rs = $DB->get_recordset_sql("SELECT q.id, q.category, qma.sequence
+                                        FROM {question} q
+                                        JOIN {question_multianswer} qma ON q.id = qma.question");
         foreach ($rs as $q) {
             if (!empty($q->sequence)) {
-                $result = $result && $DB->execute("UPDATE {question}
-                                                      SET parent = ?, category = ?
-                                                    WHERE id IN ($q->sequence) AND parent <> 0",
-                                                  array($q->id, $q->category));
+                $DB->execute("UPDATE {question}
+                                 SET parent = ?, category = ?
+                               WHERE id IN ($q->sequence) AND parent <> 0",
+                             array($q->id, $q->category));
             }
         }
         $rs->close();
-    } else {
-        $result = false;
+
+        /// multianswer savepoint reached
+        upgrade_plugin_savepoint(true, 2008050800, 'qtype', 'multianswer');
     }
-    return $result;
-}
 
+    return true;
+}
index b9ca6f3..4edf9e8 100644 (file)
@@ -24,13 +24,12 @@ function xmldb_qtype_multichoice_upgrade($oldversion) {
     global $CFG, $DB, $QTYPES;
 
     $dbman = $DB->get_manager();
-    $result = true;
 
     // This upgrade actually belongs to the random question type,
     // but that does not have a DB upgrade script. Therefore, multichoice
     // is doing it.
     // Rename random questions to give them more helpful names.
-    if ($result && $oldversion < 2008021800) {
+    if ($oldversion < 2008021800) {
         require_once($CFG->libdir . '/questionlib.php');
         // Get all categories containing random questions.
         $categories = $DB->get_recordset_sql("
@@ -44,16 +43,16 @@ function xmldb_qtype_multichoice_upgrade($oldversion) {
         $where = "qtype = 'random' AND category = ? AND " . $DB->sql_compare_text('questiontext') . " = ?";
         foreach ($categories as $cat) {
             $randomqname = $QTYPES[RANDOM]->question_name($cat, false);
-            $result = $result && $DB->set_field_select('question', 'name', $randomqname, $where, array($cat->id, '0'));
+            $DB->set_field_select('question', 'name', $randomqname, $where, array($cat->id, '0'));
 
             $randomqname = $QTYPES[RANDOM]->question_name($cat, true);
-            $result = $result && $DB->set_field_select('question', 'name', $randomqname, $where, array($cat->id, '1'));
+            $DB->set_field_select('question', 'name', $randomqname, $where, array($cat->id, '1'));
         }
 
-        upgrade_plugin_savepoint($result, 2008021800, 'qtype', 'multichoice');
+        upgrade_plugin_savepoint(true, 2008021800, 'qtype', 'multichoice');
     }
 
-    return $result;
+    return true;
 }
 
 
index f6d8946..068fc82 100644 (file)
@@ -24,10 +24,9 @@ function xmldb_qtype_numerical_upgrade($oldversion) {
     global $CFG, $DB;
 
     $dbman = $DB->get_manager();
-    $result = true;
 
 //===== 1.9.0 upgrade line ======//
-    if ($result && $oldversion < 2009100100 ) { //New version in version.php
+    if ($oldversion < 2009100100 ) { //New version in version.php
 
     /// Define table question_numerical_options to be created
         $table = new xmldb_table('question_numerical_options');
@@ -49,10 +48,10 @@ function xmldb_qtype_numerical_upgrade($oldversion) {
             // $dbman->create_table doesnt return a result, we just have to trust it
             $dbman->create_table($table);
         }//else
-        upgrade_plugin_savepoint($result, 2009100100, 'qtype', 'numerical');
+        upgrade_plugin_savepoint(true, 2009100100, 'qtype', 'numerical');
     }
 
-    return $result;
+    return true;
 }
 
 
diff --git a/question/type/randomsamatch/db/upgrade.php b/question/type/randomsamatch/db/upgrade.php
deleted file mode 100644 (file)
index 761e4a5..0000000
+++ /dev/null
@@ -1,41 +0,0 @@
-<?php
-
-// This file keeps track of upgrades to
-// the randomsamatch qtype plugin
-//
-// Sometimes, changes between versions involve
-// alterations to database structures and other
-// major things that may break installations.
-//
-// The upgrade function in this file will attempt
-// to perform all the necessary actions to upgrade
-// your older installtion to the current version.
-//
-// If there's something it cannot do itself, it
-// will tell you what you need to do.
-//
-// The commands in here will all be database-neutral,
-// using the methods of database_manager class
-//
-// Please do not forget to use upgrade_set_timeout()
-// before any action that may take longer time to finish.
-
-function xmldb_qtype_randomsamatch_upgrade($oldversion) {
-    global $CFG, $DB;
-
-    $dbman = $DB->get_manager();
-    $result = true;
-
-/// And upgrade begins here. For each one, you'll need one
-/// block of code similar to the next one. Please, delete
-/// this comment lines once this file start handling proper
-/// upgrade code.
-
-/// if ($result && $oldversion < YYYYMMDD00) { //New version in version.php
-///     $result = result of database_manager methods
-/// }
-
-    return $result;
-}
-
-
diff --git a/question/type/shortanswer/db/upgrade.php b/question/type/shortanswer/db/upgrade.php
deleted file mode 100644 (file)
index 3435156..0000000
+++ /dev/null
@@ -1,41 +0,0 @@
-<?php
-
-// This file keeps track of upgrades to
-// the shortanswer qtype plugin
-//
-// Sometimes, changes between versions involve
-// alterations to database structures and other
-// major things that may break installations.
-//
-// The upgrade function in this file will attempt
-// to perform all the necessary actions to upgrade
-// your older installtion to the current version.
-//
-// If there's something it cannot do itself, it
-// will tell you what you need to do.
-//
-// The commands in here will all be database-neutral,
-// using the methods of database_manager class
-//
-// Please do not forget to use upgrade_set_timeout()
-// before any action that may take longer time to finish.
-
-function xmldb_qtype_shortanswer_upgrade($oldversion) {
-    global $CFG, $DB;
-
-    $dbman = $DB->get_manager();
-    $result = true;
-
-/// And upgrade begins here. For each one, you'll need one
-/// block of code similar to the next one. Please, delete
-/// this comment lines once this file start handling proper
-/// upgrade code.
-
-/// if ($result && $oldversion < YYYYMMDD00) { //New version in version.php
-///     $result = result of database_manager methods
-/// }
-
-    return $result;
-}
-
-
diff --git a/question/type/truefalse/db/upgrade.php b/question/type/truefalse/db/upgrade.php
deleted file mode 100644 (file)
index 5495c62..0000000
+++ /dev/null
@@ -1,41 +0,0 @@
-<?php
-
-// This file keeps track of upgrades to
-// the truefalse qtype plugin
-//
-// Sometimes, changes between versions involve
-// alterations to database structures and other
-// major things that may break installations.
-//
-// The upgrade function in this file will attempt
-// to perform all the necessary actions to upgrade
-// your older installtion to the current version.
-//
-// If there's something it cannot do itself, it
-// will tell you what you need to do.
-//
-// The commands in here will all be database-neutral,
-// using the methods of database_manager class
-//
-// Please do not forget to use upgrade_set_timeout()
-// before any action that may take longer time to finish.
-
-function xmldb_qtype_truefalse_upgrade($oldversion) {
-    global $CFG, $DB;
-
-    $dbman = $DB->get_manager();
-    $result = true;
-
-/// And upgrade begins here. For each one, you'll need one
-/// block of code similar to the next one. Please, delete
-/// this comment lines once this file start handling proper
-/// upgrade code.
-
-/// if ($result && $oldversion < YYYYMMDD00) { //New version in version.php
-///     $result = result of database_manager methods
-/// }
-
-    return $result;
-}
-
-
diff --git a/repository/alfresco/db/upgrade.php b/repository/alfresco/db/upgrade.php
deleted file mode 100644 (file)
index b037b20..0000000
+++ /dev/null
@@ -1,20 +0,0 @@
-<?php
-
-function xmldb_repository_alfresco_upgrade($oldversion) {
-
-    global $CFG, $DB;
-
-    $dbman = $DB->get_manager();
-    $result = true;
-
-/// And upgrade begins here. For each one, you'll need one
-/// block of code similar to the next one. Please, delete
-/// this comment lines once this file start handling proper
-/// upgrade code.
-
-/// if ($result && $oldversion < YYYYMMDD00) { //New version in version.php
-///     $result = result of database_manager methods
-/// }
-
-    return $result;
-}
diff --git a/repository/boxnet/db/upgrade.php b/repository/boxnet/db/upgrade.php
deleted file mode 100644 (file)
index 2a8b38b..0000000
+++ /dev/null
@@ -1,20 +0,0 @@
-<?php
-
-function xmldb_repository_boxnet_upgrade($oldversion) {
-
-    global $CFG, $DB;
-
-    $dbman = $DB->get_manager();
-    $result = true;
-
-/// And upgrade begins here. For each one, you'll need one
-/// block of code similar to the next one. Please, delete
-/// this comment lines once this file start handling proper
-/// upgrade code.
-
-/// if ($result && $oldversion < YYYYMMDD00) { //New version in version.php
-///     $result = result of database_manager methods
-/// }
-
-    return $result;
-}
diff --git a/repository/dropbox/db/upgrade.php b/repository/dropbox/db/upgrade.php
deleted file mode 100755 (executable)
index 8b3e4e7..0000000
+++ /dev/null
@@ -1,35 +0,0 @@
-<?php
-
-// This file is part of Moodle - http://moodle.org/
-//
-// Moodle is free software: you can redistribute it and/or modify
-// it under the terms of the GNU General Public License as published by
-// the Free Software Foundation, either version 3 of the License, or
-// (at your option) any later version.
-//
-// Moodle is distributed in the hope that it will be useful,
-// but WITHOUT ANY WARRANTY; without even the implied warranty of
-// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-// GNU General Public License for more details.
-//
-// You should have received a copy of the GNU General Public License
-// along with Moodle.  If not, see <http://www.gnu.org/licenses/>.
-
-function xmldb_repository_dropbox_upgrade($oldversion) {
-
-    global $CFG, $DB;
-
-    $dbman = $DB->get_manager();
-    $result = true;
-
-/// And upgrade begins here. For each one, you'll need one
-/// block of code similar to the next one. Please, delete
-/// this comment lines once this file start handling proper
-/// upgrade code.
-
-/// if ($result && $oldversion < YYYYMMDD00) { //New version in version.php
-///     $result = result of database_manager methods
-/// }
-
-    return $result;
-}
diff --git a/repository/filesystem/db/upgrade.php b/repository/filesystem/db/upgrade.php
deleted file mode 100644 (file)
index 9e762f3..0000000
+++ /dev/null
@@ -1,20 +0,0 @@
-<?php
-
-function xmldb_repository_filesystem_upgrade($oldversion) {
-
-    global $CFG, $DB;
-
-    $dbman = $DB->get_manager();
-    $result = true;
-
-/// And upgrade begins here. For each one, you'll need one
-/// block of code similar to the next one. Please, delete
-/// this comment lines once this file start handling proper
-/// upgrade code.
-
-/// if ($result && $oldversion < YYYYMMDD00) { //New version in version.php
-///     $result = result of database_manager methods
-/// }
-
-    return $result;
-}
diff --git a/repository/flickr/db/upgrade.php b/repository/flickr/db/upgrade.php
deleted file mode 100644 (file)
index 6ad451c..0000000
+++ /dev/null
@@ -1,20 +0,0 @@
-<?php
-
-function xmldb_repository_flickr_upgrade($oldversion) {
-
-    global $CFG, $DB;
-
-    $dbman = $DB->get_manager();
-    $result = true;
-
-/// And upgrade begins here. For each one, you'll need one
-/// block of code similar to the next one. Please, delete
-/// this comment lines once this file start handling proper
-/// upgrade code.
-
-/// if ($result && $oldversion < YYYYMMDD00) { //New version in version.php
-///     $result = result of database_manager methods
-/// }
-
-    return $result;
-}
diff --git a/repository/flickr_public/db/upgrade.php b/repository/flickr_public/db/upgrade.php
deleted file mode 100644 (file)
index 05333ad..0000000
+++ /dev/null
@@ -1,20 +0,0 @@
-<?php
-
-function xmldb_repository_flickr_public_upgrade($oldversion) {
-
-    global $CFG, $DB;
-
-    $dbman = $DB->get_manager();
-    $result = true;
-
-/// And upgrade begins here. For each one, you'll need one
-/// block of code similar to the next one. Please, delete
-/// this comment lines once this file start handling proper
-/// upgrade code.
-
-/// if ($result && $oldversion < YYYYMMDD00) { //New version in version.php
-///     $result = result of database_manager methods
-/// }
-
-    return $result;
-}
diff --git a/repository/googledocs/db/upgrade.php b/repository/googledocs/db/upgrade.php
deleted file mode 100644 (file)
index 842760b..0000000
+++ /dev/null
@@ -1,20 +0,0 @@
-<?php
-
-function xmldb_repository_googledocs_upgrade($oldversion) {
-
-    global $CFG, $DB;
-
-    $dbman = $DB->get_manager();
-    $result = true;
-
-/// And upgrade begins here. For each one, you'll need one
-/// block of code similar to the next one. Please, delete
-/// this comment lines once this file start handling proper
-/// upgrade code.
-
-/// if ($result && $oldversion < YYYYMMDD00) { //New version in version.php
-///     $result = result of database_manager methods
-/// }
-
-    return $result;
-}
diff --git a/repository/local/db/upgrade.php b/repository/local/db/upgrade.php
deleted file mode 100644 (file)
index d29c39a..0000000
+++ /dev/null
@@ -1,20 +0,0 @@
-<?php
-
-function xmldb_repository_local_upgrade($oldversion) {
-
-    global $CFG, $DB;
-
-    $dbman = $DB->get_manager();
-    $result = true;
-
-/// And upgrade begins here. For each one, you'll need one
-/// block of code similar to the next one. Please, delete
-/// this comment lines once this file start handling proper
-/// upgrade code.
-
-/// if ($result && $oldversion < YYYYMMDD00) { //New version in version.php
-///     $result = result of database_manager methods
-/// }
-
-    return $result;
-}
diff --git a/repository/merlot/db/upgrade.php b/repository/merlot/db/upgrade.php
deleted file mode 100755 (executable)
index 031fbc9..0000000
+++ /dev/null
@@ -1,20 +0,0 @@
-<?php
-
-function xmldb_repository_merlot_upgrade($oldversion) {
-
-    global $CFG, $DB;
-
-    $dbman = $DB->get_manager();
-    $result = true;
-
-/// And upgrade begins here. For each one, you'll need one
-/// block of code similar to the next one. Please, delete
-/// this comment lines once this file start handling proper
-/// upgrade code.
-
-/// if ($result && $oldversion < YYYYMMDD00) { //New version in version.php
-///     $result = result of database_manager methods
-/// }
-
-    return $result;
-}
diff --git a/repository/picasa/db/upgrade.php b/repository/picasa/db/upgrade.php
deleted file mode 100644 (file)
index 7b3d54c..0000000
+++ /dev/null
@@ -1,20 +0,0 @@
-<?php
-
-function xmldb_repository_picasa_upgrade($oldversion) {
-
-    global $CFG, $DB;
-
-    $dbman = $DB->get_manager();
-    $result = true;
-
-/// And upgrade begins here. For each one, you'll need one
-/// block of code similar to the next one. Please, delete
-/// this comment lines once this file start handling proper
-/// upgrade code.
-
-/// if ($result && $oldversion < YYYYMMDD00) { //New version in version.php
-///     $result = result of database_manager methods
-/// }
-
-    return $result;
-}
diff --git a/repository/recent/db/upgrade.php b/repository/recent/db/upgrade.php
deleted file mode 100644 (file)
index 0adcf37..0000000
+++ /dev/null
@@ -1,35 +0,0 @@
-<?php
-
-// This file is part of Moodle - http://moodle.org/
-//
-// Moodle is free software: you can redistribute it and/or modify
-// it under the terms of the GNU General Public License as published by
-// the Free Software Foundation, either version 3 of the License, or
-// (at your option) any later version.
-//
-// Moodle is distributed in the hope that it will be useful,
-// but WITHOUT ANY WARRANTY; without even the implied warranty of
-// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-// GNU General Public License for more details.
-//
-// You should have received a copy of the GNU General Public License
-// along with Moodle.  If not, see <http://www.gnu.org/licenses/>.
-
-function xmldb_repository_recent_upgrade($oldversion) {
-
-    global $CFG, $DB;
-
-    $dbman = $DB->get_manager();
-    $result = true;
-
-/// And upgrade begins here. For each one, you'll need one
-/// block of code similar to the next one. Please, delete
-/// this comment lines once this file start handling proper
-/// upgrade code.
-
-/// if ($result && $oldversion < YYYYMMDD00) { //New version in version.php
-///     $result = result of database_manager methods
-/// }
-
-    return $result;
-}
diff --git a/repository/s3/db/upgrade.php b/repository/s3/db/upgrade.php
deleted file mode 100644 (file)
index cc2193e..0000000
+++ /dev/null
@@ -1,20 +0,0 @@
-<?php
-
-function xmldb_repository_s3_upgrade($oldversion) {
-
-    global $CFG, $DB;
-
-    $dbman = $DB->get_manager();
-    $result = true;
-
-/// And upgrade begins here. For each one, you'll need one
-/// block of code similar to the next one. Please, delete
-/// this comment lines once this file start handling proper
-/// upgrade code.
-
-/// if ($result && $oldversion < YYYYMMDD00) { //New version in version.php
-///     $result = result of database_manager methods
-/// }
-
-    return $result;
-}
diff --git a/repository/upload/db/upgrade.php b/repository/upload/db/upgrade.php
deleted file mode 100644 (file)
index 9b61f83..0000000
+++ /dev/null
@@ -1,19 +0,0 @@
-<?php
-function xmldb_repository_upload_upgrade($oldversion) {
-
-    global $CFG, $DB;
-
-    $dbman = $DB->get_manager();
-    $result = true;
-
-/// And upgrade begins here. For each one, you'll need one
-/// block of code similar to the next one. Please, delete
-/// this comment lines once this file start handling proper
-/// upgrade code.
-
-/// if ($result && $oldversion < YYYYMMDD00) { //New version in version.php
-///     $result = result of database_manager methods
-/// }
-
-    return $result;
-}
diff --git a/repository/url/db/upgrade.php b/repository/url/db/upgrade.php
deleted file mode 100644 (file)
index 021f9bf..0000000
+++ /dev/null
@@ -1,20 +0,0 @@
-<?php
-
-function xmldb_repository_url_upgrade($oldversion) {
-
-    global $CFG, $DB;
-
-    $dbman = $DB->get_manager();
-    $result = true;
-
-/// And upgrade begins here. For each one, you'll need one
-/// block of code similar to the next one. Please, delete
-/// this comment lines once this file start handling proper
-/// upgrade code.
-
-/// if ($result && $oldversion < YYYYMMDD00) { //New version in version.php
-///     $result = result of database_manager methods
-/// }
-
-    return $result;
-}
diff --git a/repository/user/db/upgrade.php b/repository/user/db/upgrade.php
deleted file mode 100755 (executable)
index 744fa06..0000000
+++ /dev/null
@@ -1,35 +0,0 @@
-<?php
-
-// This file is part of Moodle - http://moodle.org/
-//
-// Moodle is free software: you can redistribute it and/or modify
-// it under the terms of the GNU General Public License as published by
-// the Free Software Foundation, either version 3 of the License, or
-// (at your option) any later version.
-//
-// Moodle is distributed in the hope that it will be useful,
-// but WITHOUT ANY WARRANTY; without even the implied warranty of
-// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-// GNU General Public License for more details.
-//
-// You should have received a copy of the GNU General Public License
-// along with Moodle.  If not, see <http://www.gnu.org/licenses/>.
-
-function xmldb_repository_user_upgrade($oldversion) {
-
-    global $CFG, $DB;
-
-    $dbman = $DB->get_manager();
-    $result = true;
-
-/// And upgrade begins here. For each one, you'll need one
-/// block of code similar to the next one. Please, delete
-/// this comment lines once this file start handling proper
-/// upgrade code.
-
-/// if ($result && $oldversion < YYYYMMDD00) { //New version in version.php
-///     $result = result of database_manager methods
-/// }
-
-    return $result;
-}
diff --git a/repository/webdav/db/upgrade.php b/repository/webdav/db/upgrade.php
deleted file mode 100644 (file)
index e402c88..0000000
+++ /dev/null
@@ -1,20 +0,0 @@
-<?php
-
-function xmldb_repository_webdav_upgrade($oldversion) {
-
-    global $CFG, $DB;
-
-    $dbman = $DB->get_manager();
-    $result = true;
-
-/// And upgrade begins here. For each one, you'll need one
-/// block of code similar to the next one. Please, delete
-/// this comment lines once this file start handling proper
-/// upgrade code.
-
-/// if ($result && $oldversion < YYYYMMDD00) { //New version in version.php
-///     $result = result of database_manager methods
-/// }
-
-    return $result;
-}
diff --git a/repository/wikimedia/db/upgrade.php b/repository/wikimedia/db/upgrade.php
deleted file mode 100644 (file)
index 60302a5..0000000
+++ /dev/null
@@ -1,20 +0,0 @@
-<?php
-
-function xmldb_repository_wikimedia_upgrade($oldversion) {
-
-    global $CFG, $DB;
-
-    $dbman = $DB->get_manager();
-    $result = true;
-
-/// And upgrade begins here. For each one, you'll need one
-/// block of code similar to the next one. Please, delete
-/// this comment lines once this file start handling proper
-/// upgrade code.
-
-/// if ($result && $oldversion < YYYYMMDD00) { //New version in version.php
-///     $result = result of database_manager methods
-/// }
-
-    return $result;
-}
diff --git a/repository/youtube/db/upgrade.php b/repository/youtube/db/upgrade.php
deleted file mode 100644 (file)
index 7380860..0000000
+++ /dev/null
@@ -1,20 +0,0 @@
-<?php
-
-function xmldb_repository_youtube_upgrade($oldversion) {
-
-    global $CFG, $DB;
-
-    $dbman = $DB->get_manager();
-    $result = true;
-
-/// And upgrade begins here. For each one, you'll need one
-/// block of code similar to the next one. Please, delete
-/// this comment lines once this file start handling proper
-/// upgrade code.
-
-/// if ($result && $oldversion < YYYYMMDD00) { //New version in version.php
-///     $result = result of database_manager methods
-/// }
-
-    return $result;
-}