From 90900fc95227045b2cbc1b4a2dcddcca9199e8fd Mon Sep 17 00:00:00 2001 From: Eike Ziller Date: Tue, 4 Oct 2022 11:26:12 +0200 Subject: [PATCH] UpdateInfo: Ignore multiple update check requests The existing code would kill the currently running process, but - It's unclear why it should do that. If the user wants to cancel the currently running one, they can do that in any case. - The existing code killed the current maintenancetool, but did not cancel the task in the progress manager. Task-number: QTCREATORBUG-28236 Change-Id: I06c2d6b733d637e38cae120dc1a375f7952bfa85 Reviewed-by: Jarek Kobus --- src/plugins/updateinfo/updateinfoplugin.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/plugins/updateinfo/updateinfoplugin.cpp b/src/plugins/updateinfo/updateinfoplugin.cpp index 78d31e72910..520dff4cc7a 100644 --- a/src/plugins/updateinfo/updateinfoplugin.cpp +++ b/src/plugins/updateinfo/updateinfoplugin.cpp @@ -140,7 +140,8 @@ void UpdateInfoPlugin::doAutoCheckForUpdates() void UpdateInfoPlugin::startCheckForUpdates() { - stopCheckForUpdates(); + if (d->m_maintenanceToolProcess) + return; // do not trigger while update task is already running QFutureInterface futureIf; FutureProgress *futureProgress