Remove unused ERC20NullOwnerCappedUpgradeable contract and simplify t…

[RogerPodacter]

…okenURI function in ERC20FixedDenomination contract. The tokenURI function no longer validates token IDs, streamlining its implementation.

---

Note

Removes unused ERC20NullOwnerCappedUpgradeable and streamlines ERC20FixedDenomination.tokenURI by dropping redundant token ID validation.

• Contracts:
  • ERC20FixedDenomination: Simplifies tokenURI(uint256) by removing redundant token ID validation (relies on ownerOf to revert for invalid IDs).
  • Removal: Deletes unused contracts/src/ERC20NullOwnerCappedUpgradeable.sol.

^{Written by Cursor Bugbot for commit 7bb3566. This will update automatically on new commits. Configure here.}

[Copilot]

Pull request overview

This PR removes an unused contract and simplifies the tokenURI function by eliminating redundant validation. The removal of ERC20NullOwnerCappedUpgradeable.sol is safe as the contract is not referenced anywhere in the codebase. The tokenURI simplification is correct since ownerOf() already performs token ID validation internally.

• Removed unused ERC20NullOwnerCappedUpgradeable.sol contract (189 lines)
• Simplified tokenURI by removing redundant _validateTokenId call

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
contracts/src/ERC20NullOwnerCappedUpgradeable.sol	Complete removal of unused ERC20 contract with capping and null-owner support
contracts/src/ERC20FixedDenomination.sol	Removed redundant _validateTokenId call in tokenURI since ownerOf already validates

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.