Reisezeit aus Start/Ende, Pause pro Eintrag in Minuten
- Reisezeit wird aus Reise-Beginn/-Ende zur Laufzeit berechnet (Spalte reisezeit entfernt), Anzeige in Vorschau und Liste wie Mehrarbeit - Pause pro Eintrag in Minuten (Feld zwischen Beginn und Ende), Default aus globaler Einstellung - Eingabefelder werden nach dem Speichern eines neuen Eintrags geleert - DB-Migrationen für reise_beginn/reise_ende/pause-Spalten - Version 0.1.2 Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This commit is contained in:
@@ -25,7 +25,9 @@ export function getDb(): Database.Database {
|
||||
ort TEXT NOT NULL,
|
||||
beginn TEXT NOT NULL,
|
||||
ende TEXT NOT NULL,
|
||||
reisezeit REAL NOT NULL DEFAULT 0,
|
||||
reise_beginn TEXT NOT NULL DEFAULT '',
|
||||
reise_ende TEXT NOT NULL DEFAULT '',
|
||||
pause REAL NOT NULL DEFAULT 0.5,
|
||||
vertriebsunterstuetzung REAL NOT NULL DEFAULT 2,
|
||||
kommentar TEXT NOT NULL DEFAULT '',
|
||||
created_at TEXT NOT NULL DEFAULT (datetime('now'))
|
||||
@@ -49,11 +51,37 @@ export function getDb(): Database.Database {
|
||||
);
|
||||
`);
|
||||
|
||||
// Migration: Reise-Start/-Ende für bestehende Datenbanken ergänzen.
|
||||
const cols = (db.prepare(`PRAGMA table_info(arbeitszeit)`).all() as { name: string }[]).map(
|
||||
(c) => c.name
|
||||
);
|
||||
if (!cols.includes('reise_beginn')) {
|
||||
db.exec(`ALTER TABLE arbeitszeit ADD COLUMN reise_beginn TEXT NOT NULL DEFAULT ''`);
|
||||
}
|
||||
if (!cols.includes('reise_ende')) {
|
||||
db.exec(`ALTER TABLE arbeitszeit ADD COLUMN reise_ende TEXT NOT NULL DEFAULT ''`);
|
||||
}
|
||||
// Reisezeit wird zur Laufzeit aus reise_beginn/reise_ende berechnet, nicht mehr gespeichert.
|
||||
if (cols.includes('reisezeit')) {
|
||||
db.exec(`ALTER TABLE arbeitszeit DROP COLUMN reisezeit`);
|
||||
}
|
||||
|
||||
// Standard-Pause beim ersten Start setzen (0,5 h)
|
||||
db.prepare(
|
||||
`INSERT INTO settings (key, value) VALUES ('pause', '0.5')
|
||||
ON CONFLICT(key) DO NOTHING`
|
||||
).run();
|
||||
|
||||
// Migration: Pause pro Eintrag ergänzen. Bestehende Zeilen bekommen die aktuelle
|
||||
// globale Pause-Einstellung als Wert.
|
||||
if (!cols.includes('pause')) {
|
||||
const row = db.prepare(`SELECT value FROM settings WHERE key = 'pause'`).get() as
|
||||
| { value: string }
|
||||
| undefined;
|
||||
const globalPause = row ? parseFloat(row.value) : 0.5;
|
||||
const def = isNaN(globalPause) ? 0.5 : globalPause;
|
||||
db.exec(`ALTER TABLE arbeitszeit ADD COLUMN pause REAL NOT NULL DEFAULT ${def}`);
|
||||
}
|
||||
|
||||
return db;
|
||||
}
|
||||
|
||||
+4
-3
@@ -43,8 +43,8 @@ export function createArbeitszeit(input: ArbeitszeitInput): ArbeitszeitEintrag {
|
||||
const db = getDb();
|
||||
const info = db
|
||||
.prepare(
|
||||
`INSERT INTO arbeitszeit (datum, ort, beginn, ende, reisezeit, vertriebsunterstuetzung, kommentar)
|
||||
VALUES (@datum, @ort, @beginn, @ende, @reisezeit, @vertriebsunterstuetzung, @kommentar)`
|
||||
`INSERT INTO arbeitszeit (datum, ort, beginn, ende, reise_beginn, reise_ende, pause, vertriebsunterstuetzung, kommentar)
|
||||
VALUES (@datum, @ort, @beginn, @ende, @reise_beginn, @reise_ende, @pause, @vertriebsunterstuetzung, @kommentar)`
|
||||
)
|
||||
.run(input);
|
||||
return getArbeitszeit(Number(info.lastInsertRowid))!;
|
||||
@@ -58,7 +58,8 @@ export function updateArbeitszeit(
|
||||
db.prepare(
|
||||
`UPDATE arbeitszeit SET
|
||||
datum = @datum, ort = @ort, beginn = @beginn, ende = @ende,
|
||||
reisezeit = @reisezeit, vertriebsunterstuetzung = @vertriebsunterstuetzung,
|
||||
reise_beginn = @reise_beginn, reise_ende = @reise_ende,
|
||||
pause = @pause, vertriebsunterstuetzung = @vertriebsunterstuetzung,
|
||||
kommentar = @kommentar
|
||||
WHERE id = @id`
|
||||
).run({ ...input, id });
|
||||
|
||||
+21
-4
@@ -1,5 +1,6 @@
|
||||
import { ORTE, type ArbeitszeitInput, type Ort } from '@/types/arbeitszeit';
|
||||
import { parseHM, applyVertriebsregel } from './calc';
|
||||
import { parseHM, applyVertriebsregel, toDecimalHours } from './calc';
|
||||
import { getPause } from './repo';
|
||||
|
||||
/**
|
||||
* Validiert und normalisiert einen JSON-Body zu einem ArbeitszeitInput.
|
||||
@@ -24,9 +25,25 @@ export function parseArbeitszeitInput(body: unknown): ArbeitszeitInput {
|
||||
throw new Error('Bitte Start- und Endzeit gemeinsam angeben oder beide leer lassen.');
|
||||
}
|
||||
|
||||
const reisezeit = toNumber(b.reisezeit, 0);
|
||||
// Reise-Start/-Ende sind optional (analog Beginn/Ende). Wenn angegeben,
|
||||
// müssen sie gültig sein; entweder beide leer oder beide gesetzt.
|
||||
const reise_beginn = String(b.reise_beginn ?? '');
|
||||
const reise_ende = String(b.reise_ende ?? '');
|
||||
if (reise_beginn !== '' && parseHM(reise_beginn) === null)
|
||||
throw new Error('Ungültige Reise-Startzeit.');
|
||||
if (reise_ende !== '' && parseHM(reise_ende) === null)
|
||||
throw new Error('Ungültige Reise-Endzeit.');
|
||||
if ((reise_beginn === '') !== (reise_ende === '')) {
|
||||
throw new Error('Bitte Reise-Start und -Ende gemeinsam angeben oder beide leer lassen.');
|
||||
}
|
||||
|
||||
// Reisezeit wird zur Laufzeit aus den Reise-Zeiten berechnet (nicht gespeichert),
|
||||
// dient hier aber zur Kopplung der Vertriebsunterstützung.
|
||||
const reisezeit = toDecimalHours(reise_beginn, reise_ende);
|
||||
// Pause pro Eintrag (dezimale Stunden). Standard ist die globale Einstellung.
|
||||
const pause = toNumber(b.pause, getPause());
|
||||
let vertriebsunterstuetzung = toNumber(b.vertriebsunterstuetzung, 2);
|
||||
if (reisezeit < 0 || vertriebsunterstuetzung < 0) {
|
||||
if (pause < 0 || vertriebsunterstuetzung < 0) {
|
||||
throw new Error('Zeitwerte dürfen nicht negativ sein.');
|
||||
}
|
||||
// Regel: Vertriebsunterstützung an die Reisezeit koppeln.
|
||||
@@ -35,7 +52,7 @@ export function parseArbeitszeitInput(body: unknown): ArbeitszeitInput {
|
||||
|
||||
const kommentar = String(b.kommentar ?? '').slice(0, 500);
|
||||
|
||||
return { datum, ort, beginn, ende, reisezeit, vertriebsunterstuetzung, kommentar };
|
||||
return { datum, ort, beginn, ende, reise_beginn, reise_ende, pause, vertriebsunterstuetzung, kommentar };
|
||||
}
|
||||
|
||||
function toNumber(value: unknown, fallback: number): number {
|
||||
|
||||
Reference in New Issue
Block a user