Icons, Footer, Stundenformel

checkin-server.js

@@ -14,6 +14,7 @@ checkinApp.set('views', path.join(__dirname, 'views'));
 
 // Middleware für Check-in-Server
 checkinApp.use(express.json());
+checkinApp.use(express.static('public'));
 
 /** Erkennt Browser-Aufruf: Accept-Header enthält text/html (z. B. beim Aufruf per QR/Link im Browser). */
 function wantsHtml(req) {

doc/Stunderfassung todo.txt

@@ -3,7 +3,7 @@
 - Offset für die Verwaltung für Urlaubstage                                             -> DONE
 - Stunden pro Tag und wie viele Tage arbeit                                             -> DONE
 - Reisen für Wochenende                                                                 -> DONE                 
-- LDAP Prüfung                                                                          -> DONE GEHT?!
+- LDAP Prüfung                                                                          -> DONE
 - DSGVO Sicherheit                                                                      -> DONE
 - Feiertage müssen als ausgefüllt zählen                                                -> DONE
 - Mitarbeiter sollen PDF ansehen können.                                                -> DONE
@@ -12,6 +12,6 @@
 
 - Feiertage im PDF anzeigen                                                                                     -> DONE
 - Oben wenn woche eingereicht anzeigen als hilfestellung                                                        -> DONE              
-- Ausgefüllte Tage anhand der Tage pro woche gültig setzten                                                     -> DONE Testen
+- Ausgefüllte Tage anhand der Tage pro woche gültig setzten                                                     -> DONE 
 - Überstunden müssen anhand der Tagesstunden auch auf gültig setzten (Tag ausgefüllt wenn weniger als 8h)       -> DONE sollte passen
-- Verplante Urlaubstage müssen auf abgezogen werden, wenn die Woche die gepalnt war eingereicht wurde.          -> DONE Testen
+- Verplante Urlaubstage müssen auf abgezogen werden, wenn die Woche die gepalnt war eingereicht wurde.          -> DONE

public/css/style.css

@@ -1073,3 +1073,12 @@ table input[type="text"] {
     align-items: flex-start;
   }
 }
+
+/* App Footer */
+.app-footer {
+  text-align: center;
+  padding: 16px;
+  color: #888;
+  font-size: 13px;
+  margin-top: 24px;
+}

public/js/dashboard.js

