Прочие изменения в модулях inventory и products

myproject/inventory/forms.py

@@ -108,17 +108,18 @@ class WriteOffForm(forms.ModelForm):
 class InventoryForm(forms.ModelForm):
     class Meta:
         model = Inventory
-        fields = ['warehouse', 'conducted_by', 'notes']
+        fields = ['warehouse', 'notes']
         widgets = {
             'warehouse': forms.Select(attrs={'class': 'form-control'}),
-            'conducted_by': forms.TextInput(attrs={'class': 'form-control'}),
             'notes': forms.Textarea(attrs={'class': 'form-control', 'rows': 3}),
         }
 
     def __init__(self, *args, **kwargs):
         super().__init__(*args, **kwargs)
+        
         # Фильтруем только активные склады (исключаем скрытые)
         self.fields['warehouse'].queryset = Warehouse.objects.filter(is_active=True)
+        
         # Если есть склад по умолчанию и значение не установлено явно - предвыбираем его
         if not self.initial.get('warehouse'):
             default_warehouse = Warehouse.objects.filter(

myproject/inventory/migrations/0017_change_conducted_by_to_fk.py

@@ -0,0 +1,101 @@
+# Generated manually - Change conducted_by from CharField to ForeignKey
+
+from django.db import migrations, models
+import django.db.models.deletion
+
+
+def migrate_conducted_by_data(apps, schema_editor):
+    """
+    Миграция данных: для существующих записей инвентаризации
+    найти первого пользователя с ролью "owner" (владелец) в текущем тенанте
+    и проставить его UserRole. Если владельца нет - оставить NULL.
+    """
+    Inventory = apps.get_model('inventory', 'Inventory')
+    UserRole = apps.get_model('user_roles', 'UserRole')
+    Role = apps.get_model('user_roles', 'Role')
+    
+    # Находим первого владельца в текущем тенанте
+    try:
+        owner_role = Role.objects.get(code='owner', is_system=True)
+        owner_user_role = UserRole.objects.filter(
+            role=owner_role,
+            is_active=True
+        ).first()
+        
+        if owner_user_role:
+            # Обновляем все существующие инвентаризации, у которых есть старое текстовое значение
+            # Используем conducted_by_old (переименованное поле) для проверки
+            Inventory.objects.exclude(conducted_by_old__isnull=True).exclude(conducted_by_old='').update(
+                conducted_by_new=owner_user_role
+            )
+    except Role.DoesNotExist:
+        # Если нет роли - оставляем NULL
+        pass
+
+
+def reverse_migration(apps, schema_editor):
+    """
+    Откат: очищаем новое поле
+    """
+    Inventory = apps.get_model('inventory', 'Inventory')
+    Inventory.objects.filter(conducted_by_new__isnull=False).update(conducted_by_new=None)
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('inventory', '0016_add_document_number_to_inventory'),
+        ('user_roles', '0001_initial'),
+    ]
+
+    operations = [
+        # Шаг 1: Переименовываем старое поле в conducted_by_old
+        migrations.RenameField(
+            model_name='inventory',
+            old_name='conducted_by',
+            new_name='conducted_by_old',
+        ),
+        # Шаг 2: Добавляем новое поле как ForeignKey
+        migrations.AddField(
+            model_name='inventory',
+            name='conducted_by_new',
+            field=models.ForeignKey(
+                blank=True,
+                null=True,
+                on_delete=django.db.models.deletion.SET_NULL,
+                related_name='inventories_new',
+                to='user_roles.userrole',
+                verbose_name='Провел инвентаризацию'
+            ),
+        ),
+        # Шаг 3: Миграция данных
+        migrations.RunPython(
+            migrate_conducted_by_data,
+            reverse_code=reverse_migration
+        ),
+        # Шаг 4: Удаляем старое поле
+        migrations.RemoveField(
+            model_name='inventory',
+            name='conducted_by_old',
+        ),
+        # Шаг 5: Переименовываем новое поле в conducted_by
+        migrations.RenameField(
+            model_name='inventory',
+            old_name='conducted_by_new',
+            new_name='conducted_by',
+        ),
+        # Шаг 6: Исправляем related_name
+        migrations.AlterField(
+            model_name='inventory',
+            name='conducted_by',
+            field=models.ForeignKey(
+                blank=True,
+                null=True,
+                on_delete=django.db.models.deletion.SET_NULL,
+                related_name='inventories',
+                to='user_roles.userrole',
+                verbose_name='Провел инвентаризацию'
+            ),
+        ),
+    ]
+

