ShowcaseItem: защита от двойной продажи витринных букетов
Новая архитектура: - ShowcaseItem модель - физический экземпляр букета на витрине - OneToOneField(sold_order_item) - БД-уровневая защита от двойной продажи - Поддержка создания нескольких экземпляров одного букета - Возможность продавать N из M доступных (например 2 из 5) Изменения: - inventory/models.py: добавлена модель ShowcaseItem с методами lock/unlock/mark_sold - inventory/services/showcase_manager.py: переработан для работы с ShowcaseItem - pos/views.py: API поддерживает quantity и showcase_item_ids - pos/templates/pos/terminal.html: поле "Сколько букетов создать" - pos/static/pos/js/terminal.js: выбор количества, передача showcase_item_ids Миграции: - 0007: создание модели ShowcaseItem - 0008: data migration существующих букетов - 0009: очистка ShowcaseItem для уже проданных букетов 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This commit is contained in:
@@ -0,0 +1,63 @@
|
||||
# Generated by Django 5.0.10 on 2025-12-09 04:19
|
||||
|
||||
import django.db.models.deletion
|
||||
from django.conf import settings
|
||||
from django.db import migrations, models
|
||||
|
||||
|
||||
class Migration(migrations.Migration):
|
||||
|
||||
dependencies = [
|
||||
('inventory', '0006_reservation_cart_lock_expires_at_and_more'),
|
||||
('orders', '0006_transaction_delete_payment_and_more'),
|
||||
('products', '0010_alter_product_cost_price'),
|
||||
migrations.swappable_dependency(settings.AUTH_USER_MODEL),
|
||||
]
|
||||
|
||||
operations = [
|
||||
migrations.CreateModel(
|
||||
name='ShowcaseItem',
|
||||
fields=[
|
||||
('id', models.BigAutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')),
|
||||
('status', models.CharField(choices=[('available', 'Доступен'), ('in_cart', 'В корзине'), ('sold', 'Продан'), ('dismantled', 'Разобран')], db_index=True, default='available', max_length=20, verbose_name='Статус')),
|
||||
('sold_at', models.DateTimeField(blank=True, null=True, verbose_name='Дата продажи')),
|
||||
('cart_lock_expires_at', models.DateTimeField(blank=True, null=True, verbose_name='Блокировка истекает')),
|
||||
('cart_session_id', models.CharField(blank=True, max_length=100, null=True, verbose_name='ID сессии корзины')),
|
||||
('created_at', models.DateTimeField(auto_now_add=True, verbose_name='Создан')),
|
||||
('updated_at', models.DateTimeField(auto_now=True, verbose_name='Обновлен')),
|
||||
('locked_by_user', models.ForeignKey(blank=True, null=True, on_delete=django.db.models.deletion.SET_NULL, related_name='locked_showcase_items', to=settings.AUTH_USER_MODEL, verbose_name='Заблокировано пользователем')),
|
||||
('product_kit', models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, related_name='showcase_items', to='products.productkit', verbose_name='Шаблон комплекта')),
|
||||
('showcase', models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, related_name='showcase_items', to='inventory.showcase', verbose_name='Витрина')),
|
||||
('sold_order_item', models.OneToOneField(blank=True, null=True, on_delete=django.db.models.deletion.SET_NULL, related_name='sold_showcase_item', to='orders.orderitem', verbose_name='Позиция заказа (продажа)')),
|
||||
],
|
||||
options={
|
||||
'verbose_name': 'Экземпляр на витрине',
|
||||
'verbose_name_plural': 'Экземпляры на витрине',
|
||||
},
|
||||
),
|
||||
migrations.AddField(
|
||||
model_name='reservation',
|
||||
name='showcase_item',
|
||||
field=models.ForeignKey(blank=True, help_text='Для какого физического экземпляра создан резерв', null=True, on_delete=django.db.models.deletion.CASCADE, related_name='reservations', to='inventory.showcaseitem', verbose_name='Экземпляр на витрине'),
|
||||
),
|
||||
migrations.AddIndex(
|
||||
model_name='reservation',
|
||||
index=models.Index(fields=['showcase_item'], name='inventory_r_showcas_8cfff5_idx'),
|
||||
),
|
||||
migrations.AddIndex(
|
||||
model_name='showcaseitem',
|
||||
index=models.Index(fields=['showcase', 'status'], name='inventory_s_showcas_116f7f_idx'),
|
||||
),
|
||||
migrations.AddIndex(
|
||||
model_name='showcaseitem',
|
||||
index=models.Index(fields=['product_kit', 'status'], name='inventory_s_product_785870_idx'),
|
||||
),
|
||||
migrations.AddIndex(
|
||||
model_name='showcaseitem',
|
||||
index=models.Index(fields=['status', 'cart_lock_expires_at'], name='inventory_s_status_6acf05_idx'),
|
||||
),
|
||||
migrations.AddIndex(
|
||||
model_name='showcaseitem',
|
||||
index=models.Index(fields=['locked_by_user', 'status'], name='inventory_s_locked__88eac9_idx'),
|
||||
),
|
||||
]
|
||||
@@ -0,0 +1,64 @@
|
||||
# Generated manually - Data migration for ShowcaseItem
|
||||
|
||||
from django.db import migrations
|
||||
|
||||
|
||||
def migrate_showcase_kits_to_items(apps, schema_editor):
|
||||
"""
|
||||
Для каждого существующего витринного букета (ProductKit с is_temporary=True и showcase):
|
||||
1. Создать ShowcaseItem
|
||||
2. Привязать существующие Reservation к этому ShowcaseItem
|
||||
"""
|
||||
ProductKit = apps.get_model('products', 'ProductKit')
|
||||
ShowcaseItem = apps.get_model('inventory', 'ShowcaseItem')
|
||||
Reservation = apps.get_model('inventory', 'Reservation')
|
||||
|
||||
# Находим все витринные комплекты
|
||||
showcase_kits = ProductKit.objects.filter(
|
||||
is_temporary=True,
|
||||
showcase__isnull=False
|
||||
)
|
||||
|
||||
for kit in showcase_kits:
|
||||
# Создаём ShowcaseItem для каждого существующего витринного букета
|
||||
showcase_item = ShowcaseItem.objects.create(
|
||||
showcase=kit.showcase,
|
||||
product_kit=kit,
|
||||
status='available'
|
||||
)
|
||||
|
||||
# Привязываем существующие резервы к этому ShowcaseItem
|
||||
Reservation.objects.filter(
|
||||
product_kit=kit,
|
||||
showcase=kit.showcase,
|
||||
status='reserved'
|
||||
).update(showcase_item=showcase_item)
|
||||
|
||||
|
||||
def reverse_migration(apps, schema_editor):
|
||||
"""
|
||||
Откат: удаляем созданные ShowcaseItem и очищаем связи в Reservation
|
||||
"""
|
||||
ShowcaseItem = apps.get_model('inventory', 'ShowcaseItem')
|
||||
Reservation = apps.get_model('inventory', 'Reservation')
|
||||
|
||||
# Очищаем связи в резервах
|
||||
Reservation.objects.filter(showcase_item__isnull=False).update(showcase_item=None)
|
||||
|
||||
# Удаляем все ShowcaseItem
|
||||
ShowcaseItem.objects.all().delete()
|
||||
|
||||
|
||||
class Migration(migrations.Migration):
|
||||
|
||||
dependencies = [
|
||||
('inventory', '0007_add_showcase_item_model'),
|
||||
('products', '0001_initial'), # Убедимся что ProductKit существует
|
||||
]
|
||||
|
||||
operations = [
|
||||
migrations.RunPython(
|
||||
migrate_showcase_kits_to_items,
|
||||
reverse_code=reverse_migration
|
||||
),
|
||||
]
|
||||
@@ -0,0 +1,65 @@
|
||||
# Generated manually - Fix ShowcaseItem status for already sold kits
|
||||
|
||||
from django.db import migrations
|
||||
|
||||
|
||||
def fix_showcase_items_status(apps, schema_editor):
|
||||
"""
|
||||
Исправляем статус ShowcaseItem для уже проданных комплектов.
|
||||
|
||||
Логика:
|
||||
- Если у ShowcaseItem нет активных резервов (status='reserved') →
|
||||
это уже проданный/разобранный букет → удаляем ShowcaseItem
|
||||
"""
|
||||
ShowcaseItem = apps.get_model('inventory', 'ShowcaseItem')
|
||||
Reservation = apps.get_model('inventory', 'Reservation')
|
||||
|
||||
# Находим все ShowcaseItem в статусе 'available'
|
||||
available_items = ShowcaseItem.objects.filter(status='available')
|
||||
|
||||
items_to_delete = []
|
||||
|
||||
for item in available_items:
|
||||
# Проверяем есть ли активные резервы для этого экземпляра
|
||||
has_active_reservations = Reservation.objects.filter(
|
||||
showcase_item=item,
|
||||
status='reserved'
|
||||
).exists()
|
||||
|
||||
# Если резервы не привязаны к showcase_item, проверяем старым способом
|
||||
if not has_active_reservations:
|
||||
has_active_reservations = Reservation.objects.filter(
|
||||
product_kit=item.product_kit,
|
||||
showcase=item.showcase,
|
||||
status='reserved'
|
||||
).exists()
|
||||
|
||||
if not has_active_reservations:
|
||||
# Нет активных резервов - этот букет уже продан/разобран
|
||||
items_to_delete.append(item.id)
|
||||
|
||||
# Удаляем ShowcaseItem без активных резервов
|
||||
if items_to_delete:
|
||||
ShowcaseItem.objects.filter(id__in=items_to_delete).delete()
|
||||
|
||||
|
||||
def reverse_migration(apps, schema_editor):
|
||||
"""
|
||||
Откат невозможен - удалённые ShowcaseItem не восстановить.
|
||||
Но это безопасно - они относились к уже проданным букетам.
|
||||
"""
|
||||
pass
|
||||
|
||||
|
||||
class Migration(migrations.Migration):
|
||||
|
||||
dependencies = [
|
||||
('inventory', '0008_migrate_showcase_kits_to_items'),
|
||||
]
|
||||
|
||||
operations = [
|
||||
migrations.RunPython(
|
||||
fix_showcase_items_status,
|
||||
reverse_code=reverse_migration
|
||||
),
|
||||
]
|
||||
Reference in New Issue
Block a user