Bug 319611

Summary:	Invalid placement of TimeStamp element in kml export [patch]
Product:	[Applications] digikam	Reporter:	Glenn Washburn <development>
Component:	Geolocation-KMLExport	Assignee:	Digikam Developers <digikam-bugs-null>
Status:	RESOLVED FIXED
Severity:	normal	CC:	caulier.gilles, dave.greengas
Priority:	NOR
Version:	3.3.0
Target Milestone:	---
Platform:	Ubuntu
OS:	Linux
Latest Commit:	http://commits.kde.org/kipi-plugins/5c8934ffc91eae7be7cfa23d6a45c43adad60a4f	Version Fixed In:	4.0.0
Sentry Crash Report:
Attachments:	Patch to always put date time as child of PlaceMark

Description Glenn Washburn 2013-05-10 09:33:34 UTC

In the generated KML file, the TimeStamp element is a child of the Point element, however it should be a child of the Placemark element.  Google earth will warn about this if warnings are turned on.

Reproducible: Always

Steps to Reproduce:
1. Select some geo-referenced photos.
2. Select menu item Export > Export to KML
3. Click OK (use defaults)
4. Open KML file in Google earth
Actual Results:  
Improper KML generated

Expected Results:  
Valid KML?

Point element reference:
https://developers.google.com/kml/documentation/kmlreference#point

Placemark reference:
https://developers.google.com/kml/documentation/kmlreference#placemark

These are for the 2.2 schema, but it should parse all valid 2.1 kmls.

Comment 1 David Greengas 2013-08-12 12:23:25 UTC

Created attachment 81668 [details]
Patch to always put date time as child of PlaceMark

Comment 2 David Greengas 2013-08-12 12:24:56 UTC

I noticed that when the EXIF date time is used, it is as a child of the PlaceMark element, but when the image date time is used, it is a child of the Point element. The attached diff makes it consistent.

Comment 3 caulier.gilles 2013-10-30 07:57:01 UTC

Git commit 5c8934ffc91eae7be7cfa23d6a45c43adad60a4f by Gilles Caulier.
Committed on 30/10/2013 at 07:55.
Pushed by cgilles into branch 'master'.

KMLExport : apply patch #81668 to always put date time as child of PlaceMark
FIXED-IN: 4.0.0

M  +1    -1    kmlexport/kmlexport.cpp

http://commits.kde.org/kipi-plugins/5c8934ffc91eae7be7cfa23d6a45c43adad60a4f