MDL-9004 - Breadcrumb trail is wrong when editing a quiz question.

2007-03-22 11:46:03 +00:00
parent 803efadffc
commit 79dc9d16be
3 changed files with 78 additions and 83 deletions
@@ -1,95 +1,92 @@
 <?php  // $Id$
 /**
-* Sets up the tabs used by the quiz pages for teachers.
-*
-* @version $Id$
-* @license http://www.gnu.org/copyleft/gpl.html GNU Public License
-* @package quiz
-*/
+ * Sets up the tabs used by the quiz pages based on the users capabilites.
+ *
+ * @author Tim Hunt and others.
+ * @license http://www.gnu.org/copyleft/gpl.html GNU Public License
+ * @package quiz
+ */

-/// This file to be included so we can assume config.php has already been included.
+if (empty($quiz)) {
+    error('You cannot call this script in that way');
+}
+if (!isset($currenttab)) {
+    $currenttab = '';
+}
+if (!isset($cm)) {
+    $cm = get_coursemodule_from_instance('quiz', $quiz->id);
+}
+if (!isset($course)) {
+    $course = get_record('course', 'id', $quiz->course);
+}

-    if (empty($quiz)) {
-        error('You cannot call this script in that way');
-    }
-    if (!isset($currenttab)) {
-        $currenttab = '';
-    }
-    if (!isset($cm)) {
-        $cm = get_coursemodule_from_instance('quiz', $quiz->id);
-    }
-    if (!isset($course)) {
-        $course = get_record('course', 'id', $quiz->course);
+$context = get_context_instance(CONTEXT_MODULE, $cm->id);
+
+$tabs = array();
+$row  = array();
+$inactive = array();
+$activated = array();
+
+if (has_capability('mod/quiz:view', $context)) {
+    $row[] = new tabobject('info', "$CFG->wwwroot/mod/quiz/view.php?q=$quiz->id", get_string('info', 'quiz'));
+}
+if (has_capability('mod/quiz:viewreports', $context)) {
+    $row[] = new tabobject('reports', "$CFG->wwwroot/mod/quiz/report.php?q=$quiz->id", get_string('results', 'quiz'));  
+}
+if (has_capability('mod/quiz:preview', $context)) {
+    $row[] = new tabobject('preview', "$CFG->wwwroot/mod/quiz/attempt.php?q=$quiz->id", get_string('preview', 'quiz'));
+}
+if (has_capability('mod/quiz:manage', $context)) {
+    $row[] = new tabobject('edit', "$CFG->wwwroot/mod/quiz/edit.php?quizid=$quiz->id", get_string('edit'));
+}
+
+if ($currenttab == 'info' && count($row) == 1) {
+    // Don't show only an info tab (e.g. to students).
+} else {
+    $tabs[] = $row;
+}
+
+if ($currenttab == 'reports' and isset($mode)) {
+    $inactive[] = 'reports';
+    $activated[] = 'reports';
+    
+    $allreports = get_list_of_plugins("mod/quiz/report");
+    $reportlist = array ('overview', 'regrade', 'grading', 'analysis');   // Standard reports we want to show first
+
+    foreach ($allreports as $report) {
+        if (!in_array($report, $reportlist)) {
+            $reportlist[] = $report;
+        }
    }

-    $context = get_context_instance(CONTEXT_MODULE, $cm->id);
-    //print_heading(format_string($quiz->name));
-
-    $tabs = array();
    $row  = array();
-    $inactive = array();
-    $activated = array();
-
-    if (has_capability('mod/quiz:view', $context)) {
-        $row[] = new tabobject('info', "$CFG->wwwroot/mod/quiz/view.php?q=$quiz->id", get_string('info', 'quiz'));
-    }
-    if (has_capability('mod/quiz:viewreports', $context)) {
-        $row[] = new tabobject('reports', "$CFG->wwwroot/mod/quiz/report.php?q=$quiz->id", get_string('results', 'quiz'));  
-    }
-    if (has_capability('mod/quiz:preview', $context)) {
-        $row[] = new tabobject('preview', "$CFG->wwwroot/mod/quiz/attempt.php?q=$quiz->id", get_string('preview', 'quiz'));
-    }
-    if (has_capability('mod/quiz:manage', $context)) {
-        $row[] = new tabobject('edit', "$CFG->wwwroot/mod/quiz/edit.php?quizid=$quiz->id", get_string('edit'));
-    }
-
-    if ($currenttab == 'info' && count($row) == 1) {
-        // Don't show only an info tab (e.g. to students).
-    } else {
-        $tabs[] = $row;
-    }
-
-    if ($currenttab == 'reports' and isset($mode)) {
-        $inactive[] = 'reports';
-        $activated[] = 'reports';
-        
-        $allreports = get_list_of_plugins("mod/quiz/report");
-        $reportlist = array ('overview', 'regrade', 'grading', 'analysis');   // Standard reports we want to show first
-
-        foreach ($allreports as $report) {
-            if (!in_array($report, $reportlist)) {
-                $reportlist[] = $report;
-            }
+    $currenttab = '';
+    foreach ($reportlist as $report) {
+        $row[] = new tabobject($report, "$CFG->wwwroot/mod/quiz/report.php?q=$quiz->id&amp;mode=$report",
+                                get_string($report, 'quiz_'.$report));
+        if ($report == $mode) {
+            $currenttab = $report;
        }
-
-        $row  = array();
-        $currenttab = '';
-        foreach ($reportlist as $report) {
-            $row[] = new tabobject($report, "$CFG->wwwroot/mod/quiz/report.php?q=$quiz->id&amp;mode=$report",
-                                    get_string($report, 'quiz_'.$report));
-            if ($report == $mode) {
-                $currenttab = $report;
-            }
-        }
-        $tabs[] = $row;
    }
+    $tabs[] = $row;
+}

-    if ($currenttab == 'edit' and isset($mode)) {
-        $inactive[] = 'edit';
-        $activated[] = 'edit';
+if ($currenttab == 'edit' and isset($mode)) {
+    $inactive[] = 'edit';
+    $activated[] = 'edit';

-        $row  = array();
-        $currenttab = $mode;
+    $row  = array();
+    $currenttab = $mode;

-        $strquizzes = get_string('modulenameplural', 'quiz');
-        $strquiz = get_string('modulename', 'quiz');
-        $streditingquiz = get_string("editinga", "moodle", $strquiz);
-        $strupdate = get_string('updatethis', 'moodle', $strquiz);
-        $row[] = new tabobject('editq', "$CFG->wwwroot/mod/quiz/edit.php?quizid=$quiz->id", $strquiz, $streditingquiz);
-        questionbank_navigation_tabs($row, $context, $course->id);
-        $tabs[] = $row;
-    }
+    $strquizzes = get_string('modulenameplural', 'quiz');
+    $strquiz = get_string('modulename', 'quiz');
+    $streditingquiz = get_string("editinga", "moodle", $strquiz);
+    $strupdate = get_string('updatethis', 'moodle', $strquiz);
+    $row[] = new tabobject('editq', "$CFG->wwwroot/mod/quiz/edit.php?quizid=$quiz->id", $strquiz, $streditingquiz);
+    questionbank_navigation_tabs($row, $context, $course->id);
+    $tabs[] = $row;
+}

-    print_tabs($tabs, $currenttab, $inactive, $activated);
+print_tabs($tabs, $currenttab, $inactive, $activated);

 ?>
@@ -31,10 +31,8 @@
    require_login($course->id, false);
  
    $SESSION->returnurl = $FULLME;
-    $SESSION->fromurl = $FULLME;

    // Print basic page layout.
-
    $streditingquestions = get_string('editquestions', "quiz");

    // TODO: generalise this to any activity
@@ -47,9 +47,9 @@
    } else {
        $sortorder = isset($SESSION->questionsortorder) ? $SESSION->questionsortorder : 'qtype, name ASC';
    }
+    $SESSION->fromurl = $FULLME;

 /// Now, check for commands on this page and modify variables as necessary
-
    if (isset($_REQUEST['move']) and confirm_sesskey()) { /// Move selected questions to new category
        $tocategoryid = required_param('category', PARAM_INT);
        if (!$tocategory = get_record('question_categories', 'id', $tocategoryid)) {