Feeds integration

Subscribing, could this also be used for http://drupal.org/project/openlayers ?
greetings, Martijn

I'm still going to mark this as "needs work" - native feeds integration is still valuable to Geo, but it must be written using the API. Otherwise, resulting data will not be recognized by Views, fields or other relationships. Meanwhile, this parsing is still necessary for geo_data; so why maintain the same code in multiple places?

Thanks!

I've been meaning to roll this back in as well in some way to get this ball rolling. This makes lat/lon to geo available as a mapping option for feeds

// $Id:$

/**
 * @file
 * On behalf implementation of Feeds mapping API for geo.module (CCK).
 */

/**
 * Implementation of hook_feeds_node_processor_targets_alter()
 */
function geo_feeds_node_processor_targets_alter($targets, $content_type) {
  $info = content_types($content_type);
  $fields = array();
  if (isset($info['fields']) && count($info['fields'])) {
    foreach ($info['fields'] as $field_name => $field) {
      if (in_array($field['type'], array('geo'))) {
        $name = isset($field['widget']['label']) ? $field['widget']['label'] : $field_name;
        $targets[$field_name.'#lat'] = array(
          'name' => $name . ' (' . t('lat').')',
          'callback' => 'geo_feeds_set_target',
          'description' => t('The URL for the CCK !name field of the node.', array('!name' => $name)),
        );
        $targets[$field_name.'#lon'] = array(
          'name' => $name . ' (' . t('lon').')',
          'callback' => 'geo_feeds_set_target',
          'description' => t('The title for the CCK !name field of the node.', array('!name' => $name)),
        );
      }
    }
  }
}

/**
 * Implementation of hook_feeds_set_target().
 */
function geo_feeds_set_target($node, $target, $value) {
  list($field_name, $sub_field) = split('#', $target);

  $field_data = isset($node->$field_name) ? $node->$field_name : array();

  if(!is_array($value)) $value = array($value);
  $i = 0;
  foreach ($value as $v) {
    $field_data[$i][$sub_field] = $v;

    // once we have both lat and lon, add as point
    if (($lat = $field_data[$i]['lat']) && ($lon = $field_data[$i]['lon'])) {
      $field_data[$i]['geo'] = 'POINT('. $lon .' '. $lat .')';
    }
    $i++;
  }
  $node->$field_name = $field_data;
}

Putting this up here for review/discussion/use/combining in the interim.

Very very useful, but have not yet been able to get it working. I think we need to roll a complete patch against head so it is easier to review and test. I must be missing a few steps:

#9 contain the code that goes in geo/modules/geo_field/geo_field.feeds.inc ?
If this is correct, where is code where we include this function? geo/modules/geo_field/geo_field.module?

I just want to make sure I understand what this does. I have a CSV file with data that includes latitude and longitude columns. Will the patch and/or snippet in #9 create a feeds plug-in that will allow me to map those fields to a spatial field?

I included #9 as indicated (I think), but the spatial field is not showing up as a possible target when I set up the import mapping. Do I first need to apply the attached patch(es)?

I have added the spatial module linked in #5 to my site. Maybe I need to remove that to get #9 working?

Thanks.

Here's a modified version that works for me, but I'm only importing lat and lon. Main change is that geo_value() expects data as an array of lat and lon.

<?php
// $Id $
/**
 * @file
 * Allow Feeds data to set a Geo field.
 * From http://drupal.org/node/667080#comment-2574662.
 */

/**
 * Implements feeds_node_processor_targets_alter().
 *
 * Enable mapping to geo_field for feeds
 */
function geo_field_feeds_node_processor_targets_alter(&$targets, $content_type) {
  $info = content_types($content_type);
  $fields = array();
  if (isset($info['fields']) && count($info['fields'])) {
    foreach ($info['fields'] as $field_name => $field) {
      if ($field['type'] == 'geo') {
        $targets[$field_name .'#wkt'] = array(
          'name' => $name . ' (' . t('WKT') .')',
          'callback' => 'geo_field_feeds_set_target',
          'description' => t('The URL for the CCK !name field of the node.', array('!name' => $name)),
        );
        $targets[$field_name .'#lat'] = array(
          'name' => $name . ' (' . t('lat') .')',
          'callback' => 'geo_field_feeds_set_target',
          'description' => t('The URL for the CCK !name field of the node.', array('!name' => $name)),
        );
        $targets[$field_name .'#lon'] = array(
          'name' => $name .' (' . t('lon') .')',
          'callback' => 'geo_field_feeds_set_target',
          'description' => t('The title for the CCK !name field of the node.', array('!name' => $name)),
        );
      }
    }
  }
}

/**
 * Set Feeds Target, invoked once each for lat and lon fields.
 *
 * Callback for mapping. Here we map feeds data to the actual field of the node.
 */
function geo_field_feeds_set_target(&$node, $target, $value) {
  list($field_name, $sub_field) = split('#', $target);

  // Load the field from the node
  $field = isset($node->$field_name) ? $node->$field_name : array();

  if(!is_array($value)) {
    $value = array($value);
  }

  $i = 0;
  // Grab existing field data, as we may be gathering data from multiple invocations for lat lon.
  $field_data = $node->$field_name;
  foreach ($value as $v) {
    if ($sub_field == 'lat' || $sub_field == 'lon') {
      // We are dealing with lat / lon
      $field_data[$i][$sub_field] = $v;

      // Once we have both lat and lon, add as point
      if (($lat = $field_data[$i]['lat']) && ($lon = $field_data[$i]['lon'])) {
        //$field_data[$i]['geo'] = 'POINT('. $lon .' '. $lat .')';
        // geo_value() expects data as an array of lat and lon.
        $field_data[$i]['geo'] = array('lat' => $lat, 'lon' => $lon);
      }
    }
    elseif ($sub_field == 'wkt'){
      // We are dealing with wkt
      $field_data[$i]['geo'] = $v;
    }
    $i++;
  }

  if (!empty($field_data)) {
    $node->$field_name = $field_data;
  }
}

Ah, thanx Jonathan :)

--
Best regards from Norway, not a bad place to Kayak.