feat: 티켓 상품의 경우 별도 테이블에 정보를 저장하도록 수정

Author: MU-Software

app/admin_api/serializers/shop/orders.py

@@ -8,7 +8,7 @@
 from notification.channels import NotificationChannel
 from notification.models.base import Recipient
 from rest_framework import serializers
-from shop.order.models import CustomerInfo, Order, OrderProductOptionRelation, OrderProductRelation
+from shop.order.models import CustomerInfo, Order, OrderProductOptionRelation, OrderProductRelation, TicketInfo
 from shop.payment_history.models import PaymentHistory
 from shop.product.models import Product
 from user.models import UserExt
@@ -63,13 +63,19 @@ class Meta:
                     "custom_response",
                 )
 
+        class SimpleTicketInfoSerializer(serializers.ModelSerializer):
+            class Meta:
+                model = TicketInfo
+                fields = ("name", "phone", "email", "organization", "contribution_message")
+
         product = SimpleProductSerializer(read_only=True)
         options = SimpleOrderProductOptionRelationSerializer(many=True, read_only=True)
+        ticket_info = SimpleTicketInfoSerializer(read_only=True, allow_null=True)
 
         class Meta:
             model = OrderProductRelation
-            fields = ("id", "product", "status", "price", "donation_price", "options")
-            read_only_fields = ("id", "product", "price", "donation_price", "options")
+            fields = ("id", "product", "status", "price", "donation_price", "options", "ticket_info")
+            read_only_fields = ("id", "product", "price", "donation_price", "options", "ticket_info")
 
     user = SimpleUserSerializer(read_only=True)
     customer_info = SimpleCustomerInfoSerializer(required=False, allow_null=True)

app/admin_api/serializers/shop/products.py

@@ -12,6 +12,7 @@
 from core.util.timespan import TimeSpan
 from file.models import PublicFile
 from rest_framework import serializers
+from shop.order.models import OrderProductRelation
 from shop.product.models import Category, CategoryGroup, Option, OptionGroup, Product, Tag
 
 
@@ -29,6 +30,26 @@ class Meta:
             validators: list = []
             list_serializer_class = InstanceListSerializer
 
+        def validate(self, attrs: dict) -> dict:
+            if self.instance is not None:
+                new_is_ticket = attrs.get("is_ticket", self.instance.is_ticket)
+                if new_is_ticket != self.instance.is_ticket:
+                    self._validate_is_ticket_change(new_is_ticket=new_is_ticket)
+            return attrs
+
+        def _validate_is_ticket_change(self, *, new_is_ticket: bool) -> None:
+            purchased = OrderProductRelation.objects.filter_active().filter(
+                product__category=self.instance,
+                status__in=OrderProductRelation.PURCHASED_STOCK_STATUS,
+            )
+            if new_is_ticket:
+                if purchased.filter(ticket_info__isnull=True).exists():
+                    msg = "참가자 정보가 없는 구매 건이 있어 티켓으로 전환할 수 없습니다."
+                    raise serializers.ValidationError({"is_ticket": msg})
+            elif purchased.filter(ticket_info__isnull=False).exists():
+                msg = "참가자 정보가 수집된 티켓 구매 건이 있어 티켓 설정을 해제할 수 없습니다."
+                raise serializers.ValidationError({"is_ticket": msg})
+
     categories = CategoryAdminSerializer(many=True, required=False, source="category_set")
     category_count = serializers.IntegerField(read_only=True)
 

app/admin_api/test/shop/conftest.py

@@ -8,17 +8,20 @@
     mock_portone_register,
     mock_portone_req_cancel_payment,
     modifiable_option_relation,
+    non_ticket_opr,
+    non_ticket_product,
     option,
     option_group,
     order_factory,
     other_client,
     other_user,
-    product,
     products_by_status,
     single_product_cart,
     staff_client,
     staff_user,
     tag,
+    ticket_opr,
+    ticket_product,
 )
 
 

app/admin_api/test/shop/orders_api_test.py

@@ -56,9 +56,9 @@ def test_admin_list_filters_by_status_csv(api_client, order_factory):
 
 
 @pytest.mark.django_db
-def test_admin_list_filters_by_product_id_distinct(api_client, product, order_factory):
+def test_admin_list_filters_by_product_id_distinct(api_client, ticket_product, order_factory):
     completed_order = order_factory(status="completed")
