⚙️ :: (8452)공지 첨부물 nullable

Team-return · Nov 14, 2024 · 18e6f7a · 18e6f7a
1 parent 2a95648
commit 18e6f7a
Show file tree

Hide file tree

Showing 4 changed files with 23 additions and 10 deletions.
diff --git a/...n/src/main/java/team/retum/jobis/domain/company/usecase/TeacherQueryCompaniesUseCase.java b/...n/src/main/java/team/retum/jobis/domain/company/usecase/TeacherQueryCompaniesUseCase.java
@@ -77,7 +77,6 @@ public CompanyCountResponse countCompanies(
             .limit(1000)
             .build();
 
-
         return new CompanyCountResponse(
             queryCompanyPort.getByConditions(filter).stream().toList().size()
         );

diff --git a/.../src/main/java/team/retum/jobis/domain/notice/dto/response/QueryNoticeDetailResponse.java b/.../src/main/java/team/retum/jobis/domain/notice/dto/response/QueryNoticeDetailResponse.java
@@ -6,6 +6,7 @@
 import team.retum.jobis.domain.notice.model.NoticeAttachment;
 
 import java.time.LocalDateTime;
+import java.util.ArrayList;
 import java.util.List;
 
 @Getter

diff --git a/...application/src/main/java/team/retum/jobis/domain/notice/usecase/CreateNoticeUseCase.java b/...application/src/main/java/team/retum/jobis/domain/notice/usecase/CreateNoticeUseCase.java
@@ -9,6 +9,8 @@
 import team.retum.jobis.domain.notice.model.NoticeAttachment;
 import team.retum.jobis.domain.notice.spi.CommandNoticePort;
 
+import java.util.List;
+
 @RequiredArgsConstructor
 @UseCase
 public class CreateNoticeUseCase {
@@ -17,12 +19,15 @@ public class CreateNoticeUseCase {
     private final PublishEventPort publishEventPort;
 
     public void execute(CreateNoticeRequest request) {
+        List<NoticeAttachment> attachments = request.getAttachments().stream()
+            .filter(attachment -> attachment.getUrl() != null && attachment.getType() != null) // URL과 Type이 null이 아닌 경우만 필터링
+            .map(attachment -> new NoticeAttachment(attachment.getUrl(), attachment.getType()))
+            .toList();
+
         Notice savedNotice = commandNoticePort.save(Notice.builder()
             .title(request.getTitle())
             .content(request.getContent())
-            .attachments(request.getAttachments().stream()
-                .map(attachment -> new NoticeAttachment(attachment.getUrl(), attachment.getType()))
-                .toList())
+            .attachments(attachments)
             .build());
 
         publishEventPort.publishEvent(new NoticePostedEvent(savedNotice));

diff --git a/...ructure/src/main/java/team/retum/jobis/domain/notice/persistence/mapper/NoticeMapper.java b/...ructure/src/main/java/team/retum/jobis/domain/notice/persistence/mapper/NoticeMapper.java
@@ -14,9 +14,13 @@
 public class NoticeMapper {
 
     public NoticeEntity toEntity(Notice domain) {
-        List<NoticeAttachmentEntity> attachments = domain.getAttachments().stream()
-            .map(attachment -> new NoticeAttachmentEntity(attachment.getUrl(), attachment.getType()))
-            .toList();
+        List<NoticeAttachmentEntity> attachments = null;
+
+        if (domain.getAttachments() != null && !domain.getAttachments().isEmpty()) {
+            attachments = domain.getAttachments().stream()
+                .map(attachment -> new NoticeAttachmentEntity(attachment.getUrl(), attachment.getType()))
+                .toList();
+        }
 
         return NoticeEntity.builder()
             .id(domain.getId())
@@ -27,9 +31,13 @@ public NoticeEntity toEntity(Notice domain) {
     }
 
     public Notice toDomain(NoticeEntity entity) {
-        List<NoticeAttachment> attachments = entity.getAttachments().stream()
-            .map(attachment -> new NoticeAttachment(attachment.getAttachmentUrl(), attachment.getType()))
-            .toList();
+        List<NoticeAttachment> attachments = null;
+
+        if (entity.getAttachments() != null && !entity.getAttachments().isEmpty()) {
+            attachments = entity.getAttachments().stream()
+                .map(attachment -> new NoticeAttachment(attachment.getAttachmentUrl(), attachment.getType()))
+                .toList();
+        }
 
         return Notice.builder()
             .id(entity.getId())