fix(加班): 统一日期选择与耗时口径
This commit is contained in:
@@ -15,6 +15,7 @@ import { MonthPicker } from '@/components/MonthPicker';
|
||||
import { FilterSelect } from '@/components/FilterSelect';
|
||||
import { FieldError } from '@/components/FieldError';
|
||||
import { RouteGuard } from '@/components/auth/Guard';
|
||||
import { WorkDateTimePicker } from '@/components/WorkDateTimePicker';
|
||||
|
||||
export default function OvertimePage() {
|
||||
return (
|
||||
@@ -287,11 +288,22 @@ function OvertimeModal({ defaultPerson, products, projects, versions, reasons, r
|
||||
<div className="grid grid-cols-2 gap-3">
|
||||
<div>
|
||||
<label className="text-[12px] font-medium text-[var(--ink-soft)] mb-1 block">开始时间 *</label>
|
||||
<input type="datetime-local" value={startTime} onChange={(e) => { setStartTime(e.target.value); setEndTimeError(''); }} required className="h-9 w-full rounded-lg border border-[var(--line)] bg-[var(--bg-card)] px-3 text-[13px] focus:border-[var(--accent)] focus:outline-none" />
|
||||
<WorkDateTimePicker
|
||||
value={startTime}
|
||||
onChange={(next) => { setStartTime(next); setEndTimeError(''); }}
|
||||
placeholder="选择加班开始时间"
|
||||
defaultHour={19}
|
||||
/>
|
||||
</div>
|
||||
<div>
|
||||
<label className="text-[12px] font-medium text-[var(--ink-soft)] mb-1 block">结束时间 *</label>
|
||||
<input type="datetime-local" value={endTime} onChange={(e) => { setEndTime(e.target.value); setEndTimeError(''); }} required className="h-9 w-full rounded-lg border border-[var(--line)] bg-[var(--bg-card)] px-3 text-[13px] focus:border-[var(--accent)] focus:outline-none" />
|
||||
<WorkDateTimePicker
|
||||
value={endTime}
|
||||
onChange={(next) => { setEndTime(next); setEndTimeError(''); }}
|
||||
placeholder="选择加班结束时间"
|
||||
defaultHour={21}
|
||||
popoverAlign="right"
|
||||
/>
|
||||
<FieldError message={endTimeError} />
|
||||
</div>
|
||||
</div>
|
||||
|
||||
@@ -39,9 +39,69 @@ export const OVERTIME_REASON_LABEL: Record<string, string> = {
|
||||
'reason-11': '返工修改',
|
||||
};
|
||||
|
||||
export function calcDuration(start: string, end: string): number {
|
||||
const s = new Date(start).getTime();
|
||||
const e = new Date(end).getTime();
|
||||
if (isNaN(s) || isNaN(e) || e <= s) return 0;
|
||||
return Math.round(((e - s) / (1000 * 60 * 60)) * 10) / 10;
|
||||
const MS_PER_HOUR = 60 * 60 * 1000;
|
||||
const STANDARD_START_HOUR = 9;
|
||||
const LUNCH_START_HOUR = 12;
|
||||
const LUNCH_END_HOUR = 13;
|
||||
const STANDARD_END_HOUR = 18;
|
||||
|
||||
function startOfDay(date: Date): Date {
|
||||
const day = new Date(date);
|
||||
day.setHours(0, 0, 0, 0);
|
||||
return day;
|
||||
}
|
||||
|
||||
function dayBound(day: Date, hour: number): Date {
|
||||
const result = new Date(day);
|
||||
result.setHours(hour, 0, 0, 0);
|
||||
return result;
|
||||
}
|
||||
|
||||
function isSameLocalDay(a: Date, b: Date): boolean {
|
||||
return a.getFullYear() === b.getFullYear()
|
||||
&& a.getMonth() === b.getMonth()
|
||||
&& a.getDate() === b.getDate();
|
||||
}
|
||||
|
||||
function overlapHours(start: Date, end: Date, segmentStart: Date, segmentEnd: Date): number {
|
||||
const overlapStart = Math.max(start.getTime(), segmentStart.getTime());
|
||||
const overlapEnd = Math.min(end.getTime(), segmentEnd.getTime());
|
||||
return Math.max(0, overlapEnd - overlapStart) / MS_PER_HOUR;
|
||||
}
|
||||
|
||||
function calcProjectDayHours(start: Date, end: Date): number {
|
||||
if (end.getTime() <= start.getTime()) return 0;
|
||||
const day = startOfDay(start);
|
||||
const gross = (end.getTime() - start.getTime()) / MS_PER_HOUR;
|
||||
const lunch = overlapHours(
|
||||
start,
|
||||
end,
|
||||
dayBound(day, LUNCH_START_HOUR),
|
||||
dayBound(day, LUNCH_END_HOUR),
|
||||
);
|
||||
return Math.max(0, gross - lunch);
|
||||
}
|
||||
|
||||
export function calcDuration(start: string, end: string): number {
|
||||
const startDate = new Date(start);
|
||||
const endDate = new Date(end);
|
||||
const s = startDate.getTime();
|
||||
const e = endDate.getTime();
|
||||
if (isNaN(s) || isNaN(e) || e <= s) return 0;
|
||||
|
||||
let total = 0;
|
||||
const cursor = startOfDay(startDate);
|
||||
const lastDay = startOfDay(endDate);
|
||||
|
||||
while (cursor.getTime() <= lastDay.getTime()) {
|
||||
const isFirst = isSameLocalDay(cursor, startDate);
|
||||
const isLast = isSameLocalDay(cursor, endDate);
|
||||
const dayStart = isFirst ? new Date(startDate) : dayBound(cursor, STANDARD_START_HOUR);
|
||||
const dayEnd = isLast ? new Date(endDate) : dayBound(cursor, STANDARD_END_HOUR);
|
||||
|
||||
total += calcProjectDayHours(dayStart, dayEnd);
|
||||
cursor.setDate(cursor.getDate() + 1);
|
||||
}
|
||||
|
||||
return Math.round(total * 10) / 10;
|
||||
}
|
||||
|
||||
@@ -17,8 +17,13 @@ test('calcActualElapsedHours uses work calendar time with a half-hour minimum',
|
||||
assert.equal(calcActualElapsedHours('2026-06-22T10:00:00', '2026-06-22T09:00:00'), 0);
|
||||
});
|
||||
|
||||
test('overtime duration still counts holiday time directly', () => {
|
||||
assert.equal(calcDuration('2026-10-01T09:00:00', '2026-10-01T18:00:00'), 9);
|
||||
test('overtime duration uses project day rhythm instead of continuous clock time', () => {
|
||||
assert.equal(calcDuration('2026-07-01T09:00:00', '2026-07-02T18:00:00'), 16);
|
||||
assert.equal(calcDuration('2026-07-01T09:00:00', '2026-07-02T20:00:00'), 18);
|
||||
});
|
||||
|
||||
test('overtime duration counts holidays as selectable work days without skipping them', () => {
|
||||
assert.equal(calcDuration('2026-10-01T09:00:00', '2026-10-01T18:00:00'), 8);
|
||||
});
|
||||
|
||||
test('formatActualDuration converts hours to natural days', () => {
|
||||
|
||||
Reference in New Issue
Block a user