Please, find above a patch in order to support WKT data as lat/long fields.

This patch requires the "gml.inc" parser used on wfs.module with some modifications to convert wkt to multigeometry kml.

CommentFileSizeAuthor
#10 wkt_support-934482-10.patch12.25 KBlangworthy
#9 wkt_support-934482-9.patch12.43 KBlangworthy
#8 wkt_support-934482-8.patch11.83 KBlangworthy
wkt_parking.patch13.54 KBpaolomainardi

Comments

paolomainardi’s picture

paolomainardi commented

Feel free to pull the code from here (forked by tmcw branch): http://github.com/twinbit/kml

tmcw’s picture

tmcw commented
Status: Active » Needs work

Could you make a patch that applies with CVS? I think this is good to commit - possibly it would be good if it was purely backwards-compatible - right now it requires one to resave a view, I think, before it works. But that isn't 100% necessary, I think.

paolomainardi’s picture

paolomainardi commented

Sorry for the delay, i will post the patch that apply to CSV as soon as possible.

jeffschuler’s picture

jeffschuler
Location Boulder, Colorado
commented
Title: WKT suport » WKT support
Category: bug » feature

Changing to feature request.
I'll take a look at your github fork.

paolomainardi’s picture

paolomainardi commented

Ok, let me know if you need some informations.

langworthy’s picture

langworthy commented

I need this for a project. I'll take a look at creating a new patch in the next week or two.

jeffschuler’s picture

jeffschuler
Location Boulder, Colorado
commented

@langworthy: awesome; thanks!
A patch would take some chore out of reviewing.

langworthy’s picture

langworthy commented
Version: master » 6.x-2.x-dev
Status: Needs work » Needs review
StatusFileSize
new wkt_support-934482-8.patch11.83 KB

Attached patch contains the first 3 commits from the fork in #1.

langworthy’s picture

langworthy commented
StatusFileSize
new wkt_support-934482-9.patch12.43 KB

Last patch didn't work. This one seems to.

langworthy’s picture

langworthy commented
StatusFileSize
new wkt_support-934482-10.patch12.25 KB

wrong paths in the last patch

langworthy’s picture

langworthy commented
Status: Needs review » Needs work

I think this patch should be re-rolled and committed after #1617016: Change placemark template per view makes it in, since that issue re-writes a lot of functionality.

also, patch in #10 contains error_log().