Some unused code can be removed#
Informational
NiftyKitV3.sol#
import {ClonesUpgradeable};
can be removed as it's not used.
import {IDropKitPass};
can be removed as it's not used.
NiftyKitV3.sol#
Here: contract NiftyKitV3 is INiftyKitV3, Initializable, OwnableUpgradeable
Initializable
can be removed as it's inherited by OwnableUpgradeable
Unused functions#
Here's a list of unused code. Removing this code will:
- Help code readability
- Increase control about what the code does or not
❌ InternalERC721AUpgradeable._nextTokenId() (contracts/internals/InternalERC721AUpgradeable.sol) is never used
❌ InternalERC721AUpgradeable._baseURI() (contracts/internals/InternalERC721AUpgradeable.sol) is never used
❌ MinimalOwnableRoles._checkOwnerOrRoles(uint256) (contracts/internals/MinimalOwnableRoles.sol) is never used
❌ InternalERC721AUpgradeable._unpackedOwnership(uint256) (contracts/internals/InternalERC721AUpgradeable.sol) is never used
❌ InternalOwnable._setOwner(address) (contracts/internals/InternalOwnable.sol) is never used
❌ InternalERC721AUpgradeable._approve(address,uint256) (contracts/internals/InternalERC721AUpgradeable.sol) is never used
❌ InternalOwnableRoles._grantRoles(address,uint256) (contracts/internals/InternalOwnableRoles.sol) is never used
❌ InternalERC721AUpgradeable._exists(uint256) (contracts/internals/InternalERC721AUpgradeable.sol:438-445) is never used
❌ InternalERC721AUpgradeable._ownershipAt(uint256) (contracts/internals/InternalERC721AUpgradeable.sol) is never used
❌ InternalERC721AUpgradeable._toString(uint256) (contracts/internals/InternalERC721AUpgradeable.sol) is never used
❌ MinimalOwnableRoles._checkRoles(uint256) (contracts/internals/MinimalOwnableRoles.sol) is never used
❌ InternalERC721AUpgradeable._setExtraDataAt(uint256,uint24) (contracts/internals/InternalERC721AUpgradeable.sol) is never used
❌ InternalERC721AUpgradeable._totalBurned() (contracts/internals/InternalERC721AUpgradeable.sol) is never used
❌ InternalERC721AUpgradeable.__ERC721A_init_unchained(string,string) (contracts/internals/InternalERC721AUpgradeable.sol) is never used
❌ LibDiamond.enforceIsContractOwner() (contracts/libraries/LibDiamond.sol) is never used
❌ InternalERC721AUpgradeable._numberBurned(address) (contracts/internals/InternalERC721AUpgradeable.sol) is never used
❌ InternalERC721AUpgradeable._burn(uint256,bool) (contracts/internals/InternalERC721AUpgradeable.sol) is never used
❌ InternalERC721AUpgradeable._isApprovedForAll(address,address) (contracts/internals/InternalERC721AUpgradeable.sol) is never used
❌ InternalERC721AUpgradeable._getApprovedSlotAndAddress(uint256) (contracts/internals/InternalERC721AUpgradeable.sol) is never used
❌ InternalERC721AUpgradeable._approve(address,uint256,bool) (contracts/internals/InternalERC721AUpgradeable.sol) is never used
❌ InternalOwnableRoles._checkOwnerOrRoles(uint256) (contracts/internals/InternalOwnableRoles.sol) is never used
❌ InternalERC721AUpgradeable._totalMinted() (contracts/internals/InternalERC721AUpgradeable.sol) is never used
❌ InternalOwnableRoles._checkRoles(uint256) (contracts/internals/InternalOwnableRoles.sol) is never used
❌ InternalERC721AUpgradeable._ownershipOf(uint256) (contracts/internals/InternalERC721AUpgradeable.sol) is never used
❌ InternalERC721AUpgradeable._isSenderApprovedOrOwner(address,address,address) (contracts/internals/InternalERC721AUpgradeable.sol) is never used
❌ InternalERC721AUpgradeable._burn(uint256) (contracts/internals/InternalERC721AUpgradeable.sol) is never used
❌ InternalERC721AUpgradeable._mintERC2309(address,uint256) (contracts/internals/InternalERC721AUpgradeable.sol) is never used
❌ InternalERC721AUpgradeable._packedOwnershipOf(uint256) (contracts/internals/InternalERC721AUpgradeable.sol) is never used
❌ InternalERC721AUpgradeable._ownerOf(uint256) (contracts/internals/InternalERC721AUpgradeable.sol) is never used
❌ LibDiamond.contractOwner() (contracts/libraries/LibDiamond.sol) is never used
❌ InternalOwnableRoles._removeRoles(address,uint256) (contracts/internals/InternalOwnableRoles.sol) is never used
❌ InternalERC721AUpgradeable._initializeOwnershipAt(uint256) (contracts/internals/InternalERC721AUpgradeable.sol) is never used
❌ InternalOwnable._initializeOwner(address) (contracts/internals/InternalOwnable.sol) is never used
❌ InternalERC721AUpgradeable._numberMinted(address) (contracts/internals/InternalERC721AUpgradeable.sol) is never used
❌ InternalERC721AUpgradeable.__ERC721A_init(string,string) (contracts/internals/InternalERC721AUpgradeable.sol) is never used