Index: lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/web/controller/LearningController.java =================================================================== diff -u -rd2fae62bad612aad30c914422eea4e0316fc4be1 -rf6c29ddb268edd9f5d48a872321cf6a8b02831f3 --- lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/web/controller/LearningController.java (.../LearningController.java) (revision d2fae62bad612aad30c914422eea4e0316fc4be1) +++ lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/web/controller/LearningController.java (.../LearningController.java) (revision f6c29ddb268edd9f5d48a872321cf6a8b02831f3) @@ -880,16 +880,16 @@ questionDto.setAnswer(answer); } else if (questionType == QbQuestion.TYPE_ORDERING) { + //sort accrording to the new sequenceIds + Set sortedOptions = new TreeSet<>(); for (OptionDTO optionDto : questionDto.getOptionDtos()) { int answerSequenceId = WebUtil.readIntParam(request, AssessmentConstants.ATTR_QUESTION_PREFIX + i + "_" + optionDto.getUid()); optionDto.setDisplayOrder(answerSequenceId); + sortedOptions.add(optionDto); } - //sort accrording to the new sequenceIds - Set sortedOptions = new TreeSet<>(); - sortedOptions.addAll(questionDto.getOptionDtos()); - questionDto.setOptionDtos(sortedOptions); + questionDto.setOptionDtos(sortedOptions); } else if (questionType == QbQuestion.TYPE_MARK_HEDGING) { //store hedging marks