diff options
| author | flow <flowlnlnln@gmail.com> | 2023-01-13 16:59:37 -0300 |
|---|---|---|
| committer | flow <flowlnlnln@gmail.com> | 2023-01-13 16:59:37 -0300 |
| commit | c95c81d42f63a2807889740b89be924fd0b59083 (patch) | |
| tree | 9c272b50a663d6593855dbd0218c601ce8cf1bbb | |
| parent | 9407596b12df8cc45ddc53d3c08e495a2674199c (diff) | |
| download | PrismLauncher-c95c81d42f63a2807889740b89be924fd0b59083.tar.gz PrismLauncher-c95c81d42f63a2807889740b89be924fd0b59083.tar.bz2 PrismLauncher-c95c81d42f63a2807889740b89be924fd0b59083.zip | |
fix(ModUpdater): ensure instead of require icon_url
The spec says that this can be null, and indeed some mods have it set to
null, and should still be considered as valid.
Signed-off-by: flow <flowlnlnln@gmail.com>
| -rw-r--r-- | launcher/modplatform/modrinth/ModrinthPackIndex.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/launcher/modplatform/modrinth/ModrinthPackIndex.cpp b/launcher/modplatform/modrinth/ModrinthPackIndex.cpp index f270f470..7ade131e 100644 --- a/launcher/modplatform/modrinth/ModrinthPackIndex.cpp +++ b/launcher/modplatform/modrinth/ModrinthPackIndex.cpp @@ -27,6 +27,7 @@ static ModrinthAPI api; static ModPlatform::ProviderCapabilities ProviderCaps; +// https://docs.modrinth.com/api-spec/#tag/projects/operation/getProject void Modrinth::loadIndexedPack(ModPlatform::IndexedPack& pack, QJsonObject& obj) { pack.addonId = Json::ensureString(obj, "project_id"); @@ -44,7 +45,7 @@ void Modrinth::loadIndexedPack(ModPlatform::IndexedPack& pack, QJsonObject& obj) pack.description = Json::ensureString(obj, "description", ""); - pack.logoUrl = Json::requireString(obj, "icon_url"); + pack.logoUrl = Json::ensureString(obj, "icon_url", ""); pack.logoName = pack.addonId.toString(); ModPlatform::ModpackAuthor modAuthor; |