Add mod website button thing feature widget. It is super effective.

11 files changed, 270 insertions, 134 deletions

diff --git a/gui/LegacyModEditDialog.cpp b/gui/LegacyModEditDialog.cpp
index 45f041f3..87647e0f 100644
--- a/gui/LegacyModEditDialog.cpp
+++ b/gui/LegacyModEditDialog.cpp
@@ -24,6 +24,7 @@
 
 #include <pathutils.h>
 #include <QFileDialog>
+//#include <QMessageBox>
 #include <QDebug>
 #include <QEvent>
 #include <QKeyEvent>
@@ -342,4 +343,36 @@ void LegacyModEditDialog::on_viewTexPackBtn_clicked()
 void LegacyModEditDialog::on_buttonBox_rejected()
 {
 	close();
-}
\ No newline at end of file
+}
+
+//FIXME: too much copypasta makes peterix a sad hacker. BUT IT'S SO DELICIOUS!
+
+void LegacyModEditDialog::on_coreWebsite_clicked()
+{
+	int first, last;
+	auto list = ui->coreModsTreeView->selectionModel()->selectedRows();
+
+	if (!lastfirst(list, first, last))
+		return;
+	showWebsiteForMod(this, m_coremods->operator[](first));
+}
+
+void LegacyModEditDialog::on_jarWebsite_clicked()
+{
+	int first, last;
+	auto list = ui->jarModsTreeView->selectionModel()->selectedRows();
+
+	if (!lastfirst(list, first, last))
+		return;
+	showWebsiteForMod(this, m_jarmods->operator[](first));
+}
+
+void LegacyModEditDialog::on_loaderWebsite_clicked()
+{
+	int first, last;
+	auto list = ui->loaderModTreeView->selectionModel()->selectedRows();
+
+	if (!lastfirst(list, first, last))
+		return;
+	showWebsiteForMod(this, m_mods->operator[](first));
+}
diff --git a/gui/LegacyModEditDialog.h b/gui/LegacyModEditDialog.h
index 5f6973d3..f12d9a7b 100644
--- a/gui/LegacyModEditDialog.h
+++ b/gui/LegacyModEditDialog.h
@@ -3,7 +3,7 @@
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
- * 
+ *
  *     http://www.apache.org/licenses/LICENSE-2.0
  *
  * Unless required by applicable law or agreed to in writing, software
@@ -19,51 +19,60 @@
 #include "logic/LegacyInstance.h"
 #include <logic/net/DownloadJob.h>
 
