IsUninstaller() was evaluated to early.

Fixes the problem that the button and label was not shown
in case the tool started as uninstaller and later switched
to package manager or updater mode. This happens because
of the initalization order of the pages.

Reviewed-by: Niels Weber

1 files changed, 2 insertions, 5 deletions

diff --git a/installerbuilder/libinstaller/packagemanagergui.cpp b/installerbuilder/libinstaller/packagemanagergui.cpp
index a692556af..b9f3818bd 100644
--- a/installerbuilder/libinstaller/packagemanagergui.cpp
+++ b/installerbuilder/libinstaller/packagemanagergui.cpp
@@ -1439,11 +1439,6 @@ ReadyForInstallationPage::ReadyForInstallationPage(PackageManagerCore *core)
     bottomLayout->setStretch(1, 10);
     baseLayout->addLayout(bottomLayout);
 
-    if (packageManagerCore()->isUninstaller()) {
-        m_taskDetailsButton->setVisible(false);
-        m_taskDetailsBrowser->setVisible(false);
-    }
-
     setLayout(baseLayout);
 }
 
@@ -1457,6 +1452,8 @@ void ReadyForInstallationPage::entering()
     const QString target = packageManagerCore()->value(scTargetDir);
 
     if (packageManagerCore()->isUninstaller()) {
+        m_taskDetailsButton->setVisible(false);
+        m_taskDetailsBrowser->setVisible(false);
         setButtonText(QWizard::CommitButton, tr("U&ninstall"));
         setTitle(titleForPage(objectName(), tr("Ready to Uninstall")));
         m_msgLabel->setText(tr("Setup is now ready to begin removing %1 from your computer.<br>"