fix: 리크루트 오류 수정

fix: 리크루트 목록조회 category null 처리 fix: 리크루트 등록 시 question 빈값 처리 fix: 리크루트 질문 등록 예외처리 로직 수정 fix: 사용자 참여중 리크루트 조회시 로그인 유저는 항상 볼수있도록 수정 fix: 리크루트 참여 목록 조회시 로그인 id적용되도록 수정
SSAF-SOUND · Oct 14, 2023 · e1a084e · e1a084e
1 parent 12488b8
commit e1a084e
Show file tree

Hide file tree

Showing 5 changed files with 15 additions and 7 deletions.
diff --git a/src/main/java/com/ssafy/ssafsound/domain/recruit/controller/RecruitController.java b/src/main/java/com/ssafy/ssafsound/domain/recruit/controller/RecruitController.java
@@ -81,7 +81,7 @@ public  EnvelopeResponse<GetRecruitsResDto> getMemberJoinedRecruits(GetMemberJoi
     @GetMapping("/applied")
     public EnvelopeResponse<GetMemberAppliedRecruitsResDto> getMemberAppliedRecruits(GetMemberAppliedRecruitsReqDto recruitsReqDto, @Authentication AuthenticatedMember memberInfo) {
         return EnvelopeResponse.<GetMemberAppliedRecruitsResDto>builder()
-                .data(recruitService.getMemberAppliedRecruits(recruitsReqDto, 1L))
+                .data(recruitService.getMemberAppliedRecruits(recruitsReqDto, memberInfo.getMemberId()))
                 .build();
     }
 }
diff --git a/src/main/java/com/ssafy/ssafsound/domain/recruit/dto/PostRecruitReqDto.java b/src/main/java/com/ssafy/ssafsound/domain/recruit/dto/PostRecruitReqDto.java
@@ -18,6 +18,8 @@
 import java.util.List;
 import java.util.stream.Collectors;
 
+import org.springframework.util.StringUtils;
+
 @Getter
 @NoArgsConstructor
 @AllArgsConstructor
@@ -68,7 +70,9 @@ public Recruit to() {
     private void setRecruitQuestions (Recruit recruit) {
         if(questions == null) return;
 
-        List<RecruitQuestion> recruitQuestions = questions.stream().map((question)-> RecruitQuestion.builder()
+        List<RecruitQuestion> recruitQuestions = questions.stream()
+            .filter(StringUtils::hasText)
+            .map((question)-> RecruitQuestion.builder()
                 .recruit(recruit)
                 .content(question)
                 .build()

diff --git a/...java/com/ssafy/ssafsound/domain/recruit/repository/RecruitDynamicQueryRepositoryImpl.java b/...java/com/ssafy/ssafsound/domain/recruit/repository/RecruitDynamicQueryRepositoryImpl.java
@@ -77,7 +77,7 @@ public Slice<Recruit> findRecruitByGetRecruitsReqDto(GetRecruitsReqDto dto, Page
 
         // recruit types limitation
         List<String> recruitTypes = dto.getRecruitTypes();
-        if(dto.getCategory().toUpperCase().equals(Category.PROJECT.name()) && recruitTypes!=null && !recruitTypes.isEmpty()) {
+        if(dto.getCategory() != null && dto.getCategory().toUpperCase().equals(Category.PROJECT.name()) && recruitTypes!=null && !recruitTypes.isEmpty()) {
             String metaDataType = MetaDataType.RECRUIT_TYPE.name();
             List<MetaData> containRecruitTypes = recruitTypes.stream()
                     .map(recruitType->metaDataConsumer.getMetaData(metaDataType, recruitType))

diff --git a/src/main/java/com/ssafy/ssafsound/domain/recruit/service/RecruitService.java b/src/main/java/com/ssafy/ssafsound/domain/recruit/service/RecruitService.java
@@ -161,7 +161,8 @@ public void expiredRecruit(Long recruitId, Long memberId) {
     public GetRecruitsResDto getMemberJoinRecruits(GetMemberJoinRecruitsReqDto recruitsReqDto, Long loginMemberId) {
         Long memberId = recruitsReqDto.getMemberId();
         Member member = memberRepository.findById(memberId).orElseThrow(()->new ResourceNotFoundException(GlobalErrorInfo.NOT_FOUND));
-        if(!member.getPublicProfile()) {
+
+        if(!memberId.equals(loginMemberId) && !member.getPublicProfile()) {
             throw new MemberException(MemberErrorInfo.MEMBER_PROFILE_SECRET);
         }
 

diff --git a/...java/com/ssafy/ssafsound/domain/recruitapplication/service/RecruitApplicationService.java b/...java/com/ssafy/ssafsound/domain/recruitapplication/service/RecruitApplicationService.java
@@ -66,7 +66,10 @@ public PatchRecruitApplicationStatusResDto saveRecruitApplication(Long recruitId
                 metaDataConsumer.getMetaData(MetaDataType.RECRUIT_TYPE.name(), recruitType));
         List<RecruitQuestionReply> participantAnswers = makeRecruitQuestionReplies(postRecruitApplicationReqDto, recruit, recruitApplication);
         recruitApplicationRepository.save(recruitApplication);
-        recruitQuestionReplyRepository.saveAll(participantAnswers);
+
+        if(participantAnswers.size() != 0) {
+            recruitQuestionReplyRepository.saveAll(participantAnswers);
+        }
         return new PatchRecruitApplicationStatusResDto(recruitApplication.getId(), MatchStatus.PENDING.name());
     }
 
@@ -211,7 +214,7 @@ private List<RecruitQuestionReply>  makeRecruitQuestionReplies(PostRecruitApplic
         List<String> answers = dto.getContents();
 
         int len = recruitQuestion.size();
-        if(answers.size() != len) {
+        if(len!=0 && answers.size() != len) {
             throw new RecruitException(RecruitErrorInfo.NOT_SAME_LENGTH_RECRUIT_QUESTION_ANSWER);
         }
 
@@ -224,7 +227,7 @@ private List<RecruitQuestionReply>  makeRecruitQuestionReplies(PostRecruitApplic
                     RecruitQuestionReply.builder()
                             .application(recruitApplication)
                             .question(recruitQuestion.get(i))
-                            .content(dto.getContents().get(i))
+                            .content(answers.get(i))
                             .build()
             );
         }