[Bug] Import (JSON/Excel) ohne Validierung — Crash & fehlendes Feedback #3

New Issue

2026-06-03T14:10:01+02:00

ferdi2go commented

2026-06-03 14:10:01 +02:00

Schweregrad: MEDIUM

Dateien: js/data.js:32-42 (loadFromJSON), js/app.js:20-22, js/history.js:78, js/employees.js:70-81

Problem:

loadFromJSON übernimmt employees ohne pro-Mitarbeiter-Prüfung. Fehlt constraints (alte/handgemachte Datei), crasht js/algorithm.js:40 (c.neverDays) bei der Generierung. Doppelte IDs können Doppelbesetzung am selben Tag erzeugen.
Excel-Import (readEmployeeNames) wirft bei Nicht-XLSX-Datei oder fehlendem XLSX (CDN blockiert) — nicht gefangen, kein Nutzer-Feedback.

Fix: In loadFromJSON jeden Mitarbeiter validieren/normalisieren (fehlende constraints mit Defaults füllen, IDs deduplizieren, Urlaubsbereiche from<=to). Excel-Import in try/catch + typeof XLSX-Guard mit sichtbarer Fehlermeldung.

**Schweregrad:** MEDIUM **Dateien:** `js/data.js:32-42` (loadFromJSON), `js/app.js:20-22`, `js/history.js:78`, `js/employees.js:70-81` **Problem:** 1. `loadFromJSON` übernimmt `employees` ohne pro-Mitarbeiter-Prüfung. Fehlt `constraints` (alte/handgemachte Datei), crasht `js/algorithm.js:40` (`c.neverDays`) bei der Generierung. Doppelte IDs können Doppelbesetzung am selben Tag erzeugen. 2. Excel-Import (`readEmployeeNames`) wirft bei Nicht-XLSX-Datei oder fehlendem `XLSX` (CDN blockiert) — nicht gefangen, kein Nutzer-Feedback. **Fix:** In `loadFromJSON` jeden Mitarbeiter validieren/normalisieren (fehlende `constraints` mit Defaults füllen, IDs deduplizieren, Urlaubsbereiche `from<=to`). Excel-Import in `try/catch` + `typeof XLSX`-Guard mit sichtbarer Fehlermeldung.

ferdi2go added the severity/medium bug labels 2026-06-03 14:10:01 +02:00

ferdi2go commented

2026-06-03 14:20:41 +02:00

Behoben in 065876c: loadFromJSON und der localStorage-Start-Load normalisieren Mitarbeiter (normalizeEmployees): fehlende constraints werden gefüllt, unbrauchbare Zeilen verworfen, doppelte IDs aufgelöst, ungültige Urlaubsbereiche (from>to) entfernt. Excel-Im-/Export sind mit try/catch + typeof XLSX-Guard und Nutzer-Feedback abgesichert. Inkl. neuer Tests.

Behoben in `065876c`: `loadFromJSON` und der localStorage-Start-Load normalisieren Mitarbeiter (`normalizeEmployees`): fehlende `constraints` werden gefüllt, unbrauchbare Zeilen verworfen, doppelte IDs aufgelöst, ungültige Urlaubsbereiche (`from>to`) entfernt. Excel-Im-/Export sind mit `try/catch` + `typeof XLSX`-Guard und Nutzer-Feedback abgesichert. Inkl. neuer Tests.

ferdi2go closed this issue

2026-06-03 14:20:41 +02:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: ferdi2go/Meeting-Mixer#3