Merge pull request #284 from CodIN-INU/revert-283-develop

Revert "hotfix: 닉네임 변경 기능에서 삭제된 닉네임, 자기 자신의 닉네임 사용가능하도록 수정"

Author: gisu1102

src/main/java/inu/codin/codin/common/security/service/AuthCommonService.java

@@ -2,7 +2,6 @@
 
 import inu.codin.codin.common.exception.NotFoundException;
 import inu.codin.codin.common.security.dto.SignUpAndLoginRequestDto;
-import inu.codin.codin.common.security.util.SecurityUtils;
 import inu.codin.codin.domain.user.dto.request.UserProfileRequestDto;
 import inu.codin.codin.domain.user.entity.UserEntity;
 import inu.codin.codin.domain.user.exception.UserCreateFailException;
@@ -34,7 +33,7 @@ public AuthCommonService(UserRepository userRepository, S3Service s3Service, Jwt
     }
 
     public void completeUserProfile(UserProfileRequestDto userProfileRequestDto, MultipartFile userImage, HttpServletResponse response) {
-        Optional<UserEntity> nickNameDuplicate = userRepository.findByNicknameAndDeletedAtIsNull(SecurityUtils.getCurrentUserId(), userProfileRequestDto.getNickname());
+        Optional<UserEntity> nickNameDuplicate = userRepository.findByNicknameAndDeletedAtIsNull(userProfileRequestDto.getNickname());
         if (nickNameDuplicate.isPresent()){
             throw new UserNicknameDuplicateException("이미 사용중인 닉네임입니다.");
         }

src/main/java/inu/codin/codin/domain/user/repository/UserRepository.java

@@ -18,8 +18,7 @@ public interface UserRepository extends MongoRepository<UserEntity, ObjectId> {
     @Query("{'email':  ?0, 'deletedAt': null, 'status':  { $in:  ['ACTIVE'] }}")
     Optional<UserEntity> findByEmail(String email);
 
-    @Query("{'_id': {$ne: ?0}, 'nickname': ?1, 'deletedAt': null}")
-    Optional<UserEntity> findByNicknameAndDeletedAtIsNull(ObjectId userId, String nickname);
+    Optional<UserEntity> findByNicknameAndDeletedAtIsNull(String nickname);
 
     @Query("{'email':  ?0, 'deletedAt': null, 'status':  { $in:  ['DISABLED'] }}")
     Optional<UserEntity> findByEmailAndDisabled(String email);

src/main/java/inu/codin/codin/domain/user/service/UserService.java

@@ -30,7 +30,6 @@
 import org.springframework.data.domain.Page;
 import org.springframework.data.domain.PageRequest;
 import org.springframework.data.domain.Sort;
-import org.springframework.security.core.context.SecurityContextHolder;
 import org.springframework.stereotype.Service;
 import org.springframework.web.multipart.MultipartFile;
 
@@ -149,8 +148,9 @@ public UserInfoResponseDto getUserInfo() {
         return UserInfoResponseDto.of(user);
     }
     public void updateUserInfo(@Valid UserNicknameRequestDto userNicknameRequestDto) {
-        Optional<UserEntity> nickNameDuplicate = userRepository.findByNicknameAndDeletedAtIsNull(SecurityUtils.getCurrentUserId(), userNicknameRequestDto.getNickname());
-        if (nickNameDuplicate.isPresent()) {
+
+        Optional<UserEntity> nickNameDuplicate = userRepository.findByNicknameAndDeletedAtIsNull(userNicknameRequestDto.getNickname());
+        if (nickNameDuplicate.isPresent()){
             throw new UserNicknameDuplicateException("이미 사용중인 닉네임입니다.");
         }