Closed (fixed)
Project:
Openlayers
Version:
7.x-2.x-dev
Component:
OL Views
Priority:
Normal
Category:
Task
Assigned:
Reporter:
Created:
20 Aug 2012 at 15:39 UTC
Updated:
9 Jan 2013 at 16:30 UTC
Jump to comment: Most recent file
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 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 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) commentedUpdated issue summary.