fix: Add temporary message support for non-blocking error notifications

LiHua000 · LiHua000 · commit d8d17e9b31d4 · 2026-04-01T13:46:30.000+08:00
RAR5 errors no longer interrupt extraction, user sees floating message instead. Bug: https://pms.uniontech.com/bug-view-354871.html
diff --git a/3rdparty/clirarplugin/clirarplugin.cpp b/3rdparty/clirarplugin/clirarplugin.cpp
@@ -284,13 +284,14 @@ bool CliRarPlugin::handleLine(const QString &line, WorkType workStatus)
     }
 
     if (isWrongPasswordMsg(line)) {  // 提示密码错误
+        m_eErrorType = ET_WrongPassword;
+
         // RAR4密码错误直接结束
         if (line.startsWith(QLatin1String("Checksum error in the encrypted file"))) {
-            m_eErrorType = ET_WrongPassword;
             m_finishType = PFT_Error;
             return false;
-        } else { // RAR5密码错误反复输入新密码
-            m_eErrorType = ET_WrongPassword;
+        } else { // RAR5密码错误：发送错误提示但不中断进程，允许用户重新输入密码
+            emit error(tr("Wrong password"), tr("The password entered is incorrect. Please try again."));
             return true;
         }
     }
diff --git a/src/source/archivemanager/archivejob.h b/src/source/archivemanager/archivejob.h
@@ -110,6 +110,12 @@ class ArchiveJob : public QObject
      */
     void signalQuery(Query *query);
 
+    /**
+     * @brief signalTempMessage 发送临时消息（不中断操作）
+     * @param message 消息内容
+     */
+    void signalTempMessage(const QString &message);
+
 public:
     JobType m_eJobType = JT_NoJob;     // 操作类型
     PluginFinishType m_eFinishedType = PFT_Nomral;
