NOISSUE finalize curse modpack import work

1 files changed, 119 insertions, 25 deletions

diff --git a/api/logic/InstanceImportTask.cpp b/api/logic/InstanceImportTask.cpp
index 1b44ec7a..8304fdd4 100644
--- a/api/logic/InstanceImportTask.cpp
+++ b/api/logic/InstanceImportTask.cpp
@@ -10,6 +10,8 @@
 #include "settings/INISettingsObject.h"
 #include "icons/IIconList.h"
 #include <QtConcurrentRun>
+
+// FIXME: this does not belong here, it's Minecraft/Curse specific
 #include "minecraft/curse/FileResolvingTask.h"
 #include "minecraft/curse/PackManifest.h"
 #include "Json.h"
@@ -56,11 +58,13 @@ void InstanceImportTask::executeTask()
 void InstanceImportTask::downloadSucceeded()
 {
 	extractAndTweak();
+	m_filesNetJob.reset();
 }
 
 void InstanceImportTask::downloadFailed(QString reason)
 {
 	emitFailed(reason);
+	m_filesNetJob.reset();
 }
 
 void InstanceImportTask::downloadProgressChanged(qint64 current, qint64 total)
@@ -93,7 +97,7 @@ void InstanceImportTask::extractAndTweak()
 	setStatus(tr("Extracting modpack"));
 	m_stagingPath = m_target->getStagedInstancePath();
 	QDir extractDir(m_stagingPath);
-	qDebug() << "Attempting to create instance from" << m_archivePath;
+	qInfo() << "Attempting to create instance from" << m_archivePath;
 
 	m_extractFuture = QtConcurrent::run(QThreadPool::globalInstance(), MMCZip::extractDir, m_archivePath, extractDir.absolutePath());
 	connect(&m_extractFutureWatcher, &QFutureWatcher<QStringList>::finished, this, &InstanceImportTask::extractFinished);
@@ -110,18 +114,53 @@ void InstanceImportTask::extractFinished()
 		return;
 	}
 	QDir extractDir(m_stagingPath);
+
+	qInfo() << "Fixing permissions for extracted pack files...";
+	QDirIterator it(extractDir, QDirIterator::Subdirectories);
+	while (it.hasNext())
+	{
+		auto filepath = it.next();
+		QFileInfo file(filepath);
+		auto permissions = QFile::permissions(filepath);
+		auto origPermissions = permissions;
+		if(file.isDir())
+		{
+			// Folder +rwx for current user
+			permissions |= QFileDevice::Permission::ReadUser | QFileDevice::Permission::WriteUser | QFileDevice::Permission::ExeUser;
+		}
+		else
+		{
+			// File +rw for current user
+			permissions |= QFileDevice::Permission::ReadUser | QFileDevice::Permission::WriteUser;
+		}
+		if(origPermissions != permissions)
+		{
+			if(!QFile::setPermissions(filepath, permissions))
+			{
+				qWarning() << "Could not fix" << filepath;
+			}
+			else
+			{
+				qDebug() << "Fixed" << filepath;
+			}
+		}
+	}
+
 	const QFileInfo instanceCfgFile = findRecursive(extractDir.absolutePath(), "instance.cfg");
 	const QFileInfo curseJson = findRecursive(extractDir.absolutePath(), "manifest.json");
 	if (instanceCfgFile.isFile())
 	{
+		qInfo() << "Pack appears to be exported from MultiMC.";
 		processMultiMC(instanceCfgFile);
 	}
 	else if (curseJson.isFile())
 	{
+		qInfo() << "Pack appears to be from Curse.";
 		processCurse(curseJson);
 	}
 	else
 	{
+		qCritical() << "Archive does not contain a recognized modpack type.";
 		m_target->destroyStagingPath(m_stagingPath);
 		emitFailed(tr("Archive does not contain a recognized modpack type."));
 	}
@@ -143,46 +182,101 @@ void InstanceImportTask::processCurse(const QFileInfo & manifest)
 	}
 	catch (JSONValidationError & e)
 	{
+		m_target->destroyStagingPath(m_stagingPath);
 		emitFailed(tr("Could not understand curse manifest:\n") + e.cause());
 		return;
 	}
 	m_packRoot = manifest.absolutePath();
 	QString configPath = FS::PathCombine(m_packRoot, "instance.cfg");
