Update app.py

app.py

@@ -1,3 +1,4 @@
+
 # Импортируем необходимые библиотеки
 from flask import Flask, render_template_string, request, redirect, url_for, jsonify, flash, send_from_directory
 import json
@@ -359,13 +360,18 @@ def procurement():
             logging.error(f"Ошибка при обработке закупа: {e}", exc_info=True)
             flash(f"Произошла внутренняя ошибка при обработке закупа: {e}", "danger")
 
-    # GET запрос - рендерим шаблон
+    # --- GET запрос ---
     page_title = "Закуп материалов"
     page_content = PROCUREMENT_CONTENT # Используем переменную с контентом
     page_scripts = PROCUREMENT_SCRIPTS # Используем переменную со скриптами
-    final_html = BASE_TEMPLATE.format(title=page_title, content=page_content, scripts=page_scripts)
-    # Передаем нужные переменные в render_template_string
-    return render_template_string(final_html, categories=categories)
+
+    # Заменяем маркеры в базовом шаблоне
+    html = BASE_TEMPLATE.replace('__TITLE__', page_title)
+    html = html.replace('__CONTENT__', page_content)
+    html = html.replace('__SCRIPTS__', page_scripts)
+
+    # Передаем нужные переменные в render_template_string для рендеринга Jinja внутри контента
+    return render_template_string(html, categories=categories)
 
 
 # 2. Маршрут "Раскрой"
@@ -495,25 +501,30 @@ def cutting():
             logging.error(f"Ошибка при обработке раскроя: {e}", exc_info=True)
             flash(f"Произошла внутренняя ошибка при обработке раскроя: {e}", "danger")
 
-    # GET запрос
+    # --- GET запрос ---
     # Передаем материалы, преобразовав количество в Decimal для отображения
     fabrics_dec = []
     for f in fabrics:
         f_copy = f.copy()
-        f_copy['quantity'] = to_decimal(f_copy.get('quantity', '0'))
+        f_copy['quantity_dec'] = to_decimal(f_copy.get('quantity', '0')) # Используем новое имя ключа
         fabrics_dec.append(f_copy)
 
     fittings_dec = []
     for f in fittings:
         f_copy = f.copy()
-        f_copy['quantity'] = to_decimal(f_copy.get('quantity', '0'))
+        f_copy['quantity_dec'] = to_decimal(f_copy.get('quantity', '0')) # Используем новое имя ключа
         fittings_dec.append(f_copy)
 
     page_title = "Раскрой ткани"
     page_content = CUTTING_CONTENT
     page_scripts = CUTTING_SCRIPTS
-    final_html = BASE_TEMPLATE.format(title=page_title, content=page_content, scripts=page_scripts)
-    return render_template_string(final_html, fabrics=fabrics_dec, fittings=fittings_dec)
+
+    html = BASE_TEMPLATE.replace('__TITLE__', page_title)
+    html = html.replace('__CONTENT__', page_content)
+    html = html.replace('__SCRIPTS__', page_scripts)
+
+    # Передаем переменные в render_template_string
+    return render_template_string(html, fabrics=fabrics_dec, fittings=fittings_dec)
 
 
 # 3. Маршрут "Пошив"
@@ -701,7 +712,7 @@ def sewing():
                 'fabric_id': cutting_task['fabric_id'], # Сохраняем для справки
                 'fabric_name': cutting_task['fabric_name'],
                 'fittings_consumed': fittings_consumed_list, # Фактически использовано на пошив
-                'defects_reported': defect_log_list, # Залогированный брак (уже с правильными типами)
+                'defects_reported': [], # Заполним ниже, преобразовав Decimal в строки
                 'status': 'pending_qc', # 'pending_qc', 'completed' (packed)
                 'timestamp_created': current_time,
                 'timestamp_completed': None, # Заполнится при ОТК
@@ -709,16 +720,17 @@ def sewing():
                 'qc_defective_quantity': 0
             }
 
-            # Присваиваем ID задания на пошив записям в логе брака
+            # Присваиваем ID задания на пошив и преобразуем Decimal в строку для JSON
             for defect in defect_log_list:
                 defect['sewing_task_id'] = new_sewing_task['id']
                 # Преобразуем Decimal обратно в строку для JSON, если это ткань
-                if defect['type'] == 'fabric':
+                if isinstance(defect['quantity'], Decimal):
                     defect['quantity'] = str(defect['quantity'])
+                # Добавляем в список задания
+                new_sewing_task['defects_reported'].append(defect)
 
 
-            # 5. Обновление статуса задания на раскро�� (или частичное обновление?)
-            # Пока считаем, что отправка на пошив завершает задание на раскрой целиком.
+            # 5. Обновление статуса задания на раскрой
             task_updated = False
             for i, task in enumerate(data.get('cutting_tasks', [])):
                 if task.get('id') == cutting_task_id:
@@ -734,9 +746,10 @@ def sewing():
             if 'sewing_tasks' not in data: data['sewing_tasks'] = []
             data['sewing_tasks'].append(new_sewing_task)
 
-            if defect_log_list:
+            if defect_log_list: # Если был брак
                 if 'defect_log' not in data: data['defect_log'] = []
-                data['defect_log'].extend(defect_log_list)
+                # Добавляем уже обработанные записи из new_sewing_task['defects_reported']
+                data['defect_log'].extend(new_sewing_task['defects_reported'])
 
             # Обновляем основной список материалов
             data['materials'] = materials_list
@@ -752,20 +765,23 @@ def sewing():
             logging.error(f"Ошибка при обработке пошива: {e}", exc_info=True)
             flash(f"Произошла внутренняя ошибка при обработке пошива: {e}", "danger")
 
-    # GET запрос
+    # --- GET запрос ---
     # Преобразуем данные для шаблона
     tasks_for_template = []
     all_materials_dict = {m['id']: m for m in data.get('materials', [])} # Словарь для быстрого поиска остатков
 
     for task in pending_cutting_tasks:
         task_copy = task.copy()
