Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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.
Comment | File | Size | Author |
---|---|---|---|
#1 | 0001-Issue-1741220-Rewrite-get_group_wkt.patch | 3.5 KB | Pol |
Comments
Comment #1
PolFor OL 7.x-2.x.
Comment #1.0
PolUpdated issue summary.
Comment #2
Brandonian CreditAttribution: Brandonian commentedWouldn'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
Comment #2.0
Brandonian CreditAttribution: Brandonian commentedUpdates
Comment #3
PolTrue, it should be a requirement, I committed the patch.
http://drupalcode.org/project/openlayers.git/commit/edc9290
Thanks !
Comment #4
PolPatch committed.
Comment #5.0
(not verified) CreditAttribution: commentedUpdated issue summary.