Actually show an error when the revive request fails instead of showi…

…ng the success message.
Mephiles · Jan 5, 2025 · 3499392 · 3499392
1 parent fa2ed8d
commit 3499392
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 5 deletions.
diff --git a/extension/changelog.json b/extension/changelog.json
@@ -7,7 +7,8 @@
 			"features": [],
 			"fixes": [
 				{ "message": "Correctly count the supply pack value.", "contributor": "DeKleineKobini" },
-				{ "message": "Avoid an error on the item market when switching categories.", "contributor": "DeKleineKobini" }
+				{ "message": "Avoid an error on the item market when switching categories.", "contributor": "DeKleineKobini" },
+				{ "message": "Actually show an error when the revive request fails instead of showing the success message.", "contributor": "DeKleineKobini" }
 			],
 			"changes": [{ "message": "Better optimize calls to tornpal.", "contributor": "DeKleineKobini" }],
 			"removed": []

diff --git a/extension/scripts/global/functions/api-external.js b/extension/scripts/global/functions/api-external.js
@@ -169,10 +169,12 @@ function doRequestRevive(id, name, country, faction) {
 	const provider = REVIVE_PROVIDERS.find((p) => p.provider === providerName);
 	if (!provider) throw new Error(`Revive provider '${providerName}' not found.`);
 
-	return provider
-		.doRequest(id, name, country, faction, source)
-		.then(({ response, contract }) => ({ response, contract, provider }))
-		.catch((response) => ({ response, provider }));
+	return new Promise((resolve, reject) => {
+		provider
+			.doRequest(id, name, country, faction, source)
+			.then(({ response, contract }) => resolve({ response, contract, provider }))
+			.catch((response) => reject({ response, provider }));
+	});
 }
 
 function calculateRevivePrice({ price }) {