MDL-39910 SCORM: set correct site level defaults.
authorDan Marsden <dan@danmarsden.com>
Thu, 26 Sep 2013 09:03:00 +0000 (21:03 +1200)
committerDan Marsden <dan@danmarsden.com>
Thu, 26 Sep 2013 22:49:54 +0000 (10:49 +1200)
mod/scorm/db/upgrade.php

index 13ac724..f2171b8 100644 (file)
@@ -130,6 +130,7 @@ function xmldb_scorm_upgrade($oldversion) {
     }
 
     if ($oldversion < 2013090100) {
+        global $CFG;
         $table = new xmldb_table('scorm');
 
         $field = new xmldb_field('nav', XMLDB_TYPE_INTEGER, '1', XMLDB_UNSIGNED, true, null, 1, 'hidetoc');
@@ -160,14 +161,17 @@ function xmldb_scorm_upgrade($oldversion) {
             $dbman->drop_field($table, $field);
         }
 
-        $params = array('plugin' => 'scorm', 'name' => 'hidenav');
-        if ($DB->record_exists('config_plugins', $params)) {
-            $DB->delete_records('config_plugins', $params);
-        }
-        $params = array('plugin' => 'scorm', 'name' => 'hidenav_adv');
-        if ($DB->record_exists('config_plugins', $params)) {
-            $DB->delete_records('config_plugins', $params);
+        $hide = get_config('scorm', 'hidenav');
+        unset_config('hidenav', 'scorm');
+        if (!empty($hide)) {
+            require_once($CFG->dirroot . '/mod/scorm/lib.php');
+            set_config('nav', SCORM_NAV_DISABLED, 'scorm');
         }
+
+        $hideadv = get_config('scorm', 'hidenav_adv');
+        unset_config('hidenav_adv', 'scorm');
+        set_config('nav_adv', $hideadv, 'scorm');
+
         upgrade_mod_savepoint(true, 2013090100, 'scorm');
     }