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

Timestamp:

Nov 2, 2018 3:02:29 PM (6 years ago)

Author:

vboxsync

Message:

FE/Qt: bugref:9241: VirtualBox Manager: Tools pane: On setCurrentItem we should update last item in any case, not just when actual class of items is chosen.

File:

• trunk/src/VBox/Frontends/VirtualBox/src/manager/tools/UIToolsModel.cpp (modified) (1 diff)

trunk/src/VBox/Frontends/VirtualBox/src/manager/tools/UIToolsModel.cpp

@@ -202 +202 @@
         /* Set this item to current if navigation list contains it: */
         if (navigationList().contains(pItem))
+            m_pCurrentItem = pItem;
+        /* Update last item in any case: */
+        switch (pItem->itemClass())
         {
-            m_pCurrentItem = pItem;
-            switch (m_pCurrentItem->itemClass())
-            {
-                case UIToolClass_Global:  m_pLastItemGlobal  = m_pCurrentItem; break;
-                case UIToolClass_Machine: m_pLastItemMachine = m_pCurrentItem; break;
-                default: break;
-            }
+            case UIToolClass_Global:  m_pLastItemGlobal  = pItem; break;
+            case UIToolClass_Machine: m_pLastItemMachine = pItem; break;
+            default: break;
         }
-        /* Otherwise it's error: */
-        else
-            AssertMsgFailed(("Passed item is not in navigation list!"));
     }
     /* Otherwise reset current item: */