myproject/inventory/templates/inventory/base_inventory_minimal.html

@@ -11,7 +11,7 @@
             <nav aria-label="breadcrumb">
                 <ol class="breadcrumb mb-0">
                     <li class="breadcrumb-item"><a href="{% url 'inventory:inventory-home' %}" class="text-decoration-none">Склад</a></li>
-                    <li class="breadcrumb-item active">{% block breadcrumb_current %}{% endblock %}</li>
+                    <li class="breadcrumb-item">{% block breadcrumb_current %}{% endblock %}</li>
                 </ol>
             </nav>
 

myproject/inventory/templates/inventory/inventory/inventory_detail.html

@@ -3,7 +3,9 @@
 {% load static %}
 
 {% block inventory_title %}Детали инвентаризации{% endblock %}
-{% block breadcrumb_current %}Инвентаризация{% endblock %}
+{% block breadcrumb_current %}
+    <a href="{% url 'inventory:inventory-list' %}" class="text-decoration-none">Инвентаризация</a>
+{% endblock %}
 
 {% block inventory_content %}
 <!-- CSS для компонента поиска товаров -->
@@ -198,6 +200,9 @@
                     <button type="button" class="btn btn-primary" id="complete-inventory-btn" {% if not lines %}disabled{% endif %}>
                         <i class="bi bi-check-circle"></i> Завершить инвентаризацию
                     </button>
+                    <button type="button" class="btn btn-outline-danger" data-bs-toggle="modal" data-bs-target="#confirmDeleteModal">
+                        <i class="bi bi-trash"></i> Удалить инвентаризацию
+                    </button>
                 </div>
             {% endif %}
             <!-- Кнопка возврата всегда видна -->
@@ -241,6 +246,43 @@
     </div>
 </div>
 
+<!-- Модальное окно подтверждения удаления инвентаризации -->
+{% if inventory.status != 'completed' %}
+<div class="modal fade" id="confirmDeleteModal" tabindex="-1" aria-labelledby="confirmDeleteModalLabel" aria-hidden="true">
+    <div class="modal-dialog">
+        <div class="modal-content">
+            <div class="modal-header bg-danger text-white">
+                <h5 class="modal-title" id="confirmDeleteModalLabel">
+                    <i class="bi bi-exclamation-triangle-fill"></i> Удалить инвентаризацию?
+                </h5>
+                <button type="button" class="btn-close btn-close-white" data-bs-dismiss="modal" aria-label="Close"></button>
+            </div>
+            <div class="modal-body">
+                <p>Вы уверены, что хотите удалить эту инвентаризацию?</p>
+                <div class="alert alert-danger">
+                    <i class="bi bi-exclamation-triangle-fill"></i>
+                    <strong>Внимание!</strong> Это действие нельзя отменить. Все строки инвентаризации будут удалены.
+                </div>
+                <p class="mb-0">
+                    <strong>Склад:</strong> {{ inventory.warehouse.name }}<br>
+                    {% if inventory.document_number %}
+                    <strong>Номер документа:</strong> {{ inventory.document_number }}<br>
+                    {% endif %}
+                    <strong>Дата:</strong> {{ inventory.date|date:"d.m.Y H:i" }}
+                </p>
+            </div>
+            <div class="modal-footer">
+                <button type="button" class="btn btn-secondary" data-bs-dismiss="modal">Отмена</button>
+                <form method="post" action="{% url 'inventory:inventory-delete' inventory.pk %}">
+                    {% csrf_token %}
+                    <button type="submit" class="btn btn-danger">Удалить инвентаризацию</button>
+                </form>
+            </div>
+        </div>
+    </div>
+</div>
+{% endif %}
+
 <!-- Подключаем JavaScript -->
 <script src="{% static 'products/js/product-search-picker.js' %}"></script>
 <script src="{% static 'inventory/js/inventory_detail.js' %}"></script>

