fix : 탈퇴 회원인 경우 필터링 추가 #166
Conversation
프로필 검색, 좋아요 멤버 조회 시 필터링하여 응답하도록 수정 Related to: #165
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
| @@ -197,6 +198,7 @@ private void applyRoleTypeFilter(ProfileSearchReqDto searchCriteria, BooleanBuil | |||
| } | |||
|
|
|||
| private List<Profile> executeQuery(BooleanBuilder builder,List<OrderSpecifier<?>> orderSpecifiers, Pageable pageable) { | |||
| builder.and(member.memberStatus.ne(MemberStatus.WITHDRAW)); | |||
| return queryFactory.selectFrom(profile).distinct() | |||
| .leftJoin(profile.districts, profileArea) | |||
| .join(profile.member, member) | |||
There was a problem hiding this comment.
해당 코드 패치는 다음과 같습니다:
- 추가된 import 문으로
com.kuddy.common.member.domain.MemberStatus를 가져왔습니다. applyRoleTypeFilter메서드에서ProfileSearchReqDto와BooleanBuilder매개변수를 받습니다.executeQuery메서드에서BooleanBuilder,List<OrderSpecifier<?>>,Pageable매개변수를 받습니다.executeQuery메서드 내에builder.and(member.memberStatus.ne(MemberStatus.WITHDRAW));코드가 추가되었습니다.
버그나 개선 제안은 주어진 코드 조각만으로는 파악하기 어려우므로 전체 코드나 더 많은 컨텍스트를 제공해주시면 더 구체적인 리뷰를 제공할 수 있습니다.
| kuddyList.add(PickMemberResDto.of(heart.getMember())); | ||
| if(heart.getMember().getRoleType().equals(RoleType.TRAVELER)) | ||
| if(heart.getMember().getRoleType().equals(RoleType.TRAVELER) && !heart.getMember().getMemberStatus().equals( | ||
| MemberStatus.WITHDRAW)) | ||
| travelerList.add(PickMemberResDto.of(heart.getMember())); | ||
| } | ||
|
|
There was a problem hiding this comment.
이 코드 패치의 간단한 코드 리뷰를 도와드리겠습니다. 버그 위험 및 개선 제안사항을 환영합니다:
-
MemberStatus를 import하는 것은 좋은 개선 사항입니다.import com.kuddy.common.member.domain.MemberStatus;가 추가되었는데,MemberStatus.WITHDRAW를 사용하여 회원 상태를 확인하고 있습니다. -
responseDetailInfo메소드에서heart.getMember().getRoleType().equals(RoleType.KUDDY) && !heart.getMember().getMemberStatus().equals(MemberStatus.WITHDRAW)와heart.getMember().getRoleType().equals(RoleType.TRAVELER) && !heart.getMember().getMemberStatus().equals(MemberStatus.WITHDRAW)로 회원 상태를 검사하고 있는데, 이 부분은 정상적으로 작동할 것으로 보입니다. -
다른 부분에서 보이지 않아서 정확한 판단은 어렵지만, 이 코드 패치에 특정한 버그 위험이나 개선 사항은 없어 보입니다.
코드 패치는 회원의 역할 유형과 회원 상태를 기준으로 "KUDDY"와 "TRAVELER" 회원을 필터링하여 kuddyList와 travelerList에 추가하는 것으로 보입니다.
기능 명세
결과
api/v1/spots/1747895
api/v1/profiles/search?page=0&size=17
기존의 응답값과 동일합니다.
함께 의논할 점