fixed error message when empty site full or short name - moved $SITE reinitialisation...

diff --git a/admin/settings.php b/admin/settings.php
index 7b6b739b9e823cbe30e932e71bd374fed4718e98..3c0107d967cefe05b0c0e607455845ad01d8f9b1 100644 (file)
--- a/admin/settings.php
+++ b/admin/settings.php
@@ -78,8 +78,7 @@ if ($data = data_submitted()) {
     } else {
         error(get_string('confirmsesskeybad', 'error'));
     }
-    //reload site
-    $SITE = get_record('course', 'id', $SITE->id);
+    //update $COURSE to match changed $SITE
     $COURSE = clone($SITE);
 }
 

diff --git a/lib/adminlib.php b/lib/adminlib.php
index f5c3cb289dfb73b90b42e571181383491a48e12b..f12edf7739110e67d67f11f8730c1fde2d48dbfe 100644 (file)
--- a/lib/adminlib.php
+++ b/lib/adminlib.php
@@ -1693,7 +1693,10 @@ class admin_setting_sitesettext extends admin_setting_configtext {
         $record->id = $this->id;
         $record->{$this->name} = $data;
         $record->timemodified = time();
-        return (update_record('course', $record) ? '' : get_string('errorsetting', 'admin') . $this->visiblename . '<br />');
+        $status = update_record('course', $record) ? '' : get_string('errorsetting', 'admin') . $this->visiblename . '<br />';
+        //now update $SITE
+        global $SITE;
+        $SITE = get_record('course', 'id', $SITE->id);
     }
 
 }