ERC721_GetOwnedNFTs() => Empty

[emin-karadag]

Hello,
I am using Thirdweb v2.11.1.
I created a collection of NFTs. Before I burned the NFTs in the collection with ‘Burn’, I could get all the NFTs I created with the ERC721_GetOwnedNFTs() method. However, after ‘Burn’, I could no longer get the NFT list with the ERC721_GetOwnedNFTs() or ERC721_GetAllNFTs() methods.
Only when I give an NFT ID to the ERC721_GetNFT() method, this method works specifically.

I couldn't understand the exact cause of the problem, but as I said, whatever happened happened after the ‘Burn’ operation. I can mint a new NFT, but I cannot pull the NFT list.

=================

nft_contract_address = "0xeB0E894FFE99b43bC1Aef0b5fD8F77D27336427f"
chainId = 43113
walletAddress = "0x11F6D1935dAA4134EB3B6e3a3d56614821600Be7"

====================

var contract = await ThirdwebContract.Create(client, nft_contract_address, chainId);
var result= await contract2.ERC721_GetOwnedNFTs("0x11F6D1935dAA4134EB3B6e3a3d56614821600Be7");

Thank you...

[linear]

TOOL-2826 ERC721_GetOwnedNFTs() => Empty

[0xFirekeeper]

Hey @emin-karadag the code below seems to work (returns an empty array)

var contract = await ThirdwebContract.Create(client, "0xeB0E894FFE99b43bC1Aef0b5fD8F77D27336427f", 43113);
var result = await contract.ERC721_GetOwnedNFTs("0x11F6D1935dAA4134EB3B6e3a3d56614821600Be7");
Console.WriteLine($"NFTs: {JsonConvert.SerializeObject(result, Formatting.Indented)}");

From your snippet it looks like you're calling some other contract2 ?

I've also tested fetching nfts owned by address zero and it returns correctly as well. ERC721_GetAllNFTs also appears to work.

If contract2 is some other contract where this call fails do provide the address, if it was a typo let us know and we'll close the issue - thanks!

[0xFirekeeper]

After re-reading your original post and seeing that GetAll was also not returning burned nfts, I see the main issue here.
Your contract supports nextTokenIdToMint but only ERC1155s would call this method when checking total supply, so I've added it to ERC721 as well. This should fix both your GetOwned and GetAll. Thanks for the report! #112

[emin-karadag]

@0xFirekeeper Thank you very much for your quick solutions. :)