+ support delete multiple files

Author: hazmi-e205

Src/devicebridge.h

@@ -206,6 +206,7 @@ class DeviceBridge : public QObject
      void PullFromStorage(QString devicePath, QString localPath, QString bundleId = "");
      void PullMultipleFromStorage(QList<QPair<QString,QString>> pairs, QString bundleId = "");
      void DeleteFromStorage(QString devicePath, QString bundleId = "");
+     void DeleteMultipleFromStorage(QStringList devicePaths, QString bundleId = "");
      void MakeDirectoryToStorage(QString devicePath, QString bundleId = "");
      void RenameToStorage(QString oldPath, QString newPath, QString bundleId = "");
  private:

Src/devicebridge_filemanager.cpp

@@ -6,7 +6,7 @@
 void DeviceBridge::GetAccessibleStorage(QString startPath, QString bundleId, bool partialUpdate)
 {
     afc_filemanager_action(MobileOperation::FILE_LIST, [=, this](afc_client_t& afc){
-        if (partialUpdate) {
+        if (partialUpdate && startPath != "/") {
             QString prefix = startPath + "/";
             for (auto it = m_accessibleStorage.begin(); it != m_accessibleStorage.end(); ) {
                 if (it.key().startsWith(prefix))
@@ -168,6 +168,49 @@ void DeviceBridge::PullMultipleFromStorage(QList<QPair<QString,QString>> pairs,
     }, bundleId);
 }
 
+void DeviceBridge::DeleteMultipleFromStorage(QStringList devicePaths, QString bundleId)
+{
+    afc_filemanager_action(MobileOperation::DELETE_FILE, [=, this](afc_client_t& afc){
+        int totalFiles = devicePaths.size();
+        if (totalFiles == 0) return;
+
+        bool hasError = false;
+
+        for (int i = 0; i < totalFiles; i++) {
+            const QString& devicePath = devicePaths[i];
+            QString filename = QFileInfo(devicePath).fileName();
+            QString fileLabel = QString("[%1/%2] %3").arg(i + 1).arg(totalFiles).arg(filename);
+
+            emit FileManagerChanged(GenericStatus::IN_PROGRESS, FileOperation::DELETE_OP,
+                (i * 100) / totalFiles, fileLabel);
+
+            afc_error_t err = afc_remove_path(afc, devicePath.toUtf8().data());
+            if (err != AFC_E_SUCCESS) {
+                hasError = true;
+                emit FileManagerChanged(GenericStatus::IN_PROGRESS, FileOperation::DELETE_OP,
+                    ((i + 1) * 100) / totalFiles, fileLabel + " - FAILED - " + QString::number(err));
+            }
+        }
+
+        // Find common ancestor directory of all deleted paths for partial refresh
+        auto splitPath = [](const QString& p) { return p.split('/', Qt::SkipEmptyParts); };
+        QStringList common = splitPath(QFileInfo(devicePaths.first()).path());
+        for (const QString& path : devicePaths) {
+            QStringList parts = splitPath(QFileInfo(path).path());
+            int minLen = std::min(common.size(), parts.size());
+            int i = 0;
+            while (i < minLen && common[i] == parts[i]) i++;
+            common = common.mid(0, i);
+        }
+        QString commonDir = (common.isEmpty() ? "/" : "/" + common.join('/')) + "/";
+
+        if (hasError)
+            emit FileManagerChanged(GenericStatus::FAILED, FileOperation::DELETE_OP, 100, "Completed with errors");
+        else
+            emit FileManagerChanged(GenericStatus::SUCCESS, FileOperation::DELETE_OP, 100, commonDir);
+    }, bundleId);
+}
+
 void DeviceBridge::DeleteFromStorage(QString devicePath, QString bundleId)
 {
     afc_filemanager_action(MobileOperation::DELETE_FILE, [=, this](afc_client_t& afc){

Src/mainwindow_filemanager.cpp

@@ -305,9 +305,10 @@ void MainWindow::OnFileManagerChanged(GenericStatus status, FileOperation operat
             QString storage = ui->storageOption->currentText();
             storage = storage.contains("User's Data", Qt::CaseInsensitive) ? "" : storage;
 
-            // partial update
-            QFileInfo fileInfo(message);
-            QString dirPath = fileInfo.dir().absolutePath();
+            // partial update — trailing '/' means message is already the refresh directory
+            QString dirPath = message.endsWith('/')
+                ? message.chopped(1)
+                : QFileInfo(message).dir().absolutePath();
             DeviceBridge::Get()->GetAccessibleStorage(dirPath, storage, true);
         }
     }
@@ -426,21 +427,42 @@ void MainWindow::OnPushFileClicked()
 
 void MainWindow::OnDeleteFileClicked()
 {
-    FileManagerAction([this](QString& initialText, QString& storageAccess){
-        QMessageBox msgBox(this);
-        msgBox.setWindowTitle("Delete");
-        msgBox.setText("Are you sure to delete " + initialText + " from device?");
-        msgBox.setStandardButtons(QMessageBox::Ok | QMessageBox::Cancel);
-        msgBox.setDefaultButton(QMessageBox::Ok);
-
-        int ret = msgBox.exec();
-        if (ret == QMessageBox::Ok) {
-            DeviceBridge::Get()->DeleteFromStorage(initialText, storageAccess);
-        }
-        else {
-            m_loadingFileOperation->close();
-        }
-    }, false);
+    QString storage = ui->storageOption->currentText();
+    storage = storage.contains("User's Data", Qt::CaseInsensitive) ? "" : storage;
+
+    QModelIndexList selected = ui->fileBrowserTree->selectionModel()->selectedIndexes();
+    QStringList selectedPaths;
+    for (const QModelIndex& idx : selected) {
+        if (idx.column() != 0) continue;
+        QString path = idx.data(Qt::UserRole).toString();
+        if (path.isEmpty()) continue;
+        if (!selectedPaths.contains(path))
+            selectedPaths.append(path);
+    }
+
+    if (selectedPaths.isEmpty()) {
+        QMessageBox::critical(this, "Error", "Please choose file(s) in File Manager's Browser!", QMessageBox::Ok);
+        return;
+    }
+
+    QString confirmMsg = selectedPaths.size() == 1
+        ? "Are you sure to delete " + selectedPaths.first() + " from device?"
+        : QString("Are you sure to delete %1 items from device?").arg(selectedPaths.size());
+
+    QMessageBox msgBox(this);
+    msgBox.setWindowTitle("Delete");
+    msgBox.setText(confirmMsg);
+    msgBox.setStandardButtons(QMessageBox::Ok | QMessageBox::Cancel);
+    msgBox.setDefaultButton(QMessageBox::Ok);
+
+    if (msgBox.exec() == QMessageBox::Ok) {
+        if (selectedPaths.size() == 1)
+            DeviceBridge::Get()->DeleteFromStorage(selectedPaths.first(), storage);
+        else
+            DeviceBridge::Get()->DeleteMultipleFromStorage(selectedPaths, storage);
+    } else {
+        m_loadingFileOperation->close();
+    }
 }
 
 void MainWindow::OnRenameFileClicked()