From: jamiesensei <jamiesensei>
Date: Tue, 23 Sep 2008 10:21:32 +0000 (+0000)
Subject: MDL-16532 "Can width of column for Essay questions be varied?" Concatenate the answer... 
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=e84cb6155423594bccaad5bfe0c528f39ad54c67;p=moodle.git

MDL-16532 "Can width of column for Essay questions be varied?" Concatenate the answers displayed in all responses in responses report for answers above a certain character count (now set to 150). Also added a link to all responses to display a review of the question response. Both the concatenation and links are only showed in the report on the Moodle site and effects are not exported.
---

diff --git a/mod/quiz/report/responses/responses_table.php b/mod/quiz/report/responses/responses_table.php
index 3bfb08e000..db7d1f3cb2 100644
--- a/mod/quiz/report/responses/responses_table.php
+++ b/mod/quiz/report/responses/responses_table.php
@@ -1,4 +1,5 @@
 <?php  // $Id$
+define ('QUIZ_REPORT_RESPONSES_MAX_LEN_TO_DISPLAY', 150);
 
 class quiz_report_responses_table extends table_sql {
 
@@ -165,6 +166,13 @@ class quiz_report_responses_table extends table_sql {
                 $questionclass = "que";
                 $response = format_text($response, FORMAT_MOODLE, $format_options);
                 if ($response){
+                    if (strlen($response) > QUIZ_REPORT_RESPONSES_MAX_LEN_TO_DISPLAY){
+                        $response = shorten_text($response, QUIZ_REPORT_RESPONSES_MAX_LEN_TO_DISPLAY);
+                    }
+                    $response = link_to_popup_window('/mod/quiz/reviewquestion.php?attempt=' .
+                        $attempt->attempt . '&amp;question=' . $question->id,
+                        'reviewquestion', $response, 450, 650, get_string('reviewresponse', 'quiz'),
+                        'none', true);
                     if (question_state_is_graded($stateforqinattempt)){
                         return "<span class=\"$questionclass\"><span class=\"$qclass\">".$response."</span></span>$feedbackimg";
                     } else {