MDL-20697 removing duplicate error string, fixing get_site() which is now using excep...
authorPetr Skoda <skodak@moodle.org>
Sun, 1 Nov 2009 09:10:09 +0000 (09:10 +0000)
committerPetr Skoda <skodak@moodle.org>
Sun, 1 Nov 2009 09:10:09 +0000 (09:10 +0000)
16 files changed:
auth/ldap/ntlmsso_attempt.php
auth/ldap/ntlmsso_finish.php
backup/backup_check.html
backup/backup_execute.html
backup/backup_form.html
backup/restore_check.html
backup/restore_execute.html
backup/restore_form.html
backup/restore_precheck.html
course/category.php
course/delete.php
course/import/activities/index.php
course/index.php
lang/en_utf8/debug.php
lib/datalib.php
login/index.php

index 3b1aaaa..eb268bb 100644 (file)
@@ -8,9 +8,7 @@ httpsrequired();
 $PAGE->set_url(new moodle_url($CFG->wwwroot.'/auth/ldap/ntlmsso_attempt.php'));
 
 /// Define variables used in page
-if (!$site = get_site()) {
-    print_error("siteisnotdefined");
-}
+$site = get_site();
 
 $authsequence = get_enabled_auth_plugins(true); // auths, in sequence
 if (!in_array('ldap',$authsequence,true)) {
@@ -39,7 +37,3 @@ $msg = '<p>'.get_string('ntlmsso_attempting','auth_ldap').'</p>'
     . ' src="' . $CFG->wwwroot . '/auth/ldap/ntlmsso_magic.php?sesskey='
     . $sesskey . '" />';
 redirect($CFG->wwwroot . '/auth/ldap/ntlmsso_finish.php', $msg, 3);
-
-
-
-?>
index 0356df3..93e7455 100644 (file)
@@ -8,9 +8,7 @@ httpsrequired();
 $PAGE->set_url(new moodle_url($CFG->wwwroot.'/auth/ldap/ntlmsso_finish.php'));
 
 /// Define variables used in page
-if (!$site = get_site()) {
-    print_error("siteisnotdefined", 'debug');
-}
+$site = get_site();
 
 $authsequence = get_enabled_auth_plugins(true); // auths, in sequence
 if (!in_array('ldap',$authsequence,true)) {
@@ -36,4 +34,3 @@ if (!$authplugin->ntlmsso_finish()) {
     redirect($CFG->httpswwwroot . '/login/index.php?authldap_skipntlmsso=1', 
              get_string('ntlmsso_failed','auth_ldap'), 3);
 }
-?>
index d5b0345..0019d53 100644 (file)
@@ -31,9 +31,7 @@
 
 
     //Check site
-    if (!$site = get_site()) {
-        print_error("siteisnotdefined", 'debug');
-    }
+    $site = get_site();
     if ($count == 0) {
         notice("No backupable modules are installed!");
     }
index fd16937..44492e9 100644 (file)
@@ -24,9 +24,7 @@
 
 
     //Check site
-    if (!$site = get_site()) {
-        print_error("siteisnotdefined", 'debug');
-    }
+    $site = get_site();
 
     $preferences = new StdClass;
     backup_fetch_prefs_from_request($preferences,$count,$course);
index 2b08ff5..a4a70ce 100644 (file)
@@ -24,9 +24,7 @@
     }
 
     //Check site
-    if (!$site = get_site()) {
-        print_error("siteisnotdefined", 'debug');
-    }
+    $site = get_site();
 
     //Checks for the required files/functions to backup every mod
     //And check if there is data about it
index 528f398..a00e9cf 100644 (file)
     }
 
     //Check site
-    if (!$site = get_site()) {
-        print_error("siteisnotdefined", 'debug');
-    }
+    $site = get_site();
 
     // Non-cached - get accessinfo
     if (isset($USER->access)) {
index 083cd45..a084185 100644 (file)
@@ -60,9 +60,7 @@
     }
 
     //Check site
-    if (!$site = get_site()) {
-        print_error("siteisnotdefined", 'debug');
-    }
+    $site = get_site();
     $errorstr = '';
 
     $status = restore_execute($restore,$info,$course_header,$errorstr);
