Bug 187291

Summary:	FTP tool for uploading a selection of images
Product:	[Applications] digikam	Reporter:	Greg <noh.way.jose>
Component:	Plugin-WebService-FileTransfer	Assignee:	Digikam Developers <digikam-bugs-null>
Status:	RESOLVED FIXED
Severity:	wishlist	CC:	caulier.gilles, languitar
Priority:	NOR
Version:	0.8.0
Target Milestone:	---
Platform:	openSUSE
OS:	Linux
Latest Commit:		Version Fixed In:	0.8.0
Sentry Crash Report:
Attachments:	Export plugin based on kio capabilities little code polishing, especially kdebug area codes renamed the Plugin class to match the rest of the KIPI plugins style

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.