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

<?php
 
/**
   * 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.

Files: 
CommentFileSizeAuthor
#1 0001-Issue-1741220-Rewrite-get_group_wkt.patch3.5 KBPol
PASSED: [[SimpleTest]]: [MySQL] 238 pass(es).
[ View ]

Comments

Status:Active» Needs review
StatusFileSize
new3.5 KB
PASSED: [[SimpleTest]]: [MySQL] 238 pass(es).
[ View ]

For OL 7.x-2.x.

Issue summary:View changes

Updated issue summary.

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

Issue summary:View changes

Updates

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

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

Thanks !

Status:Needs review» Fixed

Patch committed.

Status:Fixed» Closed (fixed)

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

Issue summary:View changes

Updated issue summary.