Now that this geoPHP bug has been fixed, we can replace the function get_group_wkt() with:

  /**
   * Combine all group wkt into a single geometry collection
   *
   * openlayers_views_style_data specific
   */
  function get_group_wkt($wkt_array) {
    geophp_load();
    $geophp = geoPHP::load($wkt_array, 'wkt');
    return $geophp->out('wkt');
  }

I'm looking for people to test and give me feedback. The patch is following.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Pol’s picture

Status: Active » Needs review
FileSize
3.5 KB

For OL 7.x-2.x.

Pol’s picture

Issue summary: View changes

Updated issue summary.

Brandonian’s picture

Wouldn't this break if a site hasn't enabled geoPHP? Unless something's changed recently, geoPHP isn't a requirement for OL 7.x-2.x

Brandonian’s picture

Issue summary: View changes

Updates

Pol’s picture

True, it should be a requirement, I committed the patch.

http://drupalcode.org/project/openlayers.git/commit/edc9290

Thanks !

Pol’s picture

Status: Needs review » Fixed

Patch committed.

Status: Fixed » Closed (fixed)

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

Anonymous’s picture

Issue summary: View changes

Updated issue summary.