Update app.py

app.py

@@ -79,7 +79,7 @@ def load_json_data(file_key):
             with open(filepath, 'r', encoding='utf-8') as f:
                 return json.load(f)
         except (FileNotFoundError, json.JSONDecodeError):
-            return [] if file_key != 'users' else {}
+            return []
 
 def save_json_data(file_key, data):
     filepath, lock = DATA_FILES[file_key]
@@ -374,7 +374,6 @@ def stock_in():
         for i, variant in enumerate(product.get('variants', [])):
             if variant.get('id') == variant_id:
                 variant['stock'] = variant.get('stock', 0) + quantity
-                # Update cost price if provided, using weighted average
                 if cost_price_str:
                     new_cost = to_decimal(cost_price_str)
                     old_stock = variant.get('stock', 0) - quantity
@@ -406,7 +405,6 @@ def get_product_by_barcode(barcode):
     inventory = load_json_data('inventory')
     product = find_item_by_field(inventory, 'barcode', barcode)
     if product:
-        # Return only variants that are in stock
         active_variants = [v for v in product.get('variants', []) if v.get('stock', 0) > 0]
         if active_variants:
             product_copy = product.copy()
@@ -656,7 +654,7 @@ def product_roi_report():
                             if t['type'] == 'sale':
                                 total_qty_sold += item['quantity']
                             elif t['type'] == 'return':
-                                total_qty_sold += item['quantity'] # quantity is negative on returns
+                                total_qty_sold += item['quantity']
 
             current_stock = to_decimal(str(variant.get('stock', 0)))
             cost_price = to_decimal(variant.get('cost_price', '0'))
@@ -1006,7 +1004,10 @@ BASE_TEMPLATE = """
             .main-content { margin-left: 0; }
         }
         [data-bs-theme="dark"] body { background-color: #212529; color: #dee2e6; }
-        [data-bs-theme="dark"] .card, [data-bs-theme="dark"] .modal-content, [data-bs-theme="dark"] .list-group-item, [data-bs-theme="dark"] .table { background-color: #343a40; }
+        [data-bs-theme="dark"] .card, [data-bs-theme="dark"] .modal-content, [data-bs-theme="dark"] .list-group-item, [data-bs-theme="dark"] .table, [data-bs-theme="dark"] .accordion-item { background-color: #343a40; }
+        [data-bs-theme="dark"] .accordion-button { background-color: #3e444a; color: #fff; }
+        [data-bs-theme="dark"] .accordion-button:not(.collapsed) { background-color: #495057;}
+        [data-bs-theme="dark"] .accordion-button::after { filter: invert(1) grayscale(100) brightness(200%); }
         [data-bs-theme="dark"] .table-hover>tbody>tr:hover>* { color: var(--bs-table-hover-color); background-color: rgba(255, 255, 255, 0.075); }
         .product-card { cursor: pointer; }
         .product-card:hover { border-color: var(--bs-primary); }
@@ -1321,7 +1322,7 @@ document.addEventListener('DOMContentLoaded', () => {
         document.querySelectorAll('.product-card').forEach(card => {
             const productName = card.querySelector('.card-title').textContent.toLowerCase();
             const show = productName.includes(term) || card.dataset.barcode.includes(term);
-            card.parentElement.style.display = show ? '' : 'none'; // Target parent if inside a grid wrapper
+            card.style.display = show ? '' : 'none';
         });
     });
 
@@ -1458,7 +1459,6 @@ INVENTORY_CONTENT = """
   {% endfor %}
 </div>
 
-<!-- Add Product Modal -->
 <div class="modal fade" id="addProductModal" tabindex="-1">
   <div class="modal-dialog modal-lg">
     <div class="modal-content">
@@ -1475,7 +1475,6 @@ INVENTORY_CONTENT = """
           <hr>
           <h6>Варианты товара</h6>
           <div id="variants-container-add">
-            <!-- Variant fields will be added here by JS -->
           </div>
           <button type="button" class="btn btn-sm btn-outline-success mt-2" id="add-variant-btn-add">Добавить вариант</button>
         </div>
@@ -1485,7 +1484,6 @@ INVENTORY_CONTENT = """
   </div>
 </div>
 
-<!-- Edit Product Modals -->
 {% for p in inventory %}
 <div class="modal fade" id="editProductModal-{{ p.id }}" tabindex="-1">
   <div class="modal-dialog modal-lg">
@@ -1520,7 +1518,6 @@ INVENTORY_CONTENT = """
 </div>
 {% endfor %}
 
-<!-- Stock In Modal -->
 <div class="modal fade" id="stockInModal" tabindex="-1">
   <div class="modal-dialog">
     <div class="modal-content">
@@ -1563,7 +1560,6 @@ INVENTORY_CONTENT = """
 INVENTORY_SCRIPTS = """
 <script>
 document.addEventListener('DOMContentLoaded', () => {
-    // --- Barcode Scanner Logic for Modals ---
     let currentScanner = null;
     let currentScannerContainer = null;
     document.querySelectorAll('.scan-modal-btn').forEach(btn => {
@@ -1608,7 +1604,6 @@ document.addEventListener('DOMContentLoaded', () => {
         });
     });
     
-    // --- Dynamic Variant Fields Logic ---
     const createVariantRow = () => {
         const div = document.createElement('div');
         div.className = 'row g-2 mb-2 align-items-center variant-row';
@@ -1639,7 +1634,6 @@ document.addEventListener('DOMContentLoaded', () => {
         btn.addEventListener('click', e => e.target.closest('.variant-row').remove());
     });
     
-    // Auto-add first variant row on add modal show
     const addProductModal = document.getElementById('addProductModal');
     addProductModal.addEventListener('shown.bs.modal', () => {
         const container = document.getElementById('variants-container-add');
@@ -1648,7 +1642,6 @@ document.addEventListener('DOMContentLoaded', () => {
         }
     });
 
-    // --- Stock In Modal Logic ---
     const inventoryData = JSON.parse('{{ inventory|tojson|safe }}');
     const productSelect = document.getElementById('stockin-product');
     const variantSelect = document.getElementById('stockin-variant');