Bug 187291 - FTP tool for uploading a selection of images
Summary: FTP tool for uploading a selection of images
Status: RESOLVED FIXED
Alias: None
Product: digikam
Classification: Applications
Component: Plugin-WebService-FileTransfer (show other bugs)
Version: 0.8.0
Platform: openSUSE Linux
: NOR wishlist
Target Milestone: ---
Assignee: Digikam Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-03-16 10:34 UTC by Greg
Modified: 2018-03-23 11:22 UTC (History)
2 users (show)

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


Attachments
Export plugin based on kio capabilities (22.31 KB, patch)
2009-09-29 15:48 UTC, Johannes Wienke
Details
little code polishing, especially kdebug area codes (22.09 KB, patch)
2009-09-29 16:23 UTC, Andi Clemens
Details
renamed the Plugin class to match the rest of the KIPI plugins style (22.10 KB, patch)
2009-09-29 16:32 UTC, Andi Clemens
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Greg 2009-03-16 10:34:25 UTC
Version:           0.1.5-13.1 (using KDE 4.0.5)
Compiler:          gcc (SUSE Linux) 4.3.1 20080507 (prerelease) [gcc-4_3-branch revision 135036] none
OS:                Linux
Installed from:    SuSE RPMs

I am trying to help my mother set up Digikam to allow her to upload some of her album photos to the ftp area on www.photobox.co.uk
(ftp://ftp.photobox.com). This is her preferred site for printing photos.

I am slightly surprised to discover that KIPI and Digikam between them do not seem to be able to to do this. Using a separate FTP client is not an option for her (or other non-technical users).

The basic use cases are something like:

1/ <Main flow>: Specify the ftp URL, username and password, path and any other details required to identify a target site and save them with a name for subsequent recall and use (probably need to incorporate KWalletManager or suchlike to manage the passwords).

2/ <Main flow>: identify one or more images or albums (if it/they don't exist, create folders with the album names) from a users' collection and upload the images into the folders on the specified ftp site, from the list of saved sites created from use case 1.
2.1/ <Alternate flow>: specify a new folder name to be created to put the selection of images or albums in.
Comment 1 Greg 2009-03-16 10:39:00 UTC
It might be clearer if the following:
"(if it/they don't exist, create folders with the album names)"
actually said:
"(if it/they don't exist, create folders with the album names on the ftp site)"
Comment 2 Johannes Wienke 2009-09-29 15:48:46 UTC
Created attachment 37241 [details]
Export plugin based on kio capabilities

I created a simple plugin that can export to all file systems kio has access to. It doesn't fit directly to the described workflow but instead uses the same workflow as for most of the other export plugins.

The selection of the target is done using a KFileDialog. So the bookmarks of this dialog can be used for storing ftp locations.
Comment 3 Andi Clemens 2009-09-29 16:23:07 UTC
Created attachment 37242 [details]
little code polishing, especially kdebug area codes

Hi,

all in all I think the patch is ok.
I just polished it a little bit, it contained wrong area codes for kdebug, but in general it seems to be fine.

Gilles,
what do you think?

Andi
Comment 4 Andi Clemens 2009-09-29 16:32:08 UTC
Created attachment 37243 [details]
renamed the Plugin class to match the rest of the KIPI plugins style

just a minor change
Comment 5 caulier.gilles 2009-09-29 17:24:29 UTC
Andi,

Yes ! this is great. Let's go to add this new plugin to trunk.

We will test indeep and review/improve code later.

Johannes,

Thanks a lots for your code. It's very appreciate !

Gilles
Comment 6 Andi Clemens 2009-09-29 17:32:34 UTC
SVN commit 1029357 by aclemens:

Add new KIPIplugin "KioExport".
This plugin can be used to export images to FTP, Samba, or just local paths
using the KIO technologies.

Thanks to Johannes Wienke for the patch.

BUG:187291

 M  +1 -0      CMakeLists.txt  
 A             kioexport (directory)  
 A             kioexport/CMakeLists.txt  
 A             kioexport/KioExportWidget.cpp   [License: GPL (v2+)]
 A             kioexport/KioExportWidget.h   [License: GPL (v2+)]
 A             kioexport/KioExportWindow.cpp   [License: GPL (v2+)]
 A             kioexport/KioExportWindow.h   [License: GPL (v2+)]
 A             kioexport/Messages.sh  
 A             kioexport/kipiplugin_kioexport.desktop  
 A             kioexport/plugin_kioexport.cpp   [License: GPL (v2+)]
 A             kioexport/plugin_kioexport.h   [License: GPL (v2+)]


WebSVN link: http://websvn.kde.org/?view=rev&revision=1029357
Comment 7 caulier.gilles 2009-09-29 17:39:21 UTC
Johannes,

If you want to contribute more and more, please, ask for a KDE svn account !

It will be more easy for you to contribute, improve and test your code...

again, thanks for your help

http://techbase.kde.org/Contribute/Get_a_SVN_Account

Gilles
Comment 8 caulier.gilles 2009-09-29 20:47:26 UTC
Johannes,

With signal/sot, do not use "CLASS" directly, but "const CLASS&", as with KioExportWindow line 67...

Also, in KioExportWindow, add Help button to dialog, as it's done with other plugin. Like this users can report bugs at the right place...

Gilles
Comment 9 Johannes Wienke 2009-09-29 21:09:29 UTC
I will change this when I get the svn account.