Clean up getScopeName interface and test impls (#70)

runic-inc · Jan 12, 2024 · dad0816 · dad0816
1 parent da81765
commit dad0816
Show file tree

Hide file tree

Showing 9 changed files with 0 additions and 51 deletions.
diff --git a/src/Patchwork1155Patch.sol b/src/Patchwork1155Patch.sol
@@ -31,13 +31,6 @@ abstract contract Patchwork1155Patch is Patchwork721, IPatchwork1155Patch {
         super.supportsInterface(interfaceID); 
     }
 
-    /**
-    @dev See {IPatchwork721-getScopeName}
-    */
-    function getScopeName() public view virtual override(Patchwork721, IPatchworkScoped) returns (string memory) {
-        return _scopeName;
-    }
-
     /**
     @notice stores a patch
     @param tokenId the tokenId of the patch

diff --git a/src/PatchworkAccountPatch.sol b/src/PatchworkAccountPatch.sol
@@ -23,13 +23,6 @@ abstract contract PatchworkAccountPatch is Patchwork721, IPatchworkAccountPatch
         super.supportsInterface(interfaceID); 
     }
 
-    /**
-    @dev See {IPatchwork721-getScopeName}
-    */
-    function getScopeName() public view virtual override(Patchwork721, IPatchworkScoped) returns (string memory) {
-        return _scopeName;
-    }
-
     /**
     @notice stores a patch
     @param tokenId the tokenId of the patch

diff --git a/src/PatchworkFragmentMulti.sol b/src/PatchworkFragmentMulti.sol
@@ -20,13 +20,6 @@ abstract contract PatchworkFragmentMulti is Patchwork721, IPatchworkMultiAssigna
     /// A mapping from token IDs in this contract to their assignments.
     mapping(uint256 => AssignmentStorage) internal _assignmentStorage;
 
-    /**
-    @dev See {IPatchwork721-getScopeName}
-    */
-    function getScopeName() public view virtual override (Patchwork721, IPatchworkScoped) returns (string memory) {
-        return _scopeName;
-    }
-
     /**
     @dev See {IERC165-supportsInterface}
     */

diff --git a/src/PatchworkFragmentSingle.sol b/src/PatchworkFragmentSingle.sol
@@ -19,13 +19,6 @@ abstract contract PatchworkFragmentSingle is Patchwork721, IPatchworkSingleAssig
     /// A mapping from token IDs in this contract to their assignments.
     mapping(uint256 => Assignment) internal _assignments;
 
-    /**
-    @dev See {IPatchworkScoped-getScopeName}
-    */
-    function getScopeName() public view virtual override (Patchwork721, IPatchworkScoped) returns (string memory) {
-        return _scopeName;
-    }
-
     /**
     @dev See {IERC165-supportsInterface}
     */

diff --git a/src/PatchworkPatch.sol b/src/PatchworkPatch.sol
@@ -29,13 +29,6 @@ abstract contract PatchworkPatch is Patchwork721, IPatchworkPatch {
         super.supportsInterface(interfaceID); 
     }
 
-    /**
-    @dev See {IPatchwork721-getScopeName}
-    */
-    function getScopeName() public view virtual override(Patchwork721, IPatchworkScoped) returns (string memory) {
-        return _scopeName;
-    }
-
     /**
     @dev will return the current owner of the patched address+tokenId
     @dev See {IERC721-ownerOf}

diff --git a/test/nfts/TestDynamicArrayLiteRefNFT.sol b/test/nfts/TestDynamicArrayLiteRefNFT.sol
@@ -55,10 +55,6 @@ contract TestDynamicArrayLiteRefNFT is Patchwork721, PatchworkLiteRef, IPatchwor
         _manager = manager_;
     }
 
-    function getScopeName() public view override (Patchwork721, IPatchworkScoped) returns (string memory scopeName) {
-        return Patchwork721.getScopeName();
-    }
-
     function mint(address to, bytes calldata /* data */) public payable returns (uint256 tokenId) {
         tokenId = _nextTokenId;
         _nextTokenId++;

diff --git a/test/nfts/TestFragmentLiteRefNFT.sol b/test/nfts/TestFragmentLiteRefNFT.sol
@@ -46,10 +46,6 @@ contract TestFragmentLiteRefNFT is PatchworkFragmentSingle, PatchworkLiteRef, IP
             interfaceID == type(IPatchworkMintable).interfaceId;
     }
 
-    function getScopeName() public view override (PatchworkFragmentSingle, IPatchworkScoped) returns (string memory scopeName) {
-        return PatchworkFragmentSingle.getScopeName();
-    }
-
     function mint(address to, bytes calldata /* data */) public payable returns (uint256 tokenId) {
         tokenId = _nextTokenId;
         _nextTokenId++;

diff --git a/test/nfts/TestMultiFragmentNFT.sol b/test/nfts/TestMultiFragmentNFT.sol
@@ -20,10 +20,6 @@ contract TestMultiFragmentNFT is PatchworkFragmentMulti, IPatchworkMintable {
             interfaceID == type(IPatchworkMintable).interfaceId;
     }
 
-    function getScopeName() public view override (PatchworkFragmentMulti, IPatchworkScoped) returns (string memory scopeName) {
-        return PatchworkFragmentMulti.getScopeName();
-    }
-
     function mint(address to, bytes calldata /* data */) public payable returns (uint256 tokenId) {
         tokenId = _nextTokenId;
         _nextTokenId++;

diff --git a/test/nfts/TestPatchFragmentNFT.sol b/test/nfts/TestPatchFragmentNFT.sol
@@ -46,10 +46,6 @@ contract TestPatchFragmentNFT is PatchworkPatch, PatchworkFragmentSingle {
         _manager = manager_;
     }
 
-    function getScopeName() public view virtual override(PatchworkPatch, PatchworkFragmentSingle) returns (string memory) {
-        return _scopeName;
-    }
-
     function setLocked(uint256 tokenId, bool locked_) public view virtual override(PatchworkPatch, PatchworkFragmentSingle) {
          return PatchworkPatch.setLocked(tokenId, locked_);
     }