-namespace Ui {
+namespace Ui
+{
 class LegacyModEditDialog;
 }
 
 class LegacyModEditDialog : public QDialog
 {
 	Q_OBJECT
-	
+
 public:
-	explicit LegacyModEditDialog(LegacyInstance* inst, QWidget *parent = 0);
+	explicit LegacyModEditDialog(LegacyInstance *inst, QWidget *parent = 0);
 	~LegacyModEditDialog();
-	
-private slots:
-	
+
+private
+slots:
+
 	void on_addJarBtn_clicked();
 	void on_rmJarBtn_clicked();
 	void on_addForgeBtn_clicked();
 	void on_moveJarUpBtn_clicked();
 	void on_moveJarDownBtn_clicked();
-	
+
 	void on_addCoreBtn_clicked();
 	void on_rmCoreBtn_clicked();
 	void on_viewCoreBtn_clicked();
-	
+
 	void on_addModBtn_clicked();
 	void on_rmModBtn_clicked();
 	void on_viewModBtn_clicked();
-	
+
 	void on_addTexPackBtn_clicked();
 	void on_rmTexPackBtn_clicked();
 	void on_viewTexPackBtn_clicked();
+
+	void on_jarWebsite_clicked();
+	void on_loaderWebsite_clicked();
+	void on_coreWebsite_clicked();
+
 	// Questionable: SettingsDialog doesn't need this for some reason?
 	void on_buttonBox_rejected();
+
 protected:
 	bool eventFilter(QObject *obj, QEvent *ev);
-	bool jarListFilter( QKeyEvent* ev );
-	bool coreListFilter( QKeyEvent* ev );
-	bool loaderListFilter( QKeyEvent* ev );
-	bool texturePackListFilter( QKeyEvent* ev );
+	bool jarListFilter(QKeyEvent *ev);
+	bool coreListFilter(QKeyEvent *ev);
+	bool loaderListFilter(QKeyEvent *ev);
+	bool texturePackListFilter(QKeyEvent *ev);
+
 private:
 	Ui::LegacyModEditDialog *ui;
 	std::shared_ptr<ModList> m_mods;
 	std::shared_ptr<ModList> m_coremods;
 	std::shared_ptr<ModList> m_jarmods;
 	std::shared_ptr<ModList> m_texturepacks;
-	LegacyInstance * m_inst;
+	LegacyInstance *m_inst;
 	DownloadJobPtr forgeJob;
 };
diff --git a/gui/LegacyModEditDialog.ui b/gui/LegacyModEditDialog.ui
index 73b767dc..6a4b6d1e 100644
--- a/gui/LegacyModEditDialog.ui
+++ b/gui/LegacyModEditDialog.ui
@@ -58,6 +58,13 @@
           </widget>
          </item>
          <item>
+          <widget class="QPushButton" name="jarWebsite">
+           <property name="text">
+            <string>Website</string>
+           </property>
+          </widget>
+         </item>
+         <item>
           <spacer name="jarModsButtonSpacer">
            <property name="orientation">
             <enum>Qt::Vertical</enum>
@@ -117,6 +124,13 @@
           </widget>
          </item>
          <item>
+          <widget class="QPushButton" name="coreWebsite">
+           <property name="text">
+            <string>Website</string>
+           </property>
+          </widget>
+         </item>
+         <item>
           <spacer name="coreModsButtonSpacer">
            <property name="orientation">
             <enum>Qt::Vertical</enum>
@@ -172,6 +186,13 @@
           </widget>
          </item>
          <item>
+          <widget class="QPushButton" name="loaderWebsite">
+           <property name="text">
+            <string>Website</string>
+           </property>
+          </widget>
+         </item>
+         <item>
           <spacer name="mlModsButtonSpacer">
            <property name="orientation">
             <enum>Qt::Vertical</enum>
diff --git a/gui/ModEditDialogCommon.cpp b/gui/ModEditDialogCommon.cpp
index 5da0a039..692ac0c4 100644
--- a/gui/ModEditDialogCommon.cpp
+++ b/gui/ModEditDialogCommon.cpp
@@ -1,17 +1,40 @@
 #include "ModEditDialogCommon.h"
-
-bool lastfirst (QModelIndexList & list, int & first, int & last)
+#include <QDesktopServices>
+#include <QMessageBox>
+#include <QString>
+#include <QUrl>
+bool lastfirst(QModelIndexList &list, int &first, int &last)
 {
-	if(!list.size())
+	if (!list.size())
 		return false;
 	first = last = list[0].row();
-	for(auto item: list)
+	for (auto item : list)
 	{
 		int row = item.row();
-		if(row < first)
+		if (row < first)
 			first = row;
-		if(row > last)
+		if (row > last)
 			last = row;
 	}
 	return true;
-}
\ No newline at end of file
+}
+
+void showWebsiteForMod(QWidget *parentDlg, Mod &m)
+{
+	QString url = m.homeurl();
+	if (url.size())
+	{
+		// catch the cases where the protocol is missing
+		if(!url.startsWith("http"))
+		{
+			url = "http://" + url;
+		}
+		QDesktopServices::openUrl(url);
+	}
+	else
+	{
+		QMessageBox::warning(
+			parentDlg, parentDlg->tr("How sad!"),
+			parentDlg->tr("The mod author didn't provide a website link for this mod."));
+	}
+}
diff --git a/gui/ModEditDialogCommon.h b/gui/ModEditDialogCommon.h
index a27a8a82..bc8e223f 100644
--- a/gui/ModEditDialogCommon.h
+++ b/gui/ModEditDialogCommon.h
@@ -1,4 +1,7 @@
 #pragma once
 #include <QAbstractItemModel>
