fyb/calcurse-edge
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix handling of recurrent open-ended appointments at 00:00

Browse Source 
Prior to this commit, a recurrent appointment ending at 00:00 on a given
day was not considered to span the day. This is the correct behavior for
all cases, except appointments without end time (both starting and
ending at midnight on a given day). Handle this edge case explicitly.

Signed-off-by: Lukas Fleischer <lfleischer@calcurse.org>
This commit is contained in:
Lukas Fleischer 2023-01-01 11:39:38 -05:00
parent ce6f8bf85b
commit ac5113640d
1 changed files with 13 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
src/recur.c
View File

@ -1003,14 +1003,20 @@ static int find_occurrence(time_t start, long dur, struct rpt *rpt, llist_t *exc
if (rpt->until && t >= NEXTDAY(rpt->until))
return 0;
/* Does it span the given day? */
if (t + DUR(t) < day)
/* Does it span the given day?
*
* NOTE: An appointment ending at 00:00 is not considered to span the
* given day, unless the appointment is an appointment without
* specified end time, which is internally treated as appointment with
* duration 0.
*/
if (t + DUR(t) >= day || (t == day && dur == 0)) {
if (occurrence)
*occurrence = t;
return 1;
} else {
return 0;
if (occurrence)
*occurrence = t;
return 1;
}
}
#undef DUR