fix: Budget-Monat war bei Starttag 29 bis 31 nicht schluessig
Der Starttag wird auf den letzten Tag des Monats gekappt – einen 31. Februar gibt es nicht. period_bounds tat das, period_of nicht: Es verglich stur den Tag des Monats. Damit fiel der 28. Februar bei Starttag 31 in einen Monat, der vom 31. Januar bis zum 27. Februar reicht – der Tag lag also in keiner Auswertung. Betroffen waren elf Tage in vier Jahren, alle am Ende kurzer Monate, und nur bei Starttag 29, 30 und 31. Bei allen anderen stimmte es. Zwei Tests halten es fest: Jeder Tag von 2024 bis 2027 muss bei JEDEM Starttag in dem Monat liegen, den period_of ihm zuweist – und die Monate muessen lueckenlos aneinanderstossen, keiner doppelt, keiner dazwischen. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
@@ -56,10 +56,16 @@ def period_bounds(label: str, start_day: int) -> tuple[date, date]:
|
||||
|
||||
|
||||
def period_of(d: date, start_day: int) -> str:
|
||||
"""Label des Budget-Monats, in den ein Datum fällt."""
|
||||
"""Label des Budget-Monats, in den ein Datum fällt.
|
||||
|
||||
Der Starttag wird auf den letzten Tag des Monats gekappt – einen 31.
|
||||
Februar gibt es nicht. Ohne dieselbe Kappung wie in ``period_bounds`` fiele
|
||||
der 28. Februar bei Starttag 31 in einen Monat, der ihn gar nicht enthält:
|
||||
Die Ausgabe waere in keiner Auswertung aufgetaucht.
|
||||
"""
|
||||
if start_day <= 1:
|
||||
return f"{d.year:04d}-{d.month:02d}"
|
||||
if d.day >= start_day:
|
||||
if d >= _clamp_day(d.year, d.month, start_day):
|
||||
nxt = date(d.year, d.month, 1) + relativedelta(months=1)
|
||||
return f"{nxt.year:04d}-{nxt.month:02d}"
|
||||
return f"{d.year:04d}-{d.month:02d}"
|
||||
|
||||
Reference in New Issue
Block a user