From f062bd816846f7c28bf22741b38f1e87cbf25198 Mon Sep 17 00:00:00 2001
From: Andreas Knuth <andreas.knuth@gmail.com>
Date: Tue, 26 May 2026 13:55:27 -0500
Subject: [PATCH] Technician

---
 public/index.html                 | 36 +++++++++++++++++++----
 public/js/modals/invoice-modal.js | 46 +++++++++++++++++++++++++++--
 public/js/utils/api.js            |  8 +++++-
 public/js/views/invoice-view.js   | 48 +++++++++++++++++++++++++++++++
 public/js/views/settings-view.js  | 39 ++++++++++++++++++++++++-
 src/index.js                      |  2 +-
 src/routes/invoices.js            | 23 +++++++--------
 src/routes/settings.js            | 37 ++++++++++++++++++++++++
 8 files changed, 216 insertions(+), 23 deletions(-)

diff --git a/public/index.html b/public/index.html
index df17ea9..457257b 100644
--- a/public/index.html
+++ b/public/index.html
@@ -167,6 +167,23 @@
                     </button>
                     
                     <div id="upload-status" class="mt-4"></div>
+                    <hr class="my-8 border-gray-200">
+
+                    <h3 class="text-xl font-semibold mb-4 text-gray-800">Invoice Workers</h3>
+                    <p class="text-gray-600 mb-4">
+                        Comma-separated list of workers available in the invoice "Worker" dropdown.
+                        Example: <code class="bg-gray-100 px-1 rounded">Wallie, Elvin, Andreas</code>
+                    </p>
+                    <div class="mb-4">
+                        <input type="text" id="workers-input"
+                               class="w-full max-w-lg px-4 py-2 border border-gray-300 rounded-md focus:ring-blue-500 focus:border-blue-500"
+                               placeholder="Wallie, Elvin, Andreas">
+                    </div>
+                    <button onclick="saveWorkerSettings()"
+                            class="bg-green-600 hover:bg-green-700 text-white px-6 py-2 rounded-lg">
+                        Save Worker List
+                    </button>
+                    <div id="workers-status" class="mt-4"></div>
 
                     <hr class="my-8 border-gray-200">
 
@@ -448,11 +465,20 @@
                     </div>
                 </div>
                 
-                <div>
-                    <label class="block text-sm font-medium text-gray-700 mb-1">Authorization (optional)</label>
-                    <input type="text" id="invoice-authorization" 
-                           class="w-full px-4 py-2 border border-gray-300 rounded-md focus:ring-blue-500 focus:border-blue-500"
-                           placeholder="P.O. Number, Authorization Code, etc.">
+                <div class="grid grid-cols-3 gap-4">
+                    <div class="col-span-2">
+                        <label class="block text-sm font-medium text-gray-700 mb-1">Authorization (optional)</label>
+                        <input type="text" id="invoice-authorization" 
+                               class="w-full px-4 py-2 border border-gray-300 rounded-md focus:ring-blue-500 focus:border-blue-500"
+                               placeholder="P.O. Number, Authorization Code, etc.">
+                    </div>
+                    <div>
+                        <label class="block text-sm font-medium text-gray-700 mb-1">Worker</label>
+                        <select id="invoice-worker"
+                                class="w-full px-4 py-2 border border-gray-300 rounded-md focus:ring-blue-500 focus:border-blue-500">
+                            <option value="">— None —</option>
+                        </select>
+                    </div>
                 </div>
                 
                 <div>
diff --git a/public/js/modals/invoice-modal.js b/public/js/modals/invoice-modal.js
index 0e00996..6400665 100644
--- a/public/js/modals/invoice-modal.js
+++ b/public/js/modals/invoice-modal.js
@@ -8,9 +8,35 @@
  */
 import { addItem, getItems, resetItemCounter } from '../utils/item-editor.js';
 import { setDefaultDate, showSpinner, hideSpinner } from '../utils/helpers.js';
+import '../utils/api.js';
 
 let currentInvoiceId = null;
 let qboLaborRate = null;
