Fix more stuff. Detached tools, only MCEdit for now.

17 files changed, 307 insertions, 44 deletions

diff --git a/CMakeLists.txt b/CMakeLists.txt
index 2dd5dde5..58d01eb8 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -499,7 +499,11 @@ logic/assets/AssetsMigrateTask.cpp
 logic/assets/AssetsUtils.h
 logic/assets/AssetsUtils.cpp
 
-# Profiling
+# Profiling and tools
+logic/BaseExternalTool.h
+logic/BaseExternalTool.cpp
+logic/MCEditTool.h
+logic/MCEditTool.cpp
 logic/profiler/BaseProfiler.h
 logic/profiler/BaseProfiler.cpp
 logic/profiler/JProfiler.h
diff --git a/MultiMC.cpp b/MultiMC.cpp
index e3f60049..c813c425 100644
--- a/MultiMC.cpp
+++ b/MultiMC.cpp
@@ -33,6 +33,7 @@
 
 #include "logic/profiler/JProfiler.h"
 #include "logic/profiler/JVisualVM.h"
+#include "logic/MCEditTool.h"
 
 #include "pathutils.h"
 #include "cmdutils.h"
@@ -223,6 +224,12 @@ MultiMC::MultiMC(int &argc, char **argv, bool root_override)
 	{
 		profiler->registerSettings(m_settings.get());
 	}
+	m_tools.insert("mcedit",
+				   std::shared_ptr<BaseDetachedToolFactory>(new MCEditFactory()));
+	for (auto tool : m_tools.values())
+	{
+		tool->registerSettings(m_settings.get());
+	}
 
 	// launch instance, if that's what should be done
 	// WARNING: disabled until further notice
diff --git a/MultiMC.h b/MultiMC.h
index 854de59b..753ab3b4 100644
--- a/MultiMC.h
+++ b/MultiMC.h
@@ -23,6 +23,7 @@ class NotificationChecker;
 class NewsChecker;
 class StatusChecker;
 class BaseProfilerFactory;
+class BaseDetachedToolFactory;
 
 #if defined(MMC)
 #undef MMC
@@ -132,6 +133,10 @@ public:
 	{
 		return m_profilers;
 	}
+	QMap<QString, std::shared_ptr<BaseDetachedToolFactory>> tools()
+	{
+		return m_tools;
+	}
 
 	void installUpdates(const QString updateFilesDir, UpdateFlags flags = None);
 
@@ -205,6 +210,7 @@ private:
 	std::shared_ptr<MinecraftVersionList> m_minecraftlist;
 	std::shared_ptr<JavaVersionList> m_javalist;
 	QMap<QString, std::shared_ptr<BaseProfilerFactory>> m_profilers;
+	QMap<QString, std::shared_ptr<BaseDetachedToolFactory>> m_tools;
 	QsLogging::DestinationPtr m_fileDestination;
 	QsLogging::DestinationPtr m_debugDestination;
 
diff --git a/gui/MainWindow.cpp b/gui/MainWindow.cpp
index b05c58f7..3f469061 100644
--- a/gui/MainWindow.cpp
+++ b/gui/MainWindow.cpp
@@ -1440,7 +1440,34 @@ void MainWindow::instanceChanged(const QModelIndex &current, const QModelIndex &
 		for (auto profiler : MMC->profilers().values())
 		{
 			QAction *profilerAction = launchMenu->addAction(profiler->name());
-			connect(profilerAction, &QAction::triggered, [this, profiler](){doLaunch(true, profiler.get());});
+			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);
 
diff --git a/gui/dialogs/SettingsDialog.cpp b/gui/dialogs/SettingsDialog.cpp
index 7afb6565..9423d7eb 100644
--- a/gui/dialogs/SettingsDialog.cpp
+++ b/gui/dialogs/SettingsDialog.cpp
@@ -374,6 +374,7 @@ void SettingsDialog::applySettings(SettingsObject *s)
 	// Profilers
 	s->set("JProfilerPath", ui->jprofilerPathEdit->text());
 	s->set("JVisualVMPath", ui->jvisualvmPathEdit->text());
+	s->set("MCEditPath", ui->mceditPathEdit->text());
 }
 
 void SettingsDialog::loadSettings(SettingsObject *s)
