ferdi2go/Meeting-Mixer
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

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

New Issue
Closed
opened 2026-06-03 14:10:02 +02:00 by ferdi2go · 1 comment
ferdi2go commented 2026-06-03 14:10:02 +02:00
Owner
Copy Link
Copy Source

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/lowcode-quality labels 2026-06-03 14:10:02 +02:00
ferdi2go commented 2026-06-03 15:05:34 +02:00
Author
Owner
Copy Link
Copy Source

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.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
a11y
Barrierefreiheit
 bug
Logik-/Korrektheitsfehler
 code-quality
Code-Qualitaet / Refactoring
 docs
Dokumentation
 performance
Performance
 security
Sicherheit / Hardening
 severity/high
Hoher Schweregrad
 severity/low
Niedriger Schweregrad
 severity/medium
Mittlerer Schweregrad
 testing
Tests
No Label code-quality severity/low
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
ferdi2go
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: ferdi2go/Meeting-Mixer#14
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?