After doing a quiz that has been shuffled, present the feedback in the

same order.  Note that shuffled ANSWERS may still appear reshuffled.

When viewing reports, all shuffling is switched off so that quizzes
always appear in the default order (for easier comparison).

Also a few tweaks, robustness etc

diff --git a/mod/quiz/attempt.php b/mod/quiz/attempt.php
index 921b4247e87ef733ae5cbab7401c56bffae3ca49..2db51c8fc896e53eebc90988b5a0d21c4bff16d1 100644 (file)
--- a/mod/quiz/attempt.php
+++ b/mod/quiz/attempt.php
@@ -78,6 +78,8 @@
 
         $rawanswers = (array)$rawanswers;
 
+        $shuffleorder = NULL;
+
         unset($rawanswers["q"]);  // quiz id
         if (! count($rawanswers)) {
             print_heading(get_string("noanswers", "quiz"));
@@ -116,6 +118,8 @@
                 } else {
                     error("Answer received for non-existent question ($key)!");
                 }
+            } else if ($key == "shuffleorder") {
+                $shuffleorder = explode(",", $value);   // Actual order questions were given in
             }
         }
 
@@ -142,7 +146,9 @@
         print_continue("view.php?id=$cm->id");
 
         if ($quiz->feedback) {
-            quiz_print_quiz_questions($quiz, $result, $questions);
+            $quiz->shufflequestions = false;
+            $quiz->shuffleanswers = false;
+            quiz_print_quiz_questions($quiz, $result, $questions, $shuffleorder);
             print_continue("view.php?id=$cm->id");
         }
 

diff --git a/mod/quiz/lib.php b/mod/quiz/lib.php
index a2e1acdbf78a7a68a8d493eb510a626a8d0f6b39..26326c3a08528c8f191c5a2fca430c3a99d00934 100644 (file)
--- a/mod/quiz/lib.php
+++ b/mod/quiz/lib.php
@@ -565,14 +565,16 @@ function quiz_print_question($number, $question, $grade, $courseid,
            if (!$subquestions = get_records_list("quiz_match_sub", "id", $options->subquestions)) {
                notify("Error: Missing subquestions for this question!");
            }
-           if ($shuffleanswers) {
-               $subquestions = draw_rand_array($subquestions, count($subquestions));
+           if (!empty($question->questiontext)) {
+               echo text_to_html($question->questiontext);
            }
-           echo text_to_html($question->questiontext);
-           if ($question->image) {
+           if (!empty($question->image)) {
                print_file_picture($question->image, $courseid, QUIZ_PICTURE_DEFAULT_HEIGHT);
            }
 
+           if ($shuffleanswers) {
+               $subquestions = draw_rand_array($subquestions, count($subquestions));
+           }
            foreach ($subquestions as $subquestion) {
                $answers[$subquestion->id] = $subquestion->answertext;
            }
@@ -711,7 +713,7 @@ function quiz_print_question($number, $question, $grade, $courseid,
 
 
 
-function quiz_print_quiz_questions($quiz, $results=NULL, $questions=NULL) {
+function quiz_print_quiz_questions($quiz, $results=NULL, $questions=NULL, $shuffleorder=NULL) {
 // Prints a whole quiz on one page.
 
     /// Get the questions
@@ -727,7 +729,13 @@ function quiz_print_quiz_questions($quiz, $results=NULL, $questions=NULL) {
             return false;
         }
 
-        if (!empty($quiz->shufflequestions)) {
+        if ($shuffleorder) {                             // Order has been defined, so reorder questions
+            $oldquestions = $questions;
+            $questions = array();
+            foreach ($shuffleorder as $key) {     
+                $questions[] = $oldquestions[$key];      // This loses the index key, but doesn't matter
+            }
+        } else if (!empty($quiz->shufflequestions)) {    // Mix everything up
             $questions = swapshuffle_assoc($questions);
         }
     }
@@ -772,8 +780,11 @@ function quiz_print_quiz_questions($quiz, $results=NULL, $questions=NULL) {
     echo "<INPUT TYPE=hidden NAME=q VALUE=\"$quiz->id\">";
 
     $count = 0;
+    $questionorder = array();
+
     foreach ($questions as $question) {
         $count++;
+        $questionorder[] = $question->id;
 
         $feedback       = NULL;
         $response       = NULL;
@@ -814,13 +825,17 @@ function quiz_print_quiz_questions($quiz, $results=NULL, $questions=NULL) {
                             $feedback, $response, $actualgrades, $correct, 
                             $randomquestion, $quiz->shuffleanswers);
         print_simple_box_end();
-        echo "<BR>";
+        echo "<br \>";
     }
 
     if (empty($results)) {
-        echo "<CENTER><INPUT TYPE=submit VALUE=\"".get_string("savemyanswers", "quiz")."\"></CENTER>";
+        if (!empty($quiz->shufflequestions)) {  // Things have been mixed up, so pass the question order
+            $shuffleorder = implode(',', $questionorder);
+            echo "<input type=hidden name=shuffleorder value=\"$shuffleorder\">\n";
+        }
+        echo "<center><input type=submit value=\"".get_string("savemyanswers", "quiz")."\"></center>";
     }
-    echo "</FORM>";
+    echo "</form>";
 
     return true;
 }

diff --git a/mod/quiz/report.php b/mod/quiz/report.php
index e56da1be8cf1d91fd581ef9152e47d27d36d0991..4d9d75bbb44991b36577cdab41bb2d3a52b7786a 100644 (file)
--- a/mod/quiz/report.php
+++ b/mod/quiz/report.php
@@ -126,6 +126,8 @@
 
         $quiz->feedback = true;
         $quiz->correctanswers = true;
+        $quiz->shuffleanswers = false;
+        $quiz->shufflequestions = false;
         quiz_print_quiz_questions($quiz, $result, $questions);
 
         if (empty($review)) {