+#include <logic/Mod.h>
 
-bool lastfirst (QModelIndexList & list, int & first, int & last);
\ No newline at end of file
+bool lastfirst (QModelIndexList & list, int & first, int & last);
+
+void showWebsiteForMod(QWidget * parentDlg, Mod& m);
\ No newline at end of file
diff --git a/gui/OneSixModEditDialog.cpp b/gui/OneSixModEditDialog.cpp
index 24f01c53..788a61d0 100644
--- a/gui/OneSixModEditDialog.cpp
+++ b/gui/OneSixModEditDialog.cpp
@@ -30,6 +30,7 @@
 #include <QDebug>
 #include <QEvent>
 #include <QKeyEvent>
+#include <QDesktopServices>
 
 OneSixModEditDialog::OneSixModEditDialog(OneSixInstance *inst, QWidget *parent)
 	: m_inst(inst), QDialog(parent), ui(new Ui::OneSixModEditDialog)
@@ -296,3 +297,13 @@ void OneSixModEditDialog::on_viewResPackBtn_clicked()
 {
 	openDirInDefaultProgram(m_inst->resourcePacksDir(), true);
 }
+
+void OneSixModEditDialog::on_loaderWebsite_clicked()
+{
+	int first, last;
+	auto list = ui->loaderModTreeView->selectionModel()->selectedRows();
+
+	if (!lastfirst(list, first, last))
+		return;
+	showWebsiteForMod(this, m_mods->operator[](first));
+}
diff --git a/gui/OneSixModEditDialog.h b/gui/OneSixModEditDialog.h
index 5c65fea3..ce4da4fe 100644
--- a/gui/OneSixModEditDialog.h
+++ b/gui/OneSixModEditDialog.h
@@ -44,6 +44,7 @@ private slots:
 	void on_forgeBtn_clicked();
 	void on_customizeBtn_clicked();
 	void on_revertBtn_clicked();
+	void on_loaderWebsite_clicked();
     void updateVersionControls();
 	void disableVersionControls();
 protected:
diff --git a/gui/OneSixModEditDialog.ui b/gui/OneSixModEditDialog.ui
index 527899ca..b97d4304 100644
--- a/gui/OneSixModEditDialog.ui
+++ b/gui/OneSixModEditDialog.ui
@@ -185,6 +185,13 @@
           </widget>
          </item>
          <item>
+          <widget class="QPushButton" name="loaderWebsite">
+           <property name="text">
+            <string>Website</string>
+           </property>
+          </widget>
+         </item>
+         <item>
           <spacer name="verticalSpacer">
            <property name="orientation">
             <enum>Qt::Vertical</enum>
diff --git a/logic/Mod.cpp b/logic/Mod.cpp
index 75e0a3a9..825e663f 100644
--- a/logic/Mod.cpp
+++ b/logic/Mod.cpp
@@ -1,12 +1,12 @@
-// 
+//
 //  Copyright 2012 MultiMC Contributors
-// 
+//
 //    Licensed under the Apache License, Version 2.0 (the "License");
 //    you may not use this file except in compliance with the License.
 //    You may obtain a copy of the License at
-// 
+//
 //        http://www.apache.org/licenses/LICENSE-2.0
-// 
+//
 //    Unless required by applicable law or agreed to in writing, software
 //    distributed under the License is distributed on an "AS IS" BASIS,
 //    WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
@@ -28,17 +28,17 @@
 #include <inifile.h>
 #include <logger/QsLog.h>
 
