Fix brokness due to api changes in latest 1.9 master
authorDan Poltawski <dan.poltawski@lancaster.ac.uk>
Tue, 3 Feb 2009 09:36:27 +0000 (09:36 +0000)
committerDan Poltawski <dan.poltawski@lancaster.ac.uk>
Tue, 3 Feb 2009 09:36:27 +0000 (09:36 +0000)
As reported on http://moodle.org/mod/forum/discuss.php?d=115361&parent=506127

local/cleo_mis/cleo_mis_lib.php
local/cleo_mis/cleo_mis_listclass.php
local/cleo_mis/cleo_mis_reset.php
local/cleo_mis/cleo_mis_uploadclass.php
local/cleo_mis/cleo_mis_uploaduser.php

index b0b8fc3..09ffbb2 100644 (file)
@@ -332,9 +332,8 @@ function cleo_mis_reset_classes(){
  * Display form to set idnumber lock value if it isn't configrued to
  * be locked
  *
- * @param mixed $adminroot - adminroot so admin footer is diplayed correctly
  */
-function cleo_mis_check_idnumber_config($adminroot){
+function cleo_mis_check_idnumber_config(){
     global $CFG;
     require_once('cleo_mis_idnum_form.php');
 
@@ -353,7 +352,7 @@ function cleo_mis_check_idnumber_config($adminroot){
         }else{
             print_simple_box( get_string('idnumberexplanation', 'cleo_mis') );
             $mform->display();
-            admin_externalpage_print_footer($adminroot);
+            admin_externalpage_print_footer();
             exit;
         }
     }
@@ -364,23 +363,20 @@ function cleo_mis_check_idnumber_config($adminroot){
  * setup adminpage, check capability, setup idnumber lock value
  * @param string $pagename - name of the adminpage as specified in admin menu
  * @param string $title - title of page
- * @return mixed $adminroot the adminroot for use by admin functions
  */
 function cleo_mis_print_header($pagename, $title){
-    $adminroot = admin_get_root();
 
-    admin_externalpage_setup($pagename, $adminroot);
+    admin_externalpage_setup($pagename);
 
     // This capability check isn't *entirely accurate*, however I think its the sort of
     // permission we are after
     require_capability('moodle/site:uploadusers', get_context_instance(CONTEXT_SYSTEM));
-    admin_externalpage_print_header($adminroot);
+    admin_externalpage_print_header();
 
-    cleo_mis_check_idnumber_config($adminroot);
+    cleo_mis_check_idnumber_config();
 
     print_heading($title);
 
-    return $adminroot;
 }
 
 /**
index 95d67f6..633c692 100644 (file)
@@ -17,11 +17,11 @@ require_js('yui_connection');
 require_js($CFG->wwwroot . '/local/cleo_mis/cleo_mis_ajax.js');
 
 
-$adminroot = cleo_mis_print_header('cleo_mis_listclasses', get_string('listpagetitle', 'cleo_mis') );
+cleo_mis_print_header('cleo_mis_listclasses', get_string('listpagetitle', 'cleo_mis') );
 
 print_classes_infotable();
 
-admin_externalpage_print_footer($adminroot);
+admin_externalpage_print_footer();
 
 function print_classes_infotable(){
     global $CFG;
index 9f7c005..2de4b48 100644 (file)
@@ -16,7 +16,7 @@ require_once(dirname(__FILE__) . '/cleo_mis_lib.php');
 require_once(dirname(__FILE__) . '/cleo_mis_reset_form.php');
 
 
-$adminroot = cleo_mis_print_header('cleo_mis_reset', get_string('resetpagetitle', 'cleo_mis') );
+cleo_mis_print_header('cleo_mis_reset', get_string('resetpagetitle', 'cleo_mis') );
 
 $userform =  new cleo_mis_reset_user_form();
 $classform =  new cleo_mis_reset_class_form();
@@ -132,5 +132,5 @@ if(! empty($subjects) ){
     print_box( get_string('noclasses', 'cleo_mis') );
 }
 
-admin_externalpage_print_footer($adminroot);
+admin_externalpage_print_footer();
 ?>
index e8cbe36..a4213b4 100644 (file)
@@ -17,7 +17,7 @@ require_once(dirname(__FILE__) . '/cleo_mis_lib.php');
 
 
 
-$adminroot = cleo_mis_print_header('cleo_mis_uploadclasses', get_string('uploadclassestitle', 'cleo_mis'));
+cleo_mis_print_header('cleo_mis_uploadclasses', get_string('uploadclassestitle', 'cleo_mis'));
 
 $checklistform = new cleo_mis_checklist_form();
 $checklistform->set_data( array('notedpasswords' => 1) );
@@ -30,7 +30,7 @@ if(! ( $checklistform->get_data() or $classdata) ){
     // need to have users from MIS system already uploaded
 
     $checklistform->display();
-    admin_externalpage_print_footer($adminroot);
+    admin_externalpage_print_footer();
     exit;
 }
 
@@ -136,7 +136,7 @@ if ( $classdata ) {
 
     print_upload_problems($errored_lines);
 
-    admin_externalpage_print_footer($adminroot);
+    admin_externalpage_print_footer();
     exit;
 }
 
@@ -144,6 +144,6 @@ if ( $classdata ) {
 print_simple_box( get_string('uploadclassexplanation', 'cleo_mis') );
 $uploadform->display();
 
-admin_externalpage_print_footer($adminroot);
+admin_externalpage_print_footer();
 
 ?>
index 985a987..ec53087 100644 (file)
@@ -15,7 +15,7 @@ require_once(dirname(__FILE__) . '/cleo_mis_checklist_form.php');
 require_once(dirname(__FILE__) . '/cleo_mis_lib.php');
 require_once($CFG->libdir.'/adminlib.php');
 
-$adminroot = cleo_mis_print_header('cleo_mis_uploadusers', get_string('uploaduserfrommis', 'cleo_mis'));
+cleo_mis_print_header('cleo_mis_uploadusers', get_string('uploaduserfrommis', 'cleo_mis'));
 
 if(! $adminuser = get_admin() ){
     error("Could not find site admin");
@@ -191,7 +191,7 @@ if ( $formdata = $mform->get_data() ) {
 
     print_upload_problems($errored_lines);
 
-    admin_externalpage_print_footer($adminroot);
+    admin_externalpage_print_footer();
     exit;
 }
 
@@ -199,7 +199,7 @@ if ( $formdata = $mform->get_data() ) {
 print_simple_box( get_string('uploaduserexplanation', 'cleo_mis') );
 $mform->display();
 
-admin_externalpage_print_footer($adminroot);
+admin_externalpage_print_footer();
 
 
 /*