pimutils
All day event process correctly with --notstarted, fix #1004
When you do khal list --notstarted, the "today" all-day event is displayed. This MR avoid this problem.
Fixing the bug (that khal list now --notstarted still shows all-days) without breaking this looks to be a pain.
Hi @d7415, so if I understand, I need to create a exception to display multi-day events? For example event begins yesterday and ends in 3 day, khal list now --notstarted should display my event for tomorrow and next days but not for today?
Hi @d7415, so if I understand, I need to create a exception to display multi-day events? For example event begins yesterday and ends in 3 day,
khal list now --notstartedshould display my event for tomorrow and next days but not for today?
No. In that example the event has started so shouldn't be shown (and the current code won't show it)
My problems with this PR are:
- There should be a way to hide events like your example, while showing all-day events today (e.g. hide that I'm on holiday all week, show that it's my friend's birthday today). The current code does this, but the PR does not.
- This PR results in an inconsistency where running
khal list --nostartedat midday will hide an all-day event for today, but show an event that happened this morning.
There is still a bug from #1004 where if a time after midnight is specified (e.g. with now) all day events show in --notstarted. But that looks like we'd have to special-case a time (other than midnight) being specified, which is probably why it wasn't done in the first place. (Well, that and showing today's all-day events in --notstarted isn't likely to cause a major inconvenience)