From 7efdcbe17be389f4355a7142bfb8505abb626a1f Mon Sep 17 00:00:00 2001
From: Kenearos <kenearos@googlemail.com>
Date: Tue, 7 Jul 2026 11:38:12 +0200
Subject: [PATCH] refactor(storage): drop unreachable try/catch in
 getDutiesForMonth

new Date() and the object spread never throw; the isNaN guard already
drops invalid dates via the .filter(). The inner catch could never fire.
Outer boundary try/catch is kept.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
---
 storage.js | 16 ++++------------
 1 file changed, 4 insertions(+), 12 deletions(-)

diff --git a/storage.js b/storage.js
index 8fa8cb4..5400c9b 100644
--- a/storage.js
+++ b/storage.js
@@ -141,20 +141,12 @@ class DataStorage {
 
             // Convert date strings back to Date objects
             return allDuties[employeeName][monthKey].map(duty => {
-                try {
-                    const dateObj = new Date(duty.date);
-                    if (isNaN(dateObj.getTime())) {
-                        console.error(`Fehler: Ungültiges Datum für Dienst: ${duty.date}`);
-                        return null;
-                    }
-                    return {
-                        ...duty,
-                        date: dateObj
-                    };
-                } catch (e) {
-                    console.error('Fehler beim Konvertieren des Datums:', e);
+                const dateObj = new Date(duty.date);
+                if (isNaN(dateObj.getTime())) {
+                    console.error(`Fehler: Ungültiges Datum für Dienst: ${duty.date}`);
                     return null;
                 }
+                return { ...duty, date: dateObj };
             }).filter(duty => duty !== null); // Filter out invalid entries
         } catch (e) {
             console.error('Fehler beim Laden der Dienste für Monat:', e);