141799 2007-02-16 19:38:29 +0000 Not able to select anything after automatically adding several torrents. 2007-02-17 14:24:38 +0000 1 1 2 Applications ktorrent general unspecified Unlisted Binaries Linux RESOLVED FIXED NOR normal --- 0 laurens joris.guisson 0 oldest_to_newest 509577 0 laurens 2007-02-16 19:38:29 +0000 Version: 2.1 (using KDE KDE 3.5.6) Installed from: Unspecified Linux OS: Linux I let ktorrent automatically add .torrent files who appear in a certain folder and ktorrent should add them silently. After adding several torrents (3+) at once this way, I'm unable to select any menu option or torrent details or anything else in the ktorrent GUI. Ktorrent works as the torrents which are loaded start downloading, I'm just not able to do anything in ktorrent anymore. I can only restart ktorrent by killing at as I cannot select the option to quite ktorrent. If I just add 1 torrent this way, ktorrent is still usable. If I add multiple, ktorrent isn't usable anymore. I have reproduced this several times. 509695 1 joris.guisson 2007-02-17 13:38:20 +0000 I have a pretty good idea what the problem is 509712 2 joris.guisson 2007-02-17 14:24:37 +0000 SVN commit 634509 by guisson: - NetAccess considered harmfull, so lets not use it anymore in scanfolder plugin, this fixes bug 141799 - In case of loadSilently, if no default save location is set, the home directory of the user is used instead BUG: 141799 M +7 -7 apps/ktorrent/ktorrentcore.cpp M +3 -4 plugins/scanfolder/scanfolder.cpp --- trunk/extragear/network/ktorrent/apps/ktorrent/ktorrentcore.cpp #634508:634509 @@ -329,15 +329,15 @@ QString dir = Settings::saveDir(); if (!Settings::useSaveDir()) { - loadingFinished(url,false,false); + Out(SYS_GEN|LOG_NOTICE) << "Cannot load " << path << " silently, default save location not set !" << endl; + Out(SYS_GEN|LOG_NOTICE) << "Using home directory instead !" << endl; + dir = QDir::homeDirPath(); } + + if (dir != QString::null && load(path,dir,true)) + loadingFinished(url,true,false); else - { - if (dir != QString::null && load(path,dir,true)) - loadingFinished(url,true,false); - else - loadingFinished(url,false,true); - } + loadingFinished(url,false,true); } else { --- trunk/extragear/network/ktorrent/plugins/scanfolder/scanfolder.cpp #634508:634509 @@ -22,7 +22,7 @@ #include <kdirlister.h> #include <kfileitem.h> #include <klocale.h> -#include <kio/netaccess.h> +#include <kio/job.h> #include <qstring.h> #include <qobject.h> @@ -56,7 +56,6 @@ connect(m_dir, SIGNAL(newItems( const KFileItemList& )), this, SLOT(onNewItems( const KFileItemList& ))); connect(m_core, SIGNAL(loadingFinished( const KURL&, bool, bool )), this, SLOT(onLoadingFinished( const KURL&, bool, bool ))); -// Out() << "LOADING SCANFOLDER: " << m_dir->url().path() << endl; } @@ -76,7 +75,6 @@ QString dirname = m_dir->url().path(); QString filename = dirname + "/" + name; - if(!name.endsWith(".torrent")) continue; @@ -144,7 +142,8 @@ if(QFile::exists(dirname + "/." + name)) QFile::remove(dirname + "/." + name); - KIO::NetAccess::move(url, destination); + // NetAccess considered harmfull !!! + KIO::file_move(url, destination); break; case defaultAction: QFile f(dirname + "/." + name);