+let workerList = [];
+
+export async function loadWorkers() {
+    try {
+        const result = await window.API.settings.get('invoice_workers');
+        if (result && result.value) {
+            workerList = result.value.split(',').map(w => w.trim()).filter(Boolean);
+        } else {
+            workerList = [];
+        }
+        populateWorkerDropdown();
+        console.log(`👷 ${workerList.length} Bearbeiter geladen`);
+    } catch (e) {
+        console.log('Worker-Liste konnte nicht geladen werden.');
+    }
+}
+
+function populateWorkerDropdown() {
+    const sel = document.getElementById('invoice-worker');
+    if (!sel) return;
+    const current = sel.value;
+    sel.innerHTML = `<option value="">— None —</option>` +
+        workerList.map(w => `<option value="${w}">${w}</option>`).join('');
+    if (current) sel.value = current;
+}
 
 export async function loadLaborRate() {
     try {
@@ -179,6 +205,11 @@ async function loadInvoiceForEdit(invoiceId) {
     document.getElementById('invoice-authorization').value = data.invoice.auth_code || '';
     document.getElementById('invoice-tax-exempt').checked = data.invoice.tax_exempt;
     document.getElementById('invoice-bill-to-name').value = data.invoice.bill_to_name || '';
+    
+    // Worker
+    populateWorkerDropdown();
+    const workerEl = document.getElementById('invoice-worker');
+    if (workerEl) workerEl.value = data.invoice.worker || '';
 
     const sendDateEl = document.getElementById('invoice-send-date');
     if (sendDateEl) {
@@ -226,7 +257,12 @@ function prepareNewInvoice() {
     document.getElementById('invoice-terms').value = 'Net 14';
     document.getElementById('invoice-number').value = '';
     document.getElementById('invoice-send-date').value = '';
-
+    
+    // Worker zurücksetzen
+    populateWorkerDropdown();
+    const workerEl = document.getElementById('invoice-worker');
+    if (workerEl) workerEl.value = '';
+    
     // Reset recurring
     const recurringCb = document.getElementById('invoice-recurring');
     const recurringGroup = document.getElementById('invoice-recurring-group');
@@ -278,6 +314,7 @@ export async function handleInvoiceSubmit(e) {
         tax_exempt: document.getElementById('invoice-tax-exempt').checked,
         scheduled_send_date: document.getElementById('invoice-send-date')?.value || null,
         bill_to_name: document.getElementById('invoice-bill-to-name')?.value || null,
+        worker: document.getElementById('invoice-worker')?.value || null,
         is_recurring: isRecurring,
         recurring_interval: recurringInterval,
         items: getItems('invoice-items')
@@ -314,7 +351,9 @@ export async function handleInvoiceSubmit(e) {
 export function initInvoiceModal() {
     const form = document.getElementById('invoice-form');
     if (form) form.addEventListener('submit', handleInvoiceSubmit);
-
+    
+    loadWorkers();   // Bearbeiterliste laden
+    
     const taxExempt = document.getElementById('invoice-tax-exempt');
     if (taxExempt) taxExempt.addEventListener('change', updateInvoiceTotals);
 
@@ -342,4 +381,5 @@ export function initInvoiceModal() {
 
 window.openInvoiceModal = openInvoiceModal;
 window.closeInvoiceModal = closeInvoiceModal;
-window.addInvoiceItem = addInvoiceItem;
\ No newline at end of file
+window.addInvoiceItem = addInvoiceItem;
+window.reloadInvoiceWorkers = loadWorkers;
\ No newline at end of file
diff --git a/public/js/utils/api.js b/public/js/utils/api.js
index 311c479..a283702 100644
--- a/public/js/utils/api.js
+++ b/public/js/utils/api.js
@@ -212,7 +212,13 @@ const API = {
                 method: 'POST',
                 body: formData
             }).then(r => r.json());
-        }
+        },
+        get: (key) => fetch(`/api/settings/${encodeURIComponent(key)}`).then(r => r.json()),
+        set: (key, value) => fetch(`/api/settings/${encodeURIComponent(key)}`, {
+            method: 'PUT',
+            headers: { 'Content-Type': 'application/json' },
+            body: JSON.stringify({ value })
+        }).then(r => r.json())
     }
 };
 
diff --git a/public/js/views/invoice-view.js b/public/js/views/invoice-view.js
index aa6e7f0..d6fe2e8 100644
--- a/public/js/views/invoice-view.js
+++ b/public/js/views/invoice-view.js
@@ -5,6 +5,7 @@ let invoices = [];
 let filterCustomer = localStorage.getItem('inv_filterCustomer') || '';
 let filterStatus = localStorage.getItem('inv_filterStatus') || 'unpaid';
 let groupBy = localStorage.getItem('inv_groupBy') || 'none';
+let filterWorker = localStorage.getItem('inv_filterWorker') || '';
 
 const OVERDUE_DAYS = 30;
 
@@ -194,6 +195,7 @@ function saveSettings() {
     localStorage.setItem('inv_filterStatus', filterStatus);
     localStorage.setItem('inv_groupBy', groupBy);
     localStorage.setItem('inv_filterCustomer', filterCustomer);
+    localStorage.setItem('inv_filterWorker', filterWorker);
 }
 
 // ============================================================
@@ -237,6 +239,13 @@ function getFilteredInvoices() {
         const s = filterCustomer.toLowerCase();
         f = f.filter(i => (i.customer_name || '').toLowerCase().includes(s));
     }
+    if (filterWorker) {
+        if (filterWorker === '__none__') {
+            f = f.filter(i => !i.worker);
+        } else {
+            f = f.filter(i => i.worker === filterWorker);
+        }
+    }    
     f.sort((a, b) => (parseLocalDate(b.invoice_date) || 0) - (parseLocalDate(a.invoice_date) || 0));
     return f;
 }
@@ -563,6 +572,13 @@ export function injectToolbar() {
                        class="px-3 py-1.5 border border-gray-300 rounded-md text-sm w-48 focus:ring-blue-500 focus:border-blue-500">
             </div>
             <div class="w-px h-8 bg-gray-300"></div>
+            <div class="flex items-center gap-2">
+                <label class="text-sm font-medium text-gray-700">Worker:</label>
+                <select id="invoice-filter-worker" class="px-3 py-1.5 border border-gray-300 rounded-md text-sm bg-white">
+                    <option value="">All</option>
+                </select>
+            </div>            
+            <div class="w-px h-8 bg-gray-300"></div>
             <div class="flex items-center gap-2">
                 <label class="text-sm font-medium text-gray-700">Group:</label>
                 <select id="invoice-group-by" class="px-3 py-1.5 border border-gray-300 rounded-md text-sm bg-white">
@@ -592,6 +608,11 @@ export function injectToolbar() {
     document.getElementById('invoice-group-by').addEventListener('change', (e) => {
         groupBy = e.target.value; saveSettings(); renderInvoiceView();
     });
+    // Worker-Filter befüllen und verdrahten
+    populateWorkerFilter();
+    document.getElementById('invoice-filter-worker').addEventListener('change', (e) => {
+        filterWorker = e.target.value; saveSettings(); renderInvoiceView();
+    });    
 }
 
 // ============================================================
@@ -806,6 +827,33 @@ async function _saveSentDates(invoiceId) {
         alert('Network error.');
     }
 }
+
+async function populateWorkerFilter() {
+    const sel = document.getElementById('invoice-filter-worker');
+    if (!sel) return;
+
+    // Worker aus den geladenen Rechnungen
+    const fromInvoices = new Set(invoices.map(i => i.worker).filter(Boolean));
+
+    // Plus konfigurierte Liste aus Settings
+    try {
+        const result = await window.API.settings.get('invoice_workers');
+        if (result && result.value) {
+            result.value.split(',').map(w => w.trim()).filter(Boolean)
+                .forEach(w => fromInvoices.add(w));
+        }
+    } catch (e) { /* ignore */ }
+
+    const workers = [...fromInvoices].sort();
+    const hasNoWorker = invoices.some(i => !i.worker);
+
+    sel.innerHTML =
+        `<option value="">All</option>` +
+        workers.map(w => `<option value="${w}">${w}</option>`).join('') +
+        (hasNoWorker ? `<option value="__none__">— No worker —</option>` : '');
+
+    sel.value = filterWorker;
+}
 // ============================================================
 // Expose
 // ============================================================
diff --git a/public/js/views/settings-view.js b/public/js/views/settings-view.js
index 506e262..0ea6675 100644
--- a/public/js/views/settings-view.js
+++ b/public/js/views/settings-view.js
@@ -2,6 +2,7 @@
  * settings-view.js — Logo upload, QBO import, QBO connection test
  * Extracted from app.js
  */
+import '../utils/api.js';
 
 let currentLogoFile = null;
 
@@ -69,6 +70,8 @@ export function initSettingsView() {
             }
         });
     }
