Exercise: Fix hide_expected_answer exercise option for fill in the blank question type - refs BT#21969

pull/5934/head
NicoDucou 10 months ago
parent 8bcf1c1678
commit 2c779cbc56
  1. 27
      main/exercise/fill_blanks.class.php
  2. 3
      main/inc/lib/exercise_show_functions.lib.php

@ -1201,7 +1201,8 @@ class FillBlanks extends Question
$answer, $answer,
$feedbackType, $feedbackType,
$resultsDisabled = false, $resultsDisabled = false,
$showTotalScoreAndUserChoices = false $showTotalScoreAndUserChoices = false,
$exercise
) { ) {
$result = ''; $result = '';
$listStudentAnswerInfo = self::getAnswerInfo($answer, true); $listStudentAnswerInfo = self::getAnswerInfo($answer, true);
@ -1215,7 +1216,8 @@ class FillBlanks extends Question
$listStudentAnswerInfo['words'][$i], $listStudentAnswerInfo['words'][$i],
$feedbackType, $feedbackType,
$resultsDisabled, $resultsDisabled,
$showTotalScoreAndUserChoices $showTotalScoreAndUserChoices,
$exercise
); );
} else { } else {
$listStudentAnswerInfo['student_answer'][$i] = self::getHtmlWrongAnswer( $listStudentAnswerInfo['student_answer'][$i] = self::getHtmlWrongAnswer(
@ -1223,7 +1225,8 @@ class FillBlanks extends Question
$listStudentAnswerInfo['words'][$i], $listStudentAnswerInfo['words'][$i],
$feedbackType, $feedbackType,
$resultsDisabled, $resultsDisabled,
$showTotalScoreAndUserChoices $showTotalScoreAndUserChoices,
$exercise
); );
} }
} }
@ -1264,10 +1267,14 @@ class FillBlanks extends Question
$right, $right,
$feedbackType, $feedbackType,
$resultsDisabled = false, $resultsDisabled = false,
$showTotalScoreAndUserChoices = false $showTotalScoreAndUserChoices = false,
$exercise
) { ) {
$hideExpectedAnswer = false; $hideExpectedAnswer = false;
$hideUserSelection = false; $hideUserSelection = false;
if (!$exercise->showExpectedChoiceColumn()) {
$hideExpectedAnswer = true;
}
switch ($resultsDisabled) { switch ($resultsDisabled) {
case RESULT_DISABLE_SHOW_SCORE_AND_EXPECTED_ANSWERS_AND_RANKING: case RESULT_DISABLE_SHOW_SCORE_AND_EXPECTED_ANSWERS_AND_RANKING:
case RESULT_DISABLE_SHOW_ONLY_IN_CORRECT_ANSWER: case RESULT_DISABLE_SHOW_ONLY_IN_CORRECT_ANSWER:
@ -1356,7 +1363,8 @@ class FillBlanks extends Question
$correct, $correct,
$feedbackType, $feedbackType,
$resultsDisabled = false, $resultsDisabled = false,
$showTotalScoreAndUserChoices = false $showTotalScoreAndUserChoices = false,
$exercise
) { ) {
return self::getHtmlAnswer( return self::getHtmlAnswer(
$answer, $answer,
@ -1364,7 +1372,8 @@ class FillBlanks extends Question
true, true,
$feedbackType, $feedbackType,
$resultsDisabled, $resultsDisabled,
$showTotalScoreAndUserChoices $showTotalScoreAndUserChoices,
$exercise
); );
} }
@ -1384,7 +1393,8 @@ class FillBlanks extends Question
$correct, $correct,
$feedbackType, $feedbackType,
$resultsDisabled = false, $resultsDisabled = false,
$showTotalScoreAndUserChoices = false $showTotalScoreAndUserChoices = false,
$exercise
) { ) {
return self::getHtmlAnswer( return self::getHtmlAnswer(
$answer, $answer,
@ -1392,7 +1402,8 @@ class FillBlanks extends Question
false, false,
$feedbackType, $feedbackType,
$resultsDisabled, $resultsDisabled,
$showTotalScoreAndUserChoices $showTotalScoreAndUserChoices,
$exercise
); );
} }

@ -30,7 +30,8 @@ class ExerciseShowFunctions
$answer, $answer,
$feedbackType, $feedbackType,
$resultsDisabled, $resultsDisabled,
$showTotalScoreAndUserChoices $showTotalScoreAndUserChoices,
$exercise
); );
if (empty($id)) { if (empty($id)) {

Loading…
Cancel
Save