myproject/inventory/templates/inventory/inventory/inventory_form.html

@@ -1,7 +1,9 @@
 {% extends 'inventory/base_inventory_minimal.html' %}
 
 {% block inventory_title %}Новая инвентаризация{% endblock %}
-{% block breadcrumb_current %}Инвентаризация{% endblock %}
+{% block breadcrumb_current %}
+    <a href="{% url 'inventory:inventory-list' %}" class="text-decoration-none">Инвентаризация</a>
+{% endblock %}
 
 {% block inventory_content %}
 <div class="card">
@@ -25,19 +27,6 @@
                 {% endif %}
             </div>
 
-            <div class="mb-3">
-                <label for="{{ form.conducted_by.id_for_label }}" class="form-label">
-                    {{ form.conducted_by.label }}
-                </label>
-                {{ form.conducted_by }}
-                {% if form.conducted_by.errors %}
-                    <div class="invalid-feedback d-block">
-                        {% for error in form.conducted_by.errors %}{{ error }}{% endfor %}
-                    </div>
-                {% endif %}
-                <small class="text-muted">Кто проводит инвентаризацию?</small>
-            </div>
-
             <div class="mb-3">
                 <label for="{{ form.notes.id_for_label }}" class="form-label">
                     {{ form.notes.label }}

myproject/inventory/templates/inventory/inventory/inventory_list.html

@@ -71,9 +71,20 @@
                 <td class="text-muted">{{ inventory.conducted_by|default:"—" }}</td>
                 <td class="text-muted">{{ inventory.date|date:"d.m.Y H:i" }}</td>
                 <td class="text-end">
-                    <a href="{% url 'inventory:inventory-detail' inventory.pk %}" class="btn-icon" title="Открыть для редактирования">
+                    <div class="d-inline-flex gap-1 action-buttons-container">
-                        <i class="bi bi-pencil-square"></i>
+                        <a href="{% url 'inventory:inventory-detail' inventory.pk %}" class="btn-icon" 
-                    </a>
+                           title="{% if inventory.status == 'completed' %}Открыть для просмотра{% else %}Открыть для редактирования{% endif %}">
+                            <i class="bi {% if inventory.status == 'completed' %}bi-eye{% else %}bi-pencil-square{% endif %}"></i>
+                        </a>
+                        <button type="button" class="btn-icon text-danger {% if inventory.status == 'completed' %}invisible{% endif %}" 
+                                {% if inventory.status != 'completed' %}
+                                data-bs-toggle="modal" 
+                                data-bs-target="#confirmDeleteModal{{ inventory.pk }}"
+                                {% endif %}
+                                title="Удалить инвентаризацию">
+                            <i class="bi bi-trash"></i>
+                        </button>
+                    </div>
                 </td>
             </tr>
             {% endfor %}
@@ -116,6 +127,45 @@
 </div>
 {% endif %}
 
+<!-- Модальные окна подтверждения удаления -->
+{% for inventory in inventories %}
+{% if inventory.status != 'completed' %}
+<div class="modal fade" id="confirmDeleteModal{{ inventory.pk }}" tabindex="-1" aria-labelledby="confirmDeleteModalLabel{{ inventory.pk }}" aria-hidden="true">
+    <div class="modal-dialog">
+        <div class="modal-content">
+            <div class="modal-header bg-danger text-white">
+                <h5 class="modal-title" id="confirmDeleteModalLabel{{ inventory.pk }}">
+                    <i class="bi bi-exclamation-triangle-fill"></i> Удалить инвентаризацию?
+                </h5>
+                <button type="button" class="btn-close btn-close-white" data-bs-dismiss="modal" aria-label="Close"></button>
+            </div>
+            <div class="modal-body">
+                <p>Вы уверены, что хотите удалить эту инвентаризацию?</p>
+                <div class="alert alert-danger">
+                    <i class="bi bi-exclamation-triangle-fill"></i>
+                    <strong>Внимание!</strong> Это действие нельзя отменить. Все строки инвентаризации будут удалены.
+                </div>
+                <p class="mb-0">
+                    <strong>Склад:</strong> {{ inventory.warehouse.name }}<br>
+                    {% if inventory.document_number %}
+                    <strong>Номер документа:</strong> {{ inventory.document_number }}<br>
+                    {% endif %}
+                    <strong>Дата:</strong> {{ inventory.date|date:"d.m.Y H:i" }}
+                </p>
+            </div>
+            <div class="modal-footer">
+                <button type="button" class="btn btn-secondary" data-bs-dismiss="modal">Отмена</button>
+                <form method="post" action="{% url 'inventory:inventory-delete' inventory.pk %}" style="display: inline;">
+                    {% csrf_token %}
+                    <button type="submit" class="btn btn-danger">Удалить инвентаризацию</button>
+                </form>
+            </div>
+        </div>
+    </div>
+</div>
+{% endif %}
+{% endfor %}
+
 <!-- Подсказка для пользователя -->
 <div class="alert alert-info mt-3" style="font-size: 0.875rem;">
     <i class="bi bi-info-circle"></i>
