[BE] 삭제된 토픽, 핀이 보이는 버그 해결

.github/workflows/be-merge-dev.yml

@@ -4,7 +4,7 @@ on:
   workflow_dispatch:
 
   pull_request:
-    branches: [ develop-BE-2 ]
+    branches: [ develop-BE ]
     types: [ closed ]
     paths: backend/**
 

.github/workflows/be-pull-request.yml

@@ -4,7 +4,7 @@ on:
   workflow_dispatch:
 
   pull_request:
-    branches: [ main, develop-BE-2 ]
+    branches: [ main, develop-BE ]
     paths: backend/**
 
 permissions:

backend/src/main/java/com/mapbefine/mapbefine/admin/application/AdminCommandService.java

@@ -1,5 +1,6 @@
 package com.mapbefine.mapbefine.admin.application;
 
+import static com.mapbefine.mapbefine.pin.exception.PinErrorCode.PIN_NOT_FOUND;
 import static com.mapbefine.mapbefine.topic.exception.TopicErrorCode.TOPIC_NOT_FOUND;
 
 import com.mapbefine.mapbefine.atlas.domain.AtlasRepository;
@@ -11,14 +12,14 @@
 import com.mapbefine.mapbefine.pin.domain.Pin;
 import com.mapbefine.mapbefine.pin.domain.PinImageRepository;
 import com.mapbefine.mapbefine.pin.domain.PinRepository;
+import com.mapbefine.mapbefine.pin.exception.PinException.PinNotFoundException;
 import com.mapbefine.mapbefine.topic.domain.Topic;
 import com.mapbefine.mapbefine.topic.domain.TopicRepository;
 import com.mapbefine.mapbefine.topic.exception.TopicException;
-import org.springframework.stereotype.Service;
-import org.springframework.transaction.annotation.Transactional;
-
 import java.util.List;
 import java.util.NoSuchElementException;
+import org.springframework.stereotype.Service;
+import org.springframework.transaction.annotation.Transactional;
 
 @Service
 @Transactional
@@ -61,12 +62,15 @@ private void deleteAllRelatedMember(Member member) {
         List<Long> pinIds = extractPinIdsByMember(member);
         Long memberId = member.getId();
 
-        pinImageRepository.deleteAllByPinIds(pinIds);
-        topicRepository.deleteAllByMemberId(memberId);
-        pinRepository.deleteAllByMemberId(memberId);
         permissionRepository.deleteAllByMemberId(memberId);
+        permissionRepository.flush();
         atlasRepository.deleteAllByMemberId(memberId);
+        atlasRepository.flush();
         bookmarkRepository.deleteAllByMemberId(memberId);
+        bookmarkRepository.flush();
+        pinImageRepository.deleteAllByPinIds(pinIds);
+        pinRepository.deleteAllByMemberId(memberId);
+        topicRepository.deleteAllByMemberId(memberId);
     }
 
     private Member findMemberById(Long id) {
@@ -82,21 +86,44 @@ private List<Long> extractPinIdsByMember(Member member) {
     }
 
     public void deleteTopic(Long topicId) {
+        Topic topic = findTopicById(topicId);
+        List<Long> pinIds = extractPinIdsByTopic(topic);
+
+        permissionRepository.deleteAllByTopicId(topicId);
+        permissionRepository.flush();
+        atlasRepository.deleteAllByTopicId(topicId);
+        atlasRepository.flush();
+        bookmarkRepository.deleteAllByTopicId(topicId);
+        bookmarkRepository.flush();
+        pinImageRepository.deleteAllByPinIds(pinIds);
+        pinRepository.deleteAllByTopicId(topicId);
         topicRepository.deleteById(topicId);
     }
 
+    private List<Long> extractPinIdsByTopic(Topic topic) {
+        return topic.getPins()
+                .stream()
+                .map(Pin::getId)
+                .toList();
+    }
+
     public void deleteTopicImage(Long topicId) {
         Topic topic = findTopicById(topicId);
         topic.removeImage();
     }
 
     private Topic findTopicById(Long topicId) {
-        return topicRepository.findByIdAndIsDeletedFalse(topicId)
+        return topicRepository.findById(topicId)
                 .orElseThrow(() -> new TopicException.TopicNotFoundException(TOPIC_NOT_FOUND, List.of(topicId)));
     }
 
     public void deletePin(Long pinId) {
-        pinRepository.deleteById(pinId);
+        Pin pin = pinRepository.findById(pinId)
+                .orElseThrow(() -> new PinNotFoundException(PIN_NOT_FOUND, pinId));
+
+        pin.decreaseTopicPinCount();
+        pinImageRepository.deleteAllByPinId(pinId);
+        pinRepository.deleteById(pin.getId());
     }
 
     public void deletePinImage(Long pinImageId) {

backend/src/main/java/com/mapbefine/mapbefine/atlas/application/AtlasCommandService.java

@@ -13,11 +13,10 @@
 import com.mapbefine.mapbefine.member.domain.MemberRepository;
 import com.mapbefine.mapbefine.topic.domain.Topic;
 import com.mapbefine.mapbefine.topic.domain.TopicRepository;
-import org.springframework.stereotype.Service;
-import org.springframework.transaction.annotation.Transactional;
-
 import java.util.NoSuchElementException;
 import java.util.Objects;
+import org.springframework.stereotype.Service;
+import org.springframework.transaction.annotation.Transactional;
 
 @Service
 @Transactional
@@ -73,7 +72,7 @@ private boolean isTopicAlreadyAdded(Long topicId, Long memberId) {
     }
 
     private Topic findTopicById(Long topicId) {
-        return topicRepository.findByIdAndIsDeletedFalse(topicId)
+        return topicRepository.findById(topicId)
                 .orElseThrow(() -> new AtlasBadRequestException(ILLEGAL_TOPIC_ID));
     }
 

backend/src/main/java/com/mapbefine/mapbefine/atlas/domain/AtlasRepository.java

@@ -7,7 +7,10 @@
 public interface AtlasRepository extends JpaRepository<Atlas, Long> {
 
     boolean existsByMemberIdAndTopicId(Long memberId, Long topicId);
+
     void deleteByMemberIdAndTopicId(Long memberId, Long topicId);
 
     void deleteAllByMemberId(Long memberId);
+
+    void deleteAllByTopicId(Long topicId);
 }

backend/src/main/java/com/mapbefine/mapbefine/auth/application/AuthService.java

@@ -11,7 +11,6 @@
 import com.mapbefine.mapbefine.topic.domain.Topic;
 import java.util.List;
 import java.util.Objects;
-import java.util.Optional;
 import org.springframework.stereotype.Service;
 import org.springframework.transaction.annotation.Transactional;
 
@@ -32,9 +31,12 @@ public void validateMember(Long memberId) {
     }
 
     public AuthMember findAuthMemberByMemberId(Long memberId) {
-        return memberRepository.findById(memberId)
-                .map(this::convertToAuthMember)
+        Member member = memberRepository.findById(memberId)
                 .orElseThrow(() -> new IllegalArgumentException("findAuthMemberByMemberId; memberId= " + memberId));
+        if (member.isNormalStatus()) {
+            return convertToAuthMember(member);
+        }
+        throw new AuthUnauthorizedException(AuthErrorCode.ILLEGAL_MEMBER_ID);
     }
 
     private AuthMember convertToAuthMember(Member member) {
@@ -63,15 +65,4 @@ private List<Long> getCreatedTopics(Member member) {
                 .toList();
     }
 
-    public boolean isAdmin(Long memberId) {
-        if (Objects.isNull(memberId)) {
-            return false;
-        }
-
-        Optional<Member> member = memberRepository.findById(memberId);
-
-        return member.map(Member::isAdmin)
-                .orElse(false);
-    }
-
 }

backend/src/main/java/com/mapbefine/mapbefine/bookmark/application/BookmarkCommandService.java

@@ -15,11 +15,10 @@
 import com.mapbefine.mapbefine.member.domain.MemberRepository;
 import com.mapbefine.mapbefine.topic.domain.Topic;
 import com.mapbefine.mapbefine.topic.domain.TopicRepository;
+import java.util.NoSuchElementException;
 import org.springframework.stereotype.Service;
 import org.springframework.transaction.annotation.Transactional;
 
-import java.util.NoSuchElementException;
-
 @Service
 @Transactional
 public class BookmarkCommandService {
@@ -53,7 +52,7 @@ public Long addTopicInBookmark(AuthMember authMember, Long topicId) {
     }
 
     private Topic getTopicById(Long topicId) {
-        return topicRepository.findByIdAndIsDeletedFalse(topicId)
+        return topicRepository.findById(topicId)
                 .orElseThrow(() -> new BookmarkBadRequestException(ILLEGAL_TOPIC_ID));
     }
 

backend/src/main/java/com/mapbefine/mapbefine/bookmark/domain/BookmarkRepository.java

@@ -1,17 +1,16 @@
 package com.mapbefine.mapbefine.bookmark.domain;
 
-import org.springframework.data.jpa.repository.JpaRepository;
-import org.springframework.data.jpa.repository.Modifying;
-
 import java.util.Optional;
+import org.springframework.data.jpa.repository.JpaRepository;
 
 public interface BookmarkRepository extends JpaRepository<Bookmark, Long> {
 
+    Optional<Bookmark> findByMemberIdAndTopicId(Long memberId, Long topicId);
+
     boolean existsByMemberIdAndTopicId(Long memberId, Long topicId);
 
-    @Modifying(clearAutomatically = true)
     void deleteAllByMemberId(Long memberId);
 
-    Optional<Bookmark> findByMemberIdAndTopicId(Long memberId, Long topicId);
+    void deleteAllByTopicId(Long topicId);
 
 }

backend/src/main/java/com/mapbefine/mapbefine/member/application/MemberCommandService.java

@@ -26,7 +26,7 @@ public void updateInfoById(AuthMember authMember, MemberUpdateRequest request) {
 
         validateNicknameDuplicated(nickName);
 
-        member.update(nickName);
+        member.updateNickName(nickName);
     }
 
     private Member findMemberById(Long memberId) {

backend/src/main/java/com/mapbefine/mapbefine/member/application/MemberQueryService.java

@@ -66,7 +66,7 @@ public List<MemberResponse> findAll() {
     public List<TopicResponse> findAllTopicsInBookmark(AuthMember authMember) {
         Member member = findMemberById(authMember.getMemberId());
 
-        List<Topic> bookMarkedTopics = topicRepository.findTopicsByBookmarksMemberIdAndIsDeletedFalse(
+        List<Topic> bookMarkedTopics = topicRepository.findTopicsByBookmarksMemberId(
                 authMember.getMemberId());
         return bookMarkedTopics.stream()
                 .map(topic -> TopicResponse.from(
@@ -90,7 +90,6 @@ private boolean isInAtlas(Long memberId, Long topicId) {
 
     public List<TopicResponse> findAllTopicsInAtlas(AuthMember authMember) {
         Member member = findMemberById(authMember.getMemberId());
-
         List<Topic> topicsInAtlas = findTopicsInAtlas(member);
 
         return topicsInAtlas.stream()
@@ -107,7 +106,6 @@ private boolean isInBookmark(Long memberId, Long topicId) {
     }
 
     public List<TopicResponse> findMyAllTopics(AuthMember authMember) {
-
         Long memberId = authMember.getMemberId();
         Member member = findMemberById(memberId);
 

backend/src/main/java/com/mapbefine/mapbefine/member/domain/Member.java

@@ -104,12 +104,16 @@ private static String createNicknameSuffix() {
                 .substring(0, DEFAULT_NICKNAME_SUFFIX_LENGTH);
     }
 
-    public void update(
+    public void updateNickName(
             String nickName
     ) {
         memberInfo = memberInfo.createUpdatedMemberInfo(nickName);
     }
 
+    public void updateStatus(Status status) {
+        memberInfo = memberInfo.createUpdatedMemberInfo(status);
+    }
+
     public void addTopic(Topic topic) {
         createdTopics.add(topic);
     }
@@ -147,14 +151,4 @@ public List<Topic> getTopicsWithPermissions() {
     public boolean isNormalStatus() {
         return memberInfo.getStatus() == Status.NORMAL;
     }
-
-    public void updateStatus(Status status) {
-        memberInfo = MemberInfo.of(
-                memberInfo.getNickName(),
-                memberInfo.getEmail(),
-                memberInfo.getImageUrl(),
-                memberInfo.getRole(),
-                status
-        );
-    }
 }

backend/src/main/java/com/mapbefine/mapbefine/member/domain/MemberInfo.java

@@ -113,6 +113,11 @@ public MemberInfo createUpdatedMemberInfo(String nickName) {
         return MemberInfo.of(nickName, this.email, this.imageUrl.getImageUrl(), this.role, this.status);
     }
 
+    public MemberInfo createUpdatedMemberInfo(Status status) {
+
+        return MemberInfo.of(this.nickName, this.email, this.imageUrl.getImageUrl(), this.role, status);
+    }
+
     public String getImageUrl() {
         return imageUrl.getImageUrl();
     }

backend/src/main/java/com/mapbefine/mapbefine/permission/application/PermissionCommandService.java

@@ -15,11 +15,10 @@
 import com.mapbefine.mapbefine.permission.exception.PermissionException.PermissionForbiddenException;
 import com.mapbefine.mapbefine.topic.domain.Topic;
 import com.mapbefine.mapbefine.topic.domain.TopicRepository;
-import org.springframework.stereotype.Service;
-import org.springframework.transaction.annotation.Transactional;
-
 import java.util.List;
 import java.util.Objects;
+import org.springframework.stereotype.Service;
+import org.springframework.transaction.annotation.Transactional;
 
 @Service
 @Transactional
@@ -78,7 +77,7 @@ private boolean isNotSelfMember(AuthMember authMember, Member member) {
 
     private Topic findTopic(PermissionRequest request) {
         Long topicId = request.topicId();
-        return topicRepository.findByIdAndIsDeletedFalse(topicId)
+        return topicRepository.findById(topicId)
                 .orElseThrow(() -> new PermissionBadRequestException(ILLEGAL_TOPIC_ID, topicId));
     }
 

backend/src/main/java/com/mapbefine/mapbefine/permission/application/PermissionQueryService.java

@@ -14,11 +14,10 @@
 import com.mapbefine.mapbefine.topic.domain.TopicStatus;
 import com.mapbefine.mapbefine.topic.exception.TopicErrorCode;
 import com.mapbefine.mapbefine.topic.exception.TopicException.TopicNotFoundException;
+import java.util.List;
 import org.springframework.stereotype.Service;
 import org.springframework.transaction.annotation.Transactional;
 
-import java.util.List;
-
 @Service
 @Transactional(readOnly = true)
 public class PermissionQueryService {
@@ -43,12 +42,13 @@ public TopicAccessDetailResponse findTopicAccessDetailById(Long topicId) {
     }
 
     private Publicity findTopicPublicityById(Long topicId) {
-        return topicRepository.findByIdAndIsDeletedFalse(topicId)
+        return topicRepository.findById(topicId)
                 .map(Topic::getTopicStatus)
                 .map(TopicStatus::getPublicity)
                 .orElseThrow(() -> new TopicNotFoundException(TopicErrorCode.TOPIC_NOT_FOUND, topicId));
     }
 
+    @Deprecated(since = "2023.10.06")
     public PermissionMemberDetailResponse findPermissionById(Long permissionId) {
         Permission permission = permissionRepository.findById(permissionId)
                 .orElseThrow(() -> new PermissionNotFoundException(PERMISSION_NOT_FOUND, permissionId));

backend/src/main/java/com/mapbefine/mapbefine/permission/domain/PermissionRepository.java

@@ -10,4 +10,7 @@ public interface PermissionRepository extends JpaRepository<Permission, Long> {
     boolean existsByTopicIdAndMemberId(Long topicId, Long memberId);
 
     void deleteAllByMemberId(Long memberId);
+
+    void deleteAllByTopicId(Long topicId);
+
 }

backend/src/main/java/com/mapbefine/mapbefine/permission/presentation/PermissionController.java

@@ -56,8 +56,7 @@ public ResponseEntity<TopicAccessDetailResponse> findTopicAccessDetailByTopicId(
         return ResponseEntity.ok(response);
     }
 
-    // TODO 이 API를 쓰는 곳이 있나? + 결국 특정 회원을 조회하는 건데 어떤 API인지 알기 어렵다..
-    //  회원 정보 조회는 /members 에서 하는 걸로 충분하지 않나? 재사용성이 떨어진다. 테스트의 DisplayName도 매칭이 안된다.
+    @Deprecated(since = "2023.10.06")
     @LoginRequired
     @GetMapping("/{permissionId}")
     public ResponseEntity<PermissionMemberDetailResponse> findPermissionById(@PathVariable Long permissionId) {