Merge pull request #46 from HaRu-Developers/fix/#45-users-info-update-fix

fix/#45: 회원 정보 수정 시 jwt token에서 userId 가져오도록 수정

Author: hknhj

src/main/java/com/haru/api/domain/user/controller/UserController.java

@@ -85,11 +85,12 @@ public ApiResponse<UserResponseDTO.User> getUserInfo() {
             "# 회원 정보 수정 API 입니다. \n" +
                     "현재는 jwt token을 구현하지 않아 pathvariable로 userId를 넣어주세요.추후 jwt token이 구현되면 수정하겠습니다."
     )
-    @PatchMapping("/{userId}/info")
+    @PatchMapping("/info")
     public ApiResponse<UserResponseDTO.User> updateUserInfo(
-            @PathVariable Long userId,
             @RequestBody @Valid UserRequestDTO.UserInfoUpdateRequest request
     ) {
+        Long userId = SecurityUtil.getCurrentUserId();
+
         UserResponseDTO.User user = userCommandService.updateUserInfo(userId, request);
 
         return ApiResponse.onSuccess(user);

src/main/java/com/haru/api/domain/user/service/UserCommandServiceImpl.java

@@ -95,15 +95,17 @@ public void logout(String accessToken) {
     }
 
 
-    @Override
     @Transactional
+    @Override
     public UserResponseDTO.User updateUserInfo(Long userId, UserRequestDTO.UserInfoUpdateRequest request) {
         String name = request.getName();
-        Users user = userRepository.findById(userId)
+
+        Users foundUser = userRepository.findById(userId)
                 .orElseThrow(() -> new MemberHandler(ErrorStatus.MEMBER_NOT_FOUND));
-        user.setName(name);
 
-        return UserConverter.toUserDTO(user);
+        foundUser.setName(name);
+
+        return UserConverter.toUserDTO(foundUser);
     }
 
     private String generateAccessToken(Long userId, int accessExpTime) {