Support for the Location module

Have a look at how the taxonomy mapper works (/mappers directory in feedapi_mapper). You should be able to model your mapper after it.

Subscribing. Greetings, Martijn

Attaching a new .inc file that maps location information from FeedAPI (dependent upon #228111: Add enclosure (and location) support to FeedAPI Drupal 6) to location module.

I've fixed up a couple of comments, and now return FALSE if there are no fields, as per the instructions.

Dan's patch works for me.

This version modifies Dan's patch slightly to add a guard clause preventing a warning: Invalid argument supplied for foreach() PHP warning being issued in circumstances when $feed_element is NULL due to the feed item's latitude/longitude properties being present but empty.

Hi, not wanting to hijyack this thread, but looking for a Drupal 5 solution on this. I set my own thread about this on duplicate, because I think the location-way is the way to go for hierarchical regional info out of the feed.

I have feedapi working, location working, and the feedapi_taxonomy mapper working.
Next to this I installed the place module to get the location-taxonomy integration (www.drupal.org/project/place).

What I miss on my drupal 5 solution is the location_mapper to map the locations in my feeds to the locations, than hopefully I can place them hierarchical in taxonomy through the place_taxonomy module.

Because the place module is not on Drupal 6, and this the only module is I know of which makes the connection between location and taxonomy I need a drupal 5 solution and I think other peoples on drupal 5 would love a location mapper also!
I would very much like the location mapper into underneath feedapi_mapper list:

Description of available mappers
Only mappers for existing fields on feed items show up.

taxonomy (taxonomy module)
Maps a string or an array of strings to taxonomy terms. Chose a vocabulary from sub options.

Any possibility in getting a drupal5 feedapi_location mapper? Is there planning to backport http://drupal.org/node/228111 (#228111: Add enclosure (and location) support to FeedAPI Drupal 6) to Drupal 5?

EDIT: made a specific issue on location_mapper for drupal 5: http://drupal.org/node/333519

Thanks a lot in advance for considering this and I am willing to test this for you!
Greetings,
Martijn

Alex, the dependent patch is now included in FeedAPI, so this (Arto's modified version) should be good to go in.

Hi,

The patch worked for me also on Drupal 5. The missing part was that I also needed to enable location info on the feed-item nodes on which the feed is mapped to.

Please also add country, province, city, street and postal_code to the patch, will this work?

 else if ($op == 'list') {
      $sub_fields['country'] = 'country';
      $sub_fields['province'] = 'province';
      $sub_fields['city'] = 'city';
      $sub_fields['street'] = 'street';
      $sub_fields['postal_code'] = 'postal_code';
      $sub_fields['latitude'] = 'Latitude';
      $sub_fields['longitude'] = 'Longitude';
      return $sub_fields;

See: http://drupal.org/node/333519 for my first tests on this, I do not get the country field in the location table filled through mapping..
Greetings,
Martijn

Committed. Thanks geodaniel, Arto and others for patching and testing.

@Summit. I take it the D5 issue moved to #333519: Feedapi_mapper_location country, province, city, street and postal_code fields.

Changing status to fixed.

Hi Arto and Geodaniel,

Is in your install the mapped information also placed in the location table?
In my install this is not the case (on drupal 5).
Just to make sure it is a mistake on drupal 5, or on my site..

EDIT: I think I also have to alter this peace of code to get country etc..inserted:
http://drupal.org/node/333519#comment-1112424

Any feedback appreciated.

greetings,
Martijn

Rolled out in beta3. Thanks everybody.