diff options
| author | Niels Weber <niels.weber@digia.com> | 2014-11-24 14:34:41 +0100 |
|---|---|---|
| committer | Niels Weber <niels.weber@theqtcompany.com> | 2014-12-05 13:09:32 +0100 |
| commit | 587bc6fb7c905422c3f09bc8ea2ac91c91d25cc8 (patch) | |
| tree | c094b432ae8130ab7b77ab69e2f14de265c115e2 /src/libs/kdtools/kdupdaterpackagesinfo.cpp | |
| parent | 296c5d5b61ab0e5ce43bdb30f4c872a1db5b1483 (diff) | |
Don't set modified state without real modification
setApplicationName and setApplicationVersion probably
should not set the modified state to true as they are called
way before actual installation is started. In case of real
changes happening, modified will always be set to true by the
other changes anyway.
Without this change, an existing installation in the default
installation directory of the new installer might be destroyed
when aborting the new installer. This should probably be investigated
further so that nothing is ever written before the real install
is started.
Task-number: QTIFW-583
Change-Id: I9921a33134133ae28433c4bfb726fe79ba93c20e
Reviewed-by: Kai Koehne <kai.koehne@theqtcompany.com>
Diffstat (limited to 'src/libs/kdtools/kdupdaterpackagesinfo.cpp')
| -rw-r--r-- | src/libs/kdtools/kdupdaterpackagesinfo.cpp | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/src/libs/kdtools/kdupdaterpackagesinfo.cpp b/src/libs/kdtools/kdupdaterpackagesinfo.cpp index bae243bc5..6eb32f1e3 100644 --- a/src/libs/kdtools/kdupdaterpackagesinfo.cpp +++ b/src/libs/kdtools/kdupdaterpackagesinfo.cpp @@ -179,7 +179,6 @@ QString PackagesInfo::fileName() const void PackagesInfo::setApplicationName(const QString &name) { d->applicationName = name; - d->modified = true; } /*! @@ -197,7 +196,6 @@ QString PackagesInfo::applicationName() const void PackagesInfo::setApplicationVersion(const QString &version) { d->applicationVersion = version; - d->modified = true; } /*! |