WX: Disable a few menu items when a Wii title is running

Unsafe and keeping them enabled would allow inaccurate behaviour that
can break games.

1 files changed, 16 insertions, 1 deletions

diff --git a/Source/Core/DolphinWX/MainMenuBar.cpp b/Source/Core/DolphinWX/MainMenuBar.cpp
index 3fda28cdf6..3dc8736d14 100644
--- a/Source/Core/DolphinWX/MainMenuBar.cpp
+++ b/Source/Core/DolphinWX/MainMenuBar.cpp
@@ -514,7 +514,7 @@ void MainMenuBar::RefreshMenuLabels() const
 {
   RefreshPlayMenuLabel();
   RefreshSaveStateMenuLabels();
-  RefreshWiiSystemMenuLabel();
+  RefreshWiiToolsLabels();
 }
 
 void MainMenuBar::RefreshPlayMenuLabel() const
@@ -545,6 +545,21 @@ void MainMenuBar::RefreshSaveStateMenuLabels() const
   }
 }
 
+void MainMenuBar::RefreshWiiToolsLabels() const
+{
+  RefreshWiiSystemMenuLabel();
+
+  // The Install WAD option should not be enabled while emulation is running, because
+  // having unexpected title changes can confuse emulated software; and of course, this is
+  // not possible on a real Wii and won't be if we have IOS LLE (or simply more accurate IOS HLE).
+  //
+  // For similar reasons, it should not be possible to export or import saves, because this can
+  // result in the emulated software being confused, or even worse, exported saves having
+  // inconsistent data.
+  for (const int index : {IDM_MENU_INSTALL_WAD, IDM_EXPORT_ALL_SAVE, IDM_IMPORT_SAVE})
+    FindItem(index)->Enable(!Core::IsRunning() || !SConfig::GetInstance().bWii);
+}
+
 void MainMenuBar::RefreshWiiSystemMenuLabel() const
 {
   auto* const item = FindItem(IDM_LOAD_WII_MENU);