The 2026 Drupal Business Survey is open. Your response will help shape the future of the Drupal business ecosystem.For the development of the excellent http://drupal.org/project/feeds module an ical parser plugin.
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | 614120-6_icalparser.patch | 18.16 KB | David Goode |
| #5 | tests.zip | 6.07 KB | David Goode |
| #3 | 61412-4_icalfeeds.patch | 5.45 KB | David Goode |
Comments
Comment #1
ekes commentedDevelopment started in HEAD http://drupal.org/cvs?commit=279794 the equivalent of a date mapper will be needed and this may change how things go into the arrays.
Comment #2
whatdoesitwant commented@ekes Good to see that you started this up. Are you referring to the new Feedapi replacement module Feeds? Would your linked files allow me to re-enable Ical parser with Feeds?
Comment #3
David Goode commentedHey! I've been working on the date mapper in question for feeds here: #623448: Port FeedAPI Mappers: date. Here is the ical parser patch to go with it once that's settled.
Comment #4
alex_b commentedLooking good.
- FeedsIcalDateTimeField class needs commenting
- Needs tests (I know they're in the works). After #623448: Port FeedAPI Mappers: date will shed its TZ tests, iCal parser should contain some tests for TZ handling.
Comment #5
David Goode commentedHere is an ical parser test to go along with #3.
Comment #6
David Goode commentedHere is an improved patch, includes tests so ignore the zip file and patch of my previous posts.
Comment #7
ekes commentedGreat stuff. Think moving the TZ stuff makes sense too. Cheers! Committed to head.
Still getting a fair few warnings to investigate. I'll also look into cleaning up the Date side of it, and making it more extensible.
Comment #8
alex_b commentedekes: tks for committing.
I've opened an issue for follow up cleanups and bugs: #685942: Feeds integration cleanup. Setting this to fixed.