refactor(review): 이미지 관련 로직 제거

leebs0521 · leebs0521 · commit 6be45768b869 · 2025-01-24T13:29:21.000+09:00
diff --git a/src/main/java/com/somemore/domains/review/controller/ReviewCommandApiController.java b/src/main/java/com/somemore/domains/review/controller/ReviewCommandApiController.java
@@ -1,16 +1,12 @@
 package com.somemore.domains.review.controller;
 
-import static org.springframework.http.MediaType.MULTIPART_FORM_DATA_VALUE;
-
 import com.somemore.domains.review.dto.request.ReviewCreateRequestDto;
 import com.somemore.domains.review.dto.request.ReviewUpdateRequestDto;
 import com.somemore.domains.review.usecase.CreateReviewUseCase;
 import com.somemore.domains.review.usecase.DeleteReviewUseCase;
 import com.somemore.domains.review.usecase.UpdateReviewUseCase;
 import com.somemore.global.auth.annotation.RoleId;
 import com.somemore.global.common.response.ApiResponse;
-import com.somemore.global.imageupload.dto.ImageUploadRequestDto;
-import com.somemore.global.imageupload.usecase.ImageUploadUseCase;
 import io.swagger.v3.oas.annotations.Operation;
 import io.swagger.v3.oas.annotations.tags.Tag;
 import jakarta.validation.Valid;
@@ -23,9 +19,7 @@
 import org.springframework.web.bind.annotation.PutMapping;
 import org.springframework.web.bind.annotation.RequestBody;
 import org.springframework.web.bind.annotation.RequestMapping;
-import org.springframework.web.bind.annotation.RequestPart;
 import org.springframework.web.bind.annotation.RestController;
-import org.springframework.web.multipart.MultipartFile;
 
 @Tag(name = "Review Command API", description = "리뷰 생성 수정 삭제 API")
 @RequiredArgsConstructor
@@ -36,20 +30,17 @@ public class ReviewCommandApiController {
     private final CreateReviewUseCase createReviewUseCase;
     private final UpdateReviewUseCase updateReviewUseCase;
     private final DeleteReviewUseCase deleteReviewUseCase;
-    private final ImageUploadUseCase imageUploadUseCase;
 
     @Secured("ROLE_VOLUNTEER")
     @Operation(summary = "리뷰 등록", description = "리뷰를 등록합니다.")
-    @PostMapping(value = "/review", consumes = MULTIPART_FORM_DATA_VALUE)
+    @PostMapping(value = "/review")
     public ApiResponse<Long> createReview(
             @RoleId UUID volunteerId,
-            @Valid @RequestPart("data") ReviewCreateRequestDto requestDto,
-            @RequestPart(value = "img_file", required = false) MultipartFile image) {
+            @Valid @RequestBody ReviewCreateRequestDto requestDto) {
 
-        String imgUrl = imageUploadUseCase.uploadImage(new ImageUploadRequestDto(image));
         return ApiResponse.ok(
                 201,
-                createReviewUseCase.createReview(requestDto, volunteerId, imgUrl),
+                createReviewUseCase.createReview(requestDto, volunteerId),
                 "리뷰 등록 성공"
         );
     }
@@ -70,22 +61,6 @@ public ApiResponse<String> updateReview(
         );
     }
 
-    @Secured("ROLE_VOLUNTEER")
-    @Operation(summary = "리뷰 이미지 수정", description = "리뷰 이미지를 수정합니다.")
-    @PutMapping(value = "/review/{id}", consumes = MULTIPART_FORM_DATA_VALUE)
-    public ApiResponse<String> updateReviewImage(
-            @RoleId UUID volunteerId,
-            @PathVariable Long id,
-            @RequestPart(value = "img_file", required = false) MultipartFile image) {
-
-        String newImgUrl = imageUploadUseCase.uploadImage(new ImageUploadRequestDto(image));
-        updateReviewUseCase.updateReviewImageUrl(id, volunteerId, newImgUrl);
-        return ApiResponse.ok(
-                200,
-                "리뷰 이미지 수정 성공"
-        );
-    }
-
     @Secured("ROLE_VOLUNTEER")
     @Operation(summary = "리뷰 삭제", description = "리뷰를 삭제합니다.")
     @DeleteMapping(value = "/review/{id}")
diff --git a/src/main/java/com/somemore/domains/review/domain/Review.java b/src/main/java/com/somemore/domains/review/domain/Review.java
@@ -1,5 +1,8 @@
 package com.somemore.domains.review.domain;
 
+import static jakarta.persistence.GenerationType.IDENTITY;
+import static lombok.AccessLevel.PROTECTED;
+
 import com.somemore.domains.review.dto.request.ReviewUpdateRequestDto;
 import com.somemore.global.common.entity.BaseEntity;
 import jakarta.persistence.Column;
@@ -8,15 +11,11 @@
 import jakarta.persistence.Id;
 import jakarta.persistence.Lob;
 import jakarta.persistence.Table;
+import java.util.UUID;
 import lombok.Builder;
 import lombok.Getter;
 import lombok.NoArgsConstructor;
 
