1 files changed, 35 insertions, 47 deletions

diff --git a/api/logic/minecraft/liteloader/LiteLoaderVersionList.cpp b/api/logic/minecraft/liteloader/LiteLoaderVersionList.cpp
index 760f357a..f5c10777 100644
--- a/api/logic/minecraft/liteloader/LiteLoaderVersionList.cpp
+++ b/api/logic/minecraft/liteloader/LiteLoaderVersionList.cpp
@@ -106,17 +106,23 @@ QVariant LiteLoaderVersionList::data(const QModelIndex &index, int role) const
 	case ParentGameVersionRole:
 		return version->mcVersion;
 
-	case RecommendedRole:
+	case LatestRole:
 		return version->isLatest;
 
+	case RecommendedRole:
+		return version->isRecommended;
+
+	case TypeRole:
+		return version->isSnapshot ? tr("Snapshot") : tr("Release");
+
 	default:
 		return QVariant();
 	}
 }
 
-QList<BaseVersionList::ModelRoles> LiteLoaderVersionList::providesRoles()
+BaseVersionList::RoleList LiteLoaderVersionList::providesRoles() const
 {
-	return {VersionPointerRole, VersionRole, VersionIdRole, ParentGameVersionRole, RecommendedRole};
+	return {VersionPointerRole, VersionRole, VersionIdRole, ParentGameVersionRole, RecommendedRole, LatestRole, TypeRole};
 }
 
 BaseVersionPtr LiteLoaderVersionList::getLatestStable() const
@@ -124,7 +130,7 @@ BaseVersionPtr LiteLoaderVersionList::getLatestStable() const
 	for (int i = 0; i < m_vlist.length(); i++)
 	{
 		auto ver = std::dynamic_pointer_cast<LiteLoaderVersion>(m_vlist.at(i));
-		if (ver->isLatest)
+		if (ver->isRecommended)
 		{
 			return m_vlist.at(i);
 		}
@@ -228,9 +234,9 @@ void LLListLoadTask::listDownloaded()
 		const QString mcVersion = vIt.key();
 		const QJsonObject versionObject = vIt.value().toObject();
 
-		auto processArtefacts = [&](QJsonObject artefacts, bool notSnapshots)
+		auto processArtefacts = [&](QJsonObject artefacts, bool notSnapshots, std::shared_ptr<LiteLoaderVersion> &latest)
 		{
-			QString latest;
+			QString latestVersion;
 			QList<BaseVersionPtr> perMcVersionList;
 			for (auto aIt = artefacts.begin(); aIt != artefacts.end(); ++aIt)
 			{
@@ -238,18 +244,19 @@ void LLListLoadTask::listDownloaded()
 				const QJsonObject artefact = aIt.value().toObject();
 				if (identifier == "latest")
 				{
-					latest = artefact.value("version").toString();
+					latestVersion = artefact.value("version").toString();
 					continue;
 				}
 				LiteLoaderVersionPtr version(new LiteLoaderVersion());
 				version->version = artefact.value("version").toString();
 				version->mcVersion = mcVersion;
 				version->md5 = artefact.value("md5").toString();
-				version->timestamp = artefact.value("timestamp").toString().toInt();
+				version->timestamp = artefact.value("timestamp").toString().toLong();
 				version->tweakClass = artefact.value("tweakClass").toString();
 				version->authors = authors;
 				version->description = description;
 				version->defaultUrl = defaultUrl;
+				version->isSnapshot = !notSnapshots;
 				const QJsonArray libs = artefact.value("libraries").toArray();
 				for (auto lIt = libs.begin(); lIt != libs.end(); ++lIt)
 				{
@@ -284,58 +291,39 @@ void LLListLoadTask::listDownloaded()
 				for (auto version : perMcVersionList)
 				{
 					auto v = std::dynamic_pointer_cast<LiteLoaderVersion>(version);
-					v->isLatest = v->version == latest;
+					if(v->version == latestVersion)
+					{
+						latest = v;
+					}
 				}
 			}
 			tempList.append(perMcVersionList);
 		};
 
+		std::shared_ptr<LiteLoaderVersion> latestSnapshot;
+		std::shared_ptr<LiteLoaderVersion> latestRelease;
 		// are there actually released versions for this mc version?
 		if(versionObject.contains("artefacts"))
 		{
 			const QJsonObject artefacts = versionObject.value("artefacts").toObject().value("com.mumfrey:liteloader").toObject();
-			processArtefacts(artefacts, true);
+			processArtefacts(artefacts, true, latestRelease);
 		}
 		if(versionObject.contains("snapshots"))
 		{
 			QJsonObject artefacts = versionObject.value("snapshots").toObject().value("com.mumfrey:liteloader").toObject();
-			processArtefacts(artefacts, false);
-			/*
-			LiteLoaderVersionPtr version(new LiteLoaderVersion());
-			version->version = mcVersion + "-SNAPSHOT";
-			version->mcVersion = mcVersion;
-			version->md5 = QString(); // FIXME: not available, unable to check
-			version->timestamp = QDateTime::currentMSecsSinceEpoch() / 1000LL;
-			version->tweakClass = "com.mumfrey.liteloader.launch.LiteLoaderTweaker"; // FIXME: guessing here, no data available
-			version->authors = authors;
-			version->description = description;
-			version->defaultUrl = defaultUrl;
-			const QJsonArray libs = snapshots.value("libraries").toArray();
-			for (auto lIt = libs.begin(); lIt != libs.end(); ++lIt)
-			{
-				auto libobject = (*lIt).toObject();
-				try
-				{
-					auto lib = OneSixVersionFormat::libraryFromJson(libobject, "versions.json");
-					// hack to make liteloader 1.7.10_00 work
-					if(lib->rawName() == GradleSpecifier("org.ow2.asm:asm-all:5.0.3"))
-					{
-						lib->setRepositoryURL("http://repo.maven.apache.org/maven2/");
-					}
-					version->libraries.append(lib);
-				}
-				catch (Exception &e)
-				{
-					qCritical() << "Couldn't read JSON object:";
-					continue; // FIXME: ignores bad libraries and continues loading
-				}
-			}
-			auto liteloaderLib = std::make_shared<Library>("com.mumfrey:liteloader:" + version->version);
-			liteloaderLib->setHint("always-stale");
-			liteloaderLib->setRepositoryURL("http://dl.liteloader.com/versions/");
-			version->libraries.append(liteloaderLib);
-			tempList.append(version);
-			*/
+			processArtefacts(artefacts, false, latestSnapshot);
+		}
+		if(latestSnapshot)
+		{
+			latestSnapshot->isLatest = true;
+		}
+		else if(latestRelease)
+		{
+			latestRelease->isLatest = true;
+		}
+		if(latestRelease)
+		{
+			latestRelease->isRecommended = true;
 		}
 	}
 	m_list->updateListData(tempList);