Fixes a bug that causes ERC-721 contracts without metadata to not work.

We were only checking whether the `supportsInterface` call returned a value (rather than reverting), but we were failing to check the actual return value (a boolean).
DarkFlorist · Nov 22, 2023 · 9c177aa · 9c177aa
1 parent d859470
commit 9c177aa
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/app/ts/library/identifyTokens.ts b/app/ts/library/identifyTokens.ts
@@ -112,8 +112,8 @@ export async function itentifyAddress(address: string, id: bigint, provider: Pro
 				address,
 				id,
 				owner: nftInterface.decodeFunctionResult('ownerOf', owner.returnData)[0],
-				name: hasMetadata.success ? nftInterface.decodeFunctionResult('name', name.returnData)[0] : undefined,
-				tokenURI: hasMetadata.success ? nftInterface.decodeFunctionResult('tokenURI', tokenURI.returnData)[0] : undefined,
+				name: hasMetadata.success && nftInterface.decodeFunctionResult('supportsInterface', hasMetadata.returnData)[0] ? nftInterface.decodeFunctionResult('name', name.returnData)[0] : undefined,
+				tokenURI: hasMetadata.success && nftInterface.decodeFunctionResult('supportsInterface', hasMetadata.returnData)[0] ? nftInterface.decodeFunctionResult('tokenURI', tokenURI.returnData)[0] : undefined,
 			})
 		}
 	} catch (error) {