@@ -457,6 +458,7 @@ void SettingsDialog::loadSettings(SettingsObject *s)
 	// Profilers
 	ui->jprofilerPathEdit->setText(s->get("JProfilerPath").toString());
 	ui->jvisualvmPathEdit->setText(s->get("JVisualVMPath").toString());
+	ui->mceditPathEdit->setText(s->get("MCEditPath").toString());
 }
 
 void SettingsDialog::on_javaDetectBtn_clicked()
@@ -565,3 +567,29 @@ void SettingsDialog::on_jvisualvmCheckBtn_clicked()
 		QMessageBox::information(this, tr("OK"), tr("JVisualVM setup seems to be OK"));
 	}
 }
+
+void SettingsDialog::on_mceditPathBtn_clicked()
+{
+	QString raw_dir = QFileDialog::getOpenFileName(this, tr("MCEdit Path"),
+												   ui->jvisualvmPathEdit->text());
+	QString cooked_dir = NormalizePath(raw_dir);
+
+	// do not allow current dir - it's dirty. Do not allow dirs that don't exist
+	if (!cooked_dir.isEmpty() && QDir(cooked_dir).exists())
+	{
+		ui->mceditPathEdit->setText(cooked_dir);
+	}
+}
+void SettingsDialog::on_mceditCheckBtn_clicked()
+{
+	QString error;
+	if (!MMC->tools()["mcedit"]->check(ui->mceditPathEdit->text(), &error))
+	{
+		QMessageBox::critical(this, tr("Error"),
+								tr("Error while checking MCEdit install:\n%1").arg(error));
+	}
+	else
+	{
+		QMessageBox::information(this, tr("OK"), tr("MCEdit setup seems to be OK"));
+	}
+}
diff --git a/gui/dialogs/SettingsDialog.h b/gui/dialogs/SettingsDialog.h
index 60d569f9..d8495fdd 100644
--- a/gui/dialogs/SettingsDialog.h
+++ b/gui/dialogs/SettingsDialog.h
@@ -79,6 +79,8 @@ slots:
 	void on_jprofilerCheckBtn_clicked();
 	void on_jvisualvmPathBtn_clicked();
 	void on_jvisualvmCheckBtn_clicked();
