Torrent addition dialog size/pos are now remembered correctly

src/bittorrent.cpp

@@ -1065,6 +1065,21 @@ QTorrentHandle Bittorrent::addTorrent(QString path, bool fromScanDir, QString fr
     }
     return h;
   }
+  // Check number of files
+  if(t->num_files() < 1) {
+    addConsoleMessage(tr("Error: The torrent %1 does not contain any file.").arg(misc::toQStringU(t->name())));
+    if(fromScanDir) {
+      // Delete torrent from scan dir
+      QFile::remove(file);
+    } else {
+      if(!from_url.isNull()) {
+        // If download from url, remove temp file
+        QFile::remove(file);
+      }
+    }
+    return h;
+  }
+  // Actually add the torrent
   QString root_folder = misc::truncateRootFolder(t);
   add_torrent_params p;
   //Getting fast resume data if existing

src/torrentadditiondlg.cpp

@@ -30,7 +30,7 @@
 
 #include "torrentadditiondlg.h"
 
-torrentAdditionDialog::torrentAdditionDialog(GUI *parent, Bittorrent* _BTSession) : QDialog((QWidget*)parent), old_label("") {
+torrentAdditionDialog::torrentAdditionDialog(GUI *parent, Bittorrent* _BTSession) : QDialog((QWidget*)parent), old_label(""), hidden_height(0) {
   setupUi(this);
   setAttribute(Qt::WA_DeleteOnClose);
   connect(this, SIGNAL(torrentPaused(QTorrentHandle&)), parent->getTransferList(), SLOT(pauseTorrent(QTorrentHandle&)));
@@ -91,15 +91,17 @@ void torrentAdditionDialog::readSettings() {
 }
 
 void torrentAdditionDialog::saveSettings() {
-  if(is_magnet || t->num_files() <= 1) return;
   QSettings settings(QString::fromUtf8("qBittorrent"), QString::fromUtf8("qBittorrent"));
-  QStringList contentColsWidths;
-  // -1 because we hid PROGRESS column
-  for(int i=0; i<PropListModel->columnCount()-1; ++i) {
-    contentColsWidths << QString::number(torrentContentList->columnWidth(i));
+  if(!is_magnet && t->num_files() > 1) {
+    QStringList contentColsWidths;
+    // -1 because we hid PROGRESS column
+    for(int i=0; i<PropListModel->columnCount()-1; ++i) {
+      contentColsWidths << QString::number(torrentContentList->columnWidth(i));
+    }
+    settings.setValue(QString::fromUtf8("TorrentAdditionDlg/filesColsWidth"), contentColsWidths);
   }
-  settings.setValue(QString::fromUtf8("TorrentAdditionDlg/filesColsWidth"), contentColsWidths);
-  settings.setValue("TorrentAdditionDlg/size", size());
+  settings.setValue("TorrentAdditionDlg/size", size()+QSize(0, hidden_height));
+  qDebug("pos: (%d, %d)", pos().x(), pos().y());
   settings.setValue("TorrentAdditionDlg/pos", pos());
 }
 
@@ -110,7 +112,6 @@ void torrentAdditionDialog::renameTorrentNameInModel(QString file_path) {
 }
 
 void torrentAdditionDialog::hideTorrentContent() {
-  int hidden_height = 0;
   // Disable useless widgets
   hidden_height += torrentContentList->height();
   torrentContentList->setVisible(false);

src/torrentadditiondlg.h

@@ -102,6 +102,7 @@ private:
   boost::intrusive_ptr<torrent_info> t;
   QStringList files_path;
   bool is_magnet;
+  int hidden_height;
 };
 
 #endif