Skip to content

fix: umfassender Code-Review — 9 Bugs gefixt + Version v1.88_Build_9426#24

Merged
BEKO2210 merged 1 commit intomainfrom
claude/fix-apple-camera-barcode-5DtQy
Mar 9, 2026
Merged

fix: umfassender Code-Review — 9 Bugs gefixt + Version v1.88_Build_9426#24
BEKO2210 merged 1 commit intomainfrom
claude/fix-apple-camera-barcode-5DtQy

Conversation

@BEKO2210
Copy link
Copy Markdown
Owner

@BEKO2210 BEKO2210 commented Mar 9, 2026

CRITICAL:

  • IndexedDB Boolean-Query: .equals(0) matchte nie boolean false → Notifications waren komplett kaputt (keine Produkte gefunden) → Gefixt in notifications.ts und db.ts mit .filter()

HIGH:

  • Delete-Bestätigung: "Löschen"/"Abbrechen" Buttons hatten kein stopPropagation → öffneten gleichzeitig das Detail-Modal
  • Notification: expired-Branch war unerreichbar (daysLeft immer >0) → Threshold 0 hinzugefügt für Ablauftag-Benachrichtigung

MEDIUM:

  • ProductForm: Live-Query überschrieb Formulardaten während Bearbeitung → populatedRef verhindert mehrfaches Überschreiben
  • ImportResult: war kein Error → fehlende Stack Traces, catch-Probleme → extends Error hinzugefügt
  • BarcodeScanner: stale isOnline Closure im Scan-Callback → isOnlineRef für aktuellen Wert + Reader Cleanup via controls.stop()
  • Lagerort löschen: keine Warnung bei Produkten die den Ort nutzen → Prüfung + Hinweis wenn aktive Produkte referenziert werden

LOW:

  • Dark Mode: Default war System-Preference statt dark (wie dokumentiert)
  • Toast CSS: translate(-50%) Konflikt mit Tailwind -translate-x-1/2

Version: 1.88_Build_9426

https://claude.ai/code/session_0169CbohMRqddD1LjPg8FmXf

CRITICAL:
- IndexedDB Boolean-Query: .equals(0) matchte nie boolean false
  → Notifications waren komplett kaputt (keine Produkte gefunden)
  → Gefixt in notifications.ts und db.ts mit .filter()

HIGH:
- Delete-Bestätigung: "Löschen"/"Abbrechen" Buttons hatten kein
  stopPropagation → öffneten gleichzeitig das Detail-Modal
- Notification: expired-Branch war unerreichbar (daysLeft immer >0)
  → Threshold 0 hinzugefügt für Ablauftag-Benachrichtigung

MEDIUM:
- ProductForm: Live-Query überschrieb Formulardaten während Bearbeitung
  → populatedRef verhindert mehrfaches Überschreiben
- ImportResult: war kein Error → fehlende Stack Traces, catch-Probleme
  → extends Error hinzugefügt
- BarcodeScanner: stale isOnline Closure im Scan-Callback
  → isOnlineRef für aktuellen Wert + Reader Cleanup via controls.stop()
- Lagerort löschen: keine Warnung bei Produkten die den Ort nutzen
  → Prüfung + Hinweis wenn aktive Produkte referenziert werden

LOW:
- Dark Mode: Default war System-Preference statt dark (wie dokumentiert)
- Toast CSS: translate(-50%) Konflikt mit Tailwind -translate-x-1/2

Version: 1.88_Build_9426

https://claude.ai/code/session_0169CbohMRqddD1LjPg8FmXf
@BEKO2210 BEKO2210 merged commit 7d2e46e into main Mar 9, 2026
0 of 2 checks passed
@BEKO2210 BEKO2210 deleted the claude/fix-apple-camera-barcode-5DtQy branch March 9, 2026 18:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants