As originally fixed by @fmosca in #1053158: Add time field to "Additional" repeat dates. Separating this out as it's a bug, and that other thread is for a feature addition.

This patch fixes RDATE and EXDATE formatting to be the proper format for multiple declarations as per the iCal spec, and using timezones instead of UTC again as per the iCal spec.

CommentFileSizeAuthor
#1 date-rdate-exdate-multiples.patch2.57 KBRenee S
date-rdate-exdate-multiples.patch2.13 KBRenee S

Comments

Renee S’s picture

Renee S commented
Issue summary: View changes

clarification

Renee S’s picture

Renee S commented
StatusFileSize
new date-rdate-exdate-multiples.patch2.57 KB

Timezone part was missing.

Renee S’s picture

Renee S commented

This has been sitting for awhile, it's a pretty small fix. Can a mod take a look? Thanks!

Renee S’s picture

Renee S commented
Issue summary: View changes

moar

vijaycs85’s picture

vijaycs85
Location London, UK
commented

1: date-rdate-exdate-multiples.patch queued for re-testing.

vijaycs85’s picture

vijaycs85
Location London, UK
commented
Issue summary: View changes
Issue tags: +Needs manual testing, +sprint

Changes looks good, but need a manual testing to make sure it works for different timezone and no timezone.

andriyun queued 1: date-rdate-exdate-multiples.patch for re-testing.

Status: Needs review » Needs work

The last submitted patch, 1: date-rdate-exdate-multiples.patch, failed testing.