FS should be updated properly on labeling now

src/bittorrent.cpp

@@ -1645,7 +1645,6 @@ void Bittorrent::addConsoleMessage(QString msg, QString) {
       qDebug("Moving storage to %s", qPrintable(new_save_path));
       QDir().mkpath(new_save_path);
       h.move_storage(new_save_path);
-      emit savePathChanged(h);
     }
   }
 
@@ -1658,8 +1657,8 @@ void Bittorrent::addConsoleMessage(QString msg, QString) {
     QString new_save_path = misc::updateLabelInSavePath(defaultSavePath, old_save_path, "", label);
     if(old_save_path != new_save_path) {
       // Move storage
+      QDir().mkpath(new_save_path);
       h.move_storage(new_save_path);
-      emit savePathChanged(h);
     }
   }
 
@@ -1950,9 +1949,12 @@ void Bittorrent::addConsoleMessage(QString msg, QString) {
       }
       else if (storage_moved_alert* p = dynamic_cast<storage_moved_alert*>(a.get())) {
         QTorrentHandle h(p->handle);
-        if(h.is_valid())
+        if(h.is_valid()) {
+          TorrentPersistentData::saveSavePath(h.hash(), QString(p->path.c_str()));
+          emit savePathChanged(h);
           h.force_recheck(); //XXX: Required by libtorrent for now
         }
+      }
       else if (metadata_received_alert* p = dynamic_cast<metadata_received_alert*>(a.get())) {
         QTorrentHandle h(p->handle);
         if(h.is_valid()) {
@@ -2177,14 +2179,11 @@ void Bittorrent::addConsoleMessage(QString msg, QString) {
       if(savePath.isEmpty()) {
         savePath = defaultSavePath;
       }
-      if(appendLabelToSavePath && savePath.startsWith(defaultSavePath)) {
+      if(appendLabelToSavePath) {
         qDebug("appendLabelToSavePath is true");
         const QString &label = TorrentTempData::getLabel(hash);
         if(!label.isEmpty()) {
-          const QDir save_dir(savePath);
+          savePath = misc::updateLabelInSavePath(defaultSavePath, savePath, "", label);
-          if(save_dir.dirName() != label) {
-            savePath = save_dir.absoluteFilePath(label);
-          }
         }
       }
       qDebug("getSavePath, got save_path from temp data: %s", qPrintable(savePath));
@@ -2207,10 +2206,7 @@ void Bittorrent::addConsoleMessage(QString msg, QString) {
       if(!fromScanDir && appendLabelToSavePath) {
         const QString &label = TorrentPersistentData::getLabel(hash);
         if(!label.isEmpty()) {
-          QDir save_dir(savePath);
+          savePath = misc::updateLabelInSavePath(defaultSavePath, savePath, "", label);
-          if(save_dir.dirName() != label) {
-            savePath = save_dir.absoluteFilePath(label);
-          }
         }
       }
       if(append_root_folder && !root_folder.isEmpty()) {

src/misc.cpp

@@ -258,14 +258,17 @@ QString misc::updateLabelInSavePath(const QString& defaultSavePath, QString save
       path_parts << new_label;
   } else {
     if(old_label.isEmpty() || path_parts.first() != old_label) {
+      if(path_parts.first() != new_label)
         path_parts.prepend(new_label);
     } else {
-      if(new_label.isEmpty())
+      if(new_label.isEmpty()) {
         path_parts.removeAt(0);
-      else
+      } else {
+        if(path_parts.first() != new_label)
           path_parts.replace(0, new_label);
       }
     }
+  }
   new_save_path = defaultSavePath;
   if(!new_save_path.endsWith(QDir::separator())) new_save_path += QDir::separator();
   new_save_path += path_parts.join(QDir::separator());

src/torrentadditiondlg.h

@@ -66,6 +66,9 @@ private:
   QString hash;
   QString filePath;
   QString from_url;
+  QString defaultSavePath;
+  QString old_label;
+  bool appendLabelToSavePath;
   TorrentFilesModel *PropListModel;
   PropListDelegate *PropDelegate;
   unsigned int nbFiles;
@@ -74,7 +77,7 @@ private:
   bool is_magnet;
 
 public:
-  torrentAdditionDialog(GUI *parent, Bittorrent* _BTSession) : QDialog((QWidget*)parent) {
+  torrentAdditionDialog(GUI *parent, Bittorrent* _BTSession) : QDialog((QWidget*)parent), old_label("") {
     setupUi(this);
     setAttribute(Qt::WA_DeleteOnClose);
     connect(this, SIGNAL(torrentPaused(QTorrentHandle&)), parent->getTransferList(), SLOT(pauseTorrent(QTorrentHandle&)));
@@ -89,10 +92,14 @@ public:
     connect(torrentContentList, SIGNAL(customContextMenuRequested(const QPoint&)), this, SLOT(displayContentListMenu(const QPoint&)));
     connect(collapseAllButton, SIGNAL(clicked()), torrentContentList, SLOT(collapseAll()));
     connect(expandAllButton, SIGNAL(clicked()), torrentContentList, SLOT(expandAll()));
+    connect(comboLabel, SIGNAL(editTextChanged(QString)), this, SLOT(updateLabelInSavePath(QString)));
+    connect(comboLabel, SIGNAL(currentIndexChanged(QString)), this, SLOT(updateLabelInSavePath(QString)));
     // Remember columns width
     readSettings();
     //torrentContentList->header()->setResizeMode(0, QHeaderView::Stretch);
-    savePathTxt->setText(Preferences::getSavePath());
+    defaultSavePath = Preferences::getSavePath();
+    appendLabelToSavePath = Preferences::appendTorrentLabel();
+    savePathTxt->setText(defaultSavePath);
     if(Preferences::addTorrentsInPause()) {
       addInPause->setChecked(true);
       //addInPause->setEnabled(false);
@@ -516,6 +523,14 @@ public slots:
           close();
         }
 
+      public slots:
+        void updateLabelInSavePath(QString label) {
+          if(appendLabelToSavePath) {
+            savePathTxt->setText(misc::updateLabelInSavePath(defaultSavePath, savePathTxt->text(), old_label, label));
+            old_label = label;
+          }
+        }
+
       signals:
         void torrentPaused(QTorrentHandle &h);
       };