|Summary:||Select images graphically by GeoPosition|
|Product:||[Applications] kphotoalbum||Reporter:||Matthias Füssel <mat.fuessel>|
|Component:||Browser||Assignee:||KPhotoAlbum Bugs <kpabugs>|
|Severity:||wishlist||CC:||andreas.neustifter, blackie, miika.turkia, tl|
|Latest Commit:||Version Fixed In:|
Patch implementing this
Revised patch - now including CMakeLists.txt
Description Matthias Füssel 2009-12-28 20:55:18 UTC
Version: (using Devel) Installed from: Compiled sources This enables the user to select images with a geoposition by position using a marble widget. It creates a new icon on the overviewpage for "select by geoposition". Choosing this will show a marble widget showing the locations of all currently selected images that have a geoposition set. You can then drag a rectangle (Ctrl + mouse drag) to select the images within a region. Caveats: * with marble up do 4.3.4 kphotoalbum crashes usually on the second or third activation of the marble widget. I think this is due to bug 182470 in marble * SQL backend geoposition searching not implemented * I have tried to make everything compile (and run) fine without marble, but have not tested this
Comment 1 Matthias Füssel 2009-12-28 20:59:42 UTC
Created attachment 39416 [details] Patch implementing this
Comment 2 Jesper Pedersen 2010-01-03 16:22:21 UTC
The patch must be missing some changes to CMakeFiles.txt, I guess it should at lease compile the new files you've added
Comment 3 Matthias Füssel 2010-01-03 17:23:38 UTC
Created attachment 39521 [details] Revised patch - now including CMakeLists.txt Sorry - had forgotten CMakeLists.txt
Comment 4 Miika Turkia 2012-02-05 14:11:36 UTC
Matthias, do you happen to have a newer version of this GeoPositioning patch. It seems that the current one does not compile with GIT version of KPA. I had to first resolve a few conflicts manually but then I got a compilation error as DB/Result.h does not exist anymore.
Comment 5 Tuomas Suutari 2012-02-05 14:30:23 UTC
On 5 February 2012 16:11, Miika Turkia <firstname.lastname@example.org> wrote: > Matthias, do you happen to have a newer version of this GeoPositioning patch. > It seems that the current one does not compile with GIT version of KPA. I had > to first resolve a few conflicts manually but then I got a compilation error as > DB/Result.h does not exist anymore. DB::Result has been renamed to DB::IdList so the header is now DB/IdList.h and Result should be replaced with IdList to make the patch current again.
Comment 6 Andreas Neustifter 2012-02-05 17:07:28 UTC
I'm sorry but I think its not wise to include new functionality in the release even if there are exisiting patches. Me personally I would include only patches that fix bugs...
Comment 7 Tobias Leupold 2016-01-17 09:50:46 UTC
This functionality has been added in the meantime and will be released with KPA 4.7.