From 6dac764d182010deaf1e81828bbdb85683581648 Mon Sep 17 00:00:00 2001
From: nicolasconnault <nicolasconnault>
Date: Mon, 21 Jan 2008 13:40:47 +0000
Subject: [PATCH] MDL-13050 Reverting, it's not working out as well as planned

---
 mod/assignment/lib.php       | 21 +--------------------
 mod/exercise/submissions.php | 20 +-------------------
 mod/glossary/view.php        | 23 ++---------------------
 mod/hotpot/report.php        | 29 ++---------------------------
 mod/quiz/report/default.php  | 16 +---------------
 mod/workshop/view.php        | 25 -------------------------
 6 files changed, 7 insertions(+), 127 deletions(-)

diff --git a/mod/assignment/lib.php b/mod/assignment/lib.php
index be5f1ef4c7..c8a9c77350 100644
--- a/mod/assignment/lib.php
+++ b/mod/assignment/lib.php
@@ -1020,26 +1020,7 @@ class assignment_base {
 
         add_to_log($course->id, 'assignment', 'view submission', 'submissions.php?id='.$this->assignment->id, $this->assignment->id, $this->cm->id);
 
-        $navlinks = array();
-        $navlinks[] = array(
-                'name' => get_string('grades'),
-                'link' => $CFG->wwwroot . '/grade/report/index.php?id=' . $cm->course,
-                'type' => 'link');
-        $navlinks[] = array(
-                'name' => get_string('modulenameplural', $cm->modname),
-                'link' => $CFG->wwwroot . '/mod/' . $cm->modname . '/index.php?id=' . $cm->course,
-                'type' => 'activity');
-        $navlinks[] = array(
-                'name' => format_string($cm->name),
-                'link' => $CFG->wwwroot . '/mod/' . $cm->modname . '/view.php?id=' . $cm->id,
-                'type' => 'activityinstance');
-        $navlinks[] = array(
-                'name' => $this->strsubmissions,
-                'link' => '',
-                'type' => 'link');
-        
-        $navigation = build_navigation($navlinks);
-
+        $navigation = build_navigation($this->strsubmissions, $this->cm);
         print_header_simple(format_string($this->assignment->name,true), "", $navigation,
                 '', '', true, update_module_button($cm->id, $course->id, $this->strassignment), navmenu($course, $cm));
 
diff --git a/mod/exercise/submissions.php b/mod/exercise/submissions.php
index 14fb634770..68d5bb4d9b 100644
--- a/mod/exercise/submissions.php
+++ b/mod/exercise/submissions.php
@@ -50,25 +50,7 @@
     $strsubmissions = get_string("submissions", "exercise");
 
     // ... print the header and...
-    $navlinks = array();
-    $navlinks[] = array(
-            'name' => get_string('grades'),
-            'link' => $CFG->wwwroot . '/grade/report/index.php?id=' . $cm->course,
-            'type' => 'link');
-    $navlinks[] = array(
-            'name' => get_string('modulenameplural', $cm->modname),
-            'link' => $CFG->wwwroot . '/mod/' . $cm->modname . '/index.php?id=' . $cm->course,
-            'type' => 'activity');
-    $navlinks[] = array(
-            'name' => format_string($cm->name),
-            'link' => $CFG->wwwroot . '/mod/' . $cm->modname . '/view.php?id=' . $cm->id,
-            'type' => 'activityinstance');
-    $navlinks[] = array(
-            'name' => $strsubmissions,
-            'link' => '',
-            'type' => 'title');
-    
-    $navigation = build_navigation($navlinks);
+    $navigation = build_navigation($strsubmissions, $cm);
     print_header_simple(format_string($exercise->name), "", $navigation,
                   "", "", true);
 
diff --git a/mod/glossary/view.php b/mod/glossary/view.php
index 3dadb7c6d6..af508f2bca 100644
--- a/mod/glossary/view.php
+++ b/mod/glossary/view.php
@@ -225,35 +225,16 @@
     $strwaitingapproval = get_string('waitingapproval', 'glossary');
 
 /// If we are in approval mode, prit special header
-    $navlinks = array();
-    $navlinks[] = array(
-            'name' => get_string('grades'),
-            'link' => $CFG->wwwroot . '/grade/report/index.php?id=' . $cm->course,
-            'type' => 'link');
-    $navlinks[] = array(
-            'name' => get_string('modulenameplural', $cm->modname),
-            'link' => $CFG->wwwroot . '/mod/' . $cm->modname . '/index.php?id=' . $cm->course,
-            'type' => 'activity');
-    $navlinks[] = array(
-            'name' => format_string($cm->name),
-            'link' => $CFG->wwwroot . '/mod/' . $cm->modname . '/view.php?id=' . $cm->id,
-            'type' => 'activityinstance');
-    
     if ($tab == GLOSSARY_APPROVAL_VIEW) {
         require_capability('mod/glossary:approve', $context);
 
-        $navlinks[] = array(
-                'name' => $strwaitingapproval,
-                'link' => '',
-                'type' => 'title');
-        
-        $navigation = build_navigation($navlinks);
+        $navigation = build_navigation($strwaitingapproval, $cm);
         print_header_simple(format_string($glossary->name), "", $navigation, "", "", true,
             update_module_button($cm->id, $course->id, $strglossary), navmenu($course, $cm));
 
         print_heading($strwaitingapproval);
     } else { /// Print standard header
-        $navigation = build_navigation($navlinks);
+        $navigation = build_navigation('', $cm);
         print_header_simple(format_string($glossary->name), "", $navigation, "", "", true,
             update_module_button($cm->id, $course->id, $strglossary), navmenu($course, $cm));
     }
diff --git a/mod/hotpot/report.php b/mod/hotpot/report.php
index db8c0fc65c..8b9029f433 100644
--- a/mod/hotpot/report.php
+++ b/mod/hotpot/report.php
@@ -408,7 +408,6 @@ function hotpot_delete_selected_attempts(&$hotpot, $del) {
 /// report selector menus
 
 function hotpot_print_report_heading(&$course, &$cm, &$hotpot, &$mode) {
-    global $CFG;
     $strmodulenameplural = get_string("modulenameplural", "hotpot");
     $strmodulename  = get_string("modulename", "hotpot");
 
@@ -416,22 +415,6 @@ function hotpot_print_report_heading(&$course, &$cm, &$hotpot, &$mode) {
     $heading = $course->fullname;
     
     $modulecontext = get_context_instance(CONTEXT_MODULE, $cm->id);
-    
-    $navlinks = array();
-    $navlinks[] = array(
-            'name' => get_string('grades'),
-            'link' => $CFG->wwwroot . '/grade/report/index.php?id=' . $cm->course,
-            'type' => 'link');
-    $navlinks[] = array(
-            'name' => get_string('modulenameplural', $cm->modname),
-            'link' => $CFG->wwwroot . '/mod/' . $cm->modname . '/index.php?id=' . $cm->course,
-            'type' => 'activity');
-    $navlinks[] = array(
-            'name' => format_string($cm->name),
-            'link' => $CFG->wwwroot . '/mod/' . $cm->modname . '/view.php?id=' . $cm->id,
-            'type' => 'activityinstance');
-    
-
     if (has_capability('mod/hotpot:viewreport',$modulecontext)) {
         if ($mode=='overview' || $mode=='simplestat' || $mode=='fullstat') {
             $module = "quiz";
@@ -439,18 +422,10 @@ function hotpot_print_report_heading(&$course, &$cm, &$hotpot, &$mode) {
             $module = "hotpot";
         }
 
-        $navlinks[] = array(
-                'name' => get_string("report$mode", $module),
-                'link' => '',
-                'type' => 'title');
+        $navigation = build_navigation(get_string("report$mode", $module), $cm);
     } else {
-        $navlinks[] = array(
-                'name' => get_string("report", "quiz"),
-                'link' => '',
-                'type' => 'title');
+        $navigation = build_navigation(get_string("report", "quiz"), $cm);
     }
-    
-    $navigation = build_navigation($navlinks);
 
     $button = update_module_button($cm->id, $course->id, $strmodulename);
     print_header($title, $heading, $navigation, "", "", true, $button, navmenu($course, $cm));
diff --git a/mod/quiz/report/default.php b/mod/quiz/report/default.php
index 19545c9300..a5a05c07aa 100644
--- a/mod/quiz/report/default.php
+++ b/mod/quiz/report/default.php
@@ -27,21 +27,7 @@ class quiz_default_report {
         $strquizzes = get_string("modulenameplural", "quiz");
         $strquiz  = get_string("modulename", "quiz");
     /// Print the page header
-        $navlinks = array();
-        $navlinks[] = array(
-                'name' => get_string('grades'),
-                'link' => $CFG->wwwroot . '/grade/report/index.php?id=' . $cm->course,
-                'type' => 'link');
-        $navlinks[] = array(
-                'name' => get_string('modulenameplural', $cm->modname),
-                'link' => $CFG->wwwroot . '/mod/' . $cm->modname . '/index.php?id=' . $cm->course,
-                'type' => 'activity');
-        $navlinks[] = array(
-                'name' => format_string($cm->name),
-                'link' => $CFG->wwwroot . '/mod/' . $cm->modname . '/view.php?id=' . $cm->id,
-                'type' => 'activityinstance');
-        
-        $navigation = build_navigation($navlinks);
+        $navigation = build_navigation('', $cm);
         
         print_header_simple(format_string($quiz->name), "", $navigation,
                      '', $meta, true, update_module_button($cm->id, $course->id, $strquiz), navmenu($course, $cm));
diff --git a/mod/workshop/view.php b/mod/workshop/view.php
index a9ce44aaeb..49266aa433 100644
--- a/mod/workshop/view.php
+++ b/mod/workshop/view.php
@@ -68,32 +68,7 @@
     }
 
     // ...display header...
-
-    if ($action == 'teachersview') {
-        $navlinks = array();
-        $navlinks[] = array(
-                'name' => get_string('grades'),
-                'link' => $CFG->wwwroot . '/grade/report/index.php?id=' . $cm->course,
-                'type' => 'link');
-        $navlinks[] = array(
-                'name' => get_string('modulenameplural', $cm->modname),
-                'link' => $CFG->wwwroot . '/mod/' . $cm->modname . '/index.php?id=' . $cm->course,
-                'type' => 'activity');
-        $navlinks[] = array(
-                'name' => format_string($cm->name),
-                'link' => $CFG->wwwroot . '/mod/' . $cm->modname . '/view.php?id=' . $cm->id,
-                'type' => 'activityinstance');
-        $navlinks[] = array(
-                'name' => $action,
-                'link' => '',
-                'type' => 'title');
-        
-
-        $navigation = build_navigation($navlinks);    
-    } else {
     $navigation = build_navigation($action, $cm);    
-    }
-
     print_header_simple(format_string($workshop->name), "", $navigation,
                   "", "", true, update_module_button($cm->id, $course->id, $strworkshop), navmenu($course, $cm));
 
-- 
2.39.5