Opened 2 years ago

Closed 2 years ago

#6343 closed defect (fixed)

KML Superoverlay driver cannot handle Folders

Reported by: pphunor Owned by: warmerdam
Priority: normal Milestone: 2.1.0
Component: default Version: unspecified
Severity: normal Keywords:
Cc: Jeff McKenna

Description

The driver does not recognize the Region and GroundOverlay? entries placed under a Folder node. See https://developers.google.com/kml/documentation/kmlreference#folder.

I debugged and modified the driver so that it can handle the attached KML. I've chosen the solution with the least amount of changes, and it is not proved against any standards. Use it up if you think so.

Attachments (1)

kmlsuperoverlaydataset.cpp (94.6 KB) - added by pphunor 2 years ago.

Download all attachments as: .zip

Change History (5)

Changed 2 years ago by pphunor

Attachment: kmlsuperoverlaydataset.cpp added

comment:1 Changed 2 years ago by Even Rouault

Thanks for the patch. I gave it a quick try, but it breaks opening the https://svn.osgeo.org/gdal/trunk/autotest/gdrivers/data/kmlimage.kmz which is used in the regression test. Could you correct it so that it still works with it ?

comment:2 Changed 2 years ago by Even Rouault

Ah and please use 4 spaces instead of tabulation character as indentation.

comment:3 Changed 2 years ago by Jeff McKenna

Cc: Jeff McKenna added

comment:4 Changed 2 years ago by Even Rouault

Milestone: 2.1.0
Resolution: fixed
Status: newclosed

I came up with a different solution :

trunk r33283 "KMLSuperOverlay: recognize datasets that have a intermediate <Folder> that forms a <Document><Folder><Region/><GroundOverlay/?> structure (#6343)"

Note: See TracTickets for help on using tickets.