Bug 264423 - Export to KML drops significant figures in Lat/Long [patch]
Summary: Export to KML drops significant figures in Lat/Long [patch]
Status: RESOLVED FIXED
Alias: None
Product: digikam
Classification: Applications
Component: Geolocation-KMLExport (show other bugs)
Version: 2.3.0
Platform: Debian stable Linux
: NOR wishlist
Target Milestone: ---
Assignee: Digikam Developers
URL:
Keywords: junior-jobs
Depends on:
Blocks:
 
Reported: 2011-01-27 09:42 UTC by Catherine Chisholm
Modified: 2018-01-31 19:39 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In: 2.3.0
Sentry Crash Report:


Attachments
patch to modify the string formating of the doubles (879 bytes, patch)
2011-10-21 20:01 UTC, Vincent Batts
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Catherine Chisholm 2011-01-27 09:42:25 UTC
Version:           1.2.0 (using KDE 1.2) 
OS:                Linux

Even if original geolocation coordinates include six decimal places (when in decimal degrees), the Kipi plugin for KML export rounds to only 4 decimal places in latitude and 3 in longitude.  

Reproducible: Always



Expected Results:  
For precision (as is needed in many cases) KML export should incorporate all the significant figures in the original geocoding.

OS: Linux (x86_64) release 2.6.32-27-generic
Compiler: cc
Comment 1 Vincent Batts 2011-10-21 20:01:17 UTC
Created attachment 64768 [details]
patch to modify the string formating of the doubles

The issue is in the default toString formatting of doubles. So when passing the doubles as an arg to QString, they just need a tad of formatting.
Comment 2 Vincent Batts 2011-10-21 20:02:00 UTC
http://doc.qt.nokia.com/4.7/qstring.html#arg-20 has more documentation on it.
Comment 3 Michael G. Hansen 2011-10-25 20:14:41 UTC
Git commit 188dc7a799ddd84b4d495a6bbca7067b32684d91 by Michael Georg Hansen.
Committed on 25/10/2011 at 22:14.
Pushed by mghansen into branch 'master'.

Fix bug 264423: Export to KML drops significant figures in Lat/Long

Patch by Vincent Batts. Thanks!

BUG: 264423

M  +2    -1    NEWS
M  +7    -2    kmlexport/kmlexport.cpp

http://commits.kde.org/kipi-plugins/188dc7a799ddd84b4d495a6bbca7067b32684d91