I'm guessing that because this is just the alpha that these features are not available yet?

Any idea when the basic fields will make it in?

Thanks.

CommentFileSizeAuthor
#8 ical_parser-780940.patch1.28 KBRenee S

Comments

ekes’s picture

ekes commented

To should happen automatically if you have a from/to field and map to it. Do you want it separately?

Location - depends on what you want. The text, just as text, should be available as soon as I get to next commits. The geo but reservations about mapping issues.

pkcho’s picture

pkcho commented

Those fields do not map for me.

I'm using the Open Publish distribution. Is it possible that something in that distribution is limiting the parser?

In the past, certain things didn't work until the developers upgraded OP to the next version 1.6 to 1.7, for example using gmap and the pixel one player.

This one is driving me crazy. It should be there but I only see a limited number of mappable fields, not enough to make it usable.

Thanks.

ekes’s picture

ekes commented

The to field. You should see two drop downs. Source - this has iCal date (this is a full date object that contains the begin, end and repeat); and Target - here you should see date: Start, date: End. Use date: Start if you want Start (or start and end). Use Date: End if you just want the end (or to) date.

Do you see something different?

ekes’s picture

ekes commented

CVS version include location mapping. Note there is an error upstream with parsing feeds (like upcoming.com) with ALTREP param with an uri in them (well a : in any quoted param to be specific) so one test breaks for now. Working on a patch for date for that, but it should work for all others already.

ekes’s picture

ekes commented
Title: 6.x-2.0-alpha1 does not map "to" and "location" fields » 6.x-2.0-alpha1 does not map "location" fields
Status: Active » Reviewed & tested by the community

Assuming "to" is working as described.

Location is in CVS and is blocked on passing all tests by #282521: iCal LOCATION: param and value parsed incorrectly I'll roll an alpha2 soon with the slightly broken location, when I do close this issue and open a specific one for that.

Sagumatra’s picture

Sagumatra commented

Is there an estimate for when the location stuff will get rolled out? I'd love to use the feeds module for this project but the location field is a necessity for me.

Thanks for all your work.

Renee S’s picture

Renee S commented

Subscribing. If you're not able to get to the latest commits for alpha2 soon, is it possible to test this via a patch, or even an updated -dev?

Renee S’s picture

Renee S commented
StatusFileSize
new ical_parser-780940.patch1.28 KB

I rolled a patch. Quick and dirty. It supports basic location only. Should do for now for anybody who needs it , at least.

NOTE the patch to Date in #282521: iCal LOCATION: param and value parsed incorrectly breaks things. It's something about that patch, not my implementation, but didn't delve into it in any detail. :)

ekes’s picture

ekes commented

The code for a Location parser and tests have been in the module in CVS for months... Help checking out of CVS can be found here http://drupal.org/node/321 and in the tab at the top of all modules pages.

alpha2 on its way in minutes as well.

ekes’s picture

ekes commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.