Bug 88804

Summary: Digikam: Add images: Allows only to import local files. No sftp: ftp: :(
Product: [Applications] digikam Reporter: Achim Bohnet <ach>
Component: Import-MainViewAssignee: Digikam Developers <digikam-bugs-null>
Status: RESOLVED FIXED    
Severity: wishlist    
Priority: NOR    
Version First Reported In: 0.7.0   
Target Milestone: ---   
Platform: unspecified   
OS: Linux   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:

Description Achim Bohnet 2004-09-04 00:52:18 UTC 
Version:           0.7.0-cvs (using KDE 3.2.3,  (3.1))
Compiler:          gcc version 3.3.4 (Debian 1:3.3.4-7)
OS:                Linux (i686) release 2.6.8-ach-1-686

Album -> Import -> Add images  allows only to import local files.
Please use of kio framework ;)

(Especially anoying currently is that only in only informed about
only file:/ supported after pressing okay.
And not as soon as a directory URL != ^file:/.* is entered.)
Comment 1 Renchi Raju 2004-09-09 00:47:25 UTC 
CVS commit by pahlibar: 


allow addition of new images from external sources supported by kio
CCMAIL: 88804-done@bugs.kde.org


  M +7 -16     digikamview.cpp   1.15


--- kdeextragear-3/digikam/digikam/digikamview.cpp  #1.14:1.15
@@ -293,20 +293,11 @@ void DigikamView::slot_albumAddImages()
     PAlbum* palbum = dynamic_cast<PAlbum*>(album);
     
-    QStringList list =
-        KFileDialog::getOpenFileNames(QString::null,
+    KURL::List urls =  KFileDialog::getOpenURLs(QString::null,
                                       AlbumSettings::instance()->getImageFileFilter(),
                                       this,
                                       i18n("Add Images"));
-    KURL::List urls;
 
-    for (QStringList::Iterator it
-             = list.begin(); it != list.end(); ++it ) {
-        QFileInfo fi(*it);
-        if (!fi.isDir()) {
-            urls.append(KURL(*it));
-        }
-    }
-
-    if (!urls.isEmpty()) {
+    if (!urls.isEmpty())
+    {
         KIO::CopyJob* job =
             KIO::copy(urls, palbum->getKURL(), true);