@> function tokenURI(uint256 tokenId) public view override returns (string memory) {

@> if (ownerOf(tokenId) == address(0)) {

@> revert ERC721Metadata__URI_QueryFor_NonExistentToken();

@> }

​

ownerOf() is OpenZeppelin ERC721.ownerOf, which reverts with ERC721NonexistentToken for a token that does not exist; it never returns address(0). The == address(0) comparison can therefore never be true, and the custom error ERC721Metadata__URI_QueryFor_NonExistentToken() declared at the top of the contract can never be returned from tokenURI(). Callers querying a nonexistent token get the generic OpenZeppelin error instead of the intended one.

​

Impact is low - error reporting only - but the check is dead code that misleads readers about the behaviour of the function.

Explanation: check existence with the internal function that returns an address instead of reverting, or drop the check and rely on ownerOf()'s revert.

revert ERC721Metadata__URI_QueryFor_NonExistentToken();

}

​

_ownerOf is OpenZeppelin ERC721's internal view that returns address(0) for a nonexistent token without reverting. Alternatively, remove the check entirely and let ownerOf() revert with ERC721NonexistentToken.