Исправлена продажа нескольких экземпляров витринного букета
Проблема: При попытке продажи 2+ экземпляров одного витринного букета возникала ошибка IntegrityError, так как поле sold_order_item было OneToOneField. Это означало что к одному OrderItem мог быть привязан только один ShowcaseItem, что делало невозможной продажу нескольких экземпляров в одной позиции заказа. Решение: 1. Изменен тип поля sold_order_item с OneToOneField на ForeignKey - Теперь несколько ShowcaseItem могут относиться к одному OrderItem - related_name изменен с 'sold_showcase_item' на 'sold_showcase_items' 2. Обновлен метод mark_sold в модели ShowcaseItem - Добавлена явная проверка статуса 'sold' перед продажей - Генерируется ValidationError если экземпляр уже продан - Удален комментарий про OneToOneField защиту 3. Обновлена обработка ошибок в ShowcaseManager.sell_showcase_items - Убрана обработка IntegrityError - Добавлена обработка ValidationError от mark_sold 4. Создана миграция 0012_change_sold_order_item_to_fk Теперь можно успешно продавать 2 и более экземпляров одного витринного букета в рамках одной позиции заказа.
This commit is contained in:
@@ -0,0 +1,20 @@
|
|||||||
|
# Generated by Django 5.0.10 on 2025-12-11 19:23
|
||||||
|
|
||||||
|
import django.db.models.deletion
|
||||||
|
from django.db import migrations, models
|
||||||
|
|
||||||
|
|
||||||
|
class Migration(migrations.Migration):
|
||||||
|
|
||||||
|
dependencies = [
|
||||||
|
('inventory', '0011_add_writeoff_status_to_reservation'),
|
||||||
|
('orders', '0006_transaction_delete_payment_and_more'),
|
||||||
|
]
|
||||||
|
|
||||||
|
operations = [
|
||||||
|
migrations.AlterField(
|
||||||
|
model_name='showcaseitem',
|
||||||
|
name='sold_order_item',
|
||||||
|
field=models.ForeignKey(blank=True, null=True, on_delete=django.db.models.deletion.SET_NULL, related_name='sold_showcase_items', to='orders.orderitem', verbose_name='Позиция заказа (продажа)'),
|
||||||
|
),
|
||||||
|
]
|
||||||
@@ -542,13 +542,14 @@ class ShowcaseItem(models.Model):
|
|||||||
)
|
)
|
||||||
|
|
||||||
# === ЗАЩИТА ОТ ДВОЙНОЙ ПРОДАЖИ ===
|
# === ЗАЩИТА ОТ ДВОЙНОЙ ПРОДАЖИ ===
|
||||||
# OneToOneField гарантирует на уровне БД: 1 ShowcaseItem = max 1 OrderItem
|
# ForeignKey позволяет привязать несколько ShowcaseItem к одному OrderItem
|
||||||
sold_order_item = models.OneToOneField(
|
# (например, при продаже 2+ экземпляров одного букета)
|
||||||
|
sold_order_item = models.ForeignKey(
|
||||||
'orders.OrderItem',
|
'orders.OrderItem',
|
||||||
on_delete=models.SET_NULL,
|
on_delete=models.SET_NULL,
|
||||||
null=True,
|
null=True,
|
||||||
blank=True,
|
blank=True,
|
||||||
related_name='sold_showcase_item',
|
related_name='sold_showcase_items',
|
||||||
verbose_name="Позиция заказа (продажа)"
|
verbose_name="Позиция заказа (продажа)"
|
||||||
)
|
)
|
||||||
sold_at = models.DateTimeField(
|
sold_at = models.DateTimeField(
|
||||||
@@ -616,10 +617,13 @@ class ShowcaseItem(models.Model):
|
|||||||
def mark_sold(self, order_item):
|
def mark_sold(self, order_item):
|
||||||
"""
|
"""
|
||||||
Пометить как проданный.
|
Пометить как проданный.
|
||||||
OneToOneField автоматически выбросит IntegrityError при повторной продаже.
|
Проверяет статус перед продажей чтобы избежать дублей.
|
||||||
"""
|
"""
|
||||||
|
if self.status == 'sold':
|
||||||
|
raise ValidationError(f'Экземпляр {self} уже продан')
|
||||||
|
|
||||||
self.status = 'sold'
|
self.status = 'sold'
|
||||||
self.sold_order_item = order_item # БД защита от дублей!
|
self.sold_order_item = order_item
|
||||||
self.sold_at = timezone.now()
|
self.sold_at = timezone.now()
|
||||||
self.locked_by_user = None
|
self.locked_by_user = None
|
||||||
self.cart_lock_expires_at = None
|
self.cart_lock_expires_at = None
|
||||||
|
|||||||
@@ -199,15 +199,13 @@ class ShowcaseManager:
|
|||||||
'message': f'Продано {sold_count} экз.'
|
'message': f'Продано {sold_count} экз.'
|
||||||
}
|
}
|
||||||
|
|
||||||
except IntegrityError as e:
|
except ValidationError as e:
|
||||||
# Защита от двойной продажи сработала на уровне БД
|
# Ошибка валидации (например, экземпляр уже продан)
|
||||||
if 'sold_order_item' in str(e) or 'UNIQUE' in str(e):
|
return {
|
||||||
return {
|
'success': False,
|
||||||
'success': False,
|
'sold_count': 0,
|
||||||
'sold_count': 0,
|
'message': str(e)
|
||||||
'message': 'Один из экземпляров уже был продан. Обновите список витринных букетов.'
|
}
|
||||||
}
|
|
||||||
raise
|
|
||||||
|
|
||||||
@staticmethod
|
@staticmethod
|
||||||
def sell_from_showcase(product_kit, showcase, customer, payment_method='cash_to_courier',
|
def sell_from_showcase(product_kit, showcase, customer, payment_method='cash_to_courier',
|
||||||
|
|||||||
Reference in New Issue
Block a user