@@ -252,5 +302,19 @@
 .gap-2 {
     gap: 0.5rem !important;
 }
+
+.gap-1 {
+    gap: 0.25rem !important;
+}
+
+/* Фиксированная ширина для контейнера кнопок действий */
+.action-buttons-container {
+    width: 60px;
+    justify-content: flex-end;
+}
+
+.action-buttons-container .btn-icon {
+    flex-shrink: 0;
+}
 </style>
 {% endblock %}

myproject/inventory/urls.py

@@ -12,7 +12,7 @@ from .views import (
     # Inventory
     InventoryListView, InventoryCreateView, InventoryDetailView, InventoryLineCreateBulkView,
     InventoryLineAddView, InventoryLineUpdateView, InventoryLineDeleteView,
-    InventoryCompleteView,
+    InventoryCompleteView, InventoryDeleteView,
     # WriteOff
     WriteOffListView, WriteOffCreateView, WriteOffUpdateView, WriteOffDeleteView,
     # Transfer
@@ -87,6 +87,7 @@ urlpatterns = [
     path('inventory-ops/<int:inventory_id>/lines/<int:line_id>/update/', InventoryLineUpdateView.as_view(), name='inventory-line-update'),
     path('inventory-ops/<int:inventory_id>/lines/<int:line_id>/delete/', InventoryLineDeleteView.as_view(), name='inventory-line-delete'),
     path('inventory-ops/<int:inventory_id>/complete/', InventoryCompleteView.as_view(), name='inventory-complete'),
+    path('inventory-ops/<int:pk>/delete/', InventoryDeleteView.as_view(), name='inventory-delete'),
 
     # ==================== WRITEOFF (одиночные записи) ====================
     path('writeoffs/', WriteOffListView.as_view(), name='writeoff-list'),

myproject/inventory/views/__init__.py

@@ -25,7 +25,7 @@ from .sale import SaleListView, SaleCreateView, SaleUpdateView, SaleDeleteView,
 from .inventory_ops import (
     InventoryListView, InventoryCreateView, InventoryDetailView,
     InventoryLineCreateBulkView, InventoryLineAddView, InventoryLineUpdateView,
-    InventoryLineDeleteView, InventoryCompleteView
+    InventoryLineDeleteView, InventoryCompleteView, InventoryDeleteView
 )
 from .writeoff import WriteOffListView, WriteOffCreateView, WriteOffUpdateView, WriteOffDeleteView
 from .writeoff_document import (
@@ -67,7 +67,7 @@ __all__ = [
     # Inventory
     'InventoryListView', 'InventoryCreateView', 'InventoryDetailView', 'InventoryLineCreateBulkView',
     'InventoryLineAddView', 'InventoryLineUpdateView', 'InventoryLineDeleteView',
-    'InventoryCompleteView',
+    'InventoryCompleteView', 'InventoryDeleteView',
     # WriteOff
     'WriteOffListView', 'WriteOffCreateView', 'WriteOffUpdateView', 'WriteOffDeleteView',
     # WriteOffDocument