-        task_copy['fabric_used_decimal'] = to_decimal(task_copy.get('fabric_used', '0')) # Для возможного отображения
+        # Преобразуем числа для отображения и data-атрибутов
+        task_copy['cut_items_quantity_int'] = int(task_copy.get('cut_items_quantity', 0))
+        task_copy['fabric_used_decimal'] = to_decimal(task_copy.get('fabric_used', '0'))
         # Получаем текущие остатки фурнитуры для отображения в деталях
         if 'required_fittings' in task_copy:
             for fitting in task_copy['required_fittings']:
                  mat_data = all_materials_dict.get(fitting['fitting_id'])
                  fitting['available_quantity'] = to_decimal(mat_data.get('quantity', '0')) if mat_data else Decimal('0')
                  fitting['unit'] = mat_data.get('unit', 'шт') if mat_data else 'шт'
+                 fitting['quantity_needed_int'] = int(fitting.get('quantity_needed',0)) # Преобразуем для JSON
         tasks_for_template.append(task_copy)
 
     # Получаем все материалы для выбора брака (с преобразованными количествами)
@@ -779,8 +795,12 @@ def sewing():
     page_title = "Пошив изделий"
     page_content = SEWING_CONTENT
     page_scripts = SEWING_SCRIPTS
-    final_html = BASE_TEMPLATE.format(title=page_title, content=page_content, scripts=page_scripts)
-    return render_template_string(final_html,
+
+    html = BASE_TEMPLATE.replace('__TITLE__', page_title)
+    html = html.replace('__CONTENT__', page_content)
+    html = html.replace('__SCRIPTS__', page_scripts)
+
+    return render_template_string(html,
                                   cutting_tasks=tasks_for_template,
                                   all_materials=all_materials_dec) # Передаем материалы с quantity_decimal
 
@@ -852,7 +872,6 @@ def qc_packing():
                 data['qc_packing_items'].append(packed_item)
 
             # Регистрация брака на этапе ОТК (если есть)
-            # Здесь мы не списываем материал, а просто логируем брак готового изделия
             if quantity_defective > 0:
                 defect_log_entry = {
                     'log_id': uuid.uuid4().hex,
@@ -875,9 +894,13 @@ def qc_packing():
             task_updated = False
             for i, task in enumerate(data.get('sewing_tasks', [])):
                  if task.get('id') == sewing_task_id:
-                     # Добавляем информацию о результате ОТК в само задание
-                     task['qc_packed_quantity'] = task.get('qc_packed_quantity', 0) + quantity_packed # Накапливаем, если частичная приемка
-                     task['qc_defective_quantity'] = task.get('qc_defective_quantity', 0) + quantity_defective
+                     # Обновляем информацию о результате ОТК в само задание
+                     # Убедимся, что читаем существующие значения как int
+                     packed_before = int(task.get('qc_packed_quantity', 0))
+                     defective_before = int(task.get('qc_defective_quantity', 0))
+
+                     task['qc_packed_quantity'] = packed_before + quantity_packed
+                     task['qc_defective_quantity'] = defective_before + quantity_defective
                      task['status'] = 'completed' # Считаем завершенным после обработки
                      task['timestamp_completed'] = current_time # Время последнего ОТК
                      task_updated = True
@@ -895,13 +918,17 @@ def qc_packing():
             logging.error(f"Ошибка при обработке ОТК и упаковки: {e}", exc_info=True)
             flash(f"Произошла внутренняя ошибка при обработке ОТК и упаковки: {e}", "danger")
 
-    # GET запрос
+    # --- GET запрос ---
     page_title = "ОТК и Упаковка"
     page_content = QC_PACKING_CONTENT
     page_scripts = QC_PACKING_SCRIPTS
-    final_html = BASE_TEMPLATE.format(title=page_title, content=page_content, scripts=page_scripts)
+
+    html = BASE_TEMPLATE.replace('__TITLE__', page_title)
+    html = html.replace('__CONTENT__', page_content)
+    html = html.replace('__SCRIPTS__', page_scripts)
+
     # Передаем задачи, ожидающие ОТК, в шаблон
-    return render_template_string(final_html, sewing_tasks=pending_sewing_tasks)
+    return render_template_string(html, sewing_tasks=pending_sewing_tasks)
 
 
 # 5. Маршрут "Админ-панель"
@@ -942,6 +969,11 @@ def admin_panel():
         # Используем find_sewing_task_by_id, чтобы получить уже преобразованные числа
         task_data = find_sewing_task_by_id(t['id'])
         if task_data: # Убедимся, что задача найдена
+             # Преобразуем Decimal в строку для дефектов ткани перед передачей в шаблон
+            if 'defects_reported' in task_data:
+                for defect in task_data['defects_reported']:
+                    if isinstance(defect.get('quantity'), Decimal):
+                        defect['quantity'] = str(defect['quantity'])
             sewing_tasks_view.append(task_data)
         else:
              logging.warning(f"Задача на пошив с ID {t['id']} не найдена при подготовке для админки.")
@@ -959,7 +991,7 @@ def admin_panel():
         packed_items_view.append(item_copy)
 
     defect_log_view = []
-    total_defect_items = 0 # Будем считать штуки, метры отдельно
+    # total_defect_items = 0 # Будем считать штуки, метры отдельно
     total_defect_fabric_m = Decimal('0.00')
     total_defect_fittings_pcs = 0
     total_defect_finished_pcs = 0
@@ -991,11 +1023,14 @@ def admin_panel():
     page_title = "Админ-панель"
     page_content = ADMIN_CONTENT
     page_scripts = ADMIN_SCRIPTS # Если есть специфичные скрипты для админки
-    final_html = BASE_TEMPLATE.format(title=page_title, content=page_content, scripts=page_scripts)
+
+    html = BASE_TEMPLATE.replace('__TITLE__', page_title)
+    html = html.replace('__CONTENT__', page_content)
+    html = html.replace('__SCRIPTS__', page_scripts)
 
     # Передаем подготовленные данные
     return render_template_string(
-        final_html,
+        html,
         materials=materials_view,
         cutting_tasks=cutting_tasks_view,
         sewing_tasks=sewing_tasks_view,
@@ -1127,51 +1162,51 @@ def download_hf():
 
 # --- HTML Шаблоны (как строки Python) ---
 
-# Базовый шаблон с плейсхолдерами
+# Базовый шаблон с уникальными маркерами вместо .format() плейсхолдеров
 BASE_TEMPLATE = """
 <!DOCTYPE html>
 <html lang="ru">
 <head>
     <meta charset="UTF-8">
     <meta name="viewport" content="width=device-width, initial-scale=1.0">
-    <title>{title} - Текстиль Учет</title>
+    <title>__TITLE__ - Текстиль Учет</title>
     <link href="https://cdn.jsdelivr.net/npm/bootstrap@5.3.3/dist/css/bootstrap.min.css" rel="stylesheet" integrity="sha384-QWTKZyjpPEjISv5WaRU9OFeRpok6YctnYmDr5pNlyT2bRjXh0JMhjY6hW+ALEwIH" crossorigin="anonymous">
     <link rel="stylesheet" href="https://cdnjs.cloudflare.com/ajax/libs/font-awesome/6.5.1/css/all.min.css">
     <style>
-        body {{
+        body {
             background-color: #f8f9fa; /* Светлый фон */
             font-family: 'Segoe UI', Tahoma, Geneva, Verdana, sans-serif;
             padding-top: 70px; /* Отступ для фиксированной навигации */
             padding-bottom: 60px; /* Отступ для футера */
             position: relative;
             min-height: 100vh;
-        }}
-        .navbar {{
+        }
+        .navbar {
             background-color: #343a40; /* Темная навигация */
             box-shadow: 0 2px 4px rgba(0,0,0,.1);
-        }}
-        .navbar-brand, .nav-link {{
+        }
+        .navbar-brand, .nav-link {
             color: #f8f9fa !important; /* Светлый текст */
-        }}
-        .navbar-brand:hover, .nav-link:hover {{
+        }
+        .navbar-brand:hover, .nav-link:hover {
             color: #adb5bd !important; /* Чуть темнее при наведении */
-        }}
-        .nav-link.active {{
+        }
+        .nav-link.active {
             font-weight: bold;
             color: #ffffff !important;
             border-bottom: 2px solid #0d6efd; /* Подчеркивание активного пункта */
-        }}
-        .card {{
+        }
+        .card {
             margin-bottom: 1.5rem;
             border: none; /* Убираем стандартную рамку */
             border-radius: 0.5rem; /* Скругление углов */
             box-shadow: 0 0.5rem 1rem rgba(0, 0, 0, 0.1); /* Тень */
             transition: transform 0.2s ease-in-out;
-        }}
-         .card:hover {{
+        }
+         .card:hover {
             /* transform: translateY(-3px); Легкий подъем при наведении - убрал, может мешать */
-        }}
-        .card-header {{
+        }
+        .card-header {
             background-color: #6c757d; /* Серая шапка */
             color: white;
             font-weight: 600;
@@ -1180,86 +1215,98 @@ BASE_TEMPLATE = """
             border-top-right-radius: 0.5rem;
             display: flex;
             align-items: center;
-        }}
-         .card-header i {{
+        }
+         .card-header i {
             margin-right: 0.5rem; /* Отступ для иконки в шапке */
-         }}
-        .btn-primary {{
+         }
+        .btn-primary {
             background-color: #0d6efd;
             border-color: #0d6efd;
-        }}
-        .btn-primary:hover {{
+        }
+        .btn-primary:hover {
             background-color: #0b5ed7;
             border-color: #0a58ca;
-        }}
-        .btn-success {{
+        }
+        .btn-success {
              background-color: #198754;
              border-color: #198754;
-        }}
-        .btn-danger {{
+        }
+        .btn-danger {
             background-color: #dc3545;
             border-color: #dc3545;
-        }}
-        .btn-warning {{
+        }
+        .btn-warning {
             background-color: #ffc107;
             border-color: #ffc107;
             color: #212529; /* Темный текст для желтой кнопки */
-        }}
-        .btn-info {{
+        }
+        .btn-info {
              background-color: #0dcaf0;
              border-color: #0dcaf0;
              color: #000;
-        }}
-        .table-hover tbody tr:hover {{
+        }
+        .table-hover tbody tr:hover {
             background-color: rgba(0, 0, 0, 0.05); /* Легкое выделение строки таблицы */
             cursor: pointer; /* Намек на интерактивность (если будет) */
-        }}
-        .container-main {{ /* Переименовал, чтобы не конфликтовать с Bootstrap */
+        }
+        .container-main { /* Переименовал, чтобы не конфликтовать с Bootstrap */
             max-width: 1400px; /* Чуть шире контейнер */
             margin: 0 auto; /* Центрирование */
             padding-left: 15px;
             padding-right: 15px;
-        }}
+        }
         /* Статусы */