+	void on_mceditPathBtn_clicked();
+	void on_mceditCheckBtn_clicked();
 
 	/*!
 	 * Updates the list of update channels in the combo box.
diff --git a/gui/dialogs/SettingsDialog.ui b/gui/dialogs/SettingsDialog.ui
index acf360a3..fbfa7aed 100644
--- a/gui/dialogs/SettingsDialog.ui
+++ b/gui/dialogs/SettingsDialog.ui
@@ -863,9 +863,9 @@
        </item>
       </layout>
      </widget>
-     <widget class="QWidget" name="profilingTab">
+     <widget class="QWidget" name="externalToolsTab">
       <attribute name="title">
-       <string>Profiling</string>
+       <string>External Tools</string>
       </attribute>
       <layout class="QVBoxLayout" name="verticalLayout_13">
        <item>
@@ -929,6 +929,36 @@
         </widget>
        </item>
        <item>
+        <widget class="QGroupBox" name="groupBox_4">
+         <property name="title">
+          <string>MCEdit</string>
+         </property>
+         <layout class="QVBoxLayout" name="verticalLayout_12">
+          <item>
+           <layout class="QHBoxLayout" name="horizontalLayout_6">
+            <item>
+             <widget class="QLineEdit" name="mceditPathEdit"/>
+            </item>
+            <item>
+             <widget class="QPushButton" name="mceditPathBtn">
+              <property name="text">
+               <string>...</string>
+              </property>
+             </widget>
+            </item>
+           </layout>
+          </item>
+          <item>
+           <widget class="QPushButton" name="mceditCheckBtn">
+            <property name="text">
+             <string>Check</string>
+            </property>
+           </widget>
+          </item>
+         </layout>
+        </widget>
+       </item>
+       <item>
         <spacer name="verticalSpacer_3">
          <property name="orientation">
           <enum>Qt::Vertical</enum>
diff --git a/logic/BaseExternalTool.cpp b/logic/BaseExternalTool.cpp
new file mode 100644
index 00000000..6dffb4d6
--- /dev/null
+++ b/logic/BaseExternalTool.cpp
@@ -0,0 +1,47 @@
+#include "BaseExternalTool.h"
+
+#include <QProcess>
+#ifdef Q_OS_WIN
+#include <windows.h>
+#endif
+
+BaseExternalTool::BaseExternalTool(BaseInstance *instance, QObject *parent)
+	: QObject(parent), m_instance(instance)
+{
+}
+
+BaseExternalTool::~BaseExternalTool()
+{
+}
+
+qint64 BaseExternalTool::pid(QProcess *process)
+{
+#ifdef Q_OS_WIN
+	struct _PROCESS_INFORMATION *procinfo = process->pid();
+	return procinfo->dwProcessId;
+#else
+	return process->pid();
+#endif
+}
+
+
+BaseDetachedTool::BaseDetachedTool(BaseInstance *instance, QObject *parent)
+	: BaseExternalTool(instance, parent)
+{
+
+}
+
+void BaseDetachedTool::run()
+{
+	runImpl();
+}
+
+
+BaseExternalToolFactory::~BaseExternalToolFactory()
+{
+}
+
+BaseDetachedTool *BaseDetachedToolFactory::createDetachedTool(BaseInstance *instance, QObject *parent)
+{
+	return qobject_cast<BaseDetachedTool *>(createTool(instance, parent));
+}
diff --git a/logic/BaseExternalTool.h b/logic/BaseExternalTool.h
new file mode 100644
index 00000000..184b1f80
--- /dev/null
+++ b/logic/BaseExternalTool.h
@@ -0,0 +1,56 @@
+#pragma once
+
+#include <QObject>
+
+class BaseInstance;
+class SettingsObject;
+class MinecraftProcess;
+class QProcess;
+
+class BaseExternalTool : public QObject
+{
+	Q_OBJECT
+public:
+	explicit BaseExternalTool(BaseInstance *instance, QObject *parent = 0);
+	virtual ~BaseExternalTool();
+
+protected:
+	BaseInstance *m_instance;
+
+	qint64 pid(QProcess *process);
+};
+
+class BaseDetachedTool : public BaseExternalTool
+{
+	Q_OBJECT
+public:
+	explicit BaseDetachedTool(BaseInstance *instance, QObject *parent = 0);
+
+public
+slots:
+	void run();
+
+protected:
+	virtual void runImpl() = 0;
+};
+
+class BaseExternalToolFactory
+{
+public:
+	virtual ~BaseExternalToolFactory();
+
+	virtual QString name() const = 0;
+
+	virtual void registerSettings(SettingsObject *settings) = 0;
+
+	virtual BaseExternalTool *createTool(BaseInstance *instance, QObject *parent = 0) = 0;
+
+	virtual bool check(QString *error) = 0;
+	virtual bool check(const QString &path, QString *error) = 0;
+};
+
+class BaseDetachedToolFactory : public BaseExternalToolFactory
+{
+public:
+	virtual BaseDetachedTool *createDetachedTool(BaseInstance *instance, QObject *parent = 0);
+};
diff --git a/logic/MCEditTool.cpp b/logic/MCEditTool.cpp
new file mode 100644
index 00000000..93ffb7a5
--- /dev/null
+++ b/logic/MCEditTool.cpp
@@ -0,0 +1,62 @@
+#include "MCEditTool.h"
+
+#include <QDir>
+#include <QProcess>
+#include <QFileDialog>
+
+#include "settingsobject.h"
+#include "logic/BaseInstance.h"
+#include "MultiMC.h"
+
+MCEditTool::MCEditTool(BaseInstance *instance, QObject *parent)
+	: BaseDetachedTool(instance, parent)
+{
+}
+
+void MCEditTool::runImpl()
+{
+	const QString mceditPath = MMC->settings()->get("MCEditPath").toString();
+	const QString save = QFileDialog::getExistingDirectory(
+		MMC->activeWindow(), tr("MCEdit"),
+		QDir(m_instance->minecraftRoot()).absoluteFilePath("saves"));
+	if (save.isEmpty())
+	{
+		return;
+	}
+	const QString program =
+		QDir(mceditPath).absoluteFilePath("mcedit.py");
+	QProcess::startDetached(program, QStringList() << save, mceditPath);
+}
+
+void MCEditFactory::registerSettings(SettingsObject *settings)
+{
+	settings->registerSetting("MCEditPath");
+}
+BaseExternalTool *MCEditFactory::createTool(BaseInstance *instance, QObject *parent)
+{
+	return new MCEditTool(instance, parent);
+}
+bool MCEditFactory::check(QString *error)
+{
+	return check(MMC->settings()->get("MCEditPath").toString(), error);
+}
+bool MCEditFactory::check(const QString &path, QString *error)
+{
+	if (path.isEmpty())
+	{
+		*error = QObject::tr("Path is empty");
+		return false;
+	}
+	const QDir dir(path);
+	if (!dir.exists())
+	{
+		*error = QObject::tr("Path does not exist");
+		return false;
+	}
+	if (!dir.exists("mcedit.py"))
+	{
+		*error = QObject::tr("Path does not contain mcedit.py");
+		return false;
+	}
+	return true;
+}
diff --git a/logic/MCEditTool.h b/logic/MCEditTool.h
new file mode 100644
index 00000000..b0ed1ad4
--- /dev/null
+++ b/logic/MCEditTool.h
@@ -0,0 +1,23 @@
+#pragma once
+
+#include "BaseExternalTool.h"
+
+class MCEditTool : public BaseDetachedTool
+{
+	Q_OBJECT
+public:
+	explicit MCEditTool(BaseInstance *instance, QObject *parent = 0);
+
+protected:
+	void runImpl() override;
+};
+
+class MCEditFactory : public BaseDetachedToolFactory
+{
+public:
+	QString name() const override { return "MCEdit"; }
+	void registerSettings(SettingsObject *settings) override;
+	BaseExternalTool *createTool(BaseInstance *instance, QObject *parent = 0) override;
+	bool check(QString *error) override;
+	bool check(const QString &path, QString *error) override;
+};
diff --git a/logic/profiler/BaseProfiler.cpp b/logic/profiler/BaseProfiler.cpp
index b58e88be..9aaca793 100644
--- a/logic/profiler/BaseProfiler.cpp
+++ b/logic/profiler/BaseProfiler.cpp
@@ -1,16 +1,9 @@
 #include "BaseProfiler.h"
 
 #include <QProcess>
-#ifdef Q_OS_WIN
-#include <windows.h>
-#endif
 
 BaseProfiler::BaseProfiler(BaseInstance *instance, QObject *parent)
-	: QObject(parent), m_instance(instance)
-{
-}
-
-BaseProfiler::~BaseProfiler()
+	: BaseExternalTool(instance, parent)
 {
 }
 
@@ -36,16 +29,7 @@ void BaseProfiler::abortProfilingImpl()
 	emit abortLaunch(tr("Profiler aborted"));
 }
 
-qint64 BaseProfiler::pid(QProcess *process)
-{
-#ifdef Q_OS_WIN
-	struct _PROCESS_INFORMATION *procinfo = process->pid();
-	return procinfo->dwProcessId;
-#else
-	return process->pid();
-#endif
-}
-
-BaseProfilerFactory::~BaseProfilerFactory()
+BaseProfiler *BaseProfilerFactory::createProfiler(BaseInstance *instance, QObject *parent)
 {
+	return qobject_cast<BaseProfiler *>(createTool(instance, parent));
 }
diff --git a/logic/profiler/BaseProfiler.h b/logic/profiler/BaseProfiler.h
index e0d1d5f6..082cb65a 100644
--- a/logic/profiler/BaseProfiler.h
+++ b/logic/profiler/BaseProfiler.h
@@ -1,18 +1,17 @@
 #pragma once
 
-#include <QObject>
+#include "logic/BaseExternalTool.h"
 
 class BaseInstance;
 class SettingsObject;
 class MinecraftProcess;
 class QProcess;
 
-class BaseProfiler : public QObject
+class BaseProfiler : public BaseExternalTool
 {
 	Q_OBJECT
 public:
 	explicit BaseProfiler(BaseInstance *instance, QObject *parent = 0);
-	virtual ~BaseProfiler();
 
 public
 slots:
@@ -20,30 +19,18 @@ slots:
 	void abortProfiling();
 
 protected:
-	BaseInstance *m_instance;
 	QProcess *m_profilerProcess;
 
 	virtual void beginProfilingImpl(MinecraftProcess *process) = 0;
 	virtual void abortProfilingImpl();
 
-	qint64 pid(QProcess *process);
-
 signals:
 	void readyToLaunch(const QString &message);
 	void abortLaunch(const QString &message);
 };
 
-class BaseProfilerFactory
+class BaseProfilerFactory : public BaseExternalToolFactory
 {
 public:
-	virtual ~BaseProfilerFactory();
-
-	virtual QString name() const = 0;
-
-	virtual void registerSettings(SettingsObject *settings) = 0;
-
-	virtual BaseProfiler *createProfiler(BaseInstance *instance, QObject *parent = 0) = 0;
-
-	virtual bool check(QString *error) = 0;
-	virtual bool check(const QString &path, QString *error) = 0;
+	virtual BaseProfiler *createProfiler(BaseInstance *instance, QObject *parent = 0);
 };
diff --git a/logic/profiler/JProfiler.cpp b/logic/profiler/JProfiler.cpp
index 1cd8d128..bb851f0b 100644
--- a/logic/profiler/JProfiler.cpp
+++ b/logic/profiler/JProfiler.cpp
@@ -46,7 +46,7 @@ void JProfilerFactory::registerSettings(SettingsObject *settings)
 	settings->registerSetting("JProfilerPort", 42042);
 }
 
-BaseProfiler *JProfilerFactory::createProfiler(BaseInstance *instance, QObject *parent)
+BaseExternalTool *JProfilerFactory::createTool(BaseInstance *instance, QObject *parent)
 {
 	return new JProfiler(instance, parent);
 }
diff --git a/logic/profiler/JProfiler.h b/logic/profiler/JProfiler.h
index 8a5bc561..88a02462 100644
--- a/logic/profiler/JProfiler.h
+++ b/logic/profiler/JProfiler.h
@@ -17,7 +17,7 @@ class JProfilerFactory : public BaseProfilerFactory
 public:
 	QString name() const override { return "JProfiler"; }
 	void registerSettings(SettingsObject *settings) override;
-	BaseProfiler *createProfiler(BaseInstance *instance, QObject *parent = 0) override;
+	BaseExternalTool *createTool(BaseInstance *instance, QObject *parent = 0) override;
 	bool check(QString *error) override;
 	bool check(const QString &path, QString *error) override;
 };
diff --git a/logic/profiler/JVisualVM.cpp b/logic/profiler/JVisualVM.cpp
index c4bc851b..7b79b2e8 100644
--- a/logic/profiler/JVisualVM.cpp
+++ b/logic/profiler/JVisualVM.cpp
@@ -42,7 +42,7 @@ void JVisualVMFactory::registerSettings(SettingsObject *settings)
 	settings->registerSetting("JVisualVMPath");
 }
 
-BaseProfiler *JVisualVMFactory::createProfiler(BaseInstance *instance, QObject *parent)
+BaseExternalTool *JVisualVMFactory::createTool(BaseInstance *instance, QObject *parent)
 {
 	return new JVisualVM(instance, parent);
 }
diff --git a/logic/profiler/JVisualVM.h b/logic/profiler/JVisualVM.h
index 154180b4..af94fe55 100644
--- a/logic/profiler/JVisualVM.h
+++ b/logic/profiler/JVisualVM.h
@@ -17,7 +17,7 @@ class JVisualVMFactory : public BaseProfilerFactory
 public:
 	QString name() const override { return "JVisualVM"; }
 	void registerSettings(SettingsObject *settings) override;
-	BaseProfiler *createProfiler(BaseInstance *instance, QObject *parent = 0) override;
+	BaseExternalTool *createTool(BaseInstance *instance, QObject *parent = 0) override;
 	bool check(QString *error) override;
 	bool check(const QString &path, QString *error) override;
 };