1/2 aren't correct, however there are other bugs that have confused you. In particular, an "all day event" is the default, and for this no time will show - but the UI is confusing as the time is locked but not greyed out. There are also some bugs/inconsistencies converting timezones.
For 3 I will do a simple change, so it says it recurs regularly, but it won't try and go into detail (as that could take hours, there are so many recurring options and currently no code to convert them into an English explanation).
I know. But the time is/was included. I think you just didn't realise an event had no time, because it's not clear in the UI that an all-day event has no time. The other possibility for no time showing is if you do a long event (multi-day), in which case it trims it to just show dates.
Oh I misread, you said date was missing not time. There was a bug where an event with a start and no end just showed a random number, that could have been it. Regardless, we'll discuss in circles unless you are re-testing with the new code -- so please either grab changed files from the git revision above, or wait until the next patch release.
Anyway I applied git changes and I'm liking the output. It displays only what's needed but still displays what's needed... dates and times and whether or not it recurs.
Hey Chris. Calendar syndication I think is messed up again as of 9.0.18:
Lovinity has added an event, The Wonderbronies (happening on 12-2 AM (2 hours) and recurs regularly)
This is what the Activity feed picked up from Calendar events. This is supposed to be an event that begins March 29 8pm-10pm (EDT) and recurs weekly. But the syndication only pushed the time, not the date.
Automated response: Calendar date range display is sub-optimal
1) We need to show the date for a single-day event, not just times. 2) We don't want to strip :00 minutes from the time if we aren't able to do it for *both* start and end time (otherwise it gets confusing) 3) We should say what timezone we are displaying in
Looks good so far. Only problem is the time and date still displays in UTC on the syndication even if, for example in my case, I have it in EDT.
Lovinity has added an event, The Wonderbronies (happening on Monday 30th March 2015, 12 - 2 AM (UTC-04:00) Eastern Time (US & Canada), New York (2 hours) and recurs regularly)
Ok, that was not intentional. I have updated cms_calendar.php in the hotfix and my tests show it working correctly now. Sorry for the need for all this back and forth, the calendar code always confuses the hell out of me, so many conversions.
Alright. Thanks Chris. And don't worry... I can only imagine the pain the calendar module causes, especially when it comes to recurrence patterns and handling Daylight Savings Time.
For 3 I will do a simple change, so it says it recurs regularly, but it won't try and go into detail (as that could take hours, there are so many recurring options and currently no code to convert them into an English explanation).
Anyway I applied git changes and I'm liking the output. It displays only what's needed but still displays what's needed... dates and times and whether or not it recurs.
Lovinity has added an event, The Wonderbronies (happening on 12-2 AM (2 hours) and recurs regularly)
This is what the Activity feed picked up from Calendar events. This is supposed to be an event that begins March 29 8pm-10pm (EDT) and recurs weekly. But the syndication only pushed the time, not the date.
1) We need to show the date for a single-day event, not just times.
2) We don't want to strip :00 minutes from the time if we aren't able to do it for *both* start and end time (otherwise it gets confusing)
3) We should say what timezone we are displaying in
Lovinity has added an event, The Wonderbronies (happening on Monday 30th March 2015, 12 - 2 AM (UTC-04:00) Eastern Time (US & Canada), New York (2 hours) and recurs regularly)
(should be March 29 8pm - 10pm)