+
+    loadWorkerSettings();
 }
 
 export async function checkQboOverdue() {
@@ -175,8 +178,42 @@ export async function importFromQBO() {
         btn.disabled = false;
     }
 }
+export async function loadWorkerSettings() {
+    try {
+        const result = await window.API.settings.get('invoice_workers');
+        const input = document.getElementById('workers-input');
+        if (input) input.value = (result && result.value) ? result.value : '';
+    } catch (e) {
+        console.error('Error loading worker settings:', e);
+    }
+}
 
+export async function saveWorkerSettings() {
+    const input = document.getElementById('workers-input');
+    const statusEl = document.getElementById('workers-status');
+    if (!input) return;
+
+    const value = input.value.trim();
+    statusEl.innerHTML = '<p class="text-blue-600">Saving...</p>';
+
+    try {
+        const result = await window.API.settings.set('invoice_workers', value);
+        if (result.success) {
+            statusEl.innerHTML = '<p class="text-green-600">✓ Worker list saved.</p>';
+            // Dropdown im Invoice-Modal aktualisieren
+            if (typeof window.reloadInvoiceWorkers === 'function') {
+                window.reloadInvoiceWorkers();
+            }
+        } else {
+            statusEl.innerHTML = `<p class="text-red-600">✗ ${result.error || 'Save failed'}</p>`;
+        }
+    } catch (e) {
+        console.error('Error saving worker settings:', e);
+        statusEl.innerHTML = '<p class="text-red-600">✗ Save failed</p>';
+    }
+}
 // Expose for onclick handlers
 window.uploadLogo = uploadLogo;
 window.checkQboOverdue = checkQboOverdue;
