Bug 88804 - Digikam: Add images: Allows only to import local files. No sftp: ftp: :(
Summary: Digikam: Add images: Allows only to import local files. No sftp: ftp: :(
Status: RESOLVED FIXED
Alias: None
Product: digikam
Classification: Applications
Component: Import-MainView (show other bugs)
Version: 0.7.0
Platform: unspecified Linux
: NOR wishlist
Target Milestone: ---
Assignee: Digikam Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-09-04 00:52 UTC by Achim Bohnet
Modified: 2017-07-16 18:35 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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);