MDL-10300 - cleaning incorrectly applied to overall feedback.

moodle · Jul 30, 2007 · de26653 · de26653
1 parent 00719c0
commit de26653
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 3 deletions.
diff --git a/mod/quiz/locallib.php b/mod/quiz/locallib.php
@@ -346,7 +346,7 @@ function quiz_rescale_grade($rawgrade, $quiz) {
 
 /**
  * Get the feedback text that should be show to a student who
- * got this grade on this quiz.
+ * got this grade on this quiz. The feedback is processed ready for diplay.
  *
  * @param float $grade a grade on this quiz.
  * @param integer $quizid the id of the quiz object.
@@ -360,6 +360,11 @@ function quiz_feedback_for_grade($grade, $quizid) {
         $feedback = '';
     }
 
+    // Clean the text, ready for display.
+    $formatoptions = new stdClass;
+    $formatoptions->noclean = true;
+    $feedback = format_text($feedback, FORMAT_MOODLE, $formatoptions);
+
     return $feedback;
 }
 

diff --git a/mod/quiz/mod_form.php b/mod/quiz/mod_form.php
@@ -228,7 +228,7 @@ function definition() {
         }
         $numfeedbacks = max(count($this->_feedbacks) * 1.5, 5);
 
-        $mform->setType('feedbacktext', PARAM_TEXT);
+        $mform->setType('feedbacktext', PARAM_RAW);
         $mform->setType('feedbackboundaries', PARAM_NOTAGS);
 
         $nextel=$this->repeat_elements($repeatarray, $numfeedbacks-1,

diff --git a/mod/quiz/report/overview/report.php b/mod/quiz/report/overview/report.php
@@ -42,6 +42,10 @@ function display($quiz, $cm, $course) {
             break;
         }
 
+        // Set of format options for teacher-created content, for example overall feedback.
+        $nocleanformatoptions = new stdClass;
+        $nocleanformatoptions->noclean = true;
+
         // Set table options
         $noattempts = optional_param('noattempts', 0, PARAM_INT);
         $detailedmarks = optional_param('detailedmarks', 0, PARAM_INT);
@@ -466,7 +470,7 @@ function display($quiz, $cm, $course) {
                     }
                     if ($hasfeedback) {
                         if ($attempt->timefinish) {
-                            $row[] = $attempt->feedbacktext;
+                            $row[] = format_text($attempt->feedbacktext, FORMAT_MOODLE, $nocleanformatoptions);
                         } else {
                             $row[] = '-';
                         }