Skip to content

fix: Speichern-Button funktioniert nicht + mehrere Bugfixes#23

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

fix: Speichern-Button funktioniert nicht + mehrere Bugfixes#23
BEKO2210 merged 1 commit intomainfrom
claude/fix-apple-camera-barcode-5DtQy

Conversation

@BEKO2210
Copy link
Copy Markdown
Owner

@BEKO2210 BEKO2210 commented Mar 9, 2026

Bug 1: parseInt() zerstörte Dezimalwerte bei Menge (kg, Liter, g, ml)

  • parseInt("0.5") = 0, dann 0 || 1 = 1 → falsche Menge gespeichert
  • Gefixt: parseFloat() für quantity und minStock

Bug 2: Mengenfeld hatte min="1" fest, blockierte Dezimalwerte

  • Gefixt: dynamischer min/step je Einheit (kg=0.1, Liter=0.25, etc.)

Bug 3: Kein Error-Handling in handleSubmit

  • Bei DB-Fehlern blieb der Button auf "Speichert..." hängen
  • Gefixt: try/catch/finally mit Fehlermeldung und setSaving(false)

Bug 4: ConsumeModal startete immer bei 1, auch wenn max < 1

  • z.B. 0.5 kg Nüsse → Startwert war 1 (über dem Vorrat)
  • Gefixt: Startwert = Math.min(step, max)

https://claude.ai/code/session_0169CbohMRqddD1LjPg8FmXf

Bug 1: parseInt() zerstörte Dezimalwerte bei Menge (kg, Liter, g, ml)
- parseInt("0.5") = 0, dann 0 || 1 = 1 → falsche Menge gespeichert
- Gefixt: parseFloat() für quantity und minStock

Bug 2: Mengenfeld hatte min="1" fest, blockierte Dezimalwerte
- Gefixt: dynamischer min/step je Einheit (kg=0.1, Liter=0.25, etc.)

Bug 3: Kein Error-Handling in handleSubmit
- Bei DB-Fehlern blieb der Button auf "Speichert..." hängen
- Gefixt: try/catch/finally mit Fehlermeldung und setSaving(false)

Bug 4: ConsumeModal startete immer bei 1, auch wenn max < 1
- z.B. 0.5 kg Nüsse → Startwert war 1 (über dem Vorrat)
- Gefixt: Startwert = Math.min(step, max)

https://claude.ai/code/session_0169CbohMRqddD1LjPg8FmXf
@BEKO2210 BEKO2210 merged commit 35931b2 into main Mar 9, 2026
0 of 2 checks passed
@BEKO2210 BEKO2210 deleted the claude/fix-apple-camera-barcode-5DtQy branch March 9, 2026 17:58
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