Changeset 50326 in vbox for trunk/src/VBox/Frontends/VirtualBox

Timestamp:

Feb 5, 2014 12:40:02 PM (11 years ago)

Author:

vboxsync

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

92048

Message:

FE/Qt: 4397: Medium Manager cleanup/rework (part 2): Actions-update functionality decomposition.

Location:

trunk/src/VBox/Frontends/VirtualBox/src/medium

Files:

• UIMediumManager.cpp (modified) (2 diffs)
• UIMediumManager.h (modified) (1 diff)

trunk/src/VBox/Frontends/VirtualBox/src/medium/UIMediumManager.cpp

@@ -604 +604 @@
     }
 
-    /* Update action availability: */
-    bool fNotInEnumeration = !vboxGlobal().isMediumEnumerationInProgress();
-    bool fActionEnabledCopy    = currentTreeWidgetType() == UIMediumType_HardDisk &&
-                                 fNotInEnumeration && pMediumItem && checkMediumFor(pMediumItem, Action_Copy);
-    bool fActionEnabledModify  = currentTreeWidgetType() == UIMediumType_HardDisk &&
-                                 fNotInEnumeration && pMediumItem && checkMediumFor(pMediumItem, Action_Modify);
-    bool fActionEnabledRemove  = fNotInEnumeration && pMediumItem && checkMediumFor(pMediumItem, Action_Remove);
-    bool fActionEnabledRelease = pMediumItem && checkMediumFor(pMediumItem, Action_Release);
-    m_pActionCopy->setEnabled(fActionEnabledCopy);
-    m_pActionModify->setEnabled(fActionEnabledModify);
-    m_pActionRemove->setEnabled(fActionEnabledRemove);
-    m_pActionRelease->setEnabled(fActionEnabledRelease);
+    /* Update actions: */
+    updateActions();
 
     /* Update information-panes: */
@@ -1087 +1077 @@
 }
 
+void UIMediumManager::updateActions()
+{
+    /* Get current-item: */
+    UIMediumItem *pCurrentItem = toMediumItem(currentTreeWidget()->currentItem());
+
+    /* Calculate actions accessibility: */
+    bool fNotInEnumeration = !vboxGlobal().isMediumEnumerationInProgress();
+    bool fActionEnabledCopy = currentTreeWidgetType() == UIMediumType_HardDisk &&
+                              fNotInEnumeration && pCurrentItem && checkMediumFor(pCurrentItem, Action_Copy);
+    bool fActionEnabledModify = currentTreeWidgetType() == UIMediumType_HardDisk &&
+                                fNotInEnumeration && pCurrentItem && checkMediumFor(pCurrentItem, Action_Modify);
+    bool fActionEnabledRemove = fNotInEnumeration && pCurrentItem && checkMediumFor(pCurrentItem, Action_Remove);
+    bool fActionEnabledRelease = fNotInEnumeration && pCurrentItem && checkMediumFor(pCurrentItem, Action_Release);
+
+    /* Update actions: */
+    m_pActionCopy->setEnabled(fActionEnabledCopy);
+    m_pActionModify->setEnabled(fActionEnabledModify);
+    m_pActionRemove->setEnabled(fActionEnabledRemove);
+    m_pActionRelease->setEnabled(fActionEnabledRelease);
+}
+
 #ifdef Q_WS_MAC
 void UIMediumManager::cleanupMacWindowMenu()

trunk/src/VBox/Frontends/VirtualBox/src/medium/UIMediumManager.h

@@ -121 +121 @@
     void populateTreeWidgets();
 
+    /** Creates UIMediumItem for corresponding UIMedium. */
+    void updateActions();
+
     /* Helpers: Cleanup stuff: */
 #ifdef Q_WS_MAC