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.Some events will have multiple dates for an event or repeating dates. Currently the addtocal module only outputs the first date as an event in the ics file. Providing patch in next post.
| Comment | File | Size | Author |
|---|---|---|---|
| #13 | addtocal-support_multiple_events-2023441-13.patch | 1.33 KB | antiorario |
Comments
Comment #1
BTMash CreditAttribution: BTMash commentedAdded patch.
Comment #2
BTMash CreditAttribution: BTMash commentedSorry, revised version of patch (the paths were incorrect).
Comment #3
pixelsweatshop CreditAttribution: pixelsweatshop commentedClosed others as duplicate of this one:
#1784942: How should this work with repeating events
#1784986: Repeating Date with first date in past
Comment #4
rterman CreditAttribution: rterman commentedCan anyone confirm that this works? I applied the patch and nothing changed.
Comment #5
maen CreditAttribution: maen commentedand I couldn't patch this version
Comment #6
bdimaggioI reworked @BTMash's patch against the latest dev version of the module, and I can confirm that it's working well for iCal--don't have a copy of Outlook handy to verify. But I figure this might help some folks out.
I'd also like to deal with this situation (recurring instances of an event, some of which might be in the past) for the other calendar types. It looks like neither GCal nor Yahoo Calendar will let us add more than one instance, so the sensible thing would seem to be to find the most imminent instance and add that. Would it be helpful if I tried to make that happen in addtocal_google_link() and addtocal_yahoo_link()?
Comment #7
bdimaggioComment #8
MhueD CreditAttribution: MhueD commentedWould an alternative approach to determining how to fix which time is getting properly exported be to have an option where EACH time has it's own button/function for creating the output file? I can see this being set at the widget in Manage Display for all cases or also giving the User the choice to click a button next to each time/item OR a button at the top for 'All Times'.
Comment #9
bdimaggioI like that idea--it makes more sense to me as a user--but I'm afraid I can't take on the work that it would entail. Since I'm updating to the latest Add To Cal right now, I thought I'd just upload the patch for that version of the module. Hope this helps somebody!
Comment #10
bdimaggioComment #11
smurfxx CreditAttribution: smurfxx commentedIs this patch included in the latest non-dev release?
I have a node with 3 dates and only 1 is showed and added to calendars.
Comment #12
bdimaggioNo, it shouldn't be, smurfxx--you'd have to dl and apply it yourself.
Comment #13
antiorario CreditAttribution: antiorario commentedModifying Ben's patch at #9 to make sure it can be applied through Composer Manager and cweagans/composer-patches.