From c88c639b8efdcdfe87c2ce44fa270889826b38ef Mon Sep 17 00:00:00 2001
From: Jan Dalheimer <jan@dalheimer.de>
Date: Sun, 16 Feb 2014 11:49:55 +0100
Subject: [PATCH] Fix for windows and update tool menu after closing settings
 dialog

---
 gui/MainWindow.cpp   | 88 +++++++++++++++++++++++---------------------
 gui/MainWindow.h     |  2 +
 logic/MCEditTool.cpp | 14 +++++--
 3 files changed, 60 insertions(+), 44 deletions(-)

diff --git a/gui/MainWindow.cpp b/gui/MainWindow.cpp
index 3f4690618..16c882015 100644
--- a/gui/MainWindow.cpp
+++ b/gui/MainWindow.cpp
@@ -360,6 +360,51 @@ void MainWindow::showInstanceContextMenu(const QPoint &pos)
 	myMenu.exec(view->mapToGlobal(pos));
 }
 
+void MainWindow::updateToolsMenu()
+{
+	if (ui->actionLaunchInstance->menu())
+	{
+		ui->actionLaunchInstance->menu()->deleteLater();
+	}
+	QMenu *launchMenu = new QMenu(this);
+	QAction *normalLaunch = launchMenu->addAction(tr("Launch"));
+	connect(normalLaunch, &QAction::triggered, [this](){doLaunch();});
+	launchMenu->addSeparator()->setText(tr("Profilers"));
+	for (auto profiler : MMC->profilers().values())
+	{
+		QAction *profilerAction = launchMenu->addAction(profiler->name());
+		QString error;
+		if (!profiler->check(&error))
+		{
+			profilerAction->setDisabled(true);
+			profilerAction->setToolTip(tr("Profiler not setup correctly. Go into settings, \"External Tools\"."));
+		}
+		else
+		{
+			connect(profilerAction, &QAction::triggered, [this, profiler](){doLaunch(true, profiler.get());});
+		}
+	}
+	launchMenu->addSeparator()->setText(tr("Tools"));
+	for (auto tool : MMC->tools().values())
+	{
+		QAction *toolAction = launchMenu->addAction(tool->name());
+		QString error;
+		if (!tool->check(&error))
+		{
+			toolAction->setDisabled(true);
+			toolAction->setToolTip(tr("Tool not setup correctly. Go into settings, \"External Tools\"."));
+		}
+		else
+		{
+			connect(toolAction, &QAction::triggered, [this, tool]()
+			{
+				tool->createDetachedTool(m_selectedInstance, this)->run();
+			});
+		}
+	}
+	ui->actionLaunchInstance->setMenu(launchMenu);
+}
+
 void MainWindow::repopulateAccountsMenu()
 {
 	accountMenu->clear();
@@ -933,6 +978,7 @@ void MainWindow::on_actionSettings_triggered()
 	// FIXME: quick HACK to make this work. improve, optimize.
 	proxymodel->invalidate();
 	proxymodel->sort(0);
+	updateToolsMenu();
 }
 
 void MainWindow::on_actionManageAccounts_triggered()
@@ -1429,47 +1475,7 @@ void MainWindow::instanceChanged(const QModelIndex &current, const QModelIndex &
 		m_statusLeft->setText(m_selectedInstance->getStatusbarDescription());
 		updateInstanceToolIcon(m_selectedInstance->iconKey());
 
-		if (ui->actionLaunchInstance->menu())
-		{
-			ui->actionLaunchInstance->menu()->deleteLater();
-		}
-		QMenu *launchMenu = new QMenu;
-		QAction *normalLaunch = launchMenu->addAction(tr("Launch"));
-		connect(normalLaunch, &QAction::triggered, [this](){doLaunch();});
-		launchMenu->addSeparator()->setText(tr("Profilers"));
-		for (auto profiler : MMC->profilers().values())
-		{
-			QAction *profilerAction = launchMenu->addAction(profiler->name());
-			QString error;
-			if (!profiler->check(&error))
-			{
-				profilerAction->setDisabled(true);
-				profilerAction->setToolTip(tr("Profiler not setup correctly. Go into settings, \"External Tools\"."));
-			}
-			else
-			{
-				connect(profilerAction, &QAction::triggered, [this, profiler](){doLaunch(true, profiler.get());});
-			}
-		}
-		launchMenu->addSeparator()->setText(tr("Tools"));
-		for (auto tool : MMC->tools().values())
-		{
-			QAction *toolAction = launchMenu->addAction(tool->name());
-			QString error;
-			if (!tool->check(&error))
-			{
-				toolAction->setDisabled(true);
-				toolAction->setToolTip(tr("Tool not setup correctly. Go into settings, \"External Tools\"."));
-			}
-			else
-			{
-				connect(toolAction, &QAction::triggered, [this, tool]()
-				{
-					tool->createDetachedTool(m_selectedInstance, this)->run();
-				});
-			}
-		}
-		ui->actionLaunchInstance->setMenu(launchMenu);
+		updateToolsMenu();
 
 		MMC->settings()->set("SelectedInstance", m_selectedInstance->id());
 	}
diff --git a/gui/MainWindow.h b/gui/MainWindow.h
index 682c711d6..7a29977da 100644
--- a/gui/MainWindow.h
+++ b/gui/MainWindow.h
@@ -141,6 +141,8 @@ slots:
 
 	void showInstanceContextMenu(const QPoint&);
 
+	void updateToolsMenu();
+
 public
 slots:
 	void instanceActivated(QModelIndex);
diff --git a/logic/MCEditTool.cpp b/logic/MCEditTool.cpp
index 93ffb7a5a..14dc1b96b 100644
--- a/logic/MCEditTool.cpp
+++ b/logic/MCEditTool.cpp
@@ -23,8 +23,16 @@ void MCEditTool::runImpl()
 	{
 		return;
 	}
-	const QString program =
-		QDir(mceditPath).absoluteFilePath("mcedit.py");
+	QDir mceditDir(mceditPath);
+	QString program;
+	if (mceditDir.exists("mcedit.py"))
+	{
+		program = mceditDir.absoluteFilePath("mcedit.py");
+	}
+	else if (mceditDir.exists("mcedit.exe"))
+	{
+		program = mceditDir.absoluteFilePath("mcedit.exe");
+	}
 	QProcess::startDetached(program, QStringList() << save, mceditPath);
 }
 
@@ -53,7 +61,7 @@ bool MCEditFactory::check(const QString &path, QString *error)
 		*error = QObject::tr("Path does not exist");
 		return false;
 	}
-	if (!dir.exists("mcedit.py"))
+	if (!dir.exists("mcedit.py") && !dir.exists("mcedit.exe"))
 	{
 		*error = QObject::tr("Path does not contain mcedit.py");
 		return false;