[Code-Quality] Robustheit: parseInt-Radix, minGapDays-Guard, catch-Logs #14

New Issue

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

ferdi2go commented

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

Schweregrad: LOW

Dateien: js/calendar.js:98,104 (parseInt ohne Radix), js/algorithm.js:49 (minGapDays > 0 ohne Null-Guard), js/app.js:28-30, js/employees.js:55-57 (stille catch-Blöcke)

Fix: parseInt(x, 10) konsistent verwenden; if (c.minGapDays != null && c.minGapDays > 0); console.warn/error in catch-Blöcken ergänzen.

**Schweregrad:** LOW **Dateien:** `js/calendar.js:98,104` (parseInt ohne Radix), `js/algorithm.js:49` (`minGapDays > 0` ohne Null-Guard), `js/app.js:28-30`, `js/employees.js:55-57` (stille catch-Blöcke) **Fix:** `parseInt(x, 10)` konsistent verwenden; `if (c.minGapDays != null && c.minGapDays > 0)`; `console.warn/error` in catch-Blöcken ergänzen.

ferdi2go added the severity/low code-quality labels 2026-06-03 14:10:02 +02:00

ferdi2go commented

2026-06-03 15:05:34 +02:00

Behoben in 7ab2911: parseInt(month/year, 10) mit Radix; minGap-Check mit Null-Guard (c.minGapDays != null && c.minGapDays > 0); stille catch-Blöcke (korrupter localStorage-Load, createEmployee) loggen jetzt console.warn statt Fehler zu verschlucken.

Behoben in `7ab2911`: `parseInt(month/year, 10)` mit Radix; minGap-Check mit Null-Guard (`c.minGapDays != null && c.minGapDays > 0`); stille catch-Blöcke (korrupter localStorage-Load, `createEmployee`) loggen jetzt `console.warn` statt Fehler zu verschlucken.

ferdi2go closed this issue

2026-06-03 15:05:34 +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#14