-    response = OrdersAdminApi(http_client=api_client).list({"product_id": str(product.id)})
+    response = OrdersAdminApi(http_client=api_client).list({"product_id": str(ticket_product.id)})
     assert response.status_code == HTTP_200_OK
     assert response.json() == {
         "count": 1,
@@ -69,19 +69,19 @@ def test_admin_list_filters_by_product_id_distinct(api_client, product, order_fa
 
 
 @pytest.mark.django_db
-def test_admin_list_filters_by_active_opr_category(api_client, product, order_factory):
+def test_admin_list_filters_by_active_opr_category(api_client, ticket_product, order_factory):
     """`?category_id=` 가 active OPR 가 있는 주문만 매칭한다."""
     completed_order = order_factory(status="completed")
-    response = OrdersAdminApi(http_client=api_client).list({"category_id": str(product.category_id)})
+    response = OrdersAdminApi(http_client=api_client).list({"category_id": str(ticket_product.category_id)})
     assert response.status_code == HTTP_200_OK
     assert {row["id"] for row in response.json()["results"]} == {str(completed_order.id)}
 
 
 @pytest.mark.django_db
-def test_admin_list_filters_by_active_opr_category_group(api_client, product, order_factory):
+def test_admin_list_filters_by_active_opr_category_group(api_client, ticket_product, order_factory):
     """`?category_group_id=` 가 active OPR 가 있는 주문만 매칭한다."""
     completed_order = order_factory(status="completed")
-    response = OrdersAdminApi(http_client=api_client).list({"category_group_id": str(product.category.group_id)})
+    response = OrdersAdminApi(http_client=api_client).list({"category_group_id": str(ticket_product.category.group_id)})
     assert response.status_code == HTTP_200_OK
     assert {row["id"] for row in response.json()["results"]} == {str(completed_order.id)}
 
@@ -123,12 +123,15 @@ def test_admin_refund_action_rejects_missing_totp(api_client, mock_portone_req_c
 
 @pytest.mark.django_db
 def test_admin_refund_product_action_does_partial_refund(
-    api_client, product, mock_portone_req_cancel_payment, order_factory
+    api_client, ticket_product, mock_portone_req_cancel_payment, order_factory
 ):
     completed_order = order_factory(status="completed")
     target_opr = completed_order.products.first()
     OrderProductRelation.objects.create(
-        order=completed_order, product=product, price=product.price, status=OrderProductRelation.OrderProductStatus.paid
+        order=completed_order,
+        product=ticket_product,
+        price=ticket_product.price,
+        status=OrderProductRelation.OrderProductStatus.paid,
     )
     response = OrdersAdminApi(http_client=api_client).refund_product(
         completed_order.id, target_opr.id, totp=valid_refund_totp()
@@ -152,17 +155,17 @@ def test_admin_refund_product_action_returns_404_for_unknown_rel(api_client, ord
 @pytest.mark.django_db
 def test_admin_refund_allows_expired_window(api_client, mock_portone_req_cancel_payment, order_factory):
     completed_order = order_factory(status="completed")
-    product = completed_order.products.first().product
-    product.refundable_ends_at = datetime(2020, 1, 1, tzinfo=timezone.utc)
-    product.save()
+    ticket_product = completed_order.products.first().product
+    ticket_product.refundable_ends_at = datetime(2020, 1, 1, tzinfo=timezone.utc)
+    ticket_product.save()
 
     response = OrdersAdminApi(http_client=api_client).refund(completed_order.id, totp=valid_refund_totp())
     assert response.status_code == HTTP_204_NO_CONTENT
 
 
 @pytest.mark.django_db
-def test_admin_import_template_returns_csv(api_client, product):
-    response = OrdersAdminApi(http_client=api_client).import_template(product_id=str(product.id))
+def test_admin_import_template_returns_csv(api_client, ticket_product):
+    response = OrdersAdminApi(http_client=api_client).import_template(product_id=str(ticket_product.id))
     assert response.status_code == HTTP_200_OK
     assert "text/csv" in response.headers["Content-Type"]
 
@@ -186,15 +189,15 @@ def _csv_file(rows: str) -> BytesIO:
 
 
 @pytest.mark.django_db
-def test_admin_import_csv_persists_paid_order_from_uploaded_row(api_client, customer_user, product):
+def test_admin_import_csv_persists_paid_order_from_uploaded_row(api_client, customer_user, ticket_product):
     response = OrdersAdminApi(http_client=api_client).import_csv(
         csv_file=_csv_file(
             "name,phone,email,organization,product_id,donation_price\n"
-            f"홍길동,010-1234-5678,{customer_user.email},,{product.id},0\n"
+            f"홍길동,010-1234-5678,{customer_user.email},,{ticket_product.id},0\n"
         )
     )
     assert response.status_code == HTTP_201_CREATED
-    opr = OrderProductRelation.objects.get(product=product)
+    opr = OrderProductRelation.objects.get(product=ticket_product)
     assert opr.status == OrderProductRelation.OrderProductStatus.paid
     assert opr.order.user == customer_user
 
@@ -206,12 +209,12 @@ def test_admin_import_csv_rejects_missing_file(api_client):
 
 
 @pytest.mark.django_db
-def test_admin_import_csv_returns_400_for_invalid_rows_without_persisting(api_client, product):
+def test_admin_import_csv_returns_400_for_invalid_rows_without_persisting(api_client, ticket_product):
     # email 매칭되는 user 부재 → 모든 row validate 실패 → atomic rollback.
     response = OrdersAdminApi(http_client=api_client).import_csv(
         csv_file=_csv_file(
             "name,phone,email,organization,product_id,donation_price\n"
-            f"홍길동,010-1234-5678,nobody@example.com,,{product.id},0\n"
+            f"홍길동,010-1234-5678,nobody@example.com,,{ticket_product.id},0\n"
         )
     )
     assert response.status_code == HTTP_400_BAD_REQUEST
@@ -222,11 +225,11 @@ def test_admin_import_csv_returns_400_for_invalid_rows_without_persisting(api_cl
 @freeze_time(datetime(2026, 5, 23, 15, 30, 45, tzinfo=timezone.utc))
 @pytest.mark.django_db
 def test_admin_export_returns_xlsx_filtering_refunded_per_include_flag(
-    api_client, customer_user, product, include_refunded, order_factory
+    api_client, customer_user, ticket_product, include_refunded, order_factory
 ):
     refunded_order = order_factory(status="refunded")
     response = OrdersAdminApi(http_client=api_client).export(
-        {"product_ids": [str(product.id)], "include_refunded": include_refunded}
+        {"product_ids": [str(ticket_product.id)], "include_refunded": include_refunded}
     )
     assert response.status_code == HTTP_200_OK
     assert response.headers["Content-Type"] == "application/vnd.openxmlformats-officedocument.spreadsheetml.sheet"
@@ -272,8 +275,8 @@ def test_admin_export_returns_xlsx_filtering_refunded_per_include_flag(
     assert df_dict["주문상품"].to_dict(orient="records") == [
         {
             "주문 번호": str(refunded_order.id),
-            "상품 ID": str(product.id),
-            "상품명": product.name,
+            "상품 ID": str(ticket_product.id),
+            "상품명": ticket_product.name,
             "상태": "refunded",
             "결제 금액": opr.price,
             "추가 기부액": opr.donation_price,
@@ -314,14 +317,17 @@ def test_admin_partial_update_creates_customer_info_when_missing(api_client, ord
 
 
 @pytest.mark.django_db
-def test_admin_list_filters_by_user_id(api_client, customer_user, other_user, product, order_factory):
+def test_admin_list_filters_by_user_id(api_client, customer_user, other_user, ticket_product, order_factory):
     completed_order = order_factory(status="completed")
     other_order = Order.objects.create(user=other_user, name="other")
     OrderProductRelation.objects.create(
-        order=other_order, product=product, price=product.price, status=OrderProductRelation.OrderProductStatus.paid
+        order=other_order,
+        product=ticket_product,
+        price=ticket_product.price,
+        status=OrderProductRelation.OrderProductStatus.paid,
     )
     PaymentHistory.objects.create(
-        order=other_order, imp_id="imp_o", status=PaymentHistoryStatus.completed, price=product.price
+        order=other_order, imp_id="imp_o", status=PaymentHistoryStatus.completed, price=ticket_product.price
     )
 
     response = OrdersAdminApi(http_client=api_client).list({"user_id": str(customer_user.id)})