geo_fields in node array not correctly created [#444242]

In geo_field.module on line 169, the code to populate $node->geo_fields is incorrect. $node->geo_fields in being overwritten with a fresh array each time this is run. This results in only the last geo_field to be processed being lsited in the array. Working correct code is:


      // Identify ourselves on the node level for RSS and other postprocessing.
      if (!$node->geo_fields){
        $node->geo_fields = array();
      }
      $node->geo_fields[] = $field['field_name'];
      return array($field['field_name'] => $ret, 'geo_fields' => $node->geo_fields);

Comment	File	Size	Author
#3	444242_field_array.patch	1.03 KB	greggles

Comments

Comment #1

phayes commented 24 April 2009 at 18:42

Title:

geo_feilds in node array not correctly created

» geo_fields in node array not correctly created

Comment #2

phayes commented 29 April 2009 at 05:42

Here's some cleaned up code:

      // Identify ourselves on the node level for RSS and other postprocessing.
      if (!$node->geo_fields){
        $node->geo_fields = array();
      }
      $node->geo_fields[] = $field['field_name'];
      return array($field['field_name'] => $ret);

Thanks for the great module!