Skip to content

Commit

Permalink
bug fix for inclass no sequence quizzes
Browse files Browse the repository at this point in the history
  • Loading branch information
@ritosilva
ritosilva committed Sep 19, 2024
1 parent aacfbec commit 4cad72a
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion backend/src/main/java/pt/ulisboa/tecnico/socialsoftware/tutor/answer/domain/QuizAnswer.java
View file
Original file line number Diff line number Diff line change
Expand Up @@ -228,7 +228,8 @@ public Integer checkCorrectSequenceQuestion(Integer sequence, Integer questionId
}

public void checkIsCurrentQuestion(Integer questionId) {
if (!this.sequenceQuestionIdMap.get(currentSequenceQuestion).equals(questionId)) {
Integer currentSequenceQuestionId = this.sequenceQuestionIdMap.get(currentSequenceQuestion);
if (currentSequenceQuestionId != null && !currentSequenceQuestionId.equals(questionId)) {
throw new TutorException(INVALID_QUIZ_ANSWER_SEQUENCE, getStudent().getUsername() + " tried to submit question with ID "
+ questionId + " when they sequence is " + (currentSequenceQuestion + 1)
+ " and the question IDs sequence is " +
Expand Down

3 comments on commit 4cad72a

@AntQuintino
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The Quizzes Tutor site has security problems. Most users can not enter the site, they receive a message that the site is not secure. Please, solve it.

@ritosilva
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The certificate has expired; I am going to contact DSI to renew it. Thanks for letting me know it.

@ritosilva
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The new certificate was installed.

Please sign in to comment.