index 3e4c903..c443edf 100644 (file)
@@ -39,9 +39,7 @@
     }
 
     //Check site
-    if (!$site = get_site()) {
-        print_error("siteisnotdefined", 'debug');
-    }
+    $site = get_site();
 
     //Checks for the required files/functions to restore every mod
     $count = 0;
index e6cecba..20fcaa5 100644 (file)
@@ -33,9 +33,7 @@
     }
 
     //Check site
-    if (!$site = get_site()) {
-        print_error("siteisnotdefined", 'debug');
-    }
+    $site = get_site();
 
     $errorstr = '';
     if (!empty($SESSION->restore->importing)) {
index 8b47748..1e0ae14 100644 (file)
     $moveto = optional_param('moveto', 0, PARAM_INT);
     $resort = optional_param('resort', 0, PARAM_BOOL);
 
-    if (!$site = get_site()) {
-        print_error('siteisnotdefined', 'debug');
-    }
-
+    $site = get_site();
+    
     if (empty($id)) {
         print_error("unknowcategory");
     }
index 567b3d8..eac3987 100644 (file)
         print_error('cannotdeletecourse');
     }
 
-    if (!$site = get_site()) {
-        print_error("siteisnotdefined", 'debug');
-    }
-
+    $site = get_site();
+    
     $strdeletecourse = get_string("deletecourse");
     $stradministration = get_string("administration");
     $strcategories = get_string("categories");
index 6814aab..63f942c 100644 (file)
@@ -56,9 +56,7 @@ if (! ($course = $DB->get_record("course", array("id"=>$id)))) {
     print_error("invalidcourseid");
 }
 
-if (!$site = get_site()){
-    print_error("siteisnotdefined", 'debug');
-}
+$site = get_site();
 
 require_login($course->id);
 $tocontext = get_context_instance(CONTEXT_COURSE, $id);
index 624efb0..ff82fd5 100644 (file)
@@ -36,9 +36,7 @@ $moveto   = optional_param('moveto',-1,PARAM_INT);
 $moveup   = optional_param('moveup',0,PARAM_INT);
 $movedown = optional_param('movedown',0,PARAM_INT);
 
-if (!$site = get_site()) {
-    print_error('siteisnotdefined', 'debug');
-}
+$site = get_site();
 
 $systemcontext = get_context_instance(CONTEXT_SYSTEM);
 
index 3f7bf2b..8f5f7bc 100644 (file)
@@ -31,7 +31,6 @@ $string['modulenotexist'] = '$a module doesn\'t exist';
 $string['phpvaroff'] = 'The PHP server variable \'$a->name\' should be Off - $a->link';
 $string['phpvaron'] = 'The PHP server variable \'$a->name\' is not turned On - $a->link';
 $string['sessionmissing'] = '$a object missing from session';
-$string['siteisnotdefined'] = 'Site is not defined!';
 $string['sqlrelyonobsoletetable'] = 'This SQL relies on obsolete table(s): $a!  Your code must be fixed by a developer.';
 $string['withoutversion'] = 'Main version.php file is missing, not readable or broken.';
 $string['xmlizeunavailable'] = 'xmlize functions are not available';
index a1fce76..ab02811 100644 (file)
@@ -405,9 +405,7 @@ function get_users_confirmed() {
 /**
  * Returns $course object of the top-level site.
  *
- * @global object
- * @global object
- * @return bool|object A {@link $COURSE} object for the site
+ * @return object A {@link $COURSE} object for the site, exception if not found
  */
 function get_site() {
     global $SITE, $DB;
@@ -419,7 +417,9 @@ function get_site() {
     if ($course = $DB->get_record('course', array('category'=>0))) {
         return $course;
     } else {
-        return false;
+        // course table exists, but the site is not there,
+        // unfortunately there is no automatic way to recover
+        throw new moodle_exception('nosite', 'error');
     }
 }
 
index a8c19e4..0d03fad 100644 (file)
@@ -61,9 +61,7 @@ httpsrequired();
 $PAGE->set_url("$CFG->httpswwwroot/login/index.php");
 
 /// Define variables used in page
-if (!$site = get_site()) {
-    print_error('siteisnotdefined', 'debug');
-}
+$site = get_site();
 
 if (empty($CFG->langmenu)) {
     $langmenu = "";