-import java.util.UUID;
-
-import static jakarta.persistence.GenerationType.IDENTITY;
-import static lombok.AccessLevel.PROTECTED;
-
 @Getter
 @NoArgsConstructor(access = PROTECTED)
 @Entity
@@ -40,17 +39,12 @@ public class Review extends BaseEntity {
     @Column(name = "content", length = 1000, nullable = false)
     private String content;
 
-    @Column(name = "img_url", nullable = false)
-    private String imgUrl;
-
     @Builder
-    public Review(Long volunteerApplyId, UUID volunteerId, String title,
-                  String content, String imgUrl) {
+    public Review(Long volunteerApplyId, UUID volunteerId, String title, String content) {
         this.volunteerApplyId = volunteerApplyId;
         this.volunteerId = volunteerId;
         this.title = title;
         this.content = content;
-        this.imgUrl = imgUrl;
     }
 
     public boolean isWriter(UUID volunteerId) {
@@ -62,7 +56,4 @@ public void updateWith(ReviewUpdateRequestDto dto) {
         this.content = dto.content();
     }
 
-    public void updateWith(String imgUrl) {
-        this.imgUrl = imgUrl;
-    }
 }
diff --git a/src/main/java/com/somemore/domains/review/dto/request/ReviewCreateRequestDto.java b/src/main/java/com/somemore/domains/review/dto/request/ReviewCreateRequestDto.java
@@ -24,13 +24,12 @@ public record ReviewCreateRequestDto(
         String content
 ) {
 
-    public Review toEntity(UUID volunteerId, String imgUrl) {
+    public Review toEntity(UUID volunteerId) {
         return Review.builder()
                 .volunteerApplyId(volunteerApplyId)
                 .volunteerId(volunteerId)
                 .title(title)
                 .content(content)
-                .imgUrl(imgUrl)
                 .build();
     }
 }
diff --git a/src/main/java/com/somemore/domains/review/dto/response/ReviewDetailResponseDto.java b/src/main/java/com/somemore/domains/review/dto/response/ReviewDetailResponseDto.java
@@ -21,8 +21,6 @@ public record ReviewDetailResponseDto(
         String title,
         @Schema(description = "리뷰 내용", example = "정말 유익했습니다. 더보기..")
         String content,
-        @Schema(description = "이미지 링크", example = "https://image.domain.com/links")
-        String imgUrl,
         @Schema(description = "작성 일자", example = "2024-12-01T09:00:00", type = "string")
         LocalDateTime createdAt,
         @Schema(description = "수정 일자", example = "2024-12-01T09:00:00", type = "string")
@@ -35,7 +33,6 @@ public static ReviewDetailResponseDto from(Review review) {
                 .volunteerId(review.getVolunteerId())
                 .title(review.getTitle())
                 .content(review.getContent())
-                .imgUrl(review.getImgUrl())
                 .createdAt(review.getCreatedAt())
                 .updatedAt(review.getUpdatedAt())
                 .build();
diff --git a/src/main/java/com/somemore/domains/review/dto/response/ReviewDetailWithNicknameResponseDto.java b/src/main/java/com/somemore/domains/review/dto/response/ReviewDetailWithNicknameResponseDto.java
@@ -23,8 +23,6 @@ public record ReviewDetailWithNicknameResponseDto(
         String title,
         @Schema(description = "리뷰 내용", example = "정말 유익했습니다. 더보기..")
         String content,
-        @Schema(description = "이미지 링크", example = "https://image.domain.com/links")
-        String imgUrl,
         @Schema(description = "작성 일자", example = "2024-12-01T09:00:00", type = "string")
         LocalDateTime createdAt,
         @Schema(description = "수정 일자", example = "2024-12-01T09:00:00", type = "string")
@@ -38,7 +36,6 @@ public static ReviewDetailWithNicknameResponseDto from(Review review, String vol
                 .volunteerNickname(volunteerNickname)
                 .title(review.getTitle())
                 .content(review.getContent())
-                .imgUrl(review.getImgUrl())
                 .createdAt(review.getCreatedAt())
                 .updateAt(review.getUpdatedAt())
                 .build();
diff --git a/src/main/java/com/somemore/domains/review/service/CreateReviewService.java b/src/main/java/com/somemore/domains/review/service/CreateReviewService.java
@@ -1,5 +1,8 @@
 package com.somemore.domains.review.service;
 
+import static com.somemore.global.exception.ExceptionMessage.REVIEW_ALREADY_EXISTS;
+import static com.somemore.global.exception.ExceptionMessage.REVIEW_RESTRICTED_TO_ATTENDED;
+
 import com.somemore.domains.review.domain.Review;
 import com.somemore.domains.review.dto.request.ReviewCreateRequestDto;
 import com.somemore.domains.review.repository.ReviewRepository;
@@ -9,15 +12,11 @@
 import com.somemore.domains.volunteerapply.usecase.VolunteerApplyQueryUseCase;
 import com.somemore.global.exception.BadRequestException;
 import com.somemore.global.exception.DuplicateException;
+import java.util.UUID;
 import lombok.RequiredArgsConstructor;
 import org.springframework.stereotype.Service;
 import org.springframework.transaction.annotation.Transactional;
 
-import java.util.UUID;
-
-import static com.somemore.global.exception.ExceptionMessage.REVIEW_ALREADY_EXISTS;
-import static com.somemore.global.exception.ExceptionMessage.REVIEW_RESTRICTED_TO_ATTENDED;
-
 @RequiredArgsConstructor
 @Transactional
 @Service
@@ -28,13 +27,13 @@ public class CreateReviewService implements CreateReviewUseCase {
     private final VolunteerApplyQueryUseCase volunteerApplyQueryUseCase;
 
     @Override
-    public Long createReview(ReviewCreateRequestDto requestDto, UUID volunteerId, String imgUrl) {
+    public Long createReview(ReviewCreateRequestDto requestDto, UUID volunteerId) {
         validateDuplicateReview(requestDto.volunteerApplyId());
 
         VolunteerApply apply = volunteerApplyQueryUseCase.getById(requestDto.volunteerApplyId());
         validateActivityCompletion(apply);
 
-        Review review = requestDto.toEntity(volunteerId, imgUrl);
+        Review review = requestDto.toEntity(volunteerId);
         reviewRepository.save(review);
 
         return review.getId();
diff --git a/src/main/java/com/somemore/domains/review/service/UpdateReviewService.java b/src/main/java/com/somemore/domains/review/service/UpdateReviewService.java
@@ -5,12 +5,11 @@
 import com.somemore.domains.review.service.validator.ReviewValidator;
 import com.somemore.domains.review.usecase.ReviewQueryUseCase;
 import com.somemore.domains.review.usecase.UpdateReviewUseCase;
+import java.util.UUID;
 import lombok.RequiredArgsConstructor;
 import org.springframework.stereotype.Service;
 import org.springframework.transaction.annotation.Transactional;
 
-import java.util.UUID;
-
 @RequiredArgsConstructor
 @Transactional
 @Service
@@ -26,11 +25,4 @@ public void updateReview(Long id, UUID volunteerId, ReviewUpdateRequestDto reque
         review.updateWith(requestDto);
     }
 
-    @Override
-    public void updateReviewImageUrl(Long id, UUID volunteerId, String imgUrl) {
-        Review review = reviewQueryUseCase.getById(id);
-        reviewValidator.validateWriter(review, volunteerId);
-        review.updateWith(imgUrl);
-    }
-
 }
diff --git a/src/main/java/com/somemore/domains/review/usecase/CreateReviewUseCase.java b/src/main/java/com/somemore/domains/review/usecase/CreateReviewUseCase.java
@@ -1,12 +1,10 @@
 package com.somemore.domains.review.usecase;
 
 import com.somemore.domains.review.dto.request.ReviewCreateRequestDto;
-
 import java.util.UUID;
 
 public interface CreateReviewUseCase {
 
-    Long createReview(ReviewCreateRequestDto requestDto, UUID volunteerId, String imgUrl);
-
+    Long createReview(ReviewCreateRequestDto requestDto, UUID volunteerId);
 
 }
diff --git a/src/main/java/com/somemore/domains/review/usecase/UpdateReviewUseCase.java b/src/main/java/com/somemore/domains/review/usecase/UpdateReviewUseCase.java
@@ -1,12 +1,10 @@
 package com.somemore.domains.review.usecase;
 
 import com.somemore.domains.review.dto.request.ReviewUpdateRequestDto;
-
 import java.util.UUID;
 
 public interface UpdateReviewUseCase {
 
     void updateReview(Long id, UUID volunteerId, ReviewUpdateRequestDto requestDto);
 
-    void updateReviewImageUrl(Long id, UUID volunteerId, String imgUrl);
 }

Original file line number	Diff line number	Diff line change
`@@ -24,13 +24,12 @@ public record ReviewCreateRequestDto(`
`24`	`24`	`String content`
`25`	`25`	`) {`
`26`	`26`
`27`		`- public Review toEntity(UUID volunteerId, String imgUrl) {`
	`27`	`+ public Review toEntity(UUID volunteerId) {`
`28`	`28`	`return Review.builder()`
`29`	`29`	`.volunteerApplyId(volunteerApplyId)`
`30`	`30`	`.volunteerId(volunteerId)`
`31`	`31`	`.title(title)`
`32`	`32`	`.content(content)`
`33`		`- .imgUrl(imgUrl)`
`34`	`33`	`.build();`
`35`	`34`	`}`
`36`	`35`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1,12 +1,10 @@`
`1`	`1`	`package com.somemore.domains.review.usecase;`
`2`	`2`
`3`	`3`	`import com.somemore.domains.review.dto.request.ReviewUpdateRequestDto;`
`4`		`-`
`5`	`4`	`import java.util.UUID;`
`6`	`5`
`7`	`6`	`public interface UpdateReviewUseCase {`
`8`	`7`
`9`	`8`	`void updateReview(Long id, UUID volunteerId, ReviewUpdateRequestDto requestDto);`
`10`	`9`
`11`		`- void updateReviewImageUrl(Long id, UUID volunteerId, String imgUrl);`
`12`	`10`	`}`