diff --git a/src/source/archivemanager/archivemanager.cpp b/src/source/archivemanager/archivemanager.cpp
@@ -111,6 +111,7 @@ bool ArchiveManager::loadArchive(const QString &strArchiveFullPath, UiTools::Ass
         // 连接槽函数
         connect(pLoadJob, &LoadJob::signalJobFinshed, this, &ArchiveManager::slotJobFinished);
         connect(pLoadJob, &LoadJob::signalQuery, this, &ArchiveManager::signalQuery);
+        connect(pLoadJob, &LoadJob::signalTempMessage, this, &ArchiveManager::signalTempMessage);
 
         m_pArchiveJob = pLoadJob;
         pLoadJob->start();
@@ -139,6 +140,7 @@ bool ArchiveManager::addFiles(const QString &strArchiveFullPath, const QList<Fil
         connect(pAddJob, &AddJob::signalprogress, this, &ArchiveManager::signalprogress);
         connect(pAddJob, &AddJob::signalCurFileName, this, &ArchiveManager::signalCurFileName);
         connect(pAddJob, &AddJob::signalQuery, this, &ArchiveManager::signalQuery);
+        connect(pAddJob, &AddJob::signalTempMessage, this, &ArchiveManager::signalTempMessage);
 
         m_pArchiveJob = pAddJob;
         pAddJob->start();
@@ -171,6 +173,7 @@ bool ArchiveManager::extractFiles(const QString &strArchiveFullPath, const QList
             connect(pExtractJob, &ExtractJob::signalCurFileName, this, &ArchiveManager::signalCurFileName);
             connect(pExtractJob, &ExtractJob::signalFileWriteErrorName, this, &ArchiveManager::signalFileWriteErrorName);
             connect(pExtractJob, &ExtractJob::signalQuery, this, &ArchiveManager::signalQuery);
+            connect(pExtractJob, &ExtractJob::signalTempMessage, this, &ArchiveManager::signalTempMessage);
 
             m_pArchiveJob = pExtractJob;
             pExtractJob->start();
@@ -186,6 +189,7 @@ bool ArchiveManager::extractFiles(const QString &strArchiveFullPath, const QList
             connect(pStepExtractJob, &StepExtractJob::signalprogress, this, &ArchiveManager::signalprogress);
             connect(pStepExtractJob, &StepExtractJob::signalCurFileName, this, &ArchiveManager::signalCurFileName);
             connect(pStepExtractJob, &StepExtractJob::signalQuery, this, &ArchiveManager::signalQuery);
+            connect(pStepExtractJob, &StepExtractJob::signalTempMessage, this, &ArchiveManager::signalTempMessage);
 
             pStepExtractJob->start();
             return true;
@@ -217,6 +221,7 @@ bool ArchiveManager::extractFiles2Path(const QString &strArchiveFullPath, const
         connect(pExtractJob, &ExtractJob::signalprogress, this, &ArchiveManager::signalprogress);
         connect(pExtractJob, &ExtractJob::signalCurFileName, this, &ArchiveManager::signalCurFileName);
         connect(pExtractJob, &ExtractJob::signalQuery, this, &ArchiveManager::signalQuery);
+        connect(pExtractJob, &ExtractJob::signalTempMessage, this, &ArchiveManager::signalTempMessage);
 
         m_pArchiveJob = pExtractJob;
         pExtractJob->start();
@@ -247,6 +252,7 @@ bool ArchiveManager::deleteFiles(const QString &strArchiveFullPath, const QList<
         connect(pDeleteJob, &DeleteJob::signalprogress, this, &ArchiveManager::signalprogress);
         connect(pDeleteJob, &DeleteJob::signalCurFileName, this, &ArchiveManager::signalCurFileName);
         connect(pDeleteJob, &DeleteJob::signalQuery, this, &ArchiveManager::signalQuery);
+        connect(pDeleteJob, &DeleteJob::signalTempMessage, this, &ArchiveManager::signalTempMessage);
 
         m_pArchiveJob = pDeleteJob;
         pDeleteJob->start();
@@ -277,6 +283,7 @@ bool ArchiveManager::renameFiles(const QString &strArchiveFullPath, const QList<
         connect(pRenameJob, &RenameJob::signalprogress, this, &ArchiveManager::signalprogress);
         connect(pRenameJob, &RenameJob::signalCurFileName, this, &ArchiveManager::signalCurFileName);
         connect(pRenameJob, &RenameJob::signalQuery, this, &ArchiveManager::signalQuery);
+        connect(pRenameJob, &RenameJob::signalTempMessage, this, &ArchiveManager::signalTempMessage);
 
         m_pArchiveJob = pRenameJob;
         pRenameJob->start();
@@ -299,6 +306,7 @@ bool ArchiveManager::batchExtractFiles(const QStringList &listFiles, const QStri
         connect(pBatchExtractJob, &BatchExtractJob::signalCurFileName, this, &ArchiveManager::signalCurFileName);
         connect(pBatchExtractJob, &BatchExtractJob::signalQuery, this, &ArchiveManager::signalQuery);
         connect(pBatchExtractJob, &BatchExtractJob::signalCurArchiveName, this, &ArchiveManager::signalCurArchiveName);
+        connect(pBatchExtractJob, &BatchExtractJob::signalTempMessage, this, &ArchiveManager::signalTempMessage);
 
         m_pArchiveJob = pBatchExtractJob;
         pBatchExtractJob->start();
@@ -328,6 +336,7 @@ bool ArchiveManager::openFile(const QString &strArchiveFullPath, const FileEntry
         // 连接槽函数
         connect(pOpenJob, &OpenJob::signalJobFinshed, this, &ArchiveManager::slotJobFinished);
         connect(pOpenJob, &OpenJob::signalQuery, this, &ArchiveManager::signalQuery);
+        connect(pOpenJob, &OpenJob::signalTempMessage, this, &ArchiveManager::signalTempMessage);
 
 
         m_pArchiveJob = pOpenJob;
@@ -386,6 +395,7 @@ bool ArchiveManager::convertArchive(const QString &strOriginalArchiveFullPath, c
     connect(pConvertJob, &ConvertJob::signalprogress, this, &ArchiveManager::signalprogress);
     connect(pConvertJob, &ConvertJob::signalCurFileName, this, &ArchiveManager::signalCurFileName);
     connect(pConvertJob, &ConvertJob::signalQuery, this, &ArchiveManager::signalQuery);
+    connect(pConvertJob, &ConvertJob::signalTempMessage, this, &ArchiveManager::signalTempMessage);
 
     pConvertJob->start();
     return true;
diff --git a/src/source/archivemanager/archivemanager.h b/src/source/archivemanager/archivemanager.h
@@ -213,6 +213,12 @@ class ArchiveManager : public QObject
      */
     void signalCurArchiveName(const QString &strArchiveName);
 
+    /**
+     * @brief signalTempMessage 发送临时消息（不中断操作）
+     * @param message 消息内容
+     */
+    void signalTempMessage(const QString &message);
+
 private:
     explicit ArchiveManager(QObject *parent = nullptr);
     ~ArchiveManager() override;
diff --git a/src/source/archivemanager/batchjob.cpp b/src/source/archivemanager/batchjob.cpp
@@ -186,13 +186,15 @@ bool BatchExtractJob::addExtractItem(const QFileInfo &fileInfo)
             connect(pExtractJob, &ExtractJob::signalCurFileName, this, &BatchExtractJob::slotHandleSingleJobCurFileName);
             connect(pExtractJob, &ExtractJob::signalQuery, this, &BatchExtractJob::signalQuery);
             connect(pExtractJob, &ExtractJob::signalJobFinshed, this, &BatchExtractJob::slotHandleSingleJobFinished);
+            connect(pExtractJob, &ExtractJob::signalTempMessage, this, &BatchExtractJob::signalTempMessage);
             addSubjob(pExtractJob);
         } else {
             StepExtractJob *pStepExtractJob = new StepExtractJob(fileInfo.absoluteFilePath(), stOptions);
             connect(pStepExtractJob, &StepExtractJob::signalprogress, this, &BatchExtractJob::slotHandleSingleJobProgress);
             connect(pStepExtractJob, &StepExtractJob::signalCurFileName, this, &BatchExtractJob::slotHandleSingleJobCurFileName);
             connect(pStepExtractJob, &StepExtractJob::signalQuery, this, &BatchExtractJob::signalQuery);
             connect(pStepExtractJob, &StepExtractJob::signalJobFinshed, this, &BatchExtractJob::slotHandleSingleJobFinished);
+            connect(pStepExtractJob, &StepExtractJob::signalTempMessage, this, &BatchExtractJob::signalTempMessage);
             addSubjob(pStepExtractJob);
         }
         return true;
diff --git a/src/source/archivemanager/batchjob.h b/src/source/archivemanager/batchjob.h
@@ -114,6 +114,12 @@ class BatchExtractJob : public BatchJob
      */
     void signalCurArchiveName(const QString &strArchiveName);
 
+    /**
+     * @brief signalTempMessage 发送临时消息（不中断操作）
+     * @param message 消息内容
+     */
+    void signalTempMessage(const QString &message);
+
 private Q_SLOTS:
     /**
      * @brief slotHandleSingleJobProgress       处理单个压缩包解压进度
diff --git a/src/source/archivemanager/singlejob.cpp b/src/source/archivemanager/singlejob.cpp
@@ -116,6 +116,7 @@ void SingleJob::initConnections()
     connect(m_pInterface, &ReadOnlyArchiveInterface::signalCurFileName, this, &SingleJob::signalCurFileName, Qt::ConnectionType::UniqueConnection);
     connect(m_pInterface, &ReadOnlyArchiveInterface::signalFileWriteErrorName, this, &SingleJob::signalFileWriteErrorName, Qt::ConnectionType::UniqueConnection);
     connect(m_pInterface, &ReadOnlyArchiveInterface::signalQuery, this, &SingleJob::signalQuery, Qt::ConnectionType::AutoConnection);
+    connect(m_pInterface, &ReadOnlyArchiveInterface::error, this, &SingleJob::slotError, Qt::AutoConnection);
 }
 
 void SingleJob::slotFinished(PluginFinishType eType)
@@ -129,6 +130,13 @@ void SingleJob::slotFinished(PluginFinishType eType)
     emit signalJobFinshed();
 }
 
+void SingleJob::slotError(const QString &message, const QString &details)
+{
+    Q_UNUSED(details);
+    // 传递临时消息信号，用于显示但不中断操作的非致命错误提示
+    emit signalTempMessage(message);
+}
+
 // 加载操作
 LoadJob::LoadJob(ReadOnlyArchiveInterface *pInterface, QObject *parent)
     : SingleJob(pInterface, parent)
@@ -361,6 +369,7 @@ OpenJob::OpenJob(const FileEntry &stEntry, const QString &strTempExtractPath, co
     m_eJobType = JT_Open;
     connect(m_pInterface, &ReadOnlyArchiveInterface::signalFinished, this, &OpenJob::slotFinished, Qt::ConnectionType::UniqueConnection);
     connect(m_pInterface, &ReadOnlyArchiveInterface::signalQuery, this, &SingleJob::signalQuery, Qt::ConnectionType::AutoConnection);
+    connect(m_pInterface, &ReadOnlyArchiveInterface::error, this, &SingleJob::slotError, Qt::AutoConnection);
 }
 
 OpenJob::~OpenJob()
diff --git a/src/source/archivemanager/singlejob.h b/src/source/archivemanager/singlejob.h
@@ -70,6 +70,13 @@ class SingleJob : public ArchiveJob
      */
     bool status() override;
 
+    /**
+     * @brief slotError  处理错误消息（不中断操作）
+     * @param message 错误消息
+     * @param details 详细信息
+     */
+    void slotError(const QString &message, const QString &details);
+
     SingleJobThread *getdptr();
 protected:
     /**
diff --git a/src/source/mainwindow.cpp b/src/source/mainwindow.cpp
@@ -278,6 +278,7 @@ void MainWindow::initConnections()
     connect(ArchiveManager::get_instance(), &ArchiveManager::signalFileWriteErrorName, this, &MainWindow::slotReceiveFileWriteErrorName);
     connect(ArchiveManager::get_instance(), &ArchiveManager::signalCurArchiveName, this, &MainWindow::slotReceiveCurArchiveName);
     connect(ArchiveManager::get_instance(), &ArchiveManager::signalQuery, this, &MainWindow::slotQuery);
+    connect(ArchiveManager::get_instance(), &ArchiveManager::signalTempMessage, this, &MainWindow::slotReceiveTempMessage);
 
     connect(m_pOpenFileWatcher, &OpenFileWatcher::fileChanged, this, &MainWindow::slotOpenFileChanged);
     //定制需求不显示ctrl+shift+？快捷键菜单
@@ -1262,6 +1263,14 @@ void MainWindow::slotReceiveFileWriteErrorName(const QString &strName)
     m_fileWriteErrorName = strName;
 }
 
+void MainWindow::slotReceiveTempMessage(const QString &strMessage)
+{
+    qInfo() << "Temp message:" << strMessage;
+    QIcon icon = UiTools::renderSVG(":assets/icons/deepin/builtin/icons/compress_fail_128px.svg", QSize(30, 30));
+    // 显示 2 秒临时消息
+    sendMessage(new CustomFloatingMessage(icon, strMessage, 2000, this));
+}
+
 void MainWindow::slotQuery(Query *query)
 {
     qInfo() << " query->execute()";
diff --git a/src/source/mainwindow.h b/src/source/mainwindow.h
@@ -415,6 +415,12 @@ private Q_SLOTS:
      */
     void slotReceiveFileWriteErrorName(const QString &strName);
 
+    /**
+     * @brief slotReceiveTempMessage    接收临时消息
+     * @param strMessage    消息内容
+     */
+    void slotReceiveTempMessage(const QString &strMessage);
+
     /**
      * @brief slotQuery   发送询问信号
      * @param query 询问类型

Original file line number	Diff line number	Diff line change
`@@ -284,13 +284,14 @@ bool CliRarPlugin::handleLine(const QString &line, WorkType workStatus)`
`284`	`284`	`}`
`285`	`285`
`286`	`286`	`if (isWrongPasswordMsg(line)) { // 提示密码错误`
	`287`	`+ m_eErrorType = ET_WrongPassword;`
	`288`	`+`
`287`	`289`	`// RAR4密码错误直接结束`
`288`	`290`	`if (line.startsWith(QLatin1String("Checksum error in the encrypted file"))) {`
`289`		`- m_eErrorType = ET_WrongPassword;`
`290`	`291`	`m_finishType = PFT_Error;`
`291`	`292`	`return false;`
`292`		`- } else { // RAR5密码错误反复输入新密码`
`293`		`- m_eErrorType = ET_WrongPassword;`
	`293`	`+ } else { // RAR5密码错误：发送错误提示但不中断进程，允许用户重新输入密码`
	`294`	`+ emit error(tr("Wrong password"), tr("The password entered is incorrect. Please try again."));`
`294`	`295`	`return true;`
`295`	`296`	`}`
`296`	`297`	`}`