Bug 220446

Summary:	Select images graphically by GeoPosition
Product:	[Applications] kphotoalbum	Reporter:	Matthias Füssel <mat.fuessel>
Component:	Browser	Assignee:	KPhotoAlbum Bugs <kpabugs>
Status:	RESOLVED FIXED
Severity:	wishlist	CC:	andreas.neustifter, blackie, miika.turkia, tl
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	Compiled Sources
OS:	Unspecified
Latest Commit:		Version Fixed In:
Sentry Crash Report:
Attachments:	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 <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.