Bug 175327 - Flickr import tool wishlist [patch]
Summary: Flickr import tool wishlist [patch]
Status: REOPENED
Alias: None
Product: digikam
Classification: Applications
Component: Plugin-WebService-Flickr (show other bugs)
Version: 6.0.0
Platform: unspecified Linux
: NOR wishlist
Target Milestone: ---
Assignee: Digikam Developers
URL:
Keywords:
: 137892 (view as bug list)
Depends on:
Blocks:
 
Reported: 2008-11-16 20:52 UTC by Daniel Welch
Modified: 2021-08-23 22:46 UTC (History)
10 users (show)

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


Attachments
listPhotos method (6.58 KB, patch)
2010-02-08 12:00 UTC, Anselmo L. S. Melo (anselmolsm)
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Daniel Welch 2008-11-16 20:52:55 UTC
Version:           0.10.0-beta3 (using 4.1.2 (KDE 4.1.2), Mandriva Linux release 2009.0 (Official) for i586)
Compiler:          gcc
OS:                Linux (i686) release 2.6.27.4-desktop586-2mnb

would be good to have a flickr import wizard, similar to the camera import.

could allow user to choose set or photostream, recently loaded pics from contacts, sizes to download, etc.

I use flickr to share images with family, and this would help download high-res for printing and medium res for desktop.
Comment 1 Vardhman 2009-01-04 18:45:55 UTC
I am think of providing this feature in basic form very soon.
Comment 2 Bráulio Barros de Oliveira 2009-03-20 03:50:25 UTC
i would like this feature too.
can help in testing and would like to code it. please give some directions.
Comment 3 Anselmo L. S. Melo (anselmolsm) 2010-02-08 12:00:39 UTC
Created attachment 40601 [details]
listPhotos method