-        .status-pending {{ color: #fd7e14; font-weight: bold; }} /* Оранжевый - ожидает */
-        .status-completed {{ color: #198754; font-weight: bold; }} /* Зеленый - завершено */
-        .status-pending_qc {{ color: #ffc107; font-weight: bold; }} /* Желтый - ожидает ОТК */
+        .status-pending { color: #fd7e14; font-weight: bold; } /* Оранжевый - ожидает */
+        .status-completed { color: #198754; font-weight: bold; } /* Зеленый - завершено */
+        .status-pending_qc { color: #ffc107; font-weight: bold; } /* Желтый - ожидает ОТК */
 
-        .flash-messages .alert {{
+        .flash-messages .alert {
              margin-bottom: 1rem;
              border-radius: 0.5rem;
-        }}
+        }
          /* Стиль для динамически добавляемых строк */
-        .dynamic-row, .dynamic-fitting-row, .dynamic-defect-row {{
+        .dynamic-row, .dynamic-fitting-row, .dynamic-defect-row {
             border: 1px solid #eee;
             padding: 15px;
             margin-bottom: 15px;
             border-radius: 5px;
             background-color: #fdfdfd;
             position: relative; /* Для позиционирования кнопки удаления */
-        }}
-        .remove-row-btn, .remove-fitting-row-btn, .remove-defect-row-btn {{
+        }
+        .remove-row-btn, .remove-fitting-row-btn, .remove-defect-row-btn {
              position: absolute;
              top: 10px;
              right: 10px;
-             /* margin-top: 25px; Выравнивание кнопки удаления - старый вариант */
-        }}
+             z-index: 10; /* Чтобы кнопка была поверх инпутов */
+        }
         /* Responsive table */
-        .table-responsive {{
+        .table-responsive {
             margin-bottom: 1rem;
-        }}
-        .table th, .table td {{
+        }
+        .table th, .table td {
             vertical-align: middle; /* Выравнивание по центру */
-        }}
-         .table td small {{
+            font-size: 0.9rem; /* Уменьшил шрифт в таблицах */
+        }
+         .table td small {
              color: #6c757d; /* Серый цвет для мелкого текста */
              display: block; /* ID на новой строке */
-         }}
-         .table .badge {{ /* Стили для бэйджей в таблицах */
+             font-size: 0.75rem;
+         }
+         .table .badge { /* Стили для бэйджей в таблицах */
              font-size: 0.8em;
-         }}
+         }
+         .table th i.fa-sort,
+         .table th i.fa-sort-up,
+         .table th i.fa-sort-down {
+             margin-left: 5px;
+             color: #adb5bd;
+             cursor: pointer;
+         }
+         .table th:hover i {
+              color: #343a40;
+         }
         /* Footer */
-         .footer {{
+         .footer {
             position: absolute;
             bottom: 0;
             width: 100%;
@@ -1268,19 +1315,19 @@ BASE_TEMPLATE = """
             text-align: center;
             padding: 10px 0;
             font-size: 0.9em;
-         }}
+         }
          /* Скрытие стрелок у number инпутов */
           input[type=number]::-webkit-inner-spin-button,
-          input[type=number]::-webkit-outer-spin-button {{
+          input[type=number]::-webkit-outer-spin-button {
             -webkit-appearance: none;
             margin: 0;
-          }}
-          input[type=number] {{
+          }
+          input[type=number] {
             -moz-appearance: textfield; /* Firefox */
-          }}
-          .form-text {{
+          }
+          .form-text {
              font-size: 0.8rem;
-          }}
+          }
     </style>
 </head>
 <body>
@@ -1310,13 +1357,13 @@ BASE_TEMPLATE = """
         </li>
       </ul>
       <!-- Кнопки Hugging Face справа -->
-      <div class="d-flex">
-         <form method="POST" action="{{ url_for('backup_hf') }}" class="me-2">
+      <div class="d-flex align-items-center"> {# Выравнивание по центру #}
+         <form method="POST" action="{{ url_for('backup_hf') }}" class="me-2 mb-0"> {# Убрал нижний маргин #}
             <button type="submit" class="btn btn-sm btn-outline-light" title="Создать резервную копию на Hugging Face">
                 <i class="fas fa-cloud-upload-alt"></i> Backup
             </button>
         </form>
-        <form method="GET" action="{{ url_for('download_hf') }}" onsubmit="return confirm('ОСТОРОЖНО! Это перезапишет локальные данные последней версией с Hugging Face. Вы уверены?');">
+        <form method="GET" action="{{ url_for('download_hf') }}" onsubmit="return confirm('ОСТОРОЖНО! Это перезапишет локальные данные последней версией с Hugging Face. Вы уверены?');" class="mb-0"> {# Убрал нижний маргин #}
             <button type="submit" class="btn btn-sm btn-outline-warning" title="Скачать базу данных с Hugging Face (перезапишет локальную)">
                 <i class="fas fa-cloud-download-alt"></i> Download DB
             </button>
@@ -1332,7 +1379,13 @@ BASE_TEMPLATE = """
       {% if messages %}
         <div class="flash-messages">
         {% for category, message in messages %}
-          <div class="alert alert-{{ category if category != 'message' else 'info' }} alert-dismissible fade show" role="alert">
+          {# Используем стандартные классы bootstrap для категорий flash #}
+          {% set alert_class = 'alert-info' %} {# Default #}
+          {% if category == 'danger' %} {% set alert_class = 'alert-danger' %}
+          {% elif category == 'success' %} {% set alert_class = 'alert-success' %}
+          {% elif category == 'warning' %} {% set alert_class = 'alert-warning' %}
+          {% endif %}
+          <div class="alert {{ alert_class }} alert-dismissible fade show" role="alert">
             {{ message }}
             <button type="button" class="btn-close" data-bs-dismiss="alert" aria-label="Close"></button>
           </div>
@@ -1342,7 +1395,7 @@ BASE_TEMPLATE = """
     {% endwith %}
 
     {# Основной контент страницы #}
-    {content}
+    __CONTENT__
 
 </div>
 
@@ -1354,12 +1407,12 @@ BASE_TEMPLATE = """
 
 <script src="https://cdn.jsdelivr.net/npm/bootstrap@5.3.3/dist/js/bootstrap.bundle.min.js" integrity="sha384-YvpcrYf0tY3lHB60NNkmXc5s9fDVZLESaAA55NDzOxhy9GkcIdslK1eN7N6jIeHz" crossorigin="anonymous"></script>
 {# Скрипты для конкретной страницы #}
-{scripts}
+__SCRIPTS__
 </body>
 </html>
 """
 
-# --- Контент и скрипты для страниц ---
+# --- Контент и скрипты для страниц (остаются без изменений) ---
 
 PROCUREMENT_CONTENT = """
 <div class="card">
@@ -1430,7 +1483,10 @@ PROCUREMENT_SCRIPTS = """
     function addRow() {
         const container = document.getElementById('material-rows');
         const firstRow = container.querySelector('.dynamic-row'); // Берем первую строку как шаблон
-        if (!firstRow) return; // Защита, если первая строка удалена
+        if (!firstRow) { // Если строк нет, возможно, создать базовую? Или просто ничего не делать.
+             console.warn("No template row found to clone.");
+             return;
+        }
         const newRow = firstRow.cloneNode(true);
 
         // Очищаем значения в новой строке
@@ -1465,12 +1521,11 @@ PROCUREMENT_SCRIPTS = """
 
         container.appendChild(newRow);
         attachCategoryChangeEvent(newRow); // Добавляем обработчик для новой строки
-        // updateRemoveButtons не нужна, если кнопка всегда видима и позиционируется абсолютно
     }
 
     function removeRow(button) {
         const row = button.closest('.dynamic-row');
-         // Позволяем удалять любую строку, даже последнюю, т.к. можно добавить новую
+         // Позволяем удалять любую строку
          if (row) {
              row.remove();
          }
@@ -1509,14 +1564,16 @@ PROCUREMENT_SCRIPTS = """
 
     // Инициализация при загрузке страницы
     document.addEventListener('DOMContentLoaded', () => {
-        document.querySelectorAll('.dynamic-row').forEach(row => {
-             attachCategoryChangeEvent(row);
-             const categorySelect = row.querySelector('.category-select');
-             if (categorySelect) handleCategoryChange(categorySelect); // Инициализируем видимость поля новой категории
-         });
-
-         // Если изначально только одна строка, скроем кнопку удаления
-         // updateRemoveButtons(); // Раскомментировать, если нужна логика скрытия кнопки на 1 строке
+        // Если изначально нет строк, добавим одну пустую
+         if (!document.querySelector('#material-rows .dynamic-row')) {
+             // Не добавляем автоматически, пользователь нажмет кнопку сам
+         } else {
+            document.querySelectorAll('.dynamic-row').forEach(row => {
+                 attachCategoryChangeEvent(row);
+                 const categorySelect = row.querySelector('.category-select');
+                 if (categorySelect) handleCategoryChange(categorySelect); // Инициализируем видимость поля новой категории
+             });
+         }
     });
 </script>
 """
@@ -1596,11 +1653,11 @@ CUTTING_SCRIPTS = """
         if (selectedOption && selectedOption.value) {
             const quantity = selectedOption.getAttribute('data-quantity'); // Уже содержит запятую, если надо
             const unit = selectedOption.getAttribute('data-unit');
-            availableDiv.textContent = `В наличии: ${quantity} ${unit}`;
-            unitSpan.textContent = unit; // Обновляем единицу измерения в поле расхода
+            if (availableDiv) availableDiv.textContent = `В наличии: ${quantity} ${unit}`;
+            if (unitSpan) unitSpan.textContent = unit; // Обновляем единицу измерения в поле расхода
         } else {
-            availableDiv.textContent = '';
-            unitSpan.textContent = 'м'; // Сброс на метры по умолчанию
+           if (availableDiv) availableDiv.textContent = '';
+           if (unitSpan) unitSpan.textContent = 'м'; // Сброс на метры по умолчанию
         }
     }
 
@@ -1673,15 +1730,18 @@ CUTTING_SCRIPTS = """
     // Инициализация при загрузке
     document.addEventListener('DOMContentLoaded', () => {
         updateAvailableQuantity(); // Вызываем при загрузк��
-        document.querySelectorAll('.dynamic-fitting-row').forEach(row => {
+        const fittingRows = document.querySelectorAll('.dynamic-fitting-row');
+        fittingRows.forEach(row => {
              attachFittingChangeEvent(row);
              const fittingSelect = row.querySelector('.fitting-select');
              if (fittingSelect) handleFittingChange(fittingSelect); // Инициализируем показ доступности
          });
-         // Если нет строк фурнитуры, добавим одну пустую для удобства
-         if (!document.querySelector('.dynamic-fitting-row')) {
-            // Не будем добавлять автоматически, чтобы не мешать, если фурнитура не нужна.
-            // Пользователь может нажать кнопку "Добавить фурнитуру"
+         // Если нет строк фурнитуры ИЗНАЧАЛЬНО (например, после ошибки формы),
+         // гарантируем, что хотя бы одна строка есть для добавления.
+         // Но не добавляем, если форма загрузилась чисто.
+         if (fittingRows.length === 0 && document.getElementById('fittings-rows')) {
+            // Можно раскомментировать, если нужна хотя бы одна строка всегда
+            // addFittingRow();
          }
     });
 </script>
@@ -1794,14 +1854,14 @@ SEWING_SCRIPTS = """
 
 
         if (selectedOption && selectedOption.value) {
-            const fabricName = selectedOption.getAttribute('data-fabric-name');
-            const cutQuantity = selectedOption.getAttribute('data-cut-quantity');
+            const fabricName = selectedOption.getAttribute('data-fabric-name') || 'N/A';
+            const cutQuantity = selectedOption.getAttribute('data-cut-quantity') || '0';
 
-            document.getElementById('detail-fabric-name').textContent = fabricName;
-            document.getElementById('detail-cut-quantity').textContent = cutQuantity;
-            document.getElementById('detail-fabric-used').textContent = selectedOption.getAttribute('data-fabric-used');
+            if (document.getElementById('detail-fabric-name')) document.getElementById('detail-fabric-name').textContent = fabricName;
+            if (document.getElementById('detail-cut-quantity')) document.getElementById('detail-cut-quantity').textContent = cutQuantity;
+            if (document.getElementById('detail-fabric-used')) document.getElementById('detail-fabric-used').textContent = selectedOption.getAttribute('data-fabric-used') || 'N/A';
 
-            maxSewnSpan.textContent = cutQuantity;
+            if(maxSewnSpan) maxSewnSpan.textContent = cutQuantity;
             if(sewnInput) sewnInput.max = cutQuantity; // Устанавливаем максимум для поля ввода
 
             // Предзаполняем название изделия на основе ткани, если поле пустое
@@ -1811,34 +1871,44 @@ SEWING_SCRIPTS = """
 
 
             const fittingsList = document.getElementById('detail-fittings-list');
-            if(fittingsList) fittingsList.innerHTML = ''; // Очищаем список
-            const fittingsData = JSON.parse(selectedOption.getAttribute('data-fittings'));
-
-            if (fittingsData && fittingsData.length > 0) {
-                 fittingsData.forEach(fitting => {
-                    // Ищем фурнитуру в селекте брака, чтобы показать остаток
-                    let availableQtyStr = 'Н/Д';
-                    const defectSelect = document.querySelector('.defect-material-select'); // Берем любой селект для поиска
-                    if (defectSelect) {
-                        const option = defectSelect.querySelector(`option[value="${fitting.fitting_id}"]`);
-                        if (option) {
-                            availableQtyStr = `${option.getAttribute('data-quantity')} ${option.getAttribute('data-unit')}`;
-                        }
+            if(fittingsList) {
+                fittingsList.innerHTML = ''; // Очищаем список
+                try {
+                    const fittingsData = JSON.parse(selectedOption.getAttribute('data-fittings') || '[]');
+
+                    if (fittingsData && fittingsData.length > 0) {
+                         fittingsData.forEach(fitting => {
+                            // Ищем фурнитуру в селекте брака, чтобы показать остаток
+                            let availableQtyStr = 'Н/Д';
+                            const defectSelect = document.querySelector('.defect-material-select'); // Берем любой селект для поиска
+                            if (defectSelect) {
+                                const option = defectSelect.querySelector(`option[value="${fitting.fitting_id}"]`);
+                                if (option) {
+                                    availableQtyStr = `${option.getAttribute('data-quantity')} ${option.getAttribute('data-unit')}`;
+                                }
+                            } else {
+                                 // Если селекта брака нет, инфо об остатках не показать
+                                 availableQtyStr = '(см. склад)';
+                            }
+
+                            const listItem = document.createElement('li');
+                            listItem.textContent = `${fitting.fitting_name || '?'}: ${fitting.quantity_needed_int || '?'} шт. (Доступно: ${availableQtyStr})`;
+                            fittingsList.appendChild(listItem);
+                        });
                     } else {
-                         // Если селекта брака нет, инфо об остатках не показать
-                         availableQtyStr = '(см. склад)';
+                         const listItem = document.createElement('li');
+                         listItem.textContent = 'Фурнитура не указана в задании на раскрой.';
+                         listItem.classList.add('text-muted');
+                         fittingsList.appendChild(listItem);
                     }
-
-                    const listItem = document.createElement('li');
-                    listItem.textContent = `${fitting.fitting_name}: ${fitting.quantity_needed} шт. (Доступно: ${availableQtyStr})`;
-                    fittingsList.appendChild(listItem);
-                });
-            } else if (fittingsList) {
-                 const listItem = document.createElement('li');
-                 listItem.textContent = 'Фурнитура не указана в задании на раскрой.';
-                 listItem.classList.add('text-muted');
-                 fittingsList.appendChild(listItem);
-            }
+                } catch (e) {
+                     console.error("Error parsing fittings data:", e);
+                     const listItem = document.createElement('li');
+                     listItem.textContent = 'Ошибка загрузки данных фурнитуры.';
+                     listItem.classList.add('text-danger');
+                     fittingsList.appendChild(listItem);
+                }
+            } // end if fittingsList
 
             if(detailsDiv) detailsDiv.style.display = 'block';
         } else {
@@ -1864,6 +1934,14 @@ SEWING_SCRIPTS = """
         newRow.querySelectorAll('input[type="text"]').forEach(input => input.value = '');
         const availabilityDiv = newRow.querySelector('.defect-availability');
         if(availabilityDiv) availabilityDiv.textContent = '';
+         // Сброс инпута количества
+         const qtyInput = newRow.querySelector('.defect-quantity-input');
+         if(qtyInput){
+            qtyInput.inputMode = 'text';
+            qtyInput.placeholder = 'Количество';
+            qtyInput.step = 'any';
+         }
+
 
         // Кнопка удаления
         const removeBtn = newRow.querySelector('.remove-defect-row-btn');
@@ -1930,16 +2008,17 @@ SEWING_SCRIPTS = """
     // Инициализация
      document.addEventListener('DOMContentLoaded', () => {
         showTaskDetails(); // Показать детали для выбранного по умолчанию (если есть)
-        document.querySelectorAll('.dynamic-defect-row').forEach(row => {
+        const defectRows = document.querySelectorAll('.dynamic-defect-row');
+        defectRows.forEach(row => {
              attachDefectChangeEvent(row);
              const defectSelect = row.querySelector('.defect-material-select');
              if (defectSelect) handleDefectChange(defectSelect); // Инициализируем доступность и тип инпута
          });
-         // Если нет строк брака, добавим одну пустую
-          if (!document.querySelector('.dynamic-defect-row') && document.getElementById('defect-rows')) {
-              // Не добавляем автоматически
-              // addDefectRow();
-          }
+          // Если нет строк брака ИЗНАЧАЛЬНО, гарантируем наличие одной строки
+         if (defectRows.length === 0 && document.getElementById('defect-rows')) {
+            // Не добавляем автоматически
+             // addDefectRow();
+         }
      });
 </script>
 """
@@ -1997,7 +2076,7 @@ QC_PACKING_SCRIPTS = """
 
     function updateQcQuantities() {
         const select = document.getElementById('sewing_task_id');
-        const selectedOption = select.options[select.selectedIndex];
+        const selectedOption = select ? select.options[select.selectedIndex] : null;
         const packedInput = document.getElementById('quantity_packed');
         const defectiveInput = document.getElementById('quantity_defective');
         const totalInfo = document.getElementById('qc-total-info');
@@ -2015,7 +2094,7 @@ QC_PACKING_SCRIPTS = """
             }
              if(totalInfo) {
                  totalInfo.textContent = `Всего было сшито: ${maxAllowedQuantity} ед. Сумма упакованных и брака не должна превышать это число.`;
-                 totalInfo.classList.remove('text-danger');
+                 totalInfo.classList.remove('text-danger', 'text-success');
              }
              // Вызываем валидацию сразу
              validateQcSum();
@@ -2030,7 +2109,7 @@ QC_PACKING_SCRIPTS = """
                 defectiveInput.max = '';
                 defectiveInput.value = '0';
             }
-             if(totalInfo) totalInfo.textContent = '';
+             if(totalInfo) totalInfo.textContent = 'Выберите задание на пошив.';
         }
     }
 
@@ -2045,43 +2124,61 @@ QC_PACKING_SCRIPTS = """
         const defective = parseInt(defectiveInput.value) || 0;
         const totalProcessed = packed + defective;
 
+        totalInfo.classList.remove('text-danger', 'text-success'); // Сброс классов
+
         if (maxAllowedQuantity === 0 && totalProcessed > 0) {
             // Если задание не выбрано, но что-то введено
              totalInfo.textContent = 'Сначала выберите задание на пошив!';
              totalInfo.classList.add('text-danger');
              return false;
         }
+        if (maxAllowedQuantity === 0 && totalProcessed === 0) {
+              totalInfo.textContent = 'Выберите задание на пошив.';
+             return false; // Не валидно, если задание не выбрано
+        }
+
 
         if (totalProcessed > maxAllowedQuantity) {
             totalInfo.textContent = `Ошибка: Сумма (${totalProcessed}) превышает количество сшитых (${maxAllowedQuantity})!`;
             totalInfo.classList.add('text-danger');
             return false; // Сумма неверна
-        } else {
-             totalInfo.textContent = `Всего было сшито: ${maxAllowedQuantity} ед. Сумма упакованных и брака (${totalProcessed}) не должна превышать это число.`;
-             totalInfo.classList.remove('text-danger');
+        } else if (totalProcessed === 0) {
+             totalInfo.textContent = `Всего было сшито: ${maxAllowedQuantity} ед. Укажите кол-во упакованных или брака.`;
+             // Можно не считать ошибкой, но форма не должна отправляться
+             return false; // Не валидно, если сумма 0
+        }
+        else {
+             totalInfo.textContent = `Всего было сшито: ${maxAllowedQuantity} ед. Сумма упакованных и брака (${totalProcessed}) корректна.`;
+             totalInfo.classList.add('text-success'); // Показываем успех
              return true; // Сумма верна
         }
     }
 
     function validateQcForm() {
         // Дополнительная проверка перед отправкой формы
-        const isValidSum = validateQcSum();
+        const isValidSum = validateQcSum(); // Выполняем проверку и обновляем сообщение
         if (!isValidSum) {
-            alert('Ошибка в количестве упакованных или бракованных изделий. Проверьте введенные значения.');
+            alert('Ошибка в количестве упакованных или бракованных изделий. Проверьте введенные значения и сообщение под полями ввода.');
             return false; // Предотвратить отправку формы
         }
 
+        // Проверка, что выбрано задание
+        const select = document.getElementById('sewing_task_id');
+        if (!select || !select.value) {
+            alert('Пожалуйста, выберите задание на пошив.');
+            return false;
+        }
+
+        // Проверка, что хотя бы одно поле больше нуля (уже делается в validateQcSum)
         const packedInput = document.getElementById('quantity_packed');
         const defectiveInput = document.getElementById('quantity_defective');
         const packed = parseInt(packedInput.value) || 0;
         const defective = parseInt(defectiveInput.value) || 0;
-
         if (packed === 0 && defective === 0) {
              alert('Укажите количество упакованных или бракованных изделий (хотя бы одно должно быть больше н��ля).');
              return false;
         }
 
-
         return true; // Разрешить отправку формы
     }
 
@@ -2260,8 +2357,8 @@ ADMIN_CONTENT = """
               <tr class="material-row" data-name="{{ material.name|lower }}" data-category="{{ material.category|default('Без категории')|lower }}">
                 <td title="{{ material.id }}"><small>{{ material.id[:8] }}...</small></td>
                 <td>{{ material.name }}</td>
-                <td><span class="badge bg-info">{{ material.category | default('Без категории') }}</span></td>
-                <td>{{ 'Ткань' if material.type == 'fabric' else 'Фурнитура' }}</td>
+                <td><span class="badge bg-info text-dark">{{ material.category | default('Без категории') }}</span></td>
+                <td><span class="badge {{ 'bg-primary' if material.type == 'fabric' else 'bg-secondary' }}">{{ 'Ткань' if material.type == 'fabric' else 'Фурнитура' }}</span></td>
                 {# Используем quantity_dec для отображения #}
                 <td data-sort="{{ material.quantity_dec }}">{{ material.quantity_dec|string|replace('.', ',') }}</td>
                 <td>{{ material.unit }}</td>
@@ -2471,7 +2568,7 @@ ADMIN_CONTENT = """
                     <td><span class="badge bg-dark">{{ defect.type|replace('_', ' ')|title }}</span></td>
                     <td>{{ defect.quantity_view }}</td> {# Используем quantity_view #}
                     <td>{{ defect.unit }}</td>
-                    <td>{{ defect.stage|replace('_', ' ')|title }}</td>
+                    <td><span class="badge {{ 'bg-warning text-dark' if defect.stage == 'qc_packing' else 'bg-danger' }}">{{ defect.stage|replace('_', ' ')|title }}</span></td>
                     <td>{{ defect.reason | default('Не указана') }}</td>
                     <td>{{ defect.timestamp[:16] | replace('T', ' ') if defect.timestamp else 'N/A' }}</td>
                      <td title="{{ defect.sewing_task_id }}"><small>{{ defect.sewing_task_id[:8] if defect.sewing_task_id else 'N/A' }}...</small></td>
@@ -2495,8 +2592,8 @@ const searchInput = document.getElementById('material-search');
 const tableBody = document.getElementById('materials-table')?.querySelector('tbody'); // Добавил ? для безопасности
 
 if (searchInput && tableBody) {
-    searchInput.addEventListener('keyup', function() {
-        const searchTerm = searchInput.value.toLowerCase();
+    searchInput.addEventListener('input', function() { // Используем 'input' для немедленной реакции
+        const searchTerm = searchInput.value.toLowerCase().trim();
         const rows = tableBody.querySelectorAll('tr.material-row'); // Используем класс для точности
 
         rows.forEach(row => {
@@ -2518,23 +2615,39 @@ function sortTable(columnIndex, tableId, isNumeric = false) {
     if (!table) return;
     const tbody = table.querySelector('tbody');
     if (!tbody) return;
+
+    // Сохраняем строки заголовка, если они есть (например, <thead>)
+     const headerRow = table.querySelector('thead tr');
+     if (!headerRow) return; // Нужна строка заголовка
+
     const rows = Array.from(tbody.querySelectorAll('tr'));
-    const headerCell = table.querySelector(`thead th:nth-child(${columnIndex + 1})`); // +1 т.к. nth-child(1) - первый
+    if (rows.length < 2) return; // Нечего сортировать
+
+    const headerCell = headerRow.querySelector(`th:nth-child(${columnIndex + 1})`); // +1 т.к. nth-child(1) - первый
+    if (!headerCell) return; // Не найдена ячейка заголовка
 
     // Определяем направление сортировки
-    let currentDir = headerCell?.dataset.sortDir || 'asc'; // Используем data-атрибут для хранения направления
+    let currentDir = headerCell.dataset.sortDir || 'asc'; // Используем data-атрибут для хранения направления
     let newDir = currentDir === 'asc' ? 'desc' : 'asc';
 
-    // Сбрасываем направления у других колонок
-    table.querySelectorAll('thead th').forEach(th => th.dataset.sortDir = '');
+    // Сбрасываем направления у других колонок и иконки
+    headerRow.querySelectorAll('th').forEach((th, index) => {
+         const icon = th.querySelector('i.fa-sort, i.fa-sort-up, i.fa-sort-down');
+         if (icon && index !== columnIndex) {
+            icon.classList.remove('fa-sort-up', 'fa-sort-down');
+            icon.classList.add('fa-sort');
+         }
+         th.dataset.sortDir = ''; // Сброс направления для всех, кроме текущей
+    });
     // Устанавливаем новое направление для текущей колонки
-    if (headerCell) headerCell.dataset.sortDir = newDir;
+    headerCell.dataset.sortDir = newDir;
 
 
     rows.sort((a, b) => {
         let cellA = a.querySelector(`td:nth-child(${columnIndex + 1})`);
         let cellB = b.querySelector(`td:nth-child(${columnIndex + 1})`);
 
+        // Используем data-sort атрибут если он есть, иначе текст ячейки
         let valA = cellA ? (cellA.dataset.sort || cellA.textContent || '').trim() : '';
         let valB = cellB ? (cellB.dataset.sort || cellB.textContent || '').trim() : '';
 
@@ -2543,20 +2656,17 @@ function sortTable(columnIndex, tableId, isNumeric = false) {
             // Преобразуем в число, заменяя запятую на точку и обрабатывая ошибки
              valA = parseFloat(String(valA).replace(',', '.')) || 0;
              valB = parseFloat(String(valB).replace(',', '.')) || 0;
+             // Для корректной сортировки чисел
+             const comparison = valA - valB;
+              return newDir === 'asc' ? comparison : -comparison;
 
         } else {
             // Текстовое сравнение без учета регистра
             valA = valA.toLowerCase();
             valB = valB.toLowerCase();
+            const comparison = valA.localeCompare(valB); // Используем localeCompare для корректной сортировки строк
+             return newDir === 'asc' ? comparison : -comparison;
         }
-
-        if (valA < valB) {
-            return newDir === 'asc' ? -1 : 1;
-        }
-        if (valA > valB) {
-            return newDir === 'asc' ? 1 : -1;
-        }
-        return 0; // Равны
     });
 
     // Удаляем старые строки и вставляем отсортированные
@@ -2565,38 +2675,44 @@ function sortTable(columnIndex, tableId, isNumeric = false) {
     }
     rows.forEach(row => tbody.appendChild(row));
 
-     // Обновляем иконки сортировки
-     table.querySelectorAll('thead th i.fa-sort').forEach(icon => {
-        icon.classList.remove('fa-sort-up', 'fa-sort-down');
-        icon.classList.add('fa-sort'); // Сброс на базовую иконку
-     });
-      if (headerCell) {
-         const icon = headerCell.querySelector('i.fa-sort');
-         if (icon) {
-             icon.classList.remove('fa-sort');
-             icon.classList.add(newDir === 'asc' ? 'fa-sort-up' : 'fa-sort-down');
-         }
-      }
+     // Обновляем иконку сортировки для текущей колонки
+     const icon = headerCell.querySelector('i.fa-sort, i.fa-sort-up, i.fa-sort-down');
+     if (icon) {
+         icon.classList.remove('fa-sort', 'fa-sort-up', 'fa-sort-down');
+         icon.classList.add(newDir === 'asc' ? 'fa-sort-up' : 'fa-sort-down');
+     }
 }
 
-// --- Активация табов Bootstrap (на случай если что-то не так с авто-активацией) ---
+// --- Активация табов Bootstrap ---
 document.addEventListener('DOMContentLoaded', function () {
-    var triggerTabList = [].slice.call(document.querySelectorAll('#adminTabs button'))
+    var triggerTabList = [].slice.call(document.querySelectorAll('#adminTabs button[data-bs-toggle="tab"]'))
     triggerTabList.forEach(function (triggerEl) {
-        var tabTrigger = new bootstrap.Tab(triggerEl)
+        // Проверяем, существует ли уже экземпляр Tab
+        var tabInstance = bootstrap.Tab.getInstance(triggerEl);
+        if (!tabInstance) {
+             tabTrigger = new bootstrap.Tab(triggerEl); // Создаем, только если нет
+        } else {
+             tabTrigger = tabInstance; // Используем существующий
+        }
 
         triggerEl.addEventListener('click', function (event) {
-            event.preventDefault()
-            tabTrigger.show()
-        })
-    })
+            event.preventDefault();
+            // Переключаем таб, если он не активен
+            if (!triggerEl.classList.contains('active')) {
+                 tabTrigger.show();
+            }
+        });
+    });
 
     // Активируем первую вкладку при загрузке, если ни одна не активна
-    const firstTab = document.querySelector('#adminTabs button');
-    if (firstTab && !document.querySelector('#adminTabs button.active')) {
-        const tab = new bootstrap.Tab(firstTab);
-        tab.show();
-    }
+     // (Bootstrap 5 обычно делает это сам, но можно оставить для надежности)
+     const firstTabEl = document.querySelector('#adminTabs button[data-bs-toggle="tab"]');
+     const activeTabEl = document.querySelector('#adminTabs button[data-bs-toggle="tab"].active');
+     if (firstTabEl && !activeTabEl) {
+          var firstTab = bootstrap.Tab.getInstance(firstTabEl) || new bootstrap.Tab(firstTabEl);
+          firstTab.show();
+     }
+
 });
 
 </script>
@@ -2624,4 +2740,4 @@ if __name__ == '__main__':
     logging.info("Запуск Flask приложения на порту 7860...")
     # debug=False для продакшена
     # host='0.0.0.0' для доступа из сети
-    app.run(debug=True, host='0.0.0.0', port=7860)
+    app.run(debug=True, host='0.0.0.0', port=7860)