🚀 :: 1.2.18

jobis-application/src/main/java/team/retum/jobis/domain/application/usecase/DeleteApplicationUseCase.java

@@ -17,11 +17,13 @@ public class DeleteApplicationUseCase {
     private final SecurityPort securityPort;
 
     public void execute(Long applicationId) {
+
         Student student = securityPort.getCurrentStudent();
         Application application = queryApplicationPort.getByIdOrThrow(applicationId);
 
         application.checkIsDeletable(student);
 
+        commandApplicationPort.deleteAllByApplicationId(applicationId);
         commandApplicationPort.delete(application);
     }
 }

jobis-application/src/main/java/team/retum/jobis/domain/notice/usecase/CreateNoticeUseCase.java

@@ -20,7 +20,7 @@ public class CreateNoticeUseCase {
 
     public void execute(CreateNoticeRequest request) {
         List<NoticeAttachment> attachments = request.getAttachments().stream()
-            .filter(attachment -> attachment.getUrl() != null && attachment.getType() != null) // URL과 Type이 null이 아닌 경우만 필터링
+            .filter(attachment -> attachment.getUrl() != null && attachment.getType() != null)
             .map(attachment -> new NoticeAttachment(attachment.getUrl(), attachment.getType()))
             .toList();
 

jobis-infrastructure/src/main/java/team/retum/jobis/domain/application/persistence/ApplicationPersistenceAdapter.java

@@ -323,9 +323,7 @@ public void saveAll(List<Application> applications) {
 
     @Override
     public void deleteAllByApplicationId(Long applicationId) {
-        queryFactory.delete(applicationAttachmentEntity)
-            .where(applicationAttachmentEntity.application.id.eq(applicationId))
-            .execute();
+        applicationJpaRepository.deleteAttachmentsByApplicationId(applicationId);
     }
 
     @Override

jobis-infrastructure/src/main/java/team/retum/jobis/domain/application/persistence/entity/ApplicationEntity.java

@@ -22,6 +22,7 @@
 import org.springframework.data.annotation.LastModifiedDate;
 import org.springframework.data.jpa.domain.support.AuditingEntityListener;
 import team.retum.jobis.domain.application.model.ApplicationStatus;
+import team.retum.jobis.domain.notice.persistence.entity.NoticeAttachmentEntity;
 import team.retum.jobis.domain.recruitment.persistence.entity.RecruitmentEntity;
 import team.retum.jobis.domain.student.persistence.entity.StudentEntity;
 import team.retum.jobis.global.entity.BaseTimeEntity;
@@ -64,7 +65,7 @@ public class ApplicationEntity extends BaseTimeEntity {
     @JoinColumn(name = "recruitment_id", nullable = false)
     private RecruitmentEntity recruitment;
 
-    @OneToMany(mappedBy = "application", cascade = CascadeType.ALL, orphanRemoval = true)
+    @OneToMany(mappedBy = "application", cascade = CascadeType.ALL)
     private List<ApplicationAttachmentEntity> applicationAttachments = new ArrayList<>();
 
     @LastModifiedDate

jobis-infrastructure/src/main/java/team/retum/jobis/domain/application/persistence/repository/ApplicationJpaRepository.java

@@ -1,8 +1,8 @@
 package team.retum.jobis.domain.application.persistence.repository;
 
 import org.springframework.data.jpa.repository.JpaRepository;
+import org.springframework.data.jpa.repository.Modifying;
 import org.springframework.data.jpa.repository.Query;
-import team.retum.jobis.domain.application.model.ApplicationStatus;
 import team.retum.jobis.domain.application.persistence.entity.ApplicationEntity;
 
 import java.util.List;
@@ -14,6 +14,10 @@ public interface ApplicationJpaRepository extends JpaRepository<ApplicationEntit
 
     void deleteByIdIn(List<Long> applicationIds);
 
+    @Modifying
+    @Query("delete from ApplicationAttachmentEntity a where a.application.id = ?1")
+    void deleteAttachmentsByApplicationId(Long applicationId);
+
     @Query("select a from ApplicationEntity a join fetch a.applicationAttachments where a.id=?1")
     Optional<ApplicationEntity> findByIdFetch(Long applicationId);
 }