Changeset 86746 in vbox for trunk/src/VBox/Frontends

Timestamp:

Oct 28, 2020 5:44:51 PM (4 years ago)

Author:

vboxsync

Message:

FE/Qt: bugref:9653: VirtualBox Manager: Chooser pane: For for cloud provider uninstall logic; Cloud provider node can be removed much smarter way.

File:

• trunk/src/VBox/Frontends/VirtualBox/src/manager/chooser/UIChooserAbstractModel.cpp (modified) (2 diffs)

trunk/src/VBox/Frontends/VirtualBox/src/manager/chooser/UIChooserAbstractModel.cpp

@@ -730 +730 @@
 void UIChooserAbstractModel::sltHandleCloudProviderUninstall(const QUuid &uProviderId)
 {
-    /* Search for top-level provider node: */
-    foreach (UIChooserNode *pNode, m_pInvisibleRootNode->nodes(UIChooserNodeType_Group))
-    {
-        /* Skip unrelated nodes: */
-        AssertPtrReturnVoid(pNode);
-        UIChooserNodeGroup *pGroupNode = pNode->toGroupNode();
-        AssertPtrReturnVoid(pGroupNode);
-        if (pGroupNode->groupType() != UIChooserNodeGroupType_Provider)
-            continue;
-        const QUuid uIteratedId = pGroupNode->property("id").toUuid();
-        AssertReturnVoid(!uIteratedId.isNull());
-        if (uIteratedId != uProviderId)
-            continue;
-
-        /* Remove found provider node: */
-        delete pNode;
-    }
+    /* Search and delete corresponding cloud provider node if present: */
+    delete searchProviderNode(uProviderId);
 }
 
@@ -1085 +1070 @@
                                                            UIChooserNodeDataPrefixType_Provider,
                                                            strProviderShortName));
-        pProviderNode->setProperty("id", uProviderId);
 
         /* Iterate through provider's profiles: */