(This patch was submitted to the mailing list, I'm posting it here following recommendations)

I'm also interested in an import feature for flickr plugin. The attached patch is an initial implementation of the listPhotos method that retrieves from the Flickr API the list of pictures in a given set. In this list, we receive some parameters needed in order to find the URL to each picture. It's just a first step, there are more work needed. I have a svn account, but as it's my first contribution to kipi-plugins, I believe the right thing to do is to ask a first review of other kipi-plugins developers.
Comment 4 Vardhman 2010-02-08 19:51:45 UTC
Hi Anselmo,

Thanks for sending the patch, unfortunately I am not contributing actively
to the project and many other authors would be much more prompt and helpful
with the patching in.

Please add/cc Caulier Gilles (Digikam project lead) to this bug and he will
make sure the right people get added.

Thanks,
Vardhman
On Mon, Feb 8, 2010 at 3:00 AM, Anselmo Lacerda Silveira de Melo <
anselmo.melo@openbossa.org> wrote:

> https://bugs.kde.org/show_bug.cgi?id=175327
>
>
> Anselmo Lacerda Silveira de Melo (anselmolsm) <anselmo.melo@openbossa.org>
> changed:
>
>           What    |Removed                     |Added
>
> ----------------------------------------------------------------------------
>             Status|UNCONFIRMED                 |NEW
>         AssignedTo|vardhman@gmail.com          |anselmo.melo@openbossa.org
>     Ever Confirmed|0                           |1
>
>
>
>
> --- Comment #3 from Anselmo Lacerda Silveira de Melo (anselmolsm) <anselmo
> melo openbossa org>  2010-02-08 12:00:39 ---
> Created an attachment (id=40601)
>  --> (http://bugs.kde.org/attachment.cgi?id=40601)
> listPhotos method
>
> (This patch was submitted to the mailing list, I'm posting it here
> following
> recommendations)
>
> I'm also interested in an import feature for flickr plugin. The attached
> patch
> is an initial implementation of the listPhotos method that retrieves from
> the
> Flickr API the list of pictures in a given set. In this list, we receive
> some
> parameters needed in order to find the URL to each picture. It's just a
> first
> step, there are more work needed. I have a svn account, but as it's my
> first
> contribution to kipi-plugins, I believe the right thing to do is to ask a
> first
> review of other kipi-plugins developers.
>
> --
> Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
> ------- You are receiving this mail because: -------
> You are on the CC list for the bug.
> You are the assignee for the bug.
>
Comment 5 caulier.gilles 2010-02-08 19:57:21 UTC
Vardhman,

I'm already there. No need to CC me (:=)))

Gilles
Comment 6 Bráulio Barros de Oliveira 2010-02-09 02:50:41 UTC
There is some functional code at https://code.launchpad.net/~brauliobo/kflickr/kflickr2 for viewing albums and downloading their photos, integrated poorly into the user interface.
Comment 7 Anselmo L. S. Melo (anselmolsm) 2010-02-09 04:23:34 UTC
*** Bug 137892 has been marked as a duplicate of this bug. ***
Comment 8 Anselmo L. S. Melo (anselmolsm) 2010-02-09 04:25:59 UTC
(In reply to comment #6)
> There is some functional code at
> https://code.launchpad.net/~brauliobo/kflickr/kflickr2 for viewing albums and
> downloading their photos, integrated poorly into the user interface.

Cool! thanks =)
Comment 9 Vikas Kumar 2010-07-18 01:55:07 UTC
When are we going to see this feature?

Thanks,
Vikas.
Comment 10 caulier.gilles 2010-07-18 15:08:50 UTC
Vardhman,

We need to take a look about his patch at next coding sprint in France

Gilles Caulier
Comment 11 Ramon Antonio Parada (brainsqueezer) 2011-07-28 12:08:56 UTC
Really interested in this feature. Seems forgoten. Need help with development
or testing?
Comment 12 Anselmo L. S. Melo (anselmolsm) 2011-07-28 13:20:19 UTC
(In reply to comment #11)
> Really interested in this feature. Seems forgoten. Need help with development
> or testing?

Great!
I wrote more code than the attached patch, but it's not ready (I was able to download pictures, though). I can send this WIP code for you. I can try to help in the development, the main problem is that I'm currently in a time consuming project in my job =/
Comment 13 caulier.gilles 2011-08-01 13:46:38 UTC
Please attach a patch against git master to this bugzilla entry. 

Thanks in advance

Gilles Caulier
Comment 14 caulier.gilles 2011-12-22 13:14:47 UTC
Anselmo,

Do you see my comment #13 ?

Gilles Caulier
Comment 15 caulier.gilles 2013-10-30 10:10:33 UTC
What the status of this entry ?

There is no more patch to review here ?

Gilles Caulier
Comment 16 caulier.gilles 2015-06-28 10:03:35 UTC
New Kipiplugins 4.11.0 is available :

https://www.digikam.org/node/740

Can you reproduce the problem with this release ?

Gilles Caulier
Comment 17 caulier.gilles 2016-07-26 09:37:47 UTC
Maik,

Sound like this feature is already implemented. Right ?

Gilles
Comment 18 caulier.gilles 2017-06-23 22:26:07 UTC
digiKam + kipiplugins 5.6.0 is just released.

https://www.digikam.org/news/2017-06-21-5.6.0-release-announcement/

This file still valid with this version ?

Gilles Caulier
Comment 19 caulier.gilles 2017-12-17 14:56:21 UTC
What's about this file using last 5.8.0 pre-release bundle available here :

https://files.kde.org/digikam/

Gilles Caulier
Comment 20 caulier.gilles 2018-07-05 13:21:00 UTC
Thanh,

What's about this file where we have an older patch here ? It still suitable for 6.0.0 ?

Gilles Caulier
Comment 21 caulier.gilles 2018-08-11 12:14:25 UTC
Thanh,

For this entry, and following your recent work for GSoC 2018, we have 2 choices :

1/ Review the patch and re-use some part to integrate few ideas in current code.
2/ Ignore this patch contents, if the ideas are far than new improvements done with your project.

We need to set the future way for this entry. I vote for 2/ as all the export tools dialog step by step more and less factored and unified (thanks again to Thanh for this hard work).

Maik, your viewpoint ?

Gilles
Comment 22 Thanh Trung Dinh 2018-08-14 09:47:06 UTC
In my point of view, option 1 is better. It's true that our plan is uniting all web services to the new common wizard, and Flickr import will not be an exception. However, the core codes for import is method listPhotos(), which was written and tested (I think) by Anselmo. Since that method has API call inside (which is specific for each service as you already know), it will remain unchanged when we integrate Flickr import to the new wizard in the future. Moreover, the old patch posted here is for listPhotos(), so reviewing it in order to reuse and adapt it to new wizard will save time.

Best,

Thanh
Comment 23 caulier.gilles 2018-08-14 09:59:01 UTC
Thanks for this clarification Thanh

Gilles
Comment 24 caulier.gilles 2019-07-27 13:16:47 UTC
Please test with 6.2.0 pre-release bundles available here :

https://files.kde.org/digikam/

Thanks in advance

Gilles Caulier