From: samhemelryk Date: Mon, 7 Sep 2009 06:04:46 +0000 (+0000) Subject: gradebook MDL-19797 Updated print_header and build_navigation to OUTPUT and PAGE... X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=f3df5e144476cbf3d6675644dffc62c7d2b3ee1d;p=moodle.git gradebook MDL-19797 Updated print_header and build_navigation to OUTPUT and PAGE equivalents --- diff --git a/grade/edit/outcome/index.php b/grade/edit/outcome/index.php index 10238ffce4..3550e5ccc3 100644 --- a/grade/edit/outcome/index.php +++ b/grade/edit/outcome/index.php @@ -104,7 +104,8 @@ switch ($action) { $deleteconfirmed = optional_param('deleteconfirmed', 0, PARAM_BOOL); if(!$deleteconfirmed){ - print_header(get_string('outcomedelete', 'grades')); + $PAGE->set_title(get_string('outcomedelete', 'grades')); + echo $OUTPUT->header(); $confirmurl = new moodle_url('index.php', array( 'id' => $courseid, 'outcomeid' => $outcome->id, 'action'=> 'delete', diff --git a/grade/edit/scale/index.php b/grade/edit/scale/index.php index faf5a9ca30..d169e02333 100644 --- a/grade/edit/scale/index.php +++ b/grade/edit/scale/index.php @@ -73,7 +73,8 @@ switch ($action) { $deleteconfirmed = optional_param('deleteconfirmed', 0, PARAM_BOOL); if (!$deleteconfirmed) { - print_header(get_string('scaledelete', 'grades')); + $PAGE->set_title(get_string('scaledelete', 'grades')); + echo $OUTPUT->header(); $confirmurl = new moodle_url('index.php', array( 'id' => $courseid, 'scaleid' => $scale->id, 'action'=> 'delete', diff --git a/grade/export/key.php b/grade/export/key.php index 996184b413..33dc58395c 100644 --- a/grade/export/key.php +++ b/grade/export/key.php @@ -71,7 +71,9 @@ $returnurl = $CFG->wwwroot.'/grade/export/keymanager.php?id='.$course->id; if ($id and $delete) { if (!$confirm) { - print_header(get_string('deleteselectedkey'), get_string('deleteselectedkey')); + $PAGE->set_title(get_string('deleteselectedkey')); + $PAGE->set_heading(get_string('deleteselectedkey')); + echo $OUTPUT->header(); $optionsyes = array('id'=>$id, 'delete'=>1, 'courseid'=>$courseid, 'sesskey'=>sesskey(), 'confirm'=>1); $optionsno = array('id'=>$courseid); $formcontinue = html_form::make_button('key.php', $optionsyes, get_string('yes'), 'get'); @@ -117,14 +119,14 @@ if ($id) { $strheading = get_string('createuserkey', 'userkey'); } - -$navlinks = array(array('name'=>$strgrades, 'link'=>$CFG->wwwroot.'/grade/index.php?id='.$courseid, 'type'=>'misc'), - array('name'=>$strkeys, 'link'=>$CFG->wwwroot.'/grade/export/keymanager.php?id='.$courseid, 'type'=>'misc'), - array('name'=>$strheading, 'link'=>'', 'type'=>'misc')); -$navigation = build_navigation($navlinks); +$PAGE->navbar->add($strgrades, new moodle_url($CFG->wwwroot.'/grade/index.php', array('id'=>$courseid))); +$PAGE->navbar->add($strkeys, new moodle_url($CFG->wwwroot.'/grade/export/keymanager.php', array('id'=>$courseid))); +$PAGE->navbar->add($strheading); /// Print header -print_header_simple($strkeys, ': '.$strkeys, $navigation, '', '', true, '', navmenu($course)); +$PAGE->set_title($strkeys); +$PAGE->set_heading(': '.$strkeys); +echo $OUTPUT->header(); $editform->display(); echo $OUTPUT->footer(); diff --git a/grade/import/key.php b/grade/import/key.php index d0762f1b5f..93f3b35869 100644 --- a/grade/import/key.php +++ b/grade/import/key.php @@ -71,7 +71,9 @@ $returnurl = $CFG->wwwroot.'/grade/import/keymanager.php?id='.$course->id; if ($id and $delete) { if (!$confirm) { - print_header(get_string('deleteselectedkey'), get_string('deleteselectedkey')); + $PAGE->set_title(get_string('deleteselectedkey')); + $PAGE->set_heading(get_string('deleteselectedkey')); + echo $OUTPUT->header(); $optionsyes = array('id'=>$id, 'delete'=>1, 'courseid'=>$courseid, 'sesskey'=>sesskey(), 'confirm'=>1); $optionsno = array('id'=>$courseid); $formcontinue = html_form::make_button('key.php', $optionsyes, get_string('yes'), 'get'); @@ -117,14 +119,14 @@ if ($id) { $strheading = get_string('createuserkey', 'userkey'); } - -$navlinks = array(array('name'=>$strgrades, 'link'=>$CFG->wwwroot.'/grade/index.php?id='.$courseid, 'type'=>'misc'), - array('name'=>$strkeys, 'link'=>$CFG->wwwroot.'/grade/import/keymanager.php?id='.$courseid, 'type'=>'misc'), - array('name'=>$strheading, 'link'=>'', 'type'=>'misc')); -$navigation = build_navigation($navlinks); +$PAGE->navbar->add($strgrades, new moodle_url($CFG->wwwroot.'/grade/index.php', array('id'=>$courseid))); +$PAGE->navbar->add($strkeys, new moodle_url($CFG->wwwroot.'/grade/import/keymanager.php', array('id'=>$courseid))); +$PAGE->navbar->add($strheading); /// Print header -print_header_simple($strkeys, ': '.$strkeys, $navigation, '', '', true, '', navmenu($course)); +$PAGE->set_title($strkeys); +$PAGE->set_heading(': '.$strkeys); +echo $OUTPUT->header(); $editform->display(); echo $OUTPUT->footer(); diff --git a/grade/lib.php b/grade/lib.php index 149c92b6ae..c772da6c64 100644 --- a/grade/lib.php +++ b/grade/lib.php @@ -812,7 +812,7 @@ class grade_plugin_info { function print_grade_page_head($courseid, $active_type, $active_plugin=null, $heading = false, $return=false, $buttons=false, $extracss=array()) { - global $CFG, $COURSE, $OUTPUT; + global $CFG, $COURSE, $OUTPUT, $PAGE; $strgrades = get_string('grades'); $plugin_info = grade_get_plugin_info($courseid, $active_type, $active_plugin); @@ -820,7 +820,6 @@ function print_grade_page_head($courseid, $active_type, $active_plugin=null, $stractive_plugin = ($active_plugin) ? $plugin_info['strings']['active_plugin_str'] : $heading; $stractive_type = $plugin_info['strings'][$active_type]; - $navlinks = array(); $first_link = ''; if ($active_type == 'settings' && $active_plugin != 'coursesettings') { @@ -837,9 +836,7 @@ function print_grade_page_head($courseid, $active_type, $active_plugin=null, $CFG->stylesheets[] = $css_url; } - $navlinks[] = array('name' => $strgrades, - 'link' => $first_link, - 'type' => 'misc'); + $PAGE->navbar->add($strgrades, $first_link); $active_type_link = ''; @@ -849,24 +846,27 @@ function print_grade_page_head($courseid, $active_type, $active_plugin=null, if (!empty($plugin_info[$active_type]->parent->link)) { $active_type_link = $plugin_info[$active_type]->parent->link; - $navlinks[] = array('name' => $stractive_type, 'link' => $active_type_link, 'type' => 'misc'); + $PAGE->navbar->add($stractive_type, $active_type_link); } if (empty($plugin_info[$active_type]->id)) { - $navlinks[] = array('name' => $stractive_type, 'link' => $active_type_link, 'type' => 'misc'); + $PAGE->navbar->add($stractive_type, $active_type_link); } - $navlinks[] = array('name' => $stractive_plugin, 'link' => null, 'type' => 'misc'); - - $navigation = build_navigation($navlinks); - + $PAGE->navbar->add($stractive_plugin); + $title = ': ' . $stractive_plugin; if (empty($plugin_info[$active_type]->id) || !empty($plugin_info[$active_type]->parent)) { $title = ': ' . $stractive_type . ': ' . $stractive_plugin; } - $returnval = print_header_simple($strgrades . ': ' . $stractive_type, $title, $navigation, '', - '', true, $buttons, navmenu($COURSE), false, '', $return); + $PAGE->set_title($strgrades . ': ' . $stractive_type); + $PAGE->set_heading($title); + $PAGE->set_button($buttons); + $returnval = $OUTPUT->header(); + if (!$return) { + echo $returnval; + } // Guess heading if not given explicitly if (!$heading) { @@ -1105,7 +1105,7 @@ class grade_plugin_return { * @return string */ function grade_build_nav($path, $pagename=null, $id=null) { - global $CFG, $COURSE; + global $CFG, $COURSE, $PAGE; $strgrades = get_string('grades', 'grades'); @@ -1119,23 +1119,20 @@ function grade_build_nav($path, $pagename=null, $id=null) { $path_elements_count = count($path_elements); // First link is always 'grade' - $navlinks = array(); - $navlinks[] = array('name' => $strgrades, - 'link' => $CFG->wwwroot.'/grade/index.php?id='.$COURSE->id, - 'type' => 'misc'); + $PAGE->navbar->add($strgrades, new moodle_url($CFG->wwwroot.'/grade/index.php', array('id'=>$COURSE->id))); - $link = ''; + $link = null; $numberofelements = 3; // Prepare URL params string - $id_string = '?'; + $linkparams = array(); if (!is_null($id)) { if (is_array($id)) { foreach ($id as $idkey => $idvalue) { - $id_string .= "$idkey=$idvalue&"; + $linkparams[$idkey] = $idvalue; } } else { - $id_string .= "id=$id"; + $linkparams['id'] = $id; } } @@ -1160,7 +1157,7 @@ function grade_build_nav($path, $pagename=null, $id=null) { case 'report': // $id is required for this link. Do not print it if $id isn't given if (!is_null($id)) { - $link = $CFG->wwwroot . '/grade/report/index.php' . $id_string; + $link = new moodle_url($CFG->wwwroot.'/grade/report/index.php', $linkparams); } if ($path_elements[2] == 'grader') { @@ -1174,8 +1171,7 @@ function grade_build_nav($path, $pagename=null, $id=null) { " as the second path element after 'grade'."); return false; } - - $navlinks[] = array('name' => get_string($path_elements[1], 'grades'), 'link' => $link, 'type' => 'misc'); + $PAGE->navbar->add(get_string($path_elements[1], 'grades'), $link); // Third level links if (empty($pagename)) { @@ -1184,21 +1180,17 @@ function grade_build_nav($path, $pagename=null, $id=null) { switch ($numberofelements) { case 3: - $navlinks[] = array('name' => $pagename, 'link' => $link, 'type' => 'misc'); + $PAGE->navbar->add($pagename, $link); break; case 4: - if ($path_elements[2] == 'grader' AND $path_elements[3] != 'index.php') { - $navlinks[] = array('name' => get_string('modulename', 'gradereport_grader'), - 'link' => "$CFG->wwwroot/grade/report/grader/index.php$id_string", - 'type' => 'misc'); + $PAGE->navbar->add(get_string('modulename', 'gradereport_grader'), new moodle_url($CFG->wwwroot.'/grade/report/grader/index.php', $linkparams)); } - $navlinks[] = array('name' => $pagename, 'link' => '', 'type' => 'misc'); + $PAGE->navbar->add($pagename); break; } - $navigation = build_navigation($navlinks); - return $navigation; + return ''; } /**