-window.importFromQBO = importFromQBO;
\ No newline at end of file
+window.importFromQBO = importFromQBO;
+window.saveWorkerSettings = saveWorkerSettings;
\ No newline at end of file
diff --git a/src/index.js b/src/index.js
index d5968ef..7ca65b4 100644
--- a/src/index.js
+++ b/src/index.js
@@ -122,7 +122,7 @@ app.use('/api/invoices', invoiceRoutes);
 app.use('/api/payments', paymentRoutes);
 app.use('/api/qbo', qboRoutes);
 app.use('/api/accounting', accountingRoutes);
-app.use('/api', settingsRoutes);
+app.use('/api/settings', settingsRoutes);
 
 // Start server
 async function startServer() {
diff --git a/src/routes/invoices.js b/src/routes/invoices.js
index 96d0643..191c8bc 100644
--- a/src/routes/invoices.js
+++ b/src/routes/invoices.js
@@ -135,7 +135,7 @@ router.get('/:id', async (req, res) => {
 
 // POST create invoice
 router.post('/', async (req, res) => {
-    const { invoice_number, customer_id, invoice_date, terms, auth_code, tax_exempt, items, scheduled_send_date, bill_to_name, created_from_quote_id, is_recurring, recurring_interval } = req.body;
+    const { invoice_number, customer_id, invoice_date, terms, auth_code, tax_exempt, items, scheduled_send_date, bill_to_name, created_from_quote_id, is_recurring, recurring_interval, worker } = req.body;
 
     const client = await pool.connect();
     try {
@@ -184,9 +184,9 @@ router.post('/', async (req, res) => {
         const total = subtotal + tax_amount;
         const next_recurring_date = is_recurring ? calculateNextRecurringDate(invoice_date, recurring_interval) : null;
         const invoiceResult = await client.query(
-                `INSERT INTO invoices (invoice_number, customer_id, invoice_date, terms, auth_code, tax_exempt, tax_rate, subtotal, tax_amount, total, scheduled_send_date, bill_to_name, created_from_quote_id, is_recurring, recurring_interval, next_recurring_date)
-                    VALUES ($1, $2, $3, $4, $5, $6, $7, $8, $9, $10, $11, $12, $13, $14, $15, $16) RETURNING *`,
-                [tempNumber, customer_id, invoice_date, terms, auth_code, tax_exempt, tax_rate, subtotal, tax_amount, total, scheduled_send_date || null, bill_to_name || null, created_from_quote_id, is_recurring || false, recurring_interval || null, next_recurring_date]
+                `INSERT INTO invoices (invoice_number, customer_id, invoice_date, terms, auth_code, tax_exempt, tax_rate, subtotal, tax_amount, total, scheduled_send_date, bill_to_name, created_from_quote_id, is_recurring, recurring_interval, next_recurring_date, worker)
+                    VALUES ($1, $2, $3, $4, $5, $6, $7, $8, $9, $10, $11, $12, $13, $14, $15, $16, $17) RETURNING *`,
+                [tempNumber, customer_id, invoice_date, terms, auth_code, tax_exempt, tax_rate, subtotal, tax_amount, total, scheduled_send_date || null, bill_to_name || null, created_from_quote_id, is_recurring || false, recurring_interval || null, next_recurring_date, worker || null]
         );
         const invoiceId = invoiceResult.rows[0].id;
 
@@ -228,8 +228,7 @@ router.post('/', async (req, res) => {
 // PUT update invoice
 router.put('/:id', async (req, res) => {
     const { id } = req.params;
-    const { invoice_number, customer_id, invoice_date, terms, auth_code, tax_exempt, items, scheduled_send_date, bill_to_name, is_recurring, recurring_interval } = req.body;
-
+    const { invoice_number, customer_id, invoice_date, terms, auth_code, tax_exempt, items, scheduled_send_date, bill_to_name, is_recurring, recurring_interval, worker } = req.body;
     const client = await pool.connect();
     try {
         await client.query('BEGIN');
@@ -278,16 +277,16 @@ router.put('/:id', async (req, res) => {
         if (invoice_number) {
             await client.query(
                 `UPDATE invoices SET invoice_number = $1, customer_id = $2, invoice_date = $3, terms = $4, auth_code = $5, tax_exempt = $6,
-                 tax_rate = $7, subtotal = $8, tax_amount = $9, total = $10, scheduled_send_date = $11, bill_to_name = $12, updated_at = CURRENT_TIMESTAMP
-                 WHERE id = $13`,
-                [invoice_number, customer_id, invoice_date, terms, auth_code, tax_exempt, tax_rate, subtotal, tax_amount, total, scheduled_send_date || null, bill_to_name || null, id]
+                 tax_rate = $7, subtotal = $8, tax_amount = $9, total = $10, scheduled_send_date = $11, bill_to_name = $12, worker = $13, updated_at = CURRENT_TIMESTAMP
+                 WHERE id = $14`,
+                [invoice_number, customer_id, invoice_date, terms, auth_code, tax_exempt, tax_rate, subtotal, tax_amount, total, scheduled_send_date || null, bill_to_name || null, worker || null, id]
             );
         } else {
             await client.query(
                 `UPDATE invoices SET customer_id = $1, invoice_date = $2, terms = $3, auth_code = $4, tax_exempt = $5,
-                 tax_rate = $6, subtotal = $7, tax_amount = $8, total = $9, scheduled_send_date = $10, bill_to_name = $11, updated_at = CURRENT_TIMESTAMP
-                 WHERE id = $12`,
-                [customer_id, invoice_date, terms, auth_code, tax_exempt, tax_rate, subtotal, tax_amount, total, scheduled_send_date || null, bill_to_name || null, id]
+                 tax_rate = $6, subtotal = $7, tax_amount = $8, total = $9, scheduled_send_date = $10, bill_to_name = $11, worker = $12, updated_at = CURRENT_TIMESTAMP
+                 WHERE id = $13`,
+                [customer_id, invoice_date, terms, auth_code, tax_exempt, tax_rate, subtotal, tax_amount, total, scheduled_send_date || null, bill_to_name || null, worker || null, id]
             );
         }
 
diff --git a/src/routes/settings.js b/src/routes/settings.js
index 1802598..23f6ea3 100644
--- a/src/routes/settings.js
+++ b/src/routes/settings.js
@@ -68,4 +68,41 @@ router.post('/upload-logo', upload.single('logo'), async (req, res) => {
     }
 });
 
+// GET /api/settings/:key — liefert einen einzelnen Settings-Wert
+router.get('/:key', async (req, res) => {
+    try {
+        const result = await pool.query(
+            'SELECT value FROM settings WHERE key = $1',
+            [req.params.key]
+        );
+        if (result.rows.length === 0) {
+            return res.json({ key: req.params.key, value: null });
+        }
+        res.json({ key: req.params.key, value: result.rows[0].value });
+    } catch (error) {
+        console.error('Error fetching setting:', error);
+        res.status(500).json({ error: 'Error fetching setting' });
+    }
+});
+
+// PUT /api/settings/:key — setzt einen Settings-Wert (upsert)
+router.put('/:key', express.json(), async (req, res) => {
+    const { value } = req.body;
+    if (typeof value !== 'string') {
+        return res.status(400).json({ error: 'value must be a string' });
+    }
+    try {
+        await pool.query(
+            `INSERT INTO settings (key, value)
+             VALUES ($1, $2)
+             ON CONFLICT (key) DO UPDATE SET value = EXCLUDED.value`,
+            [req.params.key, value]
+        );
+        res.json({ success: true, key: req.params.key, value });
+    } catch (error) {
+        console.error('Error saving setting:', error);
+        res.status(500).json({ error: 'Error saving setting' });
+    }
+});
+
 module.exports = router;