Refactor TimeEntryService and StatusBox component to integrate missing break minutes into time calculations. Update logic to ensure "Offen" time reflects accurate values, including breaks, and adjust frontend display to clarify time representation without redundant pause indications. Enhance output formatting for improved clarity in time displays.

This commit is contained in:
Torsten Schulz (local)
2025-10-20 11:24:45 +02:00
parent 1f20500b5f
commit 0bcdec68ef
2 changed files with 41 additions and 12 deletions

View File

@@ -324,6 +324,8 @@ class TimeEntryService {
} }
// Berechne "Offen" basierend auf timewish (oder Standard: 8 Stunden) // Berechne "Offen" basierend auf timewish (oder Standard: 8 Stunden)
// WICHTIG: Diese Zeit wird OHNE fehlende Pausen berechnet
// Das Frontend addiert die fehlenden Pausen dann zur "Normales Arbeitsende" Berechnung
let open = null; let open = null;
const { Timewish } = database.getModels(); const { Timewish } = database.getModels();
@@ -458,6 +460,23 @@ class TimeEntryService {
// Fehlende Pausenzeit // Fehlende Pausenzeit
const missingBreakMinutes = Math.max(0, requiredBreakMinutes - alreadyTakenBreakMinutes); const missingBreakMinutes = Math.max(0, requiredBreakMinutes - alreadyTakenBreakMinutes);
// Addiere fehlende Pausen zur "Offen" Zeit
if (open && open !== '—' && open !== 'Arbeitsende erreicht') {
const openParts = open.split(':');
const openH = parseInt(openParts[0]);
const openM = parseInt(openParts[1]);
const openS = parseInt(openParts[2] || 0);
const openMinutes = openH * 60 + openM + missingBreakMinutes;
const newOpenH = Math.floor(openMinutes / 60);
const newOpenM = openMinutes % 60;
const newOpenS = openS;
console.log(`🔍 DEBUG Offen-Berechnung: Original=${open}, Fehlende Pausen=${missingBreakMinutes}min, Neu=${newOpenH}:${newOpenM.toString().padStart(2, '0')}:${newOpenS.toString().padStart(2, '0')}`);
open = `${newOpenH.toString().padStart(2, '0')}:${newOpenM.toString().padStart(2, '0')}:${newOpenS.toString().padStart(2, '0')}`;
}
// Berechne Überstunden über den gesamten Zeitraum (alle Wochen) // Berechne Überstunden über den gesamten Zeitraum (alle Wochen)
// Neue Berechnung: Timewish-basiert // Neue Berechnung: Timewish-basiert
const totalOvertimeResult = await this._calculateTotalOvertime(uid, runningEntry); const totalOvertimeResult = await this._calculateTotalOvertime(uid, runningEntry);

View File

@@ -247,19 +247,26 @@ const updateOpenTime = () => {
openTime.value = `${hours.toString().padStart(2, '0')}:${minutes.toString().padStart(2, '0')}:${seconds.toString().padStart(2, '0')}` openTime.value = `${hours.toString().padStart(2, '0')}:${minutes.toString().padStart(2, '0')}:${seconds.toString().padStart(2, '0')}`
// Berechne "Normales Arbeitsende" = Jetzt + Offen + fehlende Pausen // Berechne "Normales Arbeitsende" = Jetzt + Offen (Pausen sind bereits in Offen enthalten)
const totalRemainingSeconds = remainingSeconds + (missingBreakMinutes.value * 60) const totalRemainingSeconds = remainingSeconds
const endTimestamp = now + (totalRemainingSeconds * 1000) const endTimestamp = now + (totalRemainingSeconds * 1000)
const endDate = new Date(endTimestamp) const endDate = new Date(endTimestamp)
const endHours = endDate.getHours() const endHours = endDate.getHours()
const endMinutes = endDate.getMinutes() const endMinutes = endDate.getMinutes()
const endSeconds = endDate.getSeconds() const endSeconds = endDate.getSeconds()
// Zeige auch fehlende Pausen an (falls vorhanden) // Ausgabe ohne zusätzlichen Pausen-Hinweis; Zeit stammt bereits inkl. Pausen aus Backend
if (missingBreakMinutes.value > 0) { regularEndTime.value = `${endHours.toString().padStart(2, '0')}:${endMinutes.toString().padStart(2, '0')}:${endSeconds.toString().padStart(2, '0')} Uhr`
regularEndTime.value = `${endHours.toString().padStart(2, '0')}:${endMinutes.toString().padStart(2, '0')}:${endSeconds.toString().padStart(2, '0')} (+${missingBreakMinutes.value}min Pause)`
} else { // Override: Berechne Normales Arbeitsende deterministisch aus Start + 8h + fehlende Pausen
regularEndTime.value = `${endHours.toString().padStart(2, '0')}:${endMinutes.toString().padStart(2, '0')}:${endSeconds.toString().padStart(2, '0')}` if (workStartTime.value) {
const baseDailyMinutes = 8 * 60 // Fallback auf 8h
const endTs = workStartTime.value + (baseDailyMinutes + (missingBreakMinutes.value || 0)) * 60 * 1000
const endDt = new Date(endTs)
const eh = endDt.getHours()
const em = endDt.getMinutes()
const es = endDt.getSeconds()
regularEndTime.value = `${eh.toString().padStart(2, '0')}:${em.toString().padStart(2, '0')}:${es.toString().padStart(2, '0')} Uhr`
} }
} else { } else {
openTime.value = 'Arbeitsende erreicht' openTime.value = 'Arbeitsende erreicht'
@@ -384,9 +391,9 @@ onBeforeUnmount(() => {
const displayRows = computed(() => { const displayRows = computed(() => {
const rows = { const rows = {
'Derzeit gearbeitet': currentlyWorkedTime.value, // Verwende berechneten Wert 'Derzeit gearbeitet': currentlyWorkedTime.value === '—' ? currentlyWorkedTime.value : currentlyWorkedTime.value + ' h',
'Offen': openTime.value, // Verwende berechneten Wert 'Offen': openTime.value === '—' ? openTime.value : openTime.value + ' h',
'Normales Arbeitsende': regularEndTime.value // Verwende berechneten Wert 'Normales Arbeitsende': regularEndTime.value // Verwende berechneten Wert (hat bereits "Uhr")
} }
// Füge andere Stats hinzu // Füge andere Stats hinzu
@@ -416,16 +423,19 @@ const displayRows = computed(() => {
if (key === 'overtime') { if (key === 'overtime') {
const isNegative = val.startsWith('-'); const isNegative = val.startsWith('-');
const displayLabel = isNegative ? 'Fehlzeit (Woche)' : 'Überstunden (Woche)'; const displayLabel = isNegative ? 'Fehlzeit (Woche)' : 'Überstunden (Woche)';
const displayValue = isNegative ? val.substring(1) : val; // Entferne Minus-Zeichen const displayValue = isNegative ? val.substring(1) + ' h' : val + ' h'; // Entferne Minus-Zeichen, füge " h" hinzu
rows[displayLabel] = displayValue; rows[displayLabel] = displayValue;
} else if (key === 'totalOvertime') { } else if (key === 'totalOvertime') {
const isNegative = val.startsWith('-'); const isNegative = val.startsWith('-');
const displayLabel = isNegative ? 'Fehlzeit (Gesamt)' : 'Überstunden (Gesamt)'; const displayLabel = isNegative ? 'Fehlzeit (Gesamt)' : 'Überstunden (Gesamt)';
const displayValue = isNegative ? val.substring(1) : val; // Entferne Minus-Zeichen const displayValue = isNegative ? val.substring(1) + ' h' : val + ' h'; // Entferne Minus-Zeichen, füge " h" hinzu
rows[displayLabel] = displayValue; rows[displayLabel] = displayValue;
} else if (key === 'adjustedEndTodayGeneral' || key === 'adjustedEndTodayWeek') { } else if (key === 'adjustedEndTodayGeneral' || key === 'adjustedEndTodayWeek') {
// Füge " Uhr" zu Uhrzeiten hinzu // Füge " Uhr" zu Uhrzeiten hinzu
rows[label] = val + ' Uhr'; rows[label] = val + ' Uhr';
} else if (key === 'weekWorktime' || key === 'openForWeek' || key === 'nonWorkingHours') {
// Füge " h" zu Zeiten hinzu (außer bei "—")
rows[label] = val === '—' ? val : val + ' h';
} else { } else {
rows[label] = val; rows[label] = val;
} }