fixed exam quiz data update
This commit is contained in:
parent
b7717ed2de
commit
ceb2a53c83
5 changed files with 12 additions and 6 deletions
|
@ -16,6 +16,7 @@ import java.util.Map;
|
|||
|
||||
import javax.validation.constraints.NotNull;
|
||||
|
||||
import org.apache.commons.lang3.BooleanUtils;
|
||||
import org.apache.commons.lang3.StringUtils;
|
||||
import org.joda.time.DateTime;
|
||||
import org.joda.time.DateTimeZone;
|
||||
|
@ -295,6 +296,11 @@ public final class Exam implements GrantEntity {
|
|||
return this.lmsAvailable;
|
||||
}
|
||||
|
||||
@JsonIgnore
|
||||
public boolean isLmsAvailable() {
|
||||
return BooleanUtils.isTrue(this.lmsAvailable);
|
||||
}
|
||||
|
||||
public String getExternalId() {
|
||||
return this.externalId;
|
||||
}
|
||||
|
|
|
@ -596,7 +596,7 @@ public class ExamForm implements TemplateComposer {
|
|||
}
|
||||
|
||||
private boolean testSEBRestrictionAPI(final Exam exam) {
|
||||
if (!exam.lmsAvailable) {
|
||||
if (!exam.isLmsAvailable()) {
|
||||
return false;
|
||||
}
|
||||
|
||||
|
|
|
@ -274,7 +274,7 @@ public class ExamList implements TemplateComposer {
|
|||
final Exam exam,
|
||||
final PageService pageService) {
|
||||
|
||||
if (BooleanUtils.isFalse(exam.lmsAvailable)) {
|
||||
if (BooleanUtils.isFalse(exam.isLmsAvailable())) {
|
||||
item.setData(RWT.CUSTOM_VARIANT, CustomVariant.DISABLED.key);
|
||||
return;
|
||||
}
|
||||
|
|
|
@ -132,7 +132,7 @@ public class ExamSessionServiceImpl implements ExamSessionService {
|
|||
.getOrThrow();
|
||||
|
||||
// check lms connection
|
||||
if (!exam.lmsAvailable) {
|
||||
if (!exam.isLmsAvailable()) {
|
||||
result.add(ErrorMessage.EXAM_CONSISTENCY_VALIDATION_LMS_CONNECTION.of(exam.getModelId()));
|
||||
}
|
||||
|
||||
|
|
|
@ -108,7 +108,7 @@ class ExamUpdateHandler {
|
|||
log.error("Failed to update quiz data for exam: {}", quiz,
|
||||
updateQuizData.getError());
|
||||
} else {
|
||||
if (!exam.lmsAvailable) {
|
||||
if (!exam.isLmsAvailable()) {
|
||||
this.examDAO.markLMSAvailability(quiz.id, true, updateId);
|
||||
}
|
||||
failedOrMissing.remove(quiz.id);
|
||||
|
@ -116,7 +116,7 @@ class ExamUpdateHandler {
|
|||
}
|
||||
|
||||
} else {
|
||||
if (!exam.lmsAvailable) {
|
||||
if (!exam.isLmsAvailable()) {
|
||||
this.examDAO.markLMSAvailability(quiz.id, true, updateId);
|
||||
}
|
||||
failedOrMissing.remove(quiz.id);
|
||||
|
@ -282,7 +282,7 @@ class ExamUpdateHandler {
|
|||
}
|
||||
}
|
||||
|
||||
if (exam.lmsAvailable) {
|
||||
if (exam.isLmsAvailable()) {
|
||||
this.examDAO.markLMSAvailability(quizId, false, updateId);
|
||||
}
|
||||
throw new RuntimeException("Not Available");
|
||||
|
|
Loading…
Reference in a new issue