From 523e1a21ffbf5055c114736c27494077984b885d Mon Sep 17 00:00:00 2001
From: moodler <moodler>
Date: Mon, 23 Oct 2006 07:01:18 +0000
Subject: [PATCH] Merged choice fixes from stable

---
 mod/choice/view.php | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

diff --git a/mod/choice/view.php b/mod/choice/view.php
index 0c543fb3ab..54605de82f 100644
--- a/mod/choice/view.php
+++ b/mod/choice/view.php
@@ -38,12 +38,13 @@
                 choice_delete_responses($attemptids); //delete responses.
                 redirect("view.php?id=$cm->id");
             }
-        }    
-        
-        if (empty($form->answer)) {
+        }
+        $answer = optional_param('answer', '', PARAM_INT);
+
+        if (empty($answer)) {
             redirect("view.php?id=$cm->id", get_string('mustchooseone', 'choice'));
         } else {
-            choice_user_submit_response($form->answer, $choice, $USER->id, $course->id, $cm);
+            choice_user_submit_response($answer, $choice, $USER->id, $course->id, $cm);
         }
         redirect("view.php?id=$cm->id");
         exit;
@@ -70,7 +71,8 @@
     }
 
     //if user has already made a selection, and they are not allowed to update it, show their selected answer.
-    if (!empty($USER->id) && ($current = get_record('choice_answers', 'choiceid', $choice->id, 'userid', $USER->id))) {
+    if (!empty($USER->id) && ($current = get_record('choice_answers', 'choiceid', $choice->id, 'userid', $USER->id)) &&
+        empty($choice->allowupdate) ) {
         print_simple_box(get_string("yourselection", "choice", userdate($choice->timeopen)).": ".format_string(choice_get_option_text($choice, $current->optionid)), "center");
     }
 
-- 
2.39.5