Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
The old code didn't wrap the iCal lines with the "\r\n" as per spec.
While this works on some parsers, others break badly with this approach
including many mobile tools.
Comment | File | Size | Author |
---|---|---|---|
#2 | date_ical-wrap-long-lines-1666326-2.patch | 1.97 KB | vlad.pavlovic |
0001-wrap-ical-text-fields-as-per-iCal-specification.patch | 2.58 KB | sdague |
Comments
Comment #1
coredumperror CreditAttribution: coredumperror commentedThank you for reporting this issue, sdague! Could you please upgrade Date iCal to the 2.x-2.x build and, if this problem persists (I'm fairly sure it does), re-roll your patch against that branch? I'd really appreciate it.
Comment #2
vlad.pavlovic CreditAttribution: vlad.pavlovic commentedRe-rolled the patch for dev version.
Comment #3
coredumperror CreditAttribution: coredumperror commentedUpon reviewing this patch, I discovered that a few things didn't carry over well from it's origin as a 7.x-1.x patch. For one,
drupal_html_to_text()
was being called twice, so I took out the call that was being done in the view plugin code.I also added some comments relating to full iCal spec compliance. I did some research on it, and both #61830: Text fields in iCal feeds not escaped properly and #84740: Event feed doesn't work with Google Calendar nor iCal showed me that full compliance is a hard problem, and I don't have time to really get it right. And since no one's really complaining, I'm going to leave this patch in a somewhat half-assed state of compliance for the time being.