Bug 447815

Summary: Can't paste coordinates
Product: [Applications] digikam Reporter: yves.pratter
Component: Plugin-Generic-GeolocationEditAssignee: Digikam Developers <digikam-bugs-null>
Status: RESOLVED FIXED    
Severity: normal CC: caulier.gilles
Priority: NOR    
Version First Reported In: 7.4.0   
Target Milestone: ---   
Platform: macOS (DMG)   
OS: macOS   
Latest Commit: https://invent.kde.org/graphics/digikam/commit/fb2abe81e9d2efde81d0275f2ec3da16e1d9b7e9 Version Fixed/Implemented In: 7.5.0
Sentry Crash Report:

Description yves.pratter 2022-01-02 09:23:51 UTC 
SUMMARY
Issue with decimal separator. My local is french (coma separator) but the separator in the clipboard is dot (local is english)


STEPS TO REPRODUCE
1. copy 46.4146626, 5.7991907 (copy from JOSM OpenStreetMap editor)
2. Use "Edit Geolocation" tool (local is french)
3. check "Coordinates"
4. paste "Latitude"
5. paste "Longitude"

OBSERVED RESULT
Nothing. The fields are empty.
If I paste only the decimal digits it work (i.e. "4146626")

EXPECTED RESULT
Don't take care of the decimal separator (coma, dot...)

SOFTWARE/OS VERSIONS
macOS: Big Sur version 11.5.2
Qt Version: Qt 5.15.2 (built against 5.15.2)
KDE Version:  5.84.0

ADDITIONAL INFORMATION

Paste longitude and latitude isn't useful. Could be fine to paste the pair of coordinates once:
digikam should parse the coordinates with i.e. a regex that accept one or more spaces and an optional separator (coma, semicolon, slash…)
Comment 1 Maik Qualmann 2022-01-02 18:58:20 UTC 
Git commit fb2abe81e9d2efde81d0275f2ec3da16e1d9b7e9 by Maik Qualmann.
Committed on 02/01/2022 at 18:57.
Pushed by mqualmann into branch 'master'.

make paste coordinates more flexible
FIXED-IN: 7.5.0

M  +1    -1    NEWS
M  +41   -5    core/utilities/geolocation/geoiface/items/gpsitemlistcontextmenu.cpp

https://invent.kde.org/graphics/digikam/commit/fb2abe81e9d2efde81d0275f2ec3da16e1d9b7e9
Comment 2 Maik Qualmann 2022-01-02 19:03:40 UTC 
Git commit 9b9482ab6fe2ff7b90de97bd1390d988904e5a85 by Maik Qualmann.
Committed on 02/01/2022 at 19:02.
Pushed by mqualmann into branch 'master'.

skip empty coordinates parts

M  +2    -1    core/utilities/geolocation/geoiface/items/gpsitemlistcontextmenu.cpp

https://invent.kde.org/graphics/digikam/commit/9b9482ab6fe2ff7b90de97bd1390d988904e5a85