The validateForm function for Form/QuizTakeForm uses getCurrentQuestionNumber when it should (I think) be using getCurrentQuestion.
Line 281
Probably missed because it only matters when allowSkipping is false.
Also the assignment of $current_question should probably be inside the test as its not used in the function unless allowSkipping is false.
public function validateForm(array &$form, FormStateInterface $form_state) {
$quiz = $this->getQuiz();
if ($quiz instanceof QuizInterface) {
/** @var \Drupal\quiz_maker\QuestionInterface $current_question */
$current_question = $this->quizSession->getCurrentQuestionNumber($quiz);
if (!$quiz->allowSkipping()) {
$current_question->validateAnsweringForm($form, $form_state);
}
}
}
Active
1.0
Code
Uploaded path - Applies on 1.0.7 ok.
Hi, thanks for your comments, please fix the PHPCS and I will merge MR
Issue was fixed here - https://git.drupalcode.org/project/quiz_maker/-/commit/f38d60107207f52e1...