From fc4a5a2f63366abae8d48fa572c9da63ef5d40de Mon Sep 17 00:00:00 2001
From: moodler <moodler>
Date: Tue, 17 Feb 2004 12:14:02 +0000
Subject: [PATCH] FIxed the logging of attempts so that clicking on the link in
 the log actually shows you that attempt, rather than opening a new one!

---
 mod/quiz/attempt.php | 12 ++++++++----
 mod/quiz/lib.php     |  2 +-
 2 files changed, 9 insertions(+), 5 deletions(-)

diff --git a/mod/quiz/attempt.php b/mod/quiz/attempt.php
index 6f480e5633..959460e33d 100644
--- a/mod/quiz/attempt.php
+++ b/mod/quiz/attempt.php
@@ -77,7 +77,6 @@
 
 /// Check to see if they are submitting answers
     if ($rawanswers = data_submitted()) {
-        add_to_log($course->id, "quiz", "submit", "attempt.php?id=$cm->id", "$quiz->id", $cm->id);
 
         $rawanswers = (array)$rawanswers;
 
@@ -123,7 +122,10 @@
             error("Could not grade your quiz attempt!");
         }
 
-        if (! $attempt = quiz_save_attempt($quiz, $questions, $result, $attemptnumber)) {
+        if ($attempt = quiz_save_attempt($quiz, $questions, $result, $attemptnumber)) {
+            add_to_log($course->id, "quiz", "submit", 
+                       "review.php?q=$quiz->id&attempt=$attempt->id", "$quiz->id", $cm->id);
+        } else {
             notice(get_string("alreadysubmitted", "quiz"), "view.php?id=$cm->id");
             print_footer($course);
             exit;
@@ -154,7 +156,6 @@
         exit;
     }
 
-    add_to_log($course->id, "quiz", "attempt", "attempt.php?id=$cm->id", "$quiz->id", $cm->id);
 
 /// Print the quiz page
 
@@ -167,7 +168,10 @@
 /// Actually seeing the questions marks the start of an attempt
  
     if (!$unfinished = quiz_get_user_attempt_unfinished($quiz->id, $USER->id)) {
-        if (! quiz_start_attempt($quiz->id, $USER->id, $attemptnumber)) {
+        if ($newattemptid = quiz_start_attempt($quiz->id, $USER->id, $attemptnumber)) {
+            add_to_log($course->id, "quiz", "attempt", 
+                       "review.php?q=$quiz->id&attempt=$newattemptid", "$quiz->id", $cm->id);
+        } else {
             error("Sorry! Could not start the quiz (could not save starting time)");
         }
     }
diff --git a/mod/quiz/lib.php b/mod/quiz/lib.php
index 312a598481..4ab853c8a4 100644
--- a/mod/quiz/lib.php
+++ b/mod/quiz/lib.php
@@ -1623,7 +1623,7 @@ function quiz_save_attempt($quiz, $questions, $result, $attemptnum) {
             return false;
         }
     }
-    return true;
+    return $attempt;
 }
 
 function quiz_grade_attempt_question_result($question,
-- 
2.39.5