feat: 행사 siteMap 조회 api 구현 및 siteMap 등록 추가 - #199 #200
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
📝 WalkthroughSummary by CodeRabbit릴리스 노트
✏️ Tip: You can customize this high-level summary in your review settings. Walkthrough관리자 이벤트에 사이트맵 이미지 필드를 추가하고, 사이트맵 이미지 엔티티·리포지토리·컴포넌트·서비스·컨트롤러·예외·응답 DTO를 새로 도입해 생성/수정 시 저장·삭제 로직을 적용하고 공개 조회 엔드포인트를 추가했습니다. Changes
Sequence Diagram(s)sequenceDiagram
participant Client
participant Controller as EventSiteMapImageController
participant Service as EventSiteMapImageService
participant EventRetriever as EventRetriever
participant Retriever as SiteMapImageRetriever
participant Repo as EventSiteMapImageRepository
Client->>Controller: GET /api/events/{eventId}/sitemap
Controller->>Service: getEventSiteMapImages(eventId)
Service->>EventRetriever: getEvent(eventId)
EventRetriever-->>Service: Event or throw
alt Event not found
Service-->>Controller: throw SiteMapImageApiException(NOT_FOUND_EVENT)
else
Service->>Retriever: findAllEventSiteMapImagesByEventId(eventId)
Retriever->>Repo: findAllByEventId(eventId)
Repo-->>Retriever: List<EventSiteMapImageEntity>
Retriever-->>Service: List<EventSiteMapImage>
alt No images
Service-->>Controller: throw SiteMapImageApiException(NOT_FOUND_SITE_MAP_IMAGE)
else
Service->>Service: sort by sequence & map to DTO
Service-->>Controller: EventSiteMapGetResponse
Controller-->>Client: 200 OK (wrapped)
end
end
sequenceDiagram
participant AdminService as AdminEventService
participant EventRepo as EventRepository
participant ImageSaver as ImageSaver
participant SiteMapSaver as SiteMapImageSaver
participant SiteMapRemover as SiteMapImageRemover
participant SiteMapRepo as EventSiteMapImageRepository
AdminService->>EventRepo: save(event)
EventRepo-->>AdminService: savedEvent
AdminService->>ImageSaver: saveEventImages(images)
AdminService->>SiteMapSaver: saveEventSiteMapImages(siteMapImages)
SiteMapSaver->>SiteMapRepo: saveAll(entities)
SiteMapRepo-->>SiteMapSaver: saved entities
alt Update with new siteMapImages
AdminService->>SiteMapRemover: deleteAllSiteMapImages(eventId)
SiteMapRemover->>SiteMapRepo: deleteAllByEventId(eventId)
AdminService->>SiteMapSaver: saveEventSiteMapImages(newEntities)
SiteMapSaver->>SiteMapRepo: saveAll(newEntities)
end
Estimated code review effort🎯 4 (Complex) | ⏱️ ~45 minutes
Possibly related PRs
Pre-merge checks and finishing touches❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing touches
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (2)
src/main/java/com/permitseoul/permitserver/domain/sitemapimage/api/service/EventSiteMapImageService.java (1)
40-42: 예외 원인(cause) 체이닝 누락원본 예외(
EventNotfoundException)의 정보가 새로운 예외로 전달되지 않아 디버깅 시 원인 추적이 어려울 수 있습니다.} catch (EventNotfoundException e) { - throw new SiteMapImageApiException(ErrorCode.NOT_FOUND_EVENT); + throw new SiteMapImageApiException(ErrorCode.NOT_FOUND_EVENT, e); }
SiteMapImageApiException생성자가Throwable cause를 받는지 확인이 필요합니다.src/main/java/com/permitseoul/permitserver/domain/sitemapimage/core/domain/EventSiteMapImage.java (1)
12-14: 필드 네이밍 불일치 확인 필요
siteMapImagesId는 camelCase로 "Map"을 사용하고,sitemapImageUrl은 소문자 "sitemap"을 사용합니다. 일관성을 위해 통일하는 것이 좋습니다.- private final long siteMapImagesId; - private final int sequence; - private final String sitemapImageUrl; + private final long siteMapImageId; + private final int sequence; + private final String siteMapImageUrl;
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (19)
src/main/java/com/permitseoul/permitserver/domain/admin/event/api/dto/req/AdminEventUpdateRequest.java(1 hunks)src/main/java/com/permitseoul/permitserver/domain/admin/event/api/dto/req/AdminEventWithTicketCreateRequest.java(1 hunks)src/main/java/com/permitseoul/permitserver/domain/admin/event/api/dto/res/AdminEventDetailResponse.java(3 hunks)src/main/java/com/permitseoul/permitserver/domain/admin/event/api/service/AdminEventService.java(7 hunks)src/main/java/com/permitseoul/permitserver/domain/event/api/service/EventService.java(3 hunks)src/main/java/com/permitseoul/permitserver/domain/eventimage/core/domain/entity/EventImageEntity.java(1 hunks)src/main/java/com/permitseoul/permitserver/domain/eventtimetable/blockmedia/domain/entity/TimetableBlockMediaEntity.java(1 hunks)src/main/java/com/permitseoul/permitserver/domain/sitemapimage/SiteMapImageBaseException.java(1 hunks)src/main/java/com/permitseoul/permitserver/domain/sitemapimage/api/controller/EventSiteMapImageController.java(1 hunks)src/main/java/com/permitseoul/permitserver/domain/sitemapimage/api/dto/res/EventSiteMapGetResponse.java(1 hunks)src/main/java/com/permitseoul/permitserver/domain/sitemapimage/api/exception/SiteMapImageApiException.java(1 hunks)src/main/java/com/permitseoul/permitserver/domain/sitemapimage/api/service/EventSiteMapImageService.java(1 hunks)src/main/java/com/permitseoul/permitserver/domain/sitemapimage/core/component/SiteMapImageRemover.java(1 hunks)src/main/java/com/permitseoul/permitserver/domain/sitemapimage/core/component/SiteMapImageRetriever.java(1 hunks)src/main/java/com/permitseoul/permitserver/domain/sitemapimage/core/component/SiteMapImageSaver.java(1 hunks)src/main/java/com/permitseoul/permitserver/domain/sitemapimage/core/domain/EventSiteMapImage.java(1 hunks)src/main/java/com/permitseoul/permitserver/domain/sitemapimage/core/domain/entity/EventSiteMapImageEntity.java(1 hunks)src/main/java/com/permitseoul/permitserver/domain/sitemapimage/core/repository/EventSiteMapImageRepository.java(1 hunks)src/main/java/com/permitseoul/permitserver/global/response/code/ErrorCode.java(1 hunks)
🧰 Additional context used
🧠 Learnings (1)
📓 Common learnings
Learnt from: sjk4618
Repo: PERMIT-SEOUL/permit-server PR: 48
File: src/main/java/com/permitseoul/permitserver/domain/guest/core/domain/entity/GuestEntity.java:31-37
Timestamp: 2025-07-15T09:37:32.765Z
Learning: sjk4618 prefers to implement factory methods or public constructors for entities when they are actually needed, rather than creating them proactively.
🧬 Code graph analysis (6)
src/main/java/com/permitseoul/permitserver/domain/sitemapimage/core/component/SiteMapImageRemover.java (1)
src/main/java/com/permitseoul/permitserver/domain/sitemapimage/core/component/SiteMapImageSaver.java (1)
Component(10-18)
src/main/java/com/permitseoul/permitserver/domain/sitemapimage/api/service/EventSiteMapImageService.java (2)
src/main/java/com/permitseoul/permitserver/domain/event/core/exception/EventNotfoundException.java (1)
EventNotfoundException(3-4)src/main/java/com/permitseoul/permitserver/domain/event/api/service/EventService.java (1)
RequiredArgsConstructor(29-131)
src/main/java/com/permitseoul/permitserver/domain/sitemapimage/core/domain/entity/EventSiteMapImageEntity.java (3)
src/main/java/com/permitseoul/permitserver/domain/sitemapimage/core/domain/EventSiteMapImage.java (1)
Getter(8-25)src/main/java/com/permitseoul/permitserver/domain/eventimage/core/domain/entity/EventImageEntity.java (1)
Entity(6-36)src/main/java/com/permitseoul/permitserver/domain/eventtimetable/blockmedia/domain/entity/TimetableBlockMediaEntity.java (1)
Entity(10-41)
src/main/java/com/permitseoul/permitserver/domain/sitemapimage/api/controller/EventSiteMapImageController.java (1)
src/main/java/com/permitseoul/permitserver/global/response/ApiResponseUtil.java (1)
ApiResponseUtil(7-29)
src/main/java/com/permitseoul/permitserver/domain/sitemapimage/api/exception/SiteMapImageApiException.java (1)
src/main/java/com/permitseoul/permitserver/domain/sitemapimage/SiteMapImageBaseException.java (1)
SiteMapImageBaseException(3-4)
src/main/java/com/permitseoul/permitserver/domain/sitemapimage/core/component/SiteMapImageSaver.java (1)
src/main/java/com/permitseoul/permitserver/domain/sitemapimage/core/component/SiteMapImageRemover.java (1)
Component(7-15)
🔇 Additional comments (21)
src/main/java/com/permitseoul/permitserver/global/response/code/ErrorCode.java (1)
90-90: LGTM!새로운 에러 코드가 올바르게 추가되었습니다. 순차 번호(40429)와 메시지 형식이 기존 패턴과 일관성 있게 유지되고 있습니다.
src/main/java/com/permitseoul/permitserver/domain/sitemapimage/SiteMapImageBaseException.java (1)
1-4: LGTM!도메인 특화 기본 예외 클래스가 적절하게 정의되었습니다. 추상 클래스로 선언하여 직접 인스턴스화를 방지하고 마커 타입으로서의 역할을 수행합니다.
src/main/java/com/permitseoul/permitserver/domain/sitemapimage/core/domain/entity/EventSiteMapImageEntity.java (1)
9-42: 엔티티 구조가 잘 설계되었습니다.JPA 어노테이션 설정, 팩토리 메서드 패턴, protected 기본 생성자 등이 코드베이스의 다른 엔티티들과 일관성 있게 구현되어 있습니다.
src/main/java/com/permitseoul/permitserver/domain/sitemapimage/core/component/SiteMapImageRemover.java (1)
7-14: LGTM!컴포넌트가 단일 책임 원칙에 따라 잘 설계되었습니다. 리포지토리 삭제 로직을 캡슐화하여 재사용성과 테스트 용이성을 제공합니다.
src/main/java/com/permitseoul/permitserver/domain/sitemapimage/core/component/SiteMapImageSaver.java (1)
10-17: LGTM!컴포넌트가 일괄 저장 로직을 명확하게 캡슐화하고 있으며,
SiteMapImageRemover와 대칭적인 구조로 일관성을 유지하고 있습니다.src/main/java/com/permitseoul/permitserver/domain/sitemapimage/api/controller/EventSiteMapImageController.java (1)
14-26: LGTM!REST 컨트롤러가 명확하고 일관성 있게 구현되었습니다. 기존의
@EventIdPathVariable어노테이션을 활용하여 검증을 처리하고,ApiResponseUtil을 통해 표준화된 응답 형식을 유지하고 있습니다.src/main/java/com/permitseoul/permitserver/domain/sitemapimage/api/exception/SiteMapImageApiException.java (1)
8-12: LGTM!API 예외 클래스가 간결하고 명확하게 구현되었습니다.
ErrorCode를 캡슐화하여 일관된 에러 처리 패턴을 따르고 있습니다.src/main/java/com/permitseoul/permitserver/domain/admin/event/api/dto/req/AdminEventUpdateRequest.java (1)
54-57: LGTM!새로운
siteMapImages필드가 적절하게 추가되었습니다. 기존images필드와 동일한 타입(List<AdminEventImageRequest>)을 재사용하고@Valid어노테이션으로 중첩 검증을 보장하여 일관성을 유지하고 있습니다.src/main/java/com/permitseoul/permitserver/domain/admin/event/api/dto/req/AdminEventWithTicketCreateRequest.java (1)
90-93: LGTM! siteMapImages 필드 추가가 적절합니다.
siteMapImages가@Valid로 중첩 검증이 적용되었고,images필드와 달리@NotNull제약이 없어 선택적 필드로 처리됩니다. 사이트맵 이미지가 필수가 아닌 경우 이 설계가 적절합니다.src/main/java/com/permitseoul/permitserver/domain/sitemapimage/api/service/EventSiteMapImageService.java (1)
26-43: 전체 구조 승인기존
EventService.getEventDetail()패턴과 일관성 있게 구현되었습니다. 예외 변환, 정렬, DTO 매핑 로직이 적절합니다.src/main/java/com/permitseoul/permitserver/domain/sitemapimage/api/dto/res/EventSiteMapGetResponse.java (1)
6-21: LGTM! DTO 구조가 적절합니다.Java record를 활용한 간결한 DTO 설계와 팩토리 메서드 패턴이 기존 코드베이스(
AdminEventImageInfo.of)와 일관성을 유지합니다.src/main/java/com/permitseoul/permitserver/domain/admin/event/api/dto/res/AdminEventDetailResponse.java (1)
24-25: LGTM! siteMapImages 필드 추가가 일관성 있게 구현되었습니다.기존
AdminEventImageInfo를 재사용하여 사이트맵 이미지를 표현하고, 팩토리 메서드에도 적절히 반영되었습니다. 레코드 필드, 팩토리 메서드 시그니처, 생성자 호출의 파라미터 순서가 일치합니다.Also applies to: 44-45, 63-64
src/main/java/com/permitseoul/permitserver/domain/sitemapimage/core/component/SiteMapImageRetriever.java (1)
17-25: LGTM! 방어적 null 처리가 적용되었습니다.
Optional.ofNullable을 사용한 null 안전 처리가 적절합니다. Spring Data JPA 리포지토리는 일반적으로 빈 컬렉션을 반환하지만, 이 방어적 코딩은 안전성을 보장합니다. 엔티티-도메인 변환도 깔끔합니다.src/main/java/com/permitseoul/permitserver/domain/sitemapimage/core/repository/EventSiteMapImageRepository.java (1)
13-15: LGTM!
@Modifying에clearAutomatically와flushAutomatically옵션이 적절히 설정되어 있어 벌크 삭제 후 영속성 컨텍스트 동기화가 보장됩니다.src/main/java/com/permitseoul/permitserver/domain/admin/event/api/service/AdminEventService.java (4)
65-67: LGTM!사이트맵 이미지 관련 컴포넌트들이 적절히 의존성으로 주입되었습니다.
100-105: LGTM!사이트맵 이미지 조회 및 정렬 로직이 기존 이벤트 이미지 처리 패턴과 일관성 있게 구현되었습니다.
200-206: 사이트맵 이미지 전체 삭제 시나리오 확인 필요현재 구현에서
eventSiteMapImages가 null 또는 빈 리스트일 경우 기존 이미지가 보존됩니다. 만약 모든 사이트맵 이미지를 삭제해야 하는 요구사항이 있다면, 이 로직으로는 불가능합니다. 의도된 동작인지 확인해 주세요.
216-228: LGTM!
saveEventImages와 동일한 패턴으로 구현되었으며,IntStream을 활용한 시퀀스 할당과 URL trim 처리가 적절합니다.src/main/java/com/permitseoul/permitserver/domain/event/api/service/EventService.java (1)
58-60: 이미지 순서 정렬 로직이 올바르게 구현되었습니다.
sequence기준으로 이미지를 정렬하여 UI에 일관된 순서로 표시되도록 보장한 점이 좋습니다. 데이터베이스에서 순서가 보장되지 않을 수 있으므로 애플리케이션 레벨에서 명시적으로 정렬하는 것이 안전한 접근 방식입니다.src/main/java/com/permitseoul/permitserver/domain/eventimage/core/domain/entity/EventImageEntity.java (1)
19-19: 데이터베이스 스키마 변경이 반영되었는지 확인하세요.
image_url컬럼에columnDefinition = "TEXT"를 추가한 코드 변경은 적절합니다. 그러나 이 프로젝트는 Flyway나 Liquibase 같은 명시적 마이그레이션 도구를 사용하지 않으므로, 다음 중 하나를 확인해야 합니다:
- Hibernate의
spring.jpa.hibernate.ddl-auto설정이update또는create로 구성되어 있는지 확인- 또는 데이터베이스 스키마를 수동으로 업데이트하여
image_url컬럼이 TEXT 타입으로 생성되었는지 검증기존 프로덕션 데이터가 있다면 호환성을 사전에 확인하세요.
src/main/java/com/permitseoul/permitserver/domain/eventtimetable/blockmedia/domain/entity/TimetableBlockMediaEntity.java (1)
29-29:mediaUrl컬럼의 TEXT 타입 정의 확인 완료
mediaUrl컬럼에columnDefinition = "TEXT"가 올바르게 적용되었으며,EventImageEntity의image_url컬럼과 일관된 패턴입니다. 다만, 이 프로젝트에는 명시적인 데이터베이스 마이그레이션 스크립트(Flyway/Liquibase)가 없습니다. Hibernate의 자동 DDL 생성 기능을 사용하고 있으므로,spring.jpa.hibernate.ddl-auto설정이 올바르게 구성되어 있는지 확인하세요.Likely an incorrect or invalid review comment.
...permitseoul/permitserver/domain/sitemapimage/core/domain/entity/EventSiteMapImageEntity.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (1)
src/main/java/com/permitseoul/permitserver/domain/sitemapimage/api/service/EventSiteMapImageService.java (1)
25-43: 이벤트 조회가 불필요하게 중복되어 있습니다.Line 28에서
Event엔티티를 조회하지만, Line 29에서는 이미 파라미터로 받은eventId와 동일한event.getEventId()를 사용하고 있습니다. 이는 불필요한 데이터베이스 조회를 발생시킵니다.만약 이벤트 존재 여부 검증이 목적이라면, 다음과 같이 개선할 수 있습니다:
- 옵션 1:
eventId를 직접 사용하고,SiteMapImageRetriever에서 이벤트가 없을 때 예외를 던지도록 구현- 옵션 2: 이벤트 존재 여부만 확인하는 경량 메서드 사용 (예:
eventRetriever.validateEventExists(eventId))옵션 1 적용 (권장):
@Transactional(readOnly = true) public EventSiteMapGetResponse getEventSiteMapImages(final long eventId) { try { - final Event event = eventRetriever.findEventById(eventId); - final List<EventSiteMapImage> eventSiteMapImages = siteMapImageRetriever.findAllEventSiteMapImagesByEventId(event.getEventId()); + final List<EventSiteMapImage> eventSiteMapImages = siteMapImageRetriever.findAllEventSiteMapImagesByEventId(eventId); if (ObjectUtils.isEmpty(eventSiteMapImages)) { throw new SiteMapImageApiException(ErrorCode.NOT_FOUND_SITE_MAP_IMAGE); }옵션 2 적용 (이벤트 검증이 반드시 필요한 경우):
@Transactional(readOnly = true) public EventSiteMapGetResponse getEventSiteMapImages(final long eventId) { try { - final Event event = eventRetriever.findEventById(eventId); - final List<EventSiteMapImage> eventSiteMapImages = siteMapImageRetriever.findAllEventSiteMapImagesByEventId(event.getEventId()); + eventRetriever.validateEventExists(eventId); // 경량 검증 메서드 사용 + final List<EventSiteMapImage> eventSiteMapImages = siteMapImageRetriever.findAllEventSiteMapImagesByEventId(eventId); if (ObjectUtils.isEmpty(eventSiteMapImages)) { throw new SiteMapImageApiException(ErrorCode.NOT_FOUND_SITE_MAP_IMAGE); }
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
src/main/java/com/permitseoul/permitserver/domain/sitemapimage/api/dto/res/EventSiteMapGetResponse.java(1 hunks)src/main/java/com/permitseoul/permitserver/domain/sitemapimage/api/service/EventSiteMapImageService.java(1 hunks)
🚧 Files skipped from review as they are similar to previous changes (1)
- src/main/java/com/permitseoul/permitserver/domain/sitemapimage/api/dto/res/EventSiteMapGetResponse.java
🧰 Additional context used
🧠 Learnings (1)
📓 Common learnings
Learnt from: sjk4618
Repo: PERMIT-SEOUL/permit-server PR: 48
File: src/main/java/com/permitseoul/permitserver/domain/guest/core/domain/entity/GuestEntity.java:31-37
Timestamp: 2025-07-15T09:37:32.765Z
Learning: sjk4618 prefers to implement factory methods or public constructors for entities when they are actually needed, rather than creating them proactively.
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (3)
src/main/java/com/permitseoul/permitserver/domain/sitemapimage/api/EventSiteMapExceptionHandler.java (2)
10-11: 클래스명과 스코프의 일관성을 고려하세요.클래스명은
EventSiteMapExceptionHandler로 이벤트에 특화된 것처럼 보이지만,basePackages는 범용적인sitemapimage패키지를 대상으로 하고 있습니다. 만약 이 핸들러가 이벤트 관련 사이트맵 이미지에만 특화되어 있다면 패키지 구조나 스코프도 그에 맞춰 조정하거나, 범용적으로 사용할 예정이라면 클래스명을SiteMapImageExceptionHandler로 변경하는 것이 더 명확합니다.-public class EventSiteMapExceptionHandler { +public class SiteMapImageExceptionHandler {
13-16: 로깅 추가를 고려하세요.예외 핸들러에 로그를 추가하면 프로덕션 환경에서 디버깅에 도움이 됩니다. 또한 JavaDoc을 추가하여 핸들러의 목적과 동작을 문서화하는 것도 권장됩니다.
+import lombok.extern.slf4j.Slf4j; + +@Slf4j @RestControllerAdvice(basePackages = "com.permitseoul.permitserver.domain.sitemapimage") public class EventSiteMapExceptionHandler { + /** + * SiteMapImageApiException을 처리하여 표준화된 API 에러 응답을 반환합니다. + * + * @param e 발생한 SiteMapImageApiException + * @return 에러 코드를 포함한 표준 응답 + */ @ExceptionHandler(SiteMapImageApiException.class) public ResponseEntity<BaseResponse<?>> handleSiteMapImageApiException(final SiteMapImageApiException e) { + log.warn("SiteMapImageApiException occurred: {}", e.getErrorCode(), e); return ApiResponseUtil.failure(e.getErrorCode()); } }src/main/java/com/permitseoul/permitserver/domain/sitemapimage/core/domain/entity/EventSiteMapImageEntity.java (1)
38-42: 팩토리 메서드에 입력 검증 추가를 고려하세요.
create메서드가 입력 파라미터에 대한 검증 없이 엔티티를 생성하고 있습니다. 다음과 같은 검증을 추가하면 런타임 오류를 방지할 수 있습니다:
sitemapImageUrl이 null이거나 빈 문자열인지 확인sequence값의 유효 범위 확인 (예: 음수 불가)eventId값의 유효성 확인 (예: 양수 여부)서비스 레이어에서 이미 검증이 이루어지고 있다면 무시하셔도 됩니다.
필요시 다음과 같은 검증을 추가할 수 있습니다:
public static EventSiteMapImageEntity create(final String sitemapImageUrl, final int sequence, final long eventId) { + if (sitemapImageUrl == null || sitemapImageUrl.isBlank()) { + throw new IllegalArgumentException("sitemapImageUrl은 null이거나 빈 값일 수 없습니다"); + } + if (sequence < 0) { + throw new IllegalArgumentException("sequence는 음수일 수 없습니다"); + } + if (eventId <= 0) { + throw new IllegalArgumentException("eventId는 양수여야 합니다"); + } return new EventSiteMapImageEntity( sitemapImageUrl, sequence, eventId); }
📜 Review details
Configuration used: Repository UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
src/main/java/com/permitseoul/permitserver/domain/sitemapimage/api/EventSiteMapExceptionHandler.java(1 hunks)src/main/java/com/permitseoul/permitserver/domain/sitemapimage/core/domain/entity/EventSiteMapImageEntity.java(1 hunks)
🧰 Additional context used
🧠 Learnings (1)
📓 Common learnings
Learnt from: sjk4618
Repo: PERMIT-SEOUL/permit-server PR: 48
File: src/main/java/com/permitseoul/permitserver/domain/guest/core/domain/entity/GuestEntity.java:31-37
Timestamp: 2025-07-15T09:37:32.765Z
Learning: sjk4618 prefers to implement factory methods or public constructors for entities when they are actually needed, rather than creating them proactively.
🧬 Code graph analysis (1)
src/main/java/com/permitseoul/permitserver/domain/sitemapimage/api/EventSiteMapExceptionHandler.java (1)
src/main/java/com/permitseoul/permitserver/global/response/ApiResponseUtil.java (1)
ApiResponseUtil(7-29)
🔇 Additional comments (3)
src/main/java/com/permitseoul/permitserver/domain/sitemapimage/api/EventSiteMapExceptionHandler.java (1)
1-9: LGTM!패키지 구조와 임포트 구성이 적절합니다.
src/main/java/com/permitseoul/permitserver/domain/sitemapimage/core/domain/entity/EventSiteMapImageEntity.java (2)
15-18: 이전 리뷰 피드백이 반영되었습니다.ID 필드가 이제
private으로 올바르게 선언되어 캡슐화가 유지되고 있습니다. 이전 리뷰에서 지적된 사항이 잘 수정되었습니다.
26-27: EventSiteMapImage 삭제 로직이 서비스 레이어에 구현되어 있습니다.
eventId가 JPA 관계 매핑 없이 primitivelong으로 저장되는 것은 의도된 설계입니다. 이 설계 선택은 다음과 같이 검증되었습니다:
AdminEventService.updateEvent()메서드에서 sitemap 이미지 업데이트 시siteMapImageRemover.deleteAllSiteMapImages(eventId)를 호출하여 기존 이미지를 삭제합니다.EventSiteMapImageRepository는@Modifying애노테이션을 사용한 쿼리로deleteAllByEventId(eventId)메서드를 구현하며,clearAutomatically=true와flushAutomatically=true옵션으로 트랜잭션을 안전하게 관리합니다.- 수동 FK 관리는 잘 구현되어 있습니다.
이 설계는 순환 참조를 방지하면서도 필요한 삭제 로직을 서비스 레이어에서 명시적으로 관리하는 합리적인 접근입니다.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🔥Pull requests
⛳️ 작업한 브랜치
👷 작업한 내용
🚨 참고 사항