Minor - Try reducing the cognitive complexity (see codeclimate) of getResponseDegreeInfo() in degree of certainty

pull/2591/head
Yannick Warnier 7 years ago
parent 51662b888d
commit 52e679ddec
  1. 18
      main/exercise/MultipleAnswerTrueFalseDegreeCertainty.php

@ -337,8 +337,6 @@ class MultipleAnswerTrueFalseDegreeCertainty extends Question
'label' => get_lang('DegreeOfCertaintyDeclaredIgnorance'), 'label' => get_lang('DegreeOfCertaintyDeclaredIgnorance'),
'description' => get_lang('DegreeOfCertaintyDeclaredIgnoranceDescription'), 'description' => get_lang('DegreeOfCertaintyDeclaredIgnoranceDescription'),
]; ];
return $result;
} else { } else {
$checkResult = $studentAnswer == $expectedAnswer ? true : false; $checkResult = $studentAnswer == $expectedAnswer ? true : false;
if ($checkResult) { if ($checkResult) {
@ -350,10 +348,7 @@ class MultipleAnswerTrueFalseDegreeCertainty extends Question
'label' => get_lang('DegreeOfCertaintyVerySure'), 'label' => get_lang('DegreeOfCertaintyVerySure'),
'description' => get_lang('DegreeOfCertaintyVerySureDescription'), 'description' => get_lang('DegreeOfCertaintyVerySureDescription'),
]; ];
} elseif ($studentDegreeChoicePosition >= 4 && $studentDegreeChoicePosition <= 5) {
return $result;
}
if ($studentDegreeChoicePosition >= 4 && $studentDegreeChoicePosition <= 5) {
$result = [ $result = [
'color' => '#000000', 'color' => '#000000',
'background-color' => '#A9F5A9', 'background-color' => '#A9F5A9',
@ -361,8 +356,6 @@ class MultipleAnswerTrueFalseDegreeCertainty extends Question
'label' => get_lang('DegreeOfCertaintyPrettySure'), 'label' => get_lang('DegreeOfCertaintyPrettySure'),
'description' => get_lang('DegreeOfCertaintyPrettySureDescription'), 'description' => get_lang('DegreeOfCertaintyPrettySureDescription'),
]; ];
return $result;
} }
} else { } else {
if ($studentDegreeChoicePosition >= 6) { if ($studentDegreeChoicePosition >= 6) {
@ -373,10 +366,7 @@ class MultipleAnswerTrueFalseDegreeCertainty extends Question
'label' => get_lang('DegreeOfCertaintyVeryUnsure'), 'label' => get_lang('DegreeOfCertaintyVeryUnsure'),
'description' => get_lang('DegreeOfCertaintyVeryUnsureDescription'), 'description' => get_lang('DegreeOfCertaintyVeryUnsureDescription'),
]; ];
} elseif ($studentDegreeChoicePosition >= 4 && $studentDegreeChoicePosition <= 5) {
return $result;
}
if ($studentDegreeChoicePosition >= 4 && $studentDegreeChoicePosition <= 5) {
$result = [ $result = [
'color' => '#000000', 'color' => '#000000',
'background-color' => '#F6CECE', 'background-color' => '#F6CECE',
@ -384,11 +374,11 @@ class MultipleAnswerTrueFalseDegreeCertainty extends Question
'label' => get_lang('DegreeOfCertaintyUnsure'), 'label' => get_lang('DegreeOfCertaintyUnsure'),
'description' => get_lang('DegreeOfCertaintyUnsureDescription'), 'description' => get_lang('DegreeOfCertaintyUnsureDescription'),
]; ];
return $result;
} }
} }
} }
return $result;
} }
/** /**

Loading…
Cancel
Save