diff --git a/src/main/java/bc1/gream/domain/buy/service/command/BuyCommandService.java b/src/main/java/bc1/gream/domain/buy/service/command/BuyCommandService.java index 941fd25d..e473f744 100644 --- a/src/main/java/bc1/gream/domain/buy/service/command/BuyCommandService.java +++ b/src/main/java/bc1/gream/domain/buy/service/command/BuyCommandService.java @@ -13,6 +13,7 @@ @Service @RequiredArgsConstructor +@Transactional public class BuyCommandService { private final BuyRepository buyRepository; @@ -25,16 +26,14 @@ public void deleteBuyByIdAndUser(Buy buy, User buyer) { buyRepository.delete(buy); } - public boolean isBuyerLoggedInUser(Buy buy, User user) { + private boolean isBuyerLoggedInUser(Buy buy, User user) { return buy.getUser().getLoginId().equals(user.getLoginId()); } - @Transactional public void delete(Buy buy) { buyRepository.delete(buy); } - @Transactional public void deleteBuysOfDeadlineBefore(LocalDateTime dateTime) { buyRepository.deleteBuysOfDeadlineBefore(dateTime); } diff --git a/src/main/java/bc1/gream/domain/buy/service/query/BuyQueryService.java b/src/main/java/bc1/gream/domain/buy/service/query/BuyQueryService.java index 07ac2ebf..d3ee92bd 100644 --- a/src/main/java/bc1/gream/domain/buy/service/query/BuyQueryService.java +++ b/src/main/java/bc1/gream/domain/buy/service/query/BuyQueryService.java @@ -22,6 +22,7 @@ @Service @RequiredArgsConstructor +@Transactional(readOnly = true) public class BuyQueryService { private final BuyRepository buyRepository; @@ -39,7 +40,6 @@ public Buy findBuyById(Long buyId) { * @param pageable 페이징 요청 데이터 * @return 구매입찰가 내역 페이징 데이터 */ - @Transactional(readOnly = true) public Page findAllBuyBidsOf(Product product, Pageable pageable) { return buyRepository.findAllPriceToQuantityOf(product, pageable); } @@ -51,7 +51,6 @@ public Page findAllBuyBidsOf(Product product, Pag * @param price 구매를 원하는 상품 가격 * @return 구매입찰 */ - @Transactional(readOnly = true) public Buy getRecentBuyBidOf(Long productId, Long price) { return buyRepository.findByProductIdAndPrice(productId, price) .orElseThrow(() -> new GlobalException(BUY_BID_NOT_FOUND)); diff --git a/src/main/java/bc1/gream/domain/coupon/service/command/CouponCommandService.java b/src/main/java/bc1/gream/domain/coupon/service/command/CouponCommandService.java index ab426b43..ec09f6d3 100644 --- a/src/main/java/bc1/gream/domain/coupon/service/command/CouponCommandService.java +++ b/src/main/java/bc1/gream/domain/coupon/service/command/CouponCommandService.java @@ -7,9 +7,11 @@ import bc1.gream.domain.user.entity.User; import lombok.RequiredArgsConstructor; import org.springframework.stereotype.Service; +import org.springframework.transaction.annotation.Transactional; @Service @RequiredArgsConstructor +@Transactional public class CouponCommandService { private final CouponRepository couponRepository; diff --git a/src/main/java/bc1/gream/domain/coupon/service/qeury/CouponQueryService.java b/src/main/java/bc1/gream/domain/coupon/service/qeury/CouponQueryService.java index c933eddb..6fe8aa08 100644 --- a/src/main/java/bc1/gream/domain/coupon/service/qeury/CouponQueryService.java +++ b/src/main/java/bc1/gream/domain/coupon/service/qeury/CouponQueryService.java @@ -61,7 +61,6 @@ public List unavailableCouponList(UserDetailsImpl userDetails) { return couponRepository.unavailable(user); } - @Transactional(readOnly = true) public Coupon findCouponById(Long couponId, User user) { Coupon coupon = couponRepository.findById(couponId).orElseThrow( () -> new GlobalException(COUPON_NOT_FOUND) diff --git a/src/main/java/bc1/gream/domain/gifticon/service/command/GifticonCommandService.java b/src/main/java/bc1/gream/domain/gifticon/service/command/GifticonCommandService.java index 69260085..fdac956c 100644 --- a/src/main/java/bc1/gream/domain/gifticon/service/command/GifticonCommandService.java +++ b/src/main/java/bc1/gream/domain/gifticon/service/command/GifticonCommandService.java @@ -9,11 +9,11 @@ @Service @RequiredArgsConstructor +@Transactional public class GifticonCommandService { private final GifticonRepository gifticonRepository; - @Transactional public Gifticon saveGifticon(String gifticonUrl, Order order) { Gifticon gifticon = Gifticon.builder() .gifticonUrl(gifticonUrl) @@ -21,8 +21,7 @@ public Gifticon saveGifticon(String gifticonUrl, Order order) { .build(); return gifticonRepository.save(gifticon); } - - @Transactional + public void delete(Gifticon gifticon) { gifticonRepository.delete(gifticon); } diff --git a/src/main/java/bc1/gream/domain/gifticon/service/query/GifticonQueryService.java b/src/main/java/bc1/gream/domain/gifticon/service/query/GifticonQueryService.java index 95aefdf0..1fe431dd 100644 --- a/src/main/java/bc1/gream/domain/gifticon/service/query/GifticonQueryService.java +++ b/src/main/java/bc1/gream/domain/gifticon/service/query/GifticonQueryService.java @@ -9,9 +9,11 @@ import java.util.List; import lombok.RequiredArgsConstructor; import org.springframework.stereotype.Service; +import org.springframework.transaction.annotation.Transactional; @Service @RequiredArgsConstructor +@Transactional(readOnly = true) public class GifticonQueryService { private final GifticonRepository gifticonRepository; diff --git a/src/main/java/bc1/gream/domain/order/service/command/OrderCommandService.java b/src/main/java/bc1/gream/domain/order/service/command/OrderCommandService.java index 10ea4407..64f9ddc8 100644 --- a/src/main/java/bc1/gream/domain/order/service/command/OrderCommandService.java +++ b/src/main/java/bc1/gream/domain/order/service/command/OrderCommandService.java @@ -26,7 +26,6 @@ public class OrderCommandService { * @param coupon 쿠폰 * @return 새로운 주문 */ - @Transactional public Order saveOrderOfBuy(Buy buy, User seller, Coupon coupon) { Long finalPrice = CouponCalculator.calculateDiscount(coupon, buy.getPrice()); Order order = Order.builder() @@ -39,7 +38,6 @@ public Order saveOrderOfBuy(Buy buy, User seller, Coupon coupon) { return orderRepository.save(order); } - @Transactional public Order saveOrderOfBuyNotCoupon(Buy buy, User seller) { Long finalPrice = buy.getPrice(); @@ -51,12 +49,10 @@ public Order saveOrderOfBuyNotCoupon(Buy buy, User seller) { .finalPrice(finalPrice) .expectedPrice(buy.getPrice()) .build(); - + return orderRepository.save(order); } - - @Transactional public Order saveOrderOfSell(Sell sell, User buyer, Coupon coupon) { Long finalPrice = CouponCalculator.calculateDiscount(coupon, sell.getPrice()); @@ -71,7 +67,6 @@ public Order saveOrderOfSell(Sell sell, User buyer, Coupon coupon) { return orderRepository.save(order); } - @Transactional public Order saveOrderOfSellNotCoupon(Sell sell, User buyer) { Long finalPrice = sell.getPrice();