⚡ MarkSphere v1.0.11

북마크 갱신 시 url 필요한 버그 수정

Author: son0307

src/main/java/com/sonkim/bookmarking/domain/bookmark/controller/BookmarkController.java

@@ -3,8 +3,8 @@
 import com.sonkim.bookmarking.auth.entity.UserDetailsImpl;
 import com.sonkim.bookmarking.common.s3.dto.PresignedUrlDto;
 import com.sonkim.bookmarking.common.s3.service.S3Service;
-import com.sonkim.bookmarking.domain.bookmark.dto.BookmarkRequestDto;
 import com.sonkim.bookmarking.domain.bookmark.dto.BookmarkResponseDto;
+import com.sonkim.bookmarking.domain.bookmark.dto.BookmarkUpdateDto;
 import com.sonkim.bookmarking.domain.bookmark.service.BookmarkService;
 import com.sonkim.bookmarking.domain.bookmark.dto.BookmarkOGDto;
 import com.sonkim.bookmarking.common.util.OGUtil;
@@ -53,7 +53,7 @@ public ResponseEntity<BookmarkResponseDto> getBookmark(
     @PatchMapping("/{bookmarkId}")
     public ResponseEntity<String> updateBookmark(@AuthenticationPrincipal UserDetailsImpl userDetails,
                                             @PathVariable("bookmarkId") Long bookmarkId,
-                                            @RequestBody @Valid BookmarkRequestDto bookmarkRequestDto) {
+                                            @RequestBody @Valid BookmarkUpdateDto bookmarkRequestDto) {
         log.info("userId: {}, bookmarkId: {} 북마크 수정 요청", userDetails.getId(), bookmarkId);
         bookmarkService.updateBookmark(userDetails.getId(), bookmarkId, bookmarkRequestDto);
         return ResponseEntity.ok("bookmarkId: " + bookmarkId + " updated");

src/main/java/com/sonkim/bookmarking/domain/bookmark/dto/BookmarkUpdateDto.java

@@ -0,0 +1,33 @@
+package com.sonkim.bookmarking.domain.bookmark.dto;
+
+import jakarta.validation.constraints.Max;
+import jakarta.validation.constraints.Min;
+import jakarta.validation.constraints.Size;
+import lombok.Data;
+
+import java.util.List;
+
+@Data
+public class BookmarkUpdateDto {
+    private Long categoryId;
+    private String url;
+
+    @Size(max = 100, message = "제목은 100자를 초과할 수 없습니다.")
+    private String title;
+
+    @Size(max = 5000)
+    private String description;
+
+    @Size(max = 10, message = "태그는 최대 10개까지만 등록할 수 있습니다.")
+    private List<String> tagNames;
+
+    @Min(value = -90, message = "위도는 -90 이상이어야 합니다.")
+    @Max(value = 90, message = "위도는 90 이하여야 합니다.")
+    private Double latitude;
+
+    @Min(value = -180, message = "경도는 -180 이상이어야 합니다.")
+    @Max(value = 180, message = "경도는 180 이하여야 합니다.")
+    private Double longitude;
+    private String imageKey;            // S3에 직접 업로드한 경우의 파일 키
+    private String originalImageUrl;    // OG 정보의 이미지 URL
+}

src/main/java/com/sonkim/bookmarking/domain/bookmark/entity/Bookmark.java

@@ -1,7 +1,7 @@
 package com.sonkim.bookmarking.domain.bookmark.entity;
 
+import com.sonkim.bookmarking.domain.bookmark.dto.BookmarkUpdateDto;
 import com.sonkim.bookmarking.domain.user.entity.User;
-import com.sonkim.bookmarking.domain.bookmark.dto.BookmarkRequestDto;
 import com.sonkim.bookmarking.domain.category.entity.Category;
 import com.sonkim.bookmarking.domain.team.entity.Team;
 import jakarta.persistence.*;
@@ -54,7 +54,6 @@ public class Bookmark {
     private String originalImageUrl;
 
     // 최종 이미지 파일 키
-    @Column(length = 255)
     private String imageKey;
 
     private Double latitude;        // 위도
@@ -84,7 +83,7 @@ public void updateCategory(Category category) {
         this.category = category;
     }
 
-    public void update(BookmarkRequestDto dto) {
+    public void update(BookmarkUpdateDto dto) {
         if(dto.getTitle() != null) this.title = dto.getTitle();
         if(dto.getDescription() != null) this.description = dto.getDescription();
         if(dto.getLatitude() != null) {

src/main/java/com/sonkim/bookmarking/domain/bookmark/service/BookmarkService.java

@@ -3,9 +3,7 @@
 import com.sonkim.bookmarking.common.dto.CursorResultDto;
 import com.sonkim.bookmarking.common.s3.service.S3Service;
 import com.sonkim.bookmarking.common.service.BookmarkCreatedEvent;
-import com.sonkim.bookmarking.domain.bookmark.dto.BookmarkResponseDto;
-import com.sonkim.bookmarking.domain.bookmark.dto.BookmarkSearchCond;
-import com.sonkim.bookmarking.domain.bookmark.dto.LikeCountDto;
+import com.sonkim.bookmarking.domain.bookmark.dto.*;
 import com.sonkim.bookmarking.domain.bookmark.entity.BookmarkTag;
 import com.sonkim.bookmarking.domain.bookmark.repository.BookmarkLikeRepository;
 import com.sonkim.bookmarking.domain.bookmark.repository.BookmarkTagRepository;
@@ -14,7 +12,6 @@
 import com.sonkim.bookmarking.domain.tag.service.TagService;
 import com.sonkim.bookmarking.domain.user.entity.User;
 import com.sonkim.bookmarking.domain.user.service.UserService;
-import com.sonkim.bookmarking.domain.bookmark.dto.BookmarkRequestDto;
 import com.sonkim.bookmarking.domain.bookmark.entity.Bookmark;
 import com.sonkim.bookmarking.domain.bookmark.repository.BookmarkRepository;
 import com.sonkim.bookmarking.domain.category.service.CategoryService;
@@ -168,7 +165,7 @@ public BookmarkResponseDto getBookmarkDetails(Long userID, Long bookmarkId) {
 
     // 북마크 정보 갱신
     @Transactional
-    public void updateBookmark(Long userId, Long bookmarkId, BookmarkRequestDto dto) {
+    public void updateBookmark(Long userId, Long bookmarkId, BookmarkUpdateDto dto) {
         Bookmark bookmark = getBookmarkById(bookmarkId);
 
         // 그룹 상태 검증