diff --git a/api/src/main/java/ca/bc/gov/educ/api/dataconversion/process/StudentProcess.java b/api/src/main/java/ca/bc/gov/educ/api/dataconversion/process/StudentProcess.java index 6a9a169..626b671 100644 --- a/api/src/main/java/ca/bc/gov/educ/api/dataconversion/process/StudentProcess.java +++ b/api/src/main/java/ca/bc/gov/educ/api/dataconversion/process/StudentProcess.java @@ -53,11 +53,6 @@ public ConvGradStudent convertStudent(ConvGradStudent convGradStudent, Conversio return convGradStudent; } - // Program Completion for graduated student - if (!validateProgramCompletionDate(convGradStudent, summary)) { - return convGradStudent; - } - // Student conversion process process(convGradStudent, students, summary, reload, ongoingUpdate); @@ -619,20 +614,6 @@ private void handleAdultStartRule(Student penStudent, GraduationStudentRecord gr } } - /** - * - * @return true Valid - * false Bad data (programCompletionDate is null) - */ - private boolean validateProgramCompletionDate(ConvGradStudent convGradStudent, ConversionStudentSummaryDTO summary) { - if ("SCCP".equalsIgnoreCase(convGradStudent.getGraduationRequirementYear()) && - StringUtils.isBlank(convGradStudent.getSlpDate())) { - handleException(convGradStudent, summary, convGradStudent.getPen(), ConversionResultType.FAILURE, "Bad data: slp_date is null for SCCP"); - return false; - } - return true; - } - /** * * @return true Valid diff --git a/api/src/main/java/ca/bc/gov/educ/api/dataconversion/service/student/NewStudentEventService.java b/api/src/main/java/ca/bc/gov/educ/api/dataconversion/service/student/NewStudentEventService.java index 7c8b9ff..3e0ed08 100644 --- a/api/src/main/java/ca/bc/gov/educ/api/dataconversion/service/student/NewStudentEventService.java +++ b/api/src/main/java/ca/bc/gov/educ/api/dataconversion/service/student/NewStudentEventService.java @@ -63,6 +63,9 @@ public void processEvent(T request, Event event) { return; } if (result == null || ConversionResultType.FAILURE.equals(result.getResult())) { + if (!summary.getErrors().isEmpty()) { + summary.getErrors().forEach(e -> log.error("Load is failed for {} - {}", e.getItem(), e.getReason())); + } return; } }