Our views formatter for geofield map breaks horribly if all the variable settings aren't set before trying to clone a view.

Fatal error: Call to a member function get_human_name() on a non-object in /Users/shoyt/Sites/mappingdev/sites/all/modules/views/plugins/export_ui/views_ui.class.php on line 384

Also, we have 2 notices before variables are set.

Notice: Undefined index: geofield_map_width in geofield_map_plugin_style_map->render() (line 140 of /Users/shoyt/Sites/mappingdev/sites/all/modules/geofield/modules/geofield_map/includes/geofield_map_plugin_style_map.inc).
Notice: Undefined index: geofield_map_height in geofield_map_plugin_style_map->render() (line 143 of /Users/shoyt/Sites/mappingdev/sites/all/modules/geofield/modules/geofield_map/includes/geofield_map_plugin_style_map.inc).

In general, we should be smarter about how we handle the formatter. We already check to see if geofields are present before allowing to apply settings, we should default to the first geofield in the view instead of none.

Comments

Brandonian’s picture

See http://drupal.org/node/1345434#comment-5275498 for more things to clean up for this patch.

Brandonian’s picture

Status: Active » Fixed

These issues are fixed.

Status: Fixed » Closed (fixed)

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