Merge pull request #48 from Block-Guard/fix/#47/report-records-update-error

[Fix] 긴급신고 권장 체크 박스 수정 오류

Author: yeonju73

src/main/java/com/blockguard/server/domain/report/application/ReportRecordService.java

@@ -97,15 +97,20 @@ public ReportRecordStepResponse updateStepInfo(User user, Long reportId, int ste
 
     private void validateUpdateStepInfo(UpdateReportStepRequest updateReportStepRequest, ReportStepProgress progress, ReportStep step) {
         // 이미 완료된 step 인지 검증
-        validateProgressIsCompleted(progress);
+        validateProgressIsCompleted(updateReportStepRequest, progress);
         // 체스박스 수 검증
         validateCheckboxCounts(updateReportStepRequest, step);
         validateCompletionConsistency(updateReportStepRequest);
     }
 
-    private void validateProgressIsCompleted(ReportStepProgress progress) {
+    // 완료된 단계의 필수 체크박스는 수정이 불가능
+    // 요청된 필수 체크박스(request.getCheckBoxes())가 저장된 필수 체크박스 상태와 다를 경우에만 에러를 던진다
+    private void validateProgressIsCompleted(UpdateReportStepRequest request, ReportStepProgress progress) {
         if (progress.isCompleted()) {
-            throw new BusinessExceptionHandler(ErrorCode.REPORT_STEP_ALREADY_COMPLETED);
+            List<Boolean> currentRequired = getCheckboxes(progress, CheckboxType.REQUIRED);
+            if (!currentRequired.equals(request.getCheckBoxes())){
+                throw new BusinessExceptionHandler(ErrorCode.REPORT_STEP_ALREADY_COMPLETED);
+            }
         }
     }
 
@@ -179,8 +184,8 @@ private void validateCompletionConsistency(UpdateReportStepRequest request) {
     }
 
     private ReportRecordStepResponse buildReportRecordStepResponse(Long reportId, int stepNumber, ReportStepProgress progress, UserReportRecord record) {
-        List<Boolean> resultRequiredCheckboxes = getRequiredCheckboxes(progress, CheckboxType.REQUIRED);
-        List<Boolean> resultRecommendedCheckboxes = getRequiredCheckboxes(progress, CheckboxType.RECOMMENDED);
+        List<Boolean> resultRequiredCheckboxes = getCheckboxes(progress, CheckboxType.REQUIRED);
+        List<Boolean> resultRecommendedCheckboxes = getCheckboxes(progress, CheckboxType.RECOMMENDED);
 
         resultRecommendedCheckboxes = resultRecommendedCheckboxes.isEmpty() ? null : resultRecommendedCheckboxes;
 
@@ -194,7 +199,7 @@ private ReportRecordStepResponse buildReportRecordStepResponse(Long reportId, in
                 .build();
     }
 
-    private List<Boolean> getRequiredCheckboxes(ReportStepProgress progress, CheckboxType type) {
+    private List<Boolean> getCheckboxes(ReportStepProgress progress, CheckboxType type) {
         return progress.getCheckboxes().stream()
                 .filter(cb -> cb.getType() == type)
                 .sorted(Comparator.comparingInt(ReportStepCheckbox::getBoxIndex))

src/main/java/com/blockguard/server/global/common/codes/ErrorCode.java

@@ -34,7 +34,7 @@ public enum ErrorCode {
     REPORT_NOT_FOUND(HttpStatus.BAD_REQUEST, 4031, "존재하지 않는 신고입니다."),
     INVALID_CHECKBOX_COUNT(HttpStatus.BAD_REQUEST, 4032, "해당 단계에 유효하지 않은 체크박스 형식입니다."),
     INVALID_STEP_COMPLETION(HttpStatus.BAD_REQUEST, 4033, "해당 단계가 완료되려면 필수 체크박스가 모두 완료되어야 합니다."),
-    REPORT_STEP_ALREADY_COMPLETED(HttpStatus.BAD_REQUEST, 4034, "이미 완료된 단계의 체크박스는 수정할 수 없습니다."),
+    REPORT_STEP_ALREADY_COMPLETED(HttpStatus.BAD_REQUEST, 4034, "완료된 단계의 필수 조치 체크박스는 수정할 수 없습니다."),
     FAIL_TO_CRAWLING_NEWS(HttpStatus.BAD_REQUEST, 4035, "뉴스 크롤링에 실패하였습니다."),
     MUST_BE_POSITIVE_NUMBER(HttpStatus.BAD_REQUEST, 4036, "페이지와 사이즈는 1이상의 값을 입력해야합니다."),