Bug 220446 - Select images graphically by GeoPosition
Summary: Select images graphically by GeoPosition
Status: RESOLVED FIXED
Alias: None
Product: kphotoalbum
Classification: Applications
Component: Browser (show other bugs)
Version: unspecified
Platform: Compiled Sources Unspecified
: NOR wishlist
Target Milestone: ---
Assignee: KPhotoAlbum Bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-12-28 20:55 UTC by Matthias Füssel
Modified: 2016-01-17 09:50 UTC (History)
4 users (show)

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

Attachments
Patch implementing this (29.12 KB, patch)
2009-12-28 20:59 UTC, Matthias Füssel 		Details
Revised patch - now including CMakeLists.txt (31.22 KB, patch)
2010-01-03 17:23 UTC, Matthias Füssel 		Details
Show Obsolete (1) View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
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 <miika.turkia@gmail.com> 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.