+	if(!pack.overrides.isEmpty())
+	{
+		QString overridePath = FS::PathCombine(m_packRoot, pack.overrides);
+		QString mcPath = FS::PathCombine(m_packRoot, "minecraft");
+		if (!QFile::rename(overridePath, mcPath))
+		{
+			m_target->destroyStagingPath(m_stagingPath);
+			emitFailed(tr("Could not rename the curse overrides:\n") + pack.overrides);
+			return;
+		}
+	}
+
+	QString forgeVersion;
+	for(auto &loader: pack.minecraft.modLoaders)
+	{
+		auto id = loader.id;
+		if(id.startsWith("forge-"))
+		{
+			id.remove("forge-");
+			forgeVersion = id;
+			continue;
+		}
+		qWarning() << "Unknown mod loader in curse manifest:" << id;
+	}
+
 	auto instanceSettings = std::make_shared<INISettingsObject>(configPath);
 	instanceSettings->registerSetting("InstanceType", "Legacy");
 	instanceSettings->set("InstanceType", "OneSix");
 	OneSixInstance instance(m_globalSettings, instanceSettings, m_packRoot);
 	instance.setIntendedVersionId(pack.minecraft.version);
+	if(!forgeVersion.isEmpty())
+	{
+		instance.setComponentVersion("net.minecraftforge", forgeVersion);
+	}
 	instance.setName(m_instName);
 	instance.setIconKey(m_instIcon);
-	m_curseResolver.reset(new Curse::FileResolvingTask(pack.files));
-	connect(m_curseResolver.get(), &Curse::FileResolvingTask::succeeded, this, &InstanceImportTask::curseResolvingSucceeded);
-	connect(m_curseResolver.get(), &Curse::FileResolvingTask::failed, this, &InstanceImportTask::curseResolvingFailed);
-	m_curseResolver->start();
-}
-
-void InstanceImportTask::curseResolvingFailed(QString reason)
-{
-	m_target->destroyStagingPath(m_stagingPath);
-	m_curseResolver.reset();
-	emitFailed(tr("Unable to resolve Curse mod IDs:\n") + reason);
-}
-
-void InstanceImportTask::curseResolvingSucceeded()
-{
-	auto results = m_curseResolver->getResults();
-	for(auto result: results)
+	m_curseResolver.reset(new Curse::FileResolvingTask(pack));
+	connect(m_curseResolver.get(), &Curse::FileResolvingTask::succeeded, [&]()
 	{
-		qDebug() << result.fileName << " = " << result.url;
+		auto results = m_curseResolver->getResults();
+		m_filesNetJob.reset(new NetJob(tr("Curse mod download")));
+		for(auto result: results.files)
+		{
+			auto path = FS::PathCombine(m_packRoot, "minecraft/mods", result.fileName);
+			auto dl = Net::Download::makeFile(result.url,path);
+			m_filesNetJob->addNetAction(dl);
+		}
+		m_curseResolver.reset();
+		connect(m_filesNetJob.get(), &NetJob::succeeded, this, [&]()
+		{
+			m_filesNetJob.reset();
+			if (!m_target->commitStagedInstance(m_stagingPath, m_packRoot, m_instName, m_instGroup))
+			{
+				m_target->destroyStagingPath(m_stagingPath);
+				emitFailed(tr("Unable to commit instance"));
+				return;
+			}
+			emitSucceeded();
+		}
+		);
+		connect(m_filesNetJob.get(), &NetJob::failed, [&](QString reason)
+		{
+			m_target->destroyStagingPath(m_stagingPath);
+			m_filesNetJob.reset();
+			emitFailed(reason);
+		});
+		connect(m_filesNetJob.get(), &NetJob::progress, [&](qint64 current, qint64 total)
+		{
+			setProgress(current, total);
+		});
+		setStatus(tr("Downloading mods..."));
+		m_filesNetJob->start();
 	}
-	m_curseResolver.reset();
-	if (!m_target->commitStagedInstance(m_stagingPath, m_packRoot, m_instName, m_instGroup))
+	);
+	connect(m_curseResolver.get(), &Curse::FileResolvingTask::failed, [&](QString reason)
 	{
 		m_target->destroyStagingPath(m_stagingPath);
-		emitFailed(tr("Unable to commit instance"));
-		return;
-	}
-	emitSucceeded();
+		m_curseResolver.reset();
+		emitFailed(tr("Unable to resolve Curse mod IDs:\n") + reason);
+	});
+	connect(m_curseResolver.get(), &Curse::FileResolvingTask::progress, [&](qint64 current, qint64 total)
+	{
+		setProgress(current, total);
+	});
+	connect(m_curseResolver.get(), &Curse::FileResolvingTask::status, [&](QString status)
+	{
+		setStatus(status);
+	});
+	m_curseResolver->start();
 }
 
 void InstanceImportTask::processMultiMC(const QFileInfo & config)