-Mod::Mod( const QFileInfo& file )
+Mod::Mod(const QFileInfo &file)
 {
 	repath(file);
 }
 
-void Mod::repath ( const QFileInfo& file )
+void Mod::repath(const QFileInfo &file)
 {
 	m_file = file;
 	m_name = file.completeBaseName();
 	m_id = file.fileName();
-	
+
 	m_type = Mod::MOD_UNKNOWN;
 	if (m_file.isDir())
 		m_type = MOD_FOLDER;
@@ -50,19 +50,19 @@ void Mod::repath ( const QFileInfo& file )
 		else
 			m_type = MOD_SINGLEFILE;
 	}
-	if(m_type == MOD_ZIPFILE)
+	if (m_type == MOD_ZIPFILE)
 	{
 		QuaZip zip(m_file.filePath());
-		if(!zip.open(QuaZip::mdUnzip))
+		if (!zip.open(QuaZip::mdUnzip))
 			return;
-	
+
 		QuaZipFile file(&zip);
-		for(bool more=zip.goToFirstFile(); more; more=zip.goToNextFile())
+		for (bool more = zip.goToFirstFile(); more; more = zip.goToNextFile())
 		{
 			QString name = zip.getCurrentFileName();
-			if(name == "mcmod.info")
+			if (name == "mcmod.info")
 			{
-				if(!file.open(QIODevice::ReadOnly))
+				if (!file.open(QIODevice::ReadOnly))
 				{
 					zip.close();
 					return;
@@ -72,9 +72,9 @@ void Mod::repath ( const QFileInfo& file )
 				zip.close();
 				return;
 			}
-			else if(name == "forgeversion.properties")
+			else if (name == "forgeversion.properties")
 			{
-				if(!file.open(QIODevice::ReadOnly))
+				if (!file.open(QIODevice::ReadOnly))
 				{
 					zip.close();
 					return;
@@ -87,16 +87,16 @@ void Mod::repath ( const QFileInfo& file )
 		}
 		zip.close();
 	}
-	else if(m_type == MOD_FOLDER)
+	else if (m_type == MOD_FOLDER)
 	{
 		QFileInfo mcmod_info(PathCombine(m_file.filePath(), "mcmod.info"));
 		if (mcmod_info.isFile())
 		{
 			QFile mcmod(mcmod_info.filePath());
-			if(!mcmod.open(QIODevice::ReadOnly))
+			if (!mcmod.open(QIODevice::ReadOnly))
 				return;
 			auto data = mcmod.readAll();
-			if(data.isEmpty() || data.isNull())
+			if (data.isEmpty() || data.isNull())
 				return;
 			ReadMCModInfo(data);
 		}
@@ -110,35 +110,37 @@ void Mod::repath ( const QFileInfo& file )
 // https://github.com/MinecraftForge/FML/wiki/FML-mod-information-file/5bf6a2d05145ec79387acc0d45c958642fb049fc
 void Mod::ReadMCModInfo(QByteArray contents)
 {
-	auto getInfoFromArray = [&]( QJsonArray arr ) -> void
+	auto getInfoFromArray = [&](QJsonArray arr)->void
 	{
-		if(!arr.at(0).isObject())
+		if (!arr.at(0).isObject())
 			return;
 		auto firstObj = arr.at(0).toObject();
 		m_id = firstObj.value("modid").toString();
 		m_name = firstObj.value("name").toString();
 		m_version = firstObj.value("version").toString();
+		m_homeurl = firstObj.value("url").toString();
 		return;
-	};
+	}
+	;
 	QJsonParseError jsonError;
 	QJsonDocument jsonDoc = QJsonDocument::fromJson(contents, &jsonError);
 	// this is the very old format that had just the array
-	if(jsonDoc.isArray())
+	if (jsonDoc.isArray())
 	{
 		getInfoFromArray(jsonDoc.array());
 	}
-	else if(jsonDoc.isObject())
+	else if (jsonDoc.isObject())
 	{
 		auto val = jsonDoc.object().value("modinfoversion");
 		int version = val.toDouble();
-		if(version != 2)
+		if (version != 2)
 		{
 			QLOG_ERROR() << "BAD stuff happened to mod json:";
 			QLOG_ERROR() << contents;
 			return;
 		}
 		auto arrVal = jsonDoc.object().value("modlist");
-		if(arrVal.isArray())
+		if (arrVal.isArray())
 		{
 			getInfoFromArray(arrVal.toArray());
 		}
@@ -150,33 +152,34 @@ void Mod::ReadForgeInfo(QByteArray contents)
 	// Read the data
 	m_name = "Minecraft Forge";
 	m_id = "Forge";
+	m_homeurl = "http://www.minecraftforge.net/forum/";
 	INIFile ini;
-	if(!ini.loadFile(contents))
+	if (!ini.loadFile(contents))
 		return;
-	
-	QString major = ini.get("forge.major.number","0").toString();
-	QString minor = ini.get("forge.minor.number","0").toString();
-	QString revision = ini.get("forge.revision.number","0").toString();
-	QString build = ini.get("forge.build.number","0").toString();
-	
+
+	QString major = ini.get("forge.major.number", "0").toString();
+	QString minor = ini.get("forge.minor.number", "0").toString();
+	QString revision = ini.get("forge.revision.number", "0").toString();
+	QString build = ini.get("forge.build.number", "0").toString();
+
 	m_version = major + "." + minor + "." + revision + "." + build;
 }
 
-bool Mod::replace ( Mod& with )
+bool Mod::replace(Mod &with)
 {
-	if(!destroy())
+	if (!destroy())
 		return false;
 	bool success = false;
 	auto t = with.type();
-	if(t == MOD_ZIPFILE || t == MOD_SINGLEFILE)
+	if (t == MOD_ZIPFILE || t == MOD_SINGLEFILE)
 	{
 		success = QFile::copy(with.m_file.filePath(), m_file.path());
 	}
-	if(t == MOD_FOLDER)
+	if (t == MOD_FOLDER)
 	{
 		success = copyPath(with.m_file.filePath(), m_file.path());
 	}
-	if(success)
+	if (success)
 	{
 		m_id = with.m_id;
 		m_mcversion = with.m_mcversion;
@@ -189,10 +192,10 @@ bool Mod::replace ( Mod& with )
 
 bool Mod::destroy()
 {
-	if(m_type == MOD_FOLDER)
+	if (m_type == MOD_FOLDER)
 	{
 		QDir d(m_file.filePath());
-		if(d.removeRecursively())
+		if (d.removeRecursively())
 		{
 			m_type = MOD_UNKNOWN;
 			return true;
@@ -202,7 +205,7 @@ bool Mod::destroy()
 	else if (m_type == MOD_SINGLEFILE || m_type == MOD_ZIPFILE)
 	{
 		QFile f(m_file.filePath());
-		if(f.remove())
+		if (f.remove())
 		{
 			m_type = MOD_UNKNOWN;
 			return true;
@@ -212,18 +215,17 @@ bool Mod::destroy()
 	return true;
 }
 
-
 QString Mod::version() const
 {
-	switch(type())
+	switch (type())
 	{
-		case MOD_ZIPFILE:
-			return m_version;
-		case MOD_FOLDER:
-			return "Folder";
-		case MOD_SINGLEFILE:
-			return "File";
-		default:
-			return "VOID";
+	case MOD_ZIPFILE:
+		return m_version;
+	case MOD_FOLDER:
+		return "Folder";
+	case MOD_SINGLEFILE:
+		return "File";
+	default:
+		return "VOID";
 	}
 }
diff --git a/logic/Mod.h b/logic/Mod.h
index fcfcc139..9831fdc0 100644
--- a/logic/Mod.h
+++ b/logic/Mod.h
@@ -1,12 +1,12 @@
-// 
+//
 //  Copyright 2012 MultiMC Contributors
-// 
+//
 //    Licensed under the Apache License, Version 2.0 (the "License");
 //    you may not use this file except in compliance with the License.
 //    You may obtain a copy of the License at
-// 
+//
 //        http://www.apache.org/licenses/LICENSE-2.0
-// 
+//
 //    Unless required by applicable law or agreed to in writing, software
 //    distributed under the License is distributed on an "AS IS" BASIS,
 //    WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
@@ -22,46 +22,72 @@ class Mod
 public:
 	enum ModType
 	{
-		MOD_UNKNOWN, //!< Indicates an unspecified mod type.
-		MOD_ZIPFILE, //!< The mod is a zip file containing the mod's class files.
+		MOD_UNKNOWN,	//!< Indicates an unspecified mod type.
+		MOD_ZIPFILE,	//!< The mod is a zip file containing the mod's class files.
 		MOD_SINGLEFILE, //!< The mod is a single file (not a zip file).
-		MOD_FOLDER, //!< The mod is in a folder on the filesystem.
+		MOD_FOLDER,	 //!< The mod is in a folder on the filesystem.
 	};
 
 	Mod(const QFileInfo &file);
-	
-	QFileInfo filename() const { return m_file; }
-	QString id() const { return m_id; }
-	ModType type() const { return m_type; }
-	QString mcversion() const { return m_mcversion; };
-	bool valid() {return m_type != MOD_UNKNOWN;}
-	QString name() const {return m_name; };
-	
+
+	QFileInfo filename() const
+	{
+		return m_file;
+	}
+	QString id() const
+	{
+		return m_id;
+	}
+	ModType type() const
+	{
+		return m_type;
+	}
+	QString mcversion() const
+	{
+		return m_mcversion;
+	}
+	;
+	bool valid()
+	{
+		return m_type != MOD_UNKNOWN;
+	}
+	QString name() const
+	{
+		return m_name;
+	}
+
 	QString version() const;
-	
-	
+
+	QString homeurl() const
+	{
+		return m_homeurl;
+	}
+
 	// delete all the files of this mod
 	bool destroy();
 	// replace this mod with a copy of the other
-	bool replace(Mod & with);
+	bool replace(Mod &with);
 	// change the mod's filesystem path (used by mod lists for *MAGIC* purposes)
 	void repath(const QFileInfo &file);
 
 	// WEAK compare operator - used for replacing mods
-	bool operator ==(const Mod &other) const
+	bool operator==(const Mod &other) const
 	{
 		return filename() == other.filename();
 	}
 	bool strongCompare(const Mod &other) const
 	{
-		return filename() == other.filename() && id() == other.id() && version() == other.version() && type() == other.type();
+		return filename() == other.filename() && id() == other.id() &&
+			   version() == other.version() && type() == other.type();
 	}
+
 private:
 	void ReadMCModInfo(QByteArray contents);
 	void ReadForgeInfo(QByteArray contents);
+
 protected:
 
-	//FIXME: what do do with those? HMM...
+	// FIXME: what do do with those? HMM...
 	/*
 	void ReadModInfoData(QString info);
 	void ReadForgeInfoData(QString infoFileData);
@@ -72,6 +98,7 @@ protected:
 	QString m_name;
 	QString m_version;
 	QString m_mcversion;
+	QString m_homeurl;
 
 	ModType m_type;
 };
diff --git a/logic/lists/MinecraftVersionList.cpp b/logic/lists/MinecraftVersionList.cpp
index fbf609b5..dd26714a 100644
--- a/logic/lists/MinecraftVersionList.cpp
+++ b/logic/lists/MinecraftVersionList.cpp
@@ -3,7 +3,7 @@
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
- * 
+ *
  *     http://www.apache.org/licenses/LICENSE-2.0
  *
  * Unless required by applicable law or agreed to in writing, software
@@ -32,10 +32,8 @@
 #define ASSETS_URLBASE "http://assets.minecraft.net/"
 #define MCN_URLBASE "http://sonicrules.org/mcnweb.py"
 
-MinecraftVersionList::MinecraftVersionList(QObject *parent) :
-	BaseVersionList(parent)
+MinecraftVersionList::MinecraftVersionList(QObject *parent) : BaseVersionList(parent)
 {
-	
 }
 
 Task *MinecraftVersionList::getLoadTask()
@@ -76,7 +74,7 @@ BaseVersionPtr MinecraftVersionList::getLatestStable() const
 {
 	for (int i = 0; i < m_vlist.length(); i++)
 	{
-		auto ver =std::dynamic_pointer_cast<MinecraftVersion>(m_vlist.at(i));
+		auto ver = std::dynamic_pointer_cast<MinecraftVersion>(m_vlist.at(i));
 		if (ver->is_latest && !ver->is_snapshot)
 		{
 			return m_vlist.at(i);
@@ -85,7 +83,7 @@ BaseVersionPtr MinecraftVersionList::getLatestStable() const
 	return BaseVersionPtr();
 }
 
-void MinecraftVersionList::updateListData(QList<BaseVersionPtr > versions)
+void MinecraftVersionList::updateListData(QList<BaseVersionPtr> versions)
 {
 	beginResetModel();
 	m_vlist = versions;
@@ -109,7 +107,6 @@ inline QDateTime timeFromS3Time(QString str)
 	return QDateTime::fromString(str, Qt::ISODate);
 }
 
-
 MCVListLoadTask::MCVListLoadTask(MinecraftVersionList *vlist)
 {
 	m_list = vlist;
@@ -151,91 +148,91 @@ void MCVListLoadTask::executeTask()
 	connect(vlistReply, SIGNAL(finished()), this, SLOT(list_downloaded()));
 }
 
-
 void MCVListLoadTask::list_downloaded()
 {
-	if(vlistReply->error() != QNetworkReply::NoError)
+	if (vlistReply->error() != QNetworkReply::NoError)
 	{
 		vlistReply->deleteLater();
 		emitFailed("Failed to load Minecraft main version list" + vlistReply->errorString());
 		return;
 	}
-	
+
 	QJsonParseError jsonError;
 	QJsonDocument jsonDoc = QJsonDocument::fromJson(vlistReply->readAll(), &jsonError);
 	vlistReply->deleteLater();
-	
+
 	if (jsonError.error != QJsonParseError::NoError)
 	{
 		emitFailed("Error parsing version list JSON:" + jsonError.errorString());
 		return;
 	}
 
-	if(!jsonDoc.isObject())
+	if (!jsonDoc.isObject())
 	{
 		emitFailed("Error parsing version list JSON: jsonDoc is not an object");
 		return;
 	}
-	
+
 	QJsonObject root = jsonDoc.object();
-	
+
 	// Get the ID of the latest release and the latest snapshot.
-	if(!root.value("latest").isObject())
+	if (!root.value("latest").isObject())
 	{
 		emitFailed("Error parsing version list JSON: version list is missing 'latest' object");
 		return;
 	}
-	
+
 	QJsonObject latest = root.value("latest").toObject();
-	
+
 	QString latestReleaseID = latest.value("release").toString("");
 	QString latestSnapshotID = latest.value("snapshot").toString("");
-	if(latestReleaseID.isEmpty())
+	if (latestReleaseID.isEmpty())
 	{
 		emitFailed("Error parsing version list JSON: latest release field is missing");
 		return;
 	}
-	if(latestSnapshotID.isEmpty())
+	if (latestSnapshotID.isEmpty())
 	{
 		emitFailed("Error parsing version list JSON: latest snapshot field is missing");
 		return;
 	}
 
 	// Now, get the array of versions.
-	if(!root.value("versions").isArray())
+	if (!root.value("versions").isArray())
 	{
-		emitFailed("Error parsing version list JSON: version list object is missing 'versions' array");
+		emitFailed(
+			"Error parsing version list JSON: version list object is missing 'versions' array");
 		return;
 	}
 	QJsonArray versions = root.value("versions").toArray();
-	
-	QList<BaseVersionPtr > tempList;
+
+	QList<BaseVersionPtr> tempList;
 	for (int i = 0; i < versions.count(); i++)
 	{
 		bool is_snapshot = false;
 		bool is_latest = false;
-		
+
 		// Load the version info.
-		if(!versions[i].isObject())
+		if (!versions[i].isObject())
 		{
-			//FIXME: log this somewhere
+			// FIXME: log this somewhere
 			continue;
 		}
 		QJsonObject version = versions[i].toObject();
 		QString versionID = version.value("id").toString("");
 		QString versionTimeStr = version.value("releaseTime").toString("");
 		QString versionTypeStr = version.value("type").toString("");
-		if(versionID.isEmpty() || versionTimeStr.isEmpty() || versionTypeStr.isEmpty())
+		if (versionID.isEmpty() || versionTimeStr.isEmpty() || versionTypeStr.isEmpty())
 		{
-			//FIXME: log this somewhere
+			// FIXME: log this somewhere
 			continue;
 		}
-		
+
 		// Parse the timestamp.
 		QDateTime versionTime = timeFromS3Time(versionTimeStr);
-		if(!versionTime.isValid())
+		if (!versionTime.isValid())
 		{
-			//FIXME: log this somewhere
+			// FIXME: log this somewhere
 			continue;
 		}
 		// Parse the type.
@@ -243,23 +240,25 @@ void MCVListLoadTask::list_downloaded()
 		// OneSix or Legacy. use filter to determine type
 		if (versionTypeStr == "release")
 		{
-			versionType = legacyWhitelist.contains(versionID)?MinecraftVersion::Legacy:MinecraftVersion::OneSix;
+			versionType = legacyWhitelist.contains(versionID) ? MinecraftVersion::Legacy
+															  : MinecraftVersion::OneSix;
 			is_latest = (versionID == latestReleaseID);
 			is_snapshot = false;
 		}
-		else if(versionTypeStr == "snapshot") // It's a snapshot... yay
+		else if (versionTypeStr == "snapshot") // It's a snapshot... yay
 		{
-			versionType = legacyWhitelist.contains(versionID)?MinecraftVersion::Legacy:MinecraftVersion::OneSix;
+			versionType = legacyWhitelist.contains(versionID) ? MinecraftVersion::Legacy
+															  : MinecraftVersion::OneSix;
 			is_latest = (versionID == latestSnapshotID);
 			is_snapshot = true;
 		}
-		else if(versionTypeStr == "old_alpha")
+		else if (versionTypeStr == "old_alpha")
 		{
 			versionType = MinecraftVersion::Nostalgia;
 			is_latest = false;
 			is_snapshot = false;
 		}
-		else if(versionTypeStr == "old_beta")
+		else if (versionTypeStr == "old_beta")
 		{
 			versionType = MinecraftVersion::Legacy;
 			is_latest = false;
@@ -267,12 +266,12 @@ void MCVListLoadTask::list_downloaded()
 		}
 		else
 		{
-			//FIXME: log this somewhere
+			// FIXME: log this somewhere
 			continue;
 		}
 		// Get the download URL.
 		QString dlUrl = QString(MCVLIST_URLBASE) + versionID + "/";
-		
+
 		// Now, we construct the version object and add it to the list.
 		std::shared_ptr<MinecraftVersion> mcVersion(new MinecraftVersion());
 		mcVersion->m_name = mcVersion->m_descriptor = versionID;
@@ -284,7 +283,7 @@ void MCVListLoadTask::list_downloaded()
 		tempList.append(mcVersion);
 	}
 	m_list->updateListData(tempList);
-	
+
 	emitSucceeded();
 	return;
 }