Merge pull request #6696 from jacinta-stacks/chore/remove-metadata-already-set

Remove unused try_set_metadata

Author: jacinta-stacks

clarity-types/src/errors/mod.rs

@@ -184,8 +184,6 @@ pub enum RuntimeError {
     BadBlockHash(Vec<u8>),
     /// Failed to unwrap an `Optional` (`none`) or `Response` (`err` or `ok`) Clarity value.
     UnwrapFailure,
-    /// Attempt to set metadata (e.g., for NFTs or tokens) that was already initialized.
-    MetadataAlreadySet,
     /// Interaction with a deprecated or inactive Proof of Transfer (PoX) contract.
     DefunctPoxContract,
     /// Attempt to lock STX for stacking when already locked in an active PoX cycle.

clarity/src/vm/database/clarity_db.rs

@@ -686,25 +686,6 @@ impl<'a> ClarityDatabase<'a> {
             .map_err(|e| e.into())
     }
 
-    /// Set a metadata entry if it hasn't already been set, yielding
-    ///  a runtime error if it was. This should only be called by post-nakamoto
-    ///  contexts.
-    pub fn try_set_metadata(
-        &mut self,
-        contract_identifier: &QualifiedContractIdentifier,
-        key: &str,
-        data: &str,
-    ) -> Result<(), VmExecutionError> {
-        if self.store.has_metadata_entry(contract_identifier, key) {
-            Err(VmExecutionError::Runtime(
-                RuntimeError::MetadataAlreadySet,
-                None,
-            ))
-        } else {
-            Ok(self.store.insert_metadata(contract_identifier, key, data)?)
-        }
-    }
-
     fn insert_metadata<T: ClaritySerializable>(
         &mut self,
         contract_identifier: &QualifiedContractIdentifier,