From 42ea3df6224f78829ce7723837b2e0fc8ccb361f Mon Sep 17 00:00:00 2001
From: Eric Marguin <e.marguin@elixir-interactive.com>
Date: Wed, 6 Jun 2007 10:17:24 +0200
Subject: [PATCH] [svn r12542] fix a bug when displaying result of multiple
 answers

---
 main/exercice/exercise_result.php | 20 +++++++++-----------
 1 file changed, 9 insertions(+), 11 deletions(-)

diff --git a/main/exercice/exercise_result.php b/main/exercice/exercise_result.php
index b8a9ba12c6..bbcce2162d 100644
--- a/main/exercice/exercise_result.php
+++ b/main/exercice/exercise_result.php
@@ -27,7 +27,7 @@
 *	@package dokeos.exercise
 *	@author Olivier Brouckaert, main author
 *	@author Roan Embrechts, some refactoring
-* 	@version $Id: exercise_result.php 12503 2007-05-30 10:27:42Z elixir_inter $
+* 	@version $Id: exercise_result.php 12542 2007-06-06 08:17:24Z elixir_inter $
 *
 *	@todo	split more code up in functions, move functions to library?
 */
@@ -209,11 +209,11 @@ function display_unique_or_multiple_answer($answerType, $studentChoice, $answer,
 		echo $answer; ?>
 	</td>
 	<td width="45%" style="border-bottom: 1px solid #4171B5;">
-		<?php
-		$answerComment=api_parse_tex($answerComment);
-		if($studentChoice) echo nl2br(make_clickable($answerComment)); else echo '&nbsp;'; ?>
-	</td>
-	</tr>
+ 		<?php
+ 		$answerComment=api_parse_tex($answerComment);
+		if($studentChoice || $answerType == MULTIPLE_ANSWER) echo nl2br(make_clickable($answerComment)); else echo '&nbsp;'; ?>
+ 	</td>
+ 	</tr>
 	<?php
 }
 
@@ -467,11 +467,9 @@ $exerciseTitle=api_parse_tex($exerciseTitle);
 				case MULTIPLE_ANSWER :
 
 										$studentChoice=$choice[$answerId];
-
-										if($studentChoice)
-										{
-											$questionScore+=$answerWeighting;
-											$totalScore+=$answerWeighting;
+										if(intval($studentChoice) == intval($answerCorrect)) {
+ 											$questionScore+=$answerWeighting;
+ 											$totalScore+=$answerWeighting;
 										}
 
 										break;