Changeset 102814 in vbox

Timestamp:

Jan 10, 2024 1:10:55 PM (15 months ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

161002

Message:

FE/Qt: A bit of cleanup for UIVirtualBoxManagerWidget; Split doxy stuff related to Common and Tools section into separate sections; A bit of corresponding reordering.

Location:

trunk/src/VBox/Frontends/VirtualBox/src/manager

Files:

• UIVirtualBoxManagerWidget.cpp (modified) (2 diffs)
• UIVirtualBoxManagerWidget.h (modified) (2 diffs)

trunk/src/VBox/Frontends/VirtualBox/src/manager/UIVirtualBoxManagerWidget.cpp

@@ -1083 +1083 @@
 }
 
+void UIVirtualBoxManagerWidget::recacheCurrentMachineItemInformation(bool fDontRaiseErrorPane /* = false */)
+{
+    /* Sanity check, this method is for machine or group of machine items: */
+    if (!isMachineItemSelected() && !isGroupItemSelected())
+        return;
+
+    /* Get current item: */
+    UIVirtualMachineItem *pItem = currentItem();
+    const bool fCurrentItemIsOk = pItem && pItem->accessible();
+
+    /* If current item is Ok: */
+    if (fCurrentItemIsOk)
+    {
+        /* If Error-pane is chosen currently => switch to tool currently chosen in Tools-menu: */
+        if (m_pPaneToolsMachine->currentTool() == UIToolType_Error)
+            switchMachineToolTo(m_pMenuToolsMachine->toolsType());
+
+        /* Propagate current items to the Tools pane: */
+        m_pPaneToolsMachine->setItems(currentItems());
+    }
+    /* Otherwise if we were not asked separately to calm down: */
+    else if (!fDontRaiseErrorPane)
+    {
+        /* Make sure Error pane raised: */
+        if (m_pPaneToolsMachine->currentTool() != UIToolType_Error)
+            m_pPaneToolsMachine->openTool(UIToolType_Error);
+
+        /* Propagate last access error to the Error-pane: */
+        if (pItem)
+            m_pPaneToolsMachine->setErrorDetails(pItem->accessError());
+    }
+}
+
 void UIVirtualBoxManagerWidget::updateToolsMenuGlobal()
 {
@@ -1126 +1159 @@
         m_pPaneToolsMachine->closeTool(enmRestrictedType);
 }
-
-void UIVirtualBoxManagerWidget::recacheCurrentMachineItemInformation(bool fDontRaiseErrorPane /* = false */)
-{
-    /* Sanity check, this method is for machine or group of machine items: */
-    if (!isMachineItemSelected() && !isGroupItemSelected())
-        return;
-
-    /* Get current item: */
-    UIVirtualMachineItem *pItem = currentItem();
-    const bool fCurrentItemIsOk = pItem && pItem->accessible();
-
-    /* If current item is Ok: */
-    if (fCurrentItemIsOk)
-    {
-        /* If Error-pane is chosen currently => switch to tool currently chosen in Tools-menu: */
-        if (m_pPaneToolsMachine->currentTool() == UIToolType_Error)
-            switchMachineToolTo(m_pMenuToolsMachine->toolsType());
-
-        /* Propagate current items to the Tools pane: */
-        m_pPaneToolsMachine->setItems(currentItems());
-    }
-    /* Otherwise if we were not asked separately to calm down: */
-    else if (!fDontRaiseErrorPane)
-    {
-        /* Make sure Error pane raised: */
-        if (m_pPaneToolsMachine->currentTool() != UIToolType_Error)
-            m_pPaneToolsMachine->openTool(UIToolType_Error);
-
-        /* Propagate last access error to the Error-pane: */
-        if (pItem)
-            m_pPaneToolsMachine->setErrorDetails(pItem->accessError());
-    }
-}

trunk/src/VBox/Frontends/VirtualBox/src/manager/UIVirtualBoxManagerWidget.h

@@ -361 +361 @@
     /** @} */
 
-    /** @name Tools / Common stuff.
+    /** @name Common stuff.
+      * @{ */
+        /** Recaches current machine item information.
+          * @param  fDontRaiseErrorPane  Brings whether we should not raise error-pane. */
+        void recacheCurrentMachineItemInformation(bool fDontRaiseErrorPane = false);
+    /** @} */
+
+    /** @name Tools stuff.
       * @{ */
         /** Updates Global tools menu. */
@@ -367 +374 @@
         /** Updates Machine tools menu for @a pItem specified. */
         void updateToolsMenuMachine(UIVirtualMachineItem *pItem);
-
-        /** Recaches current machine item information.
-          * @param  fDontRaiseErrorPane  Brings whether we should not raise error-pane. */
-        void recacheCurrentMachineItemInformation(bool fDontRaiseErrorPane = false);
     /** @} */