@@ -440,12 +440,17 @@ function renderWeek() {
     
     // Stunden zur Summe hinzufügen
     // Bei ganztägigem Urlaub oder Krank sollten es bereits 8 Stunden sein (vom Backend gesetzt)
-    // Feiertag: 8h Basis + gearbeitete Stunden (jede gearbeitete Stunde = Überstunde)
+    // Feiertag Werktag: 8h Basis + gearbeitete Stunden (jede gearbeitete Stunde = Überstunde)
+    // Feiertag am Wochenende: keine Tagesarbeitsstunden, nur gearbeitete Stunden (z. B. Reise)
     // Bei halbem Tag Urlaub werden die Urlaubsstunden später in der Überstunden-Berechnung hinzugezählt
     // Wochenend-Prozentsätze: Nur auf tatsächlich gearbeitete Stunden anwenden (nicht auf Urlaub, Krankheit, Feiertage)
     let hoursToAdd = 0;
     if (isHoliday) {
+      if (isWeekend) {
+        hoursToAdd = hours || 0; // Feiertag am Wochenende: keine Tagesarbeitsstunden
+      } else {
         hoursToAdd = fullDayHours + (hours || 0); // (Wochenarbeitszeit / Arbeitstage) Feiertag + gearbeitete Stunden (= Überstunden)
+      }
     } else {
       hoursToAdd = hours || 0;
       // Wochenend-Prozentsatz anwenden (nur wenn weekend_travel aktiviert UND es ist ein Wochenendtag)
@@ -488,6 +493,9 @@ function renderWeek() {
       }
     } else if (isSick) {
       hoursDisplay = fullDayHours.toFixed(2) + ' h (Krank)';
+    } else if (isHoliday && isWeekend) {
+      // Feiertag am Wochenende: keine Tagesarbeitsstunden
+      hoursDisplay = (hours ? hours.toFixed(2) : '0') + ' h (Feiertag)';
     } else if (isHoliday && !hours) {
       hoursDisplay = fullDayHours.toFixed(2) + ' h (Feiertag)';
     } else if (isHoliday && hours) {
@@ -818,7 +826,9 @@ function updateOvertimeDisplay() {
     } else if (sickStatus) {
       totalHours += fullDayHours; // Krank = (Wochenarbeitszeit / Arbeitstage) Stunden
     } else if (currentHolidayDates.has(dateStr)) {
-      // Feiertag: (Wochenarbeitszeit / Arbeitstage) Basis + gearbeitete Stunden (jede Stunde = Überstunde)
+      // Feiertag: Werktag = Basis + gearbeitete Stunden; Wochenende = nur gearbeitete Stunden (keine Tagesarbeitsstunden)
+      const dayOfWeek = date.getDay();
+      const isWeekendHoliday = (dayOfWeek === 6 || dayOfWeek === 0);
       const startInput = document.querySelector(`input[data-date="${dateStr}"][data-field="start_time"]`);
       const endInput = document.querySelector(`input[data-date="${dateStr}"][data-field="end_time"]`);
       const startTime = startInput ? startInput.value : '';
@@ -833,7 +843,11 @@ function updateOvertimeDisplay() {
       } else if (currentEntries[dateStr]?.total_hours) {
         worked = parseFloat(currentEntries[dateStr].total_hours) || 0;
       }
+      if (isWeekendHoliday) {
+        totalHours += worked; // Feiertag am Wochenende: keine Tagesarbeitsstunden
+      } else {
         totalHours += fullDayHours + worked; // (Wochenarbeitszeit / Arbeitstage) Feiertag + gearbeitete Stunden (= Überstunden)
+      }
     } else {
       // Wenn 8 Überstunden (ganzer Tag) eingetragen sind, zählt der Tag als 0 Stunden
       if (isFullDayOvertime) {
@@ -922,7 +936,8 @@ function updateOvertimeDisplay() {
   const totalHoursWithVacation = totalHours + vacationHours;
   const adjustedSollStunden = sollStunden - (fullDayOvertimeDays * fullDayHours);
   // overtimeHours = Überstunden diese Woche (wie im Backend berechnet)
-  const overtimeHours = totalHoursWithVacation - adjustedSollStunden;
+  // Genommene Überstunden werden abgezogen, um die Netto-Überstunden zu erhalten
+  const overtimeHours = totalHoursWithVacation - adjustedSollStunden - overtimeTaken;
   
   // Überstunden-Anzeige aktualisieren
   const overtimeSummaryItem = document.getElementById('overtimeSummaryItem');
@@ -1206,8 +1221,12 @@ async function saveEntry(input) {
         currentEntries[date].total_hours = totalHours;
       } else {
         // Zurück zu normaler Anzeige basierend auf anderen Status
+        const d = new Date(date);
+        const isWeekendHoliday = isHoliday && (d.getDay() === 6 || d.getDay() === 0);
         if (isSick) {
           hoursElement.textContent = fullDayHours.toFixed(2) + ' h (Krank)';
+        } else if (isWeekendHoliday) {
+          hoursElement.textContent = (hours ? hours.toFixed(2) : '0') + ' h (Feiertag)';
         } else if (isHoliday && !hours) {
           hoursElement.textContent = fullDayHours.toFixed(2) + ' h (Feiertag)';
         } else if (isHoliday && hours) {
@@ -1369,12 +1388,18 @@ async function saveEntry(input) {
           currentEntries[date].total_hours = totalHours;
         } else if (isSick) {
           hoursText = fullDayHours.toFixed(2) + ' h (Krank)';
-        } else if (isHoliday && result.total_hours <= fullDayHours) {
+        } else if (isHoliday) {
+          const d = new Date(date);
+          const isWeekendHoliday = (d.getDay() === 6 || d.getDay() === 0);
+          if (isWeekendHoliday) {
+            hoursText = (result.total_hours || 0).toFixed(2) + ' h (Feiertag)';
+          } else if (result.total_hours <= fullDayHours) {
             hoursText = fullDayHours.toFixed(2) + ' h (Feiertag)';
-        } else if (isHoliday && result.total_hours > fullDayHours) {
+          } else {
             const overtime = result.total_hours - fullDayHours;
             hoursText = fullDayHours.toFixed(2) + ' + ' + overtime.toFixed(2) + ' h (Überst.)';
           }
+        }
         
         hoursElement.textContent = hoursText;
         // total_hours wurde bereits für halben Tag Urlaub gesetzt, sonst verwende result.total_hours
@@ -1973,8 +1998,12 @@ function toggleSickStatus(dateStr) {
       currentEntries[dateStr].total_hours = fullDayHours;
     } else {
       // Zurück zu normaler Anzeige basierend auf anderen Status
+      const d = new Date(dateStr);
+      const isWeekendHoliday = isHoliday && (d.getDay() === 6 || d.getDay() === 0);
       if (isFullDayVacation) {
         hoursElement.textContent = fullDayHours.toFixed(2) + ' h (Urlaub)';
+      } else if (isWeekendHoliday) {
+        hoursElement.textContent = (hours ? hours.toFixed(2) : '0') + ' h (Feiertag)';
       } else if (isHoliday && !hours) {
         hoursElement.textContent = fullDayHours.toFixed(2) + ' h (Feiertag)';
       } else if (isHoliday && hours) {

public/manifest.json

@@ -7,12 +7,12 @@
     "theme_color": "#0a5ea8",
     "icons": [
       {
-        "src": "/public/images/icons/icon-192x192.png",
+        "src": "/images/icons/icon-192x192.png",
         "sizes": "192x192",
         "type": "image/png"
       },
       {
-        "src": "/public/images/icons/icon-512x512.png",
+        "src": "/images/icons/icon-512x512.png",
         "sizes": "512x512",
         "type": "image/png"
       }

routes/verwaltung-routes.js

@@ -300,17 +300,33 @@ function registerVerwaltungRoutes(app) {
           }
           
           function processCurrentWeek(totalVacationDays) {
-            // Einträge für die Woche abrufen
+            // Einträge für die Woche abrufen (id/updated_at für neuesten pro Tag)
-            db.all(`SELECT date, total_hours, overtime_taken_hours, vacation_type, sick_status 
+            db.all(`SELECT id, date, updated_at, total_hours, overtime_taken_hours, vacation_type, sick_status 
                     FROM timesheet_entries 
                     WHERE user_id = ? AND date >= ? AND date <= ?
-                    ORDER BY date`,
+                    ORDER BY date, updated_at DESC, id DESC`,
               [userId, week_start, week_end],
-              (err, entries) => {
+              (err, allEntries) => {
                 if (err) {
                   return res.status(500).json({ error: 'Fehler beim Abrufen der Einträge' });
                 }
                 
+                // Nur neuesten Eintrag pro Tag zählen (wie PDF/Submit), sonst Doppelzählung bei Duplikaten
+                const entriesByDate = {};
+                (allEntries || []).forEach(entry => {
+                  const existing = entriesByDate[entry.date];
+                  if (!existing) {
+                    entriesByDate[entry.date] = entry;
+                  } else {
+                    const existingTime = existing.updated_at ? new Date(existing.updated_at).getTime() : 0;
+                    const currentTime = entry.updated_at ? new Date(entry.updated_at).getTime() : 0;
+                    if (currentTime > existingTime || (currentTime === existingTime && entry.id > existing.id)) {
+                      entriesByDate[entry.date] = entry;
+                    }
+                  }
+                });
+                const entries = Object.values(entriesByDate);
+                
                 // Berechnungen
                 let totalHours = 0;
                 let overtimeTaken = 0;
@@ -320,7 +336,7 @@ function registerVerwaltungRoutes(app) {
                 
                 entries.forEach(entry => {
                   if (entry.overtime_taken_hours) {
-                    overtimeTaken += entry.overtime_taken_hours;
+                    overtimeTaken += parseFloat(entry.overtime_taken_hours) || 0;
                   }
                   
                   // Krankheitstage zählen

services/pdf-service.js

@@ -187,7 +187,11 @@ function generatePDF(timesheetId, req, res) {
           if (row.type === 'holiday') {
             y = doc.y;
             x = 50;
-            const rowData = [formatDate(row.date), '-', '-', '-', fullDayHours.toFixed(2) + ' h (Feiertag)'];
+            // Feiertag am Wochenende: keine Tagesarbeitsstunden anzeigen
+            const holidayDay = new Date(row.date + 'T12:00:00').getDay();
+            const isWeekendHoliday = (holidayDay === 0 || holidayDay === 6);
+            const holidayHoursStr = isWeekendHoliday ? '0 h (Feiertag)' : fullDayHours.toFixed(2) + ' h (Feiertag)';
+            const rowData = [formatDate(row.date), '-', '-', '-', holidayHoursStr];
             rowData.forEach((data, i) => {
               doc.text(data, x, y, { width: colWidths[i], align: 'left' });
               x += colWidths[i];
@@ -455,7 +459,11 @@ function generatePDFToBuffer(timesheetId, req) {
             if (row.type === 'holiday') {
               y = doc.y;
               x = 50;
-              const rowDataBuf = [formatDate(row.date), '-', '-', '-', fullDayHoursBuf.toFixed(2) + ' h (Feiertag)'];
+              // Feiertag am Wochenende: keine Tagesarbeitsstunden anzeigen
+              const holidayDay = new Date(row.date + 'T12:00:00').getDay();
+              const isWeekendHoliday = (holidayDay === 0 || holidayDay === 6);
+              const holidayHoursStr = isWeekendHoliday ? '0 h (Feiertag)' : fullDayHoursBuf.toFixed(2) + ' h (Feiertag)';
+              const rowDataBuf = [formatDate(row.date), '-', '-', '-', holidayHoursStr];
               rowDataBuf.forEach((data, i) => {
                 doc.text(data, x, y, { width: colWidths[i], align: 'left' });
                 x += colWidths[i];

views/admin.ejs

@@ -484,5 +484,6 @@
       }
     });
   </script>
+  <%- include('footer') %>
 </body>
 </html>

views/checkin-result.ejs

@@ -53,5 +53,6 @@
     <h1><%= title %></h1>
     <p class="message"><%= message %></p>
   </div>
+  <%- include('footer') %>
 </body>
 </html>

views/dashboard.ejs

@@ -507,5 +507,6 @@
       }
     }
   </script>
+  <%- include('footer') %>
 </body>
 </html>

views/footer.ejs

@@ -0,0 +1,3 @@
+  <footer class="app-footer">
+    Made with &#10084;&#65039; by Carsten Graf
+  </footer>

views/header.ejs

@@ -2,14 +2,14 @@
 <link rel="manifest" href="/manifest.json">
 
 <!-- Apple Icons -->
-<link rel="apple-touch-icon" sizes="120x120" href="/icons/icon-120x120.png">
+<link rel="apple-touch-icon" sizes="120x120" href="/images/icons//icon-120x120.png">
-<link rel="apple-touch-icon" sizes="152x152" href="/icons/icon-152x152.png">
+<link rel="apple-touch-icon" sizes="152x152" href="/images/icons/icon-152x152.png">
-<link rel="apple-touch-icon" sizes="167x167" href="/icons/icon-167x167.png">
+<link rel="apple-touch-icon" sizes="167x167" href="/images/icons/icon-167x167.png">
-<link rel="apple-touch-icon" sizes="180x180" href="/icons/icon-180x180.png">
+<link rel="apple-touch-icon" sizes="180x180" href="/images/icons/icon-180x180.png">
 
 <!-- Android / PWA -->
-<link rel="icon" type="image/png" sizes="192x192" href="/icons/icon-192x192.png">
+<link rel="icon" type="image/png" sizes="192x192" href="/images/icons/favicon.png">
-<link rel="icon" type="image/png" sizes="512x512" href="/icons/icon-512x512.png">
+<link rel="icon" type="image/png" sizes="512x512" href="/images/icons/favicon.png">
 
 <!-- iOS App Mode -->
 <meta name="apple-mobile-web-app-capable" content="yes">

views/login.ejs

@@ -93,5 +93,6 @@
       </script>
     </div>
   </div>
+  <%- include('footer') %>
 </body>
 </html>

views/overtime-breakdown.ejs

@@ -309,5 +309,6 @@
       loadOvertimeBreakdown();
     });
   </script>
+  <%- include('footer') %>
 </body>
 </html>

views/verwaltung.ejs

@@ -867,5 +867,6 @@
       }
     });
   </script>
+  <%- include('footer') %>
 </body>
 </html>