diff options
author | Robert Griebl <robert.griebl@qt.io> | 2022-03-24 12:53:38 +0100 |
---|---|---|
committer | Qt Cherry-pick Bot <cherrypick_bot@qt-project.org> | 2022-03-25 07:25:40 +0000 |
commit | e5bdbb08eb75d3ed9b0a55932273230221b2f0a4 (patch) | |
tree | 38b38de239f860cb1180aeba9cc196ef979fc87f | |
parent | a056e53da86cd61fec962ec1821a1dfca050a8e1 (diff) | |
download | qtapplicationmanager-e5bdbb08eb75d3ed9b0a55932273230221b2f0a4.tar.gz |
Really fix package removal if documentDir is not set
The fix in bc06a04dd845d039ba802f94b5eae57519ec63a8 was not correct in
the package removal case.
Change-Id: I65645a1895b6e5cba6dfa6bd61be1b1a638fe67a
Reviewed-by: Bernd Weimer <bernd.weimer@qt.io>
(cherry picked from commit 074d1a70d273b72a369009a6105a7dcad2efc48b)
Reviewed-by: Qt Cherry-pick Bot <cherrypick_bot@qt-project.org>
-rw-r--r-- | src/manager-lib/deinstallationtask.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/manager-lib/deinstallationtask.cpp b/src/manager-lib/deinstallationtask.cpp index 4de582a1..23bc0204 100644 --- a/src/manager-lib/deinstallationtask.cpp +++ b/src/manager-lib/deinstallationtask.cpp @@ -114,7 +114,7 @@ void DeinstallationTask::execute() ScopedRenamer docDirRename; ScopedRenamer appDirRename; - if (!m_keepDocuments || !m_documentPath.isEmpty()) { + if (!m_keepDocuments && !m_documentPath.isEmpty()) { if (!docDirRename.rename(QDir(m_documentPath).absoluteFilePath(packageId()), ScopedRenamer::NameToNameMinus)) { throw Exception(Error::IO, "could not rename %1 to %1-").arg(docDirRename.baseName()); |