diff options
author | Katja Marttila <katja.marttila@qt.io> | 2020-08-25 13:00:58 +0300 |
---|---|---|
committer | Katja Marttila <katja.marttila@qt.io> | 2020-08-26 11:05:24 +0300 |
commit | eedb0ee312c092a623f5e55bcaf94788290161bb (patch) | |
tree | 33b268d829e7627db06b935272a8c5fb4f97c996 /src/libs/installer/packagemanagergui.cpp | |
parent | 2efc27fc698e18afc98f1c414e5519cf455d95c2 (diff) |
Fix crash when fetching categories
Component scripts are re-evaluated when new categories are fetched.
Remove refereces to already added custom widgets from pages as those
are re-added when scripts are re-evaluated.
Task-number: QTIFW-1928
Change-Id: Iad52232920eaf3859d710d6fe06e80eeee90797d
Reviewed-by: Arttu Tarkiainen <arttu.tarkiainen@qt.io>
Diffstat (limited to 'src/libs/installer/packagemanagergui.cpp')
-rw-r--r-- | src/libs/installer/packagemanagergui.cpp | 18 |
1 files changed, 18 insertions, 0 deletions
diff --git a/src/libs/installer/packagemanagergui.cpp b/src/libs/installer/packagemanagergui.cpp index 26af00bdc..8c0877d3e 100644 --- a/src/libs/installer/packagemanagergui.cpp +++ b/src/libs/installer/packagemanagergui.cpp @@ -828,6 +828,12 @@ void PackageManagerGui::wizardWidgetInsertionRequested(QWidget *widget, void PackageManagerGui::wizardWidgetRemovalRequested(QWidget *widget) { Q_ASSERT(widget); + + const QList<int> pages = pageIds(); + foreach (int id, pages) { + PackageManagerPage *managerPage = qobject_cast<PackageManagerPage *>(page(id)); + managerPage->removeCustomWidget(widget); + } widget->setParent(nullptr); packageManagerCore()->controlScriptEngine()->removeFromGlobalObject(widget); packageManagerCore()->componentScriptEngine()->removeFromGlobalObject(widget); @@ -1357,6 +1363,18 @@ bool PackageManagerPage::validatePage() } /*! + \internal +*/ +void PackageManagerPage::removeCustomWidget(const QWidget *widget) +{ + for (auto it = m_customWidgets.begin(); it != m_customWidgets.end();) { + if (it.value() == widget) + it = m_customWidgets.erase(it); + else + ++it; + } +} +/*! Inserts \a widget at the position specified by \a offset in relation to another widget specified by \a siblingName. The default position is directly behind the sibling. |