- Torrent folders can also be renamed

Changelog

@@ -4,7 +4,7 @@
     - FEATURE: Disk cache size can be set from preferences
     - FEATURE: Peer Exchange (PeX) can be disabled from preferences
     - FEATURE: Append !.qB extension to incomplete files option (libtorrent >= v0.15 only)
-    - FEATURE: Torrent files can be renamed
+    - FEATURE: Torrent files/folders can be renamed
 
 * Thu Dec 10 2009 - Christophe Dumez <chris@qbittorrent.org> - v2.0.0
     - FEATURE: Added program option to disable splash screen

src/propertieswidget.cpp

@@ -479,7 +479,7 @@ void PropertiesWidget::displayFilesListMenu(const QPoint&){
   QMenu myFilesLlistMenu;
   QModelIndexList selectedRows = filesList->selectionModel()->selectedRows(0);
   QAction *actRename = 0;
-  if(selectedRows.size() == 1 && PropListModel->getType(selectedRows.first())==TFILE) {
+  if(selectedRows.size() == 1) {
     actRename = myFilesLlistMenu.addAction(QIcon(QString::fromUtf8(":/Icons/oxygen/edit_clear.png")), tr("Rename..."));
     myFilesLlistMenu.addSeparator();
   }
@@ -500,43 +500,70 @@ void PropertiesWidget::displayFilesListMenu(const QPoint&){
 void PropertiesWidget::renameSelectedFile() {
   QModelIndexList selectedIndexes = filesList->selectionModel()->selectedRows(0);
   Q_ASSERT(selectedIndexes.size() == 1);
+  QModelIndex index = selectedIndexes.first();
   // Ask for new name
   bool ok;
   QString new_name_last = QInputDialog::getText(this, tr("Rename torrent file"),
-                                           tr("New name:"), QLineEdit::Normal,
+                                                tr("New name:"), QLineEdit::Normal,
-                                           selectedIndexes.first().data().toString(), &ok);
+                                                index.data().toString(), &ok);
   if (ok && !new_name_last.isEmpty()) {
-    int file_index = PropListModel->getFileIndex(selectedIndexes.first());
+    if(PropListModel->getType(index)==TFILE) {
-    if(!h.is_valid() || !h.has_metadata()) return;
+      // File renaming
-    QString old_name = misc::toQString(h.get_torrent_info().file_at(file_index).path.string());
+      int file_index = PropListModel->getFileIndex(index);
-    if(old_name.endsWith(".!qB") && !new_name_last.endsWith(".!qB")) {
+      if(!h.is_valid() || !h.has_metadata()) return;
-      new_name_last += ".!qB";
+      QString old_name = misc::toQString(h.get_torrent_info().file_at(file_index).path.string());
-    }
+      if(old_name.endsWith(".!qB") && !new_name_last.endsWith(".!qB")) {
-    QStringList path_items = old_name.split(QDir::separator());
+        new_name_last += ".!qB";
-    path_items.removeLast();
+      }
-    path_items << new_name_last;
+      QStringList path_items = old_name.split(QDir::separator());
-    QString new_name = path_items.join(QDir::separator());
+      path_items.removeLast();
-    if(old_name == new_name) {
+      path_items << new_name_last;
-      qDebug("Name did not change");
+      QString new_name = path_items.join(QDir::separator());
-      return;
+      if(old_name == new_name) {
-    }
+        qDebug("Name did not change");
-    // Check if that name is already used
-    for(int i=0; i<h.num_files(); ++i) {
-      if(i == file_index) continue;
-      if(misc::toQString(h.get_torrent_info().file_at(i).path.string()) == new_name) {
-        // Display error message
-        QMessageBox::warning(this, tr("The file could not be renamed"),
-                         tr("This name is already in use in this folder. Please use a different name."),
-                         QMessageBox::Ok);
         return;
       }
+      // Check if that name is already used
+      for(int i=0; i<h.num_files(); ++i) {
+        if(i == file_index) continue;
+        if(misc::toQString(h.get_torrent_info().file_at(i).path.string()) == new_name) {
+          // Display error message
+          QMessageBox::warning(this, tr("The file could not be renamed"),
+                               tr("This name is already in use in this folder. Please use a different name."),
+                               QMessageBox::Ok);
+          return;
+        }
+      }
+      qDebug("Renaming %s to %s", old_name.toLocal8Bit().data(), new_name.toLocal8Bit().data());
+      h.rename_file(file_index, new_name);
+      // Rename if torrent files model too
+      if(new_name_last.endsWith(".!qB"))
+        new_name_last.chop(4);
+      PropListModel->setData(index, new_name_last);
+    } else {
+      // Folder renaming
+      QStringList path_items;
+      path_items << index.data().toString();
+      QModelIndex parent = PropListModel->parent(index);
+      while(parent.isValid()) {
+        path_items.prepend(parent.data().toString());
+        parent = PropListModel->parent(parent);
+      }
+      QString old_path = path_items.join(QDir::separator());
+      path_items.removeLast();
+      path_items << new_name_last;
+      QString new_path = path_items.join(QDir::separator());
+      // XXX: Check for overwriting
+      // Replace path in all files
+      for(int i=0; i<h.num_files(); ++i) {
+        QString current_name = misc::toQString(h.get_torrent_info().file_at(i).path.string());
+        QString new_name = current_name.replace(old_path, new_path);
+        qDebug("Rename %s to %s", current_name.toLocal8Bit().data(), new_name.toLocal8Bit().data());
+        h.rename_file(i, new_name);
+      }
+      // Rename folder in torrent files model too
+      PropListModel->setData(index, new_name_last);
     }
-    qDebug("Renaming %s to %s", old_name.toLocal8Bit().data(), new_name.toLocal8Bit().data());
-    h.rename_file(file_index, new_name);
-    // Rename if torrent files model too
-    if(new_name_last.endsWith(".!qB"))
-      new_name_last.chop(4);
-    PropListModel->setData(selectedIndexes.first(), new_name_last);
   }
 }
 

src/torrentfilesmodel.h

@@ -329,7 +329,7 @@ public:
 
   void updateFilesPriorities(std::vector<int> fprio) {
     for(unsigned int i=0; i<fprio.size(); ++i) {
-      qDebug("Called updateFilesPriorities with %d", fprio[i]);
+      //qDebug("Called updateFilesPriorities with %d", fprio[i]);
       files_index[i]->setPriority(fprio[i]);
     }
     emit layoutChanged();
@@ -338,7 +338,7 @@ public:
   std::vector<int> getFilesPriorities(unsigned int nbFiles) const {
     std::vector<int> prio;
     for(unsigned int i=0; i<nbFiles; ++i) {
-      qDebug("Called getFilesPriorities: %d", files_index[i]->getPriority());
+      //qDebug("Called getFilesPriorities: %d", files_index[i]->getPriority());
       prio.push_back(files_index[i]->getPriority());
     }
     return prio;