git.delta.rocks / unique-network / refs/commits / 7d542e4157ec

--- a/crates/evm-coder/procedural/src/solidity_interface.rs
+++ b/crates/evm-coder/procedural/src/solidity_interface.rs
@@ -999,7 +999,7 @@
 						)*),
 					};
 
-					let mut out = ::evm_coder::types::string::new();
+					let mut out = ::evm_coder::types::String::new();
 					if #solidity_name.starts_with("Inline") {
 						out.push_str("/// @dev inlined interface\n");
 					}

--- a/crates/evm-coder/procedural/src/to_log.rs
+++ b/crates/evm-coder/procedural/src/to_log.rs
@@ -222,7 +222,7 @@
 							#solidity_functions,
 						)*),
 					};
-					let mut out = string::new();
+					let mut out = ::evm_coder::types::String::new();
 					out.push_str("/// @dev inlined interface\n");
 					let _ = interface.format(is_impl, &mut out, tc);
 					tc.collect(out);

--- a/crates/evm-coder/src/abi/impls.rs
+++ b/crates/evm-coder/src/abi/impls.rs
@@ -63,7 +63,7 @@
 impl_abi!(u128, uint128, false);
 impl_abi!(U256, uint256, false);
 impl_abi!(H160, address, false);
-impl_abi!(string, string, true);
+impl_abi!(String, string, true);
 
 impl_abi_writeable!(&str, string);
 

--- a/crates/evm-coder/src/abi/mod.rs
+++ b/crates/evm-coder/src/abi/mod.rs
@@ -148,8 +148,8 @@
 	}
 
 	/// Read [`string`] at current position, then advance
-	pub fn string(&mut self) -> Result<string> {
-		string::from_utf8(self.bytes()?).map_err(|_| Error::Error(ExitError::InvalidRange))
+	pub fn string(&mut self) -> Result<String> {
+		String::from_utf8(self.bytes()?).map_err(|_| Error::Error(ExitError::InvalidRange))
 	}
 
 	/// Read [`u8`] at current position, then advance

--- a/crates/evm-coder/src/abi/test.rs
+++ b/crates/evm-coder/src/abi/test.rs
@@ -138,7 +138,7 @@
 
 #[test]
 fn encode_decode_vec_tuple_uint256_string() {
-	test_impl::<Vec<(U256, string)>>(
+	test_impl::<Vec<(U256, String)>>(
         0xdeadbeef,
         vec![
             (1.into(), "Test URI 0".to_string()),
@@ -261,7 +261,7 @@
 	let (call, mut decoder) = AbiReader::new_call(encoded_data).unwrap();
 	assert_eq!(call, u32::to_be_bytes(decoded_data.0));
 	let address = decoder.address().unwrap();
-	let data = <Vec<(U256, string)>>::abi_read(&mut decoder).unwrap();
+	let data = <Vec<(U256, String)>>::abi_read(&mut decoder).unwrap();
 	assert_eq!(data, decoded_data.1);
 
 	let mut writer = AbiWriter::new_call(decoded_data.0);
@@ -273,7 +273,7 @@
 
 #[test]
 fn encode_decode_vec_tuple_string_bytes() {
-	test_impl::<Vec<(string, bytes)>>(
+	test_impl::<Vec<(String, bytes)>>(
 		0xdeadbeef,
 		vec![
 			(

--- a/crates/evm-coder/src/lib.rs
+++ b/crates/evm-coder/src/lib.rs
@@ -136,9 +136,9 @@
 	pub type Topic = H256;
 
 	#[cfg(not(feature = "std"))]
-	pub type string = ::alloc::string::String;
+	pub type String = ::alloc::string::String;
 	#[cfg(feature = "std")]
-	pub type string = ::std::string::String;
+	pub type String = ::std::string::String;
 
 	#[derive(Default, Debug, PartialEq, Eq, Clone)]
 	pub struct bytes(pub Vec<u8>);

--- a/crates/evm-coder/src/solidity/impls.rs
+++ b/crates/evm-coder/src/solidity/impls.rs
@@ -29,7 +29,7 @@
 	U256 => "uint256" true = "0",
 	Bytes4 => "bytes4" true = "bytes4(0)",
 	H160 => "address" true = "0x0000000000000000000000000000000000000000",
-	string => "string" false = "\"\"",
+	String => "string" false = "\"\"",
 	bytes => "bytes" false = "hex\"\"",
 	bool => "bool" true = "false",
 }
@@ -72,10 +72,10 @@
 macro_rules! impl_tuples {
 	($($ident:ident)+) => {
 		impl<$($ident: SolidityTypeName + 'static),+> SolidityTupleTy for ($($ident,)+) {
-			fn fields(tc: &TypeCollector) -> Vec<string> {
+			fn fields(tc: &TypeCollector) -> Vec<String> {
 				let mut collected = Vec::with_capacity(Self::len());
 				$({
-					let mut out = string::new();
+					let mut out = String::new();
 					$ident::solidity_name(&mut out, tc).expect("no fmt error");
 					collected.push(out);
 				})*;

--- a/crates/evm-coder/src/solidity/mod.rs
+++ b/crates/evm-coder/src/solidity/mod.rs
@@ -26,7 +26,7 @@
 mod impls;
 
 #[cfg(not(feature = "std"))]
-use alloc::{string::String, vec::Vec, collections::BTreeMap, format};
+use alloc::{vec::Vec, collections::BTreeMap, format};
 #[cfg(feature = "std")]
 use std::collections::BTreeMap;
 use core::{
@@ -42,16 +42,16 @@
 pub struct TypeCollector {
 	/// Code => id
 	/// id ordering is required to perform topo-sort on the resulting data
-	structs: RefCell<BTreeMap<string, usize>>,
-	anonymous: RefCell<BTreeMap<Vec<string>, usize>>,
-	// generic: RefCell<BTreeMap<string, usize>>,
+	structs: RefCell<BTreeMap<String, usize>>,
+	anonymous: RefCell<BTreeMap<Vec<String>, usize>>,
+	// generic: RefCell<BTreeMap<String, usize>>,
 	id: Cell<usize>,
 }
 impl TypeCollector {
 	pub fn new() -> Self {
 		Self::default()
 	}
-	pub fn collect(&self, item: string) {
+	pub fn collect(&self, item: String) {
 		let id = self.next_id();
 		self.structs.borrow_mut().insert(item, id);
 	}
@@ -84,7 +84,7 @@
 	pub fn collect_enum<T: SolidityEnumTy>(&self) -> String {
 		T::generate_solidity_interface(self)
 	}
-	pub fn finish(self) -> Vec<string> {
+	pub fn finish(self) -> Vec<String> {
 		let mut data = self.structs.into_inner().into_iter().collect::<Vec<_>>();
 		data.sort_by_key(|(_, id)| Reverse(*id));
 		data.into_iter().map(|(code, _)| code).collect()

--- a/pallets/common/src/erc.rs
+++ b/pallets/common/src/erc.rs
@@ -94,7 +94,7 @@
 	/// @param value Propery value.
 	#[solidity(hide)]
 	#[weight(<SelfWeightOf<T>>::set_collection_properties(1))]
-	fn set_collection_property(&mut self, caller: caller, key: string, value: bytes) -> Result<()> {
+	fn set_collection_property(&mut self, caller: caller, key: String, value: bytes) -> Result<()> {
 		let caller = T::CrossAccountId::from_eth(caller);
 		let key = <Vec<u8>>::from(key)
 			.try_into()
@@ -130,7 +130,7 @@
 	/// @param key Property key.
 	#[solidity(hide)]
 	#[weight(<SelfWeightOf<T>>::delete_collection_properties(1))]
-	fn delete_collection_property(&mut self, caller: caller, key: string) -> Result<()> {
+	fn delete_collection_property(&mut self, caller: caller, key: String) -> Result<()> {
 		let caller = T::CrossAccountId::from_eth(caller);
 		let key = <Vec<u8>>::from(key)
 			.try_into()
@@ -143,7 +143,7 @@
 	///
 	/// @param keys Properties keys.
 	#[weight(<SelfWeightOf<T>>::delete_collection_properties(keys.len() as u32))]
-	fn delete_collection_properties(&mut self, caller: caller, keys: Vec<string>) -> Result<()> {
+	fn delete_collection_properties(&mut self, caller: caller, keys: Vec<String>) -> Result<()> {
 		let caller = T::CrossAccountId::from_eth(caller);
 		let keys = keys
 			.into_iter()
@@ -164,7 +164,7 @@
 	///
 	/// @param key Property key.
 	/// @return bytes The property corresponding to the key.
-	fn collection_property(&self, key: string) -> Result<bytes> {
+	fn collection_property(&self, key: String) -> Result<bytes> {
 		let key = <Vec<u8>>::from(key)
 			.try_into()
 			.map_err(|_| "key too large")?;
@@ -179,7 +179,7 @@
 	///
 	/// @param keys Properties keys. Empty keys for all propertyes.
 	/// @return Vector of properties key/value pairs.
-	fn collection_properties(&self, keys: Vec<string>) -> Result<Vec<eth::Property>> {
+	fn collection_properties(&self, keys: Vec<String>) -> Result<Vec<eth::Property>> {
 		let keys = keys
 			.into_iter()
 			.map(|key| {
@@ -616,7 +616,7 @@
 	/// Returns collection type
 	///
 	/// @return `Fungible` or `NFT` or `ReFungible`
-	fn unique_collection_type(&self) -> Result<string> {
+	fn unique_collection_type(&self) -> Result<String> {
 		let mode = match self.collection.mode {
 			CollectionMode::Fungible(_) => "Fungible",
 			CollectionMode::NFT => "NFT",

--- a/pallets/common/src/eth.rs
+++ b/pallets/common/src/eth.rs
@@ -118,7 +118,7 @@
 /// Ethereum representation of collection [`PropertyKey`](up_data_structs::PropertyKey) and [`PropertyValue`](up_data_structs::PropertyValue).
 #[derive(Debug, Default, AbiCoder)]
 pub struct Property {
-	key: evm_coder::types::string,
+	key: evm_coder::types::String,
 	value: evm_coder::types::bytes,
 }
 
@@ -126,7 +126,7 @@
 	type Error = evm_coder::execution::Error;
 
 	fn try_from(from: up_data_structs::Property) -> Result<Self, Self::Error> {
-		let key = evm_coder::types::string::from_utf8(from.key.into())
+		let key = evm_coder::types::String::from_utf8(from.key.into())
 			.map_err(|e| Self::Error::Revert(format!("utf8 conversion error: {}", e)))?;
 		let value = evm_coder::types::bytes(from.value.to_vec());
 		Ok(Property { key, value })
@@ -342,7 +342,7 @@
 #[derive(Debug, Default, AbiCoder)]
 pub struct TokenPropertyPermission {
 	/// Token property key.
-	key: evm_coder::types::string,
+	key: evm_coder::types::String,
 	/// Token property permissions.
 	permissions: Vec<PropertyPermission>,
 }
@@ -360,7 +360,7 @@
 		),
 	) -> Self {
 		let (key, permission) = value;
-		let key = evm_coder::types::string::from_utf8(key.into_inner())
+		let key = evm_coder::types::String::from_utf8(key.into_inner())
 			.expect("Stored key must be valid");
 		let permissions = PropertyPermission::into_vec(permission);
 		Self { key, permissions }

--- a/pallets/fungible/src/erc.rs
+++ b/pallets/fungible/src/erc.rs
@@ -59,13 +59,13 @@
 
 #[solidity_interface(name = ERC20, events(ERC20Events))]
 impl<T: Config> FungibleHandle<T> {
-	fn name(&self) -> Result<string> {
+	fn name(&self) -> Result<String> {
 		Ok(decode_utf16(self.name.iter().copied())
 			.map(|r| r.unwrap_or(REPLACEMENT_CHARACTER))
-			.collect::<string>())
+			.collect::<String>())
 	}
-	fn symbol(&self) -> Result<string> {
-		Ok(string::from_utf8_lossy(&self.token_prefix).into())
+	fn symbol(&self) -> Result<String> {
+		Ok(String::from_utf8_lossy(&self.token_prefix).into())
 	}
 	fn total_supply(&self) -> Result<U256> {
 		self.consume_store_reads(1)?;
@@ -167,10 +167,10 @@
 	T::AccountId: From<[u8; 32]>,
 {
 	/// @notice A description for the collection.
-	fn description(&self) -> Result<string> {
+	fn description(&self) -> Result<String> {
 		Ok(decode_utf16(self.description.iter().copied())
 			.map(|r| r.unwrap_or(REPLACEMENT_CHARACTER))
-			.collect::<string>())
+			.collect::<String>())
 	}
 
 	#[weight(<SelfWeightOf<T>>::create_item())]

64	fn set_token_property_permission(64	fn set_token_property_permission(
65		&mut self,65		&mut self,
66		caller: caller,66		caller: caller,
67		key: string,67		key: String,
68		is_mutable: bool,68		is_mutable: bool,
69		collection_admin: bool,69		collection_admin: bool,
70		token_owner: bool,70		token_owner: bool,
123		&mut self,123		&mut self,
124		caller: caller,124		caller: caller,
125		token_id: U256,125		token_id: U256,
126		key: string,126		key: String,
127		value: bytes,127		value: bytes,
128	) -> Result<()> {128	) -> Result<()> {
129		let caller = T::CrossAccountId::from_eth(caller);129		let caller = T::CrossAccountId::from_eth(caller);
187	/// @param key Property key.187	/// @param key Property key.
188	#[solidity(hide)]188	#[solidity(hide)]
189	#[weight(<SelfWeightOf<T>>::delete_token_properties(1))]189	#[weight(<SelfWeightOf<T>>::delete_token_properties(1))]
190	fn delete_property(&mut self, token_id: U256, caller: caller, key: string) -> Result<()> {190	fn delete_property(&mut self, token_id: U256, caller: caller, key: String) -> Result<()> {
191		let caller = T::CrossAccountId::from_eth(caller);191		let caller = T::CrossAccountId::from_eth(caller);
192		let token_id: u32 = token_id.try_into().map_err(|_| "token id overflow")?;192		let token_id: u32 = token_id.try_into().map_err(|_| "token id overflow")?;
193		let key = <Vec<u8>>::from(key)193		let key = <Vec<u8>>::from(key)
211		&mut self,211		&mut self,
212		token_id: U256,212		token_id: U256,
213		caller: caller,213		caller: caller,
214		keys: Vec<string>,214		keys: Vec<String>,
215	) -> Result<()> {215	) -> Result<()> {
216		let caller = T::CrossAccountId::from_eth(caller);216		let caller = T::CrossAccountId::from_eth(caller);
217		let token_id: u32 = token_id.try_into().map_err(|_| "token id overflow")?;217		let token_id: u32 = token_id.try_into().map_err(|_| "token id overflow")?;
239	/// @param tokenId ID of the token.239	/// @param tokenId ID of the token.
240	/// @param key Property key.240	/// @param key Property key.
241	/// @return Property value bytes241	/// @return Property value bytes
242	fn property(&self, token_id: U256, key: string) -> Result<bytes> {242	fn property(&self, token_id: U256, key: String) -> Result<bytes> {
243		let token_id: u32 = token_id.try_into().map_err(|_| "token id overflow")?;243		let token_id: u32 = token_id.try_into().map_err(|_| "token id overflow")?;
244		let key = <Vec<u8>>::from(key)244		let key = <Vec<u8>>::from(key)
245			.try_into()245			.try_into()
301	/// @notice A descriptive name for a collection of NFTs in this contract301	/// @notice A descriptive name for a collection of NFTs in this contract
302	/// @dev real implementation of this function lies in `ERC721UniqueExtensions`302	/// @dev real implementation of this function lies in `ERC721UniqueExtensions`
303	#[solidity(hide, rename_selector = "name")]303	#[solidity(hide, rename_selector = "name")]
304	fn name_proxy(&self) -> Result<string> {304	fn name_proxy(&self) -> Result<String> {
305		self.name()305		self.name()
306	}306	}
307307
308	/// @notice An abbreviated name for NFTs in this contract308	/// @notice An abbreviated name for NFTs in this contract
309	/// @dev real implementation of this function lies in `ERC721UniqueExtensions`309	/// @dev real implementation of this function lies in `ERC721UniqueExtensions`
310	#[solidity(hide, rename_selector = "symbol")]310	#[solidity(hide, rename_selector = "symbol")]
311	fn symbol_proxy(&self) -> Result<string> {311	fn symbol_proxy(&self) -> Result<String> {
312		self.symbol()312		self.symbol()
313	}313	}
314314
322	///322	///
323	/// @return token's const_metadata323	/// @return token's const_metadata
324	#[solidity(rename_selector = "tokenURI")]324	#[solidity(rename_selector = "tokenURI")]
325	fn token_uri(&self, token_id: U256) -> Result<string> {325	fn token_uri(&self, token_id: U256) -> Result<String> {
326		let token_id_u32: u32 = token_id.try_into().map_err(|_| "token id overflow")?;326		let token_id_u32: u32 = token_id.try_into().map_err(|_| "token id overflow")?;
327327
328		match get_token_property(self, token_id_u32, &key::url()).as_deref() {328		match get_token_property(self, token_id_u32, &key::url()).as_deref() {
335		let base_uri =335		let base_uri =
336			pallet_common::Pallet::<T>::get_collection_property(self.id, &key::base_uri())336			pallet_common::Pallet::<T>::get_collection_property(self.id, &key::base_uri())
337				.map(BoundedVec::into_inner)337				.map(BoundedVec::into_inner)
338				.map(string::from_utf8)338				.map(String::from_utf8)
339				.transpose()339				.transpose()
340				.map_err(|e| {340				.map_err(|e| {
341					Error::Revert(alloc::format!(341					Error::Revert(alloc::format!(
595		&mut self,595		&mut self,
596		caller: caller,596		caller: caller,
597		to: Address,597		to: Address,
598		token_uri: string,598		token_uri: String,
599	) -> Result<U256> {599	) -> Result<U256> {
600		let token_id: U256 = <TokensMinted<T>>::get(self.id)600		let token_id: U256 = <TokensMinted<T>>::get(self.id)
601			.checked_add(1)601			.checked_add(1)
618		caller: caller,618		caller: caller,
619		to: Address,619		to: Address,
620		token_id: U256,620		token_id: U256,
621		token_uri: string,621		token_uri: String,
622	) -> Result<bool> {622	) -> Result<bool> {
623		let key = key::url();623		let key = key::url();
624		let permission = get_token_permission::<T>(self.id, &key)?;624		let permission = get_token_permission::<T>(self.id, &key)?;
670	collection: &CollectionHandle<T>,670	collection: &CollectionHandle<T>,
671	token_id: u32,671	token_id: u32,
672	key: &up_data_structs::PropertyKey,672	key: &up_data_structs::PropertyKey,
673) -> Result<string> {673) -> Result<String> {
674	collection.consume_store_reads(1)?;674	collection.consume_store_reads(1)?;
675	let properties = <TokenProperties<T>>::try_get((collection.id, token_id))675	let properties = <TokenProperties<T>>::try_get((collection.id, token_id))
676		.map_err(|_| Error::Revert("Token properties not found".into()))?;676		.map_err(|_| Error::Revert("Token properties not found".into()))?;
677	if let Some(property) = properties.get(key) {677	if let Some(property) = properties.get(key) {
678		return Ok(string::from_utf8_lossy(property).into());678		return Ok(String::from_utf8_lossy(property).into());
679	}679	}
680680
681	Err("Property tokenURI not found".into())681	Err("Property tokenURI not found".into())
691		.get(key)691		.get(key)
692		.map(Clone::clone)692		.map(Clone::clone)
693		.ok_or_else(|| {693		.ok_or_else(|| {
694			let key = string::from_utf8(key.clone().into_inner()).unwrap_or_default();694			let key = String::from_utf8(key.clone().into_inner()).unwrap_or_default();
695			Error::Revert(alloc::format!("No permission for key {}", key))695			Error::Revert(alloc::format!("No permission for key {}", key))
696		})?;696		})?;
697	Ok(a)697	Ok(a)
704	T::AccountId: From<[u8; 32]> + AsRef<[u8; 32]>,704	T::AccountId: From<[u8; 32]> + AsRef<[u8; 32]>,
705{705{
706	/// @notice A descriptive name for a collection of NFTs in this contract706	/// @notice A descriptive name for a collection of NFTs in this contract
707	fn name(&self) -> Result<string> {707	fn name(&self) -> Result<String> {
708		Ok(decode_utf16(self.name.iter().copied())708		Ok(decode_utf16(self.name.iter().copied())
709			.map(|r| r.unwrap_or(REPLACEMENT_CHARACTER))709			.map(|r| r.unwrap_or(REPLACEMENT_CHARACTER))
710			.collect::<string>())710			.collect::<String>())
711	}711	}
712712
713	/// @notice An abbreviated name for NFTs in this contract713	/// @notice An abbreviated name for NFTs in this contract
714	fn symbol(&self) -> Result<string> {714	fn symbol(&self) -> Result<String> {
715		Ok(string::from_utf8_lossy(&self.token_prefix).into())715		Ok(String::from_utf8_lossy(&self.token_prefix).into())
716	}716	}
717717
718	/// @notice A description for the collection.718	/// @notice A description for the collection.
719	fn description(&self) -> Result<string> {719	fn description(&self) -> Result<String> {
720		Ok(decode_utf16(self.description.iter().copied())720		Ok(decode_utf16(self.description.iter().copied())
721			.map(|r| r.unwrap_or(REPLACEMENT_CHARACTER))721			.map(|r| r.unwrap_or(REPLACEMENT_CHARACTER))
722			.collect::<string>())722			.collect::<String>())
723	}723	}
724724
725	/// Returns the owner (in cross format) of the token.725	/// Returns the owner (in cross format) of the token.
736	/// @param tokenId Id for the token.736	/// @param tokenId Id for the token.
737	/// @param keys Properties keys. Empty keys for all propertyes.737	/// @param keys Properties keys. Empty keys for all propertyes.
738	/// @return Vector of properties key/value pairs.738	/// @return Vector of properties key/value pairs.
739	fn properties(&self, token_id: U256, keys: Vec<string>) -> Result<Vec<eth::Property>> {739	fn properties(&self, token_id: U256, keys: Vec<String>) -> Result<Vec<eth::Property>> {
740		let keys = keys740		let keys = keys
741			.into_iter()741			.into_iter()
742			.map(|key| {742			.map(|key| {
948		&mut self,948		&mut self,
949		caller: caller,949		caller: caller,
950		to: Address,950		to: Address,
951		tokens: Vec<(U256, string)>,951		tokens: Vec<(U256, String)>,
952	) -> Result<bool> {952	) -> Result<bool> {
953		let key = key::url();953		let key = key::url();
954		let caller = T::CrossAccountId::from_eth(caller);954		let caller = T::CrossAccountId::from_eth(caller);

--- a/pallets/refungible/src/erc.rs
+++ b/pallets/refungible/src/erc.rs
@@ -67,7 +67,7 @@
 	fn set_token_property_permission(
 		&mut self,
 		caller: caller,
-		key: string,
+		key: String,
 		is_mutable: bool,
 		collection_admin: bool,
 		token_owner: bool,
@@ -126,7 +126,7 @@
 		&mut self,
 		caller: caller,
 		token_id: U256,
-		key: string,
+		key: String,
 		value: bytes,
 	) -> Result<()> {
 		let caller = T::CrossAccountId::from_eth(caller);
@@ -190,7 +190,7 @@
 	/// @param key Property key.
 	#[solidity(hide)]
 	#[weight(<SelfWeightOf<T>>::delete_token_properties(1))]
-	fn delete_property(&mut self, token_id: U256, caller: caller, key: string) -> Result<()> {
+	fn delete_property(&mut self, token_id: U256, caller: caller, key: String) -> Result<()> {
 		let caller = T::CrossAccountId::from_eth(caller);
 		let token_id: u32 = token_id.try_into().map_err(|_| "token id overflow")?;
 		let key = <Vec<u8>>::from(key)
@@ -214,7 +214,7 @@
 		&mut self,
 		token_id: U256,
 		caller: caller,
-		keys: Vec<string>,
+		keys: Vec<String>,
 	) -> Result<()> {
 		let caller = T::CrossAccountId::from_eth(caller);
 		let token_id: u32 = token_id.try_into().map_err(|_| "token id overflow")?;
@@ -242,7 +242,7 @@
 	/// @param tokenId ID of the token.
 	/// @param key Property key.
 	/// @return Property value bytes
-	fn property(&self, token_id: U256, key: string) -> Result<bytes> {
+	fn property(&self, token_id: U256, key: String) -> Result<bytes> {
 		let token_id: u32 = token_id.try_into().map_err(|_| "token id overflow")?;
 		let key = <Vec<u8>>::from(key)
 			.try_into()
@@ -298,14 +298,14 @@
 	/// @notice A descriptive name for a collection of NFTs in this contract
 	/// @dev real implementation of this function lies in `ERC721UniqueExtensions`
 	#[solidity(hide, rename_selector = "name")]
-	fn name_proxy(&self) -> Result<string> {
+	fn name_proxy(&self) -> Result<String> {
 		self.name()
 	}
 
 	/// @notice An abbreviated name for NFTs in this contract
 	/// @dev real implementation of this function lies in `ERC721UniqueExtensions`
 	#[solidity(hide, rename_selector = "symbol")]
-	fn symbol_proxy(&self) -> Result<string> {
+	fn symbol_proxy(&self) -> Result<String> {
 		self.symbol()
 	}
 
@@ -319,7 +319,7 @@
 	///
 	/// @return token's const_metadata
 	#[solidity(rename_selector = "tokenURI")]
-	fn token_uri(&self, token_id: U256) -> Result<string> {
+	fn token_uri(&self, token_id: U256) -> Result<String> {
 		let token_id_u32: u32 = token_id.try_into().map_err(|_| "token id overflow")?;
 
 		match get_token_property(self, token_id_u32, &key::url()).as_deref() {
@@ -332,7 +332,7 @@
 		let base_uri =
 			pallet_common::Pallet::<T>::get_collection_property(self.id, &key::base_uri())
 				.map(BoundedVec::into_inner)
-				.map(string::from_utf8)
+				.map(String::from_utf8)
 				.transpose()
 				.map_err(|e| {
 					Error::Revert(alloc::format!(
@@ -631,7 +631,7 @@
 		&mut self,
 		caller: caller,
 		to: Address,
-		token_uri: string,
+		token_uri: String,
 	) -> Result<U256> {
 		let token_id: U256 = <TokensMinted<T>>::get(self.id)
 			.checked_add(1)
@@ -654,7 +654,7 @@
 		caller: caller,
 		to: Address,
 		token_id: U256,
-		token_uri: string,
+		token_uri: String,
 	) -> Result<bool> {
 		let key = key::url();
 		let permission = get_token_permission::<T>(self.id, &key)?;
@@ -708,12 +708,12 @@
 	collection: &CollectionHandle<T>,
 	token_id: u32,
 	key: &up_data_structs::PropertyKey,
-) -> Result<string> {
+) -> Result<String> {
 	collection.consume_store_reads(1)?;
 	let properties = <TokenProperties<T>>::try_get((collection.id, token_id))
 		.map_err(|_| Error::Revert("Token properties not found".into()))?;
 	if let Some(property) = properties.get(key) {
-		return Ok(string::from_utf8_lossy(property).into());
+		return Ok(String::from_utf8_lossy(property).into());
 	}
 
 	Err("Property tokenURI not found".into())
@@ -729,7 +729,7 @@
 		.get(key)
 		.map(Clone::clone)
 		.ok_or_else(|| {
-			let key = string::from_utf8(key.clone().into_inner()).unwrap_or_default();
+			let key = String::from_utf8(key.clone().into_inner()).unwrap_or_default();
 			Error::Revert(alloc::format!("No permission for key {}", key))
 		})?;
 	Ok(a)
@@ -742,22 +742,22 @@
 	T::AccountId: From<[u8; 32]> + AsRef<[u8; 32]>,
 {
 	/// @notice A descriptive name for a collection of NFTs in this contract
-	fn name(&self) -> Result<string> {
+	fn name(&self) -> Result<String> {
 		Ok(decode_utf16(self.name.iter().copied())
 			.map(|r| r.unwrap_or(REPLACEMENT_CHARACTER))
-			.collect::<string>())
+			.collect::<String>())
 	}
 
 	/// @notice An abbreviated name for NFTs in this contract
-	fn symbol(&self) -> Result<string> {
-		Ok(string::from_utf8_lossy(&self.token_prefix).into())
+	fn symbol(&self) -> Result<String> {
+		Ok(String::from_utf8_lossy(&self.token_prefix).into())
 	}
 
 	/// @notice A description for the collection.
-	fn description(&self) -> Result<string> {
+	fn description(&self) -> Result<String> {
 		Ok(decode_utf16(self.description.iter().copied())
 			.map(|r| r.unwrap_or(REPLACEMENT_CHARACTER))
-			.collect::<string>())
+			.collect::<String>())
 	}
 
 	/// Returns the owner (in cross format) of the token.
@@ -774,7 +774,7 @@
 	/// @param tokenId Id for the token.
 	/// @param keys Properties keys. Empty keys for all propertyes.
 	/// @return Vector of properties key/value pairs.
-	fn properties(&self, token_id: U256, keys: Vec<string>) -> Result<Vec<eth::Property>> {
+	fn properties(&self, token_id: U256, keys: Vec<String>) -> Result<Vec<eth::Property>> {
 		let keys = keys
 			.into_iter()
 			.map(|key| {
@@ -991,7 +991,7 @@
 		&mut self,
 		caller: caller,
 		to: Address,
-		tokens: Vec<(U256, string)>,
+		tokens: Vec<(U256, String)>,
 	) -> Result<bool> {
 		let key = key::url();
 		let caller = T::CrossAccountId::from_eth(caller);

--- a/pallets/refungible/src/erc_token.rs
+++ b/pallets/refungible/src/erc_token.rs
@@ -92,15 +92,15 @@
 #[solidity_interface(name = ERC20, events(ERC20Events))]
 impl<T: Config> RefungibleTokenHandle<T> {
 	/// @return the name of the token.
-	fn name(&self) -> Result<string> {
+	fn name(&self) -> Result<String> {
 		Ok(decode_utf16(self.name.iter().copied())
 			.map(|r| r.unwrap_or(REPLACEMENT_CHARACTER))
-			.collect::<string>())
+			.collect::<String>())
 	}
 
 	/// @return the symbol of the token.
-	fn symbol(&self) -> Result<string> {
-		Ok(string::from_utf8_lossy(&self.token_prefix).into())
+	fn symbol(&self) -> Result<String> {
+		Ok(String::from_utf8_lossy(&self.token_prefix).into())
 	}
 
 	/// @dev Total number of tokens in existence

--- a/pallets/unique/src/eth/mod.rs
+++ b/pallets/unique/src/eth/mod.rs
@@ -57,9 +57,9 @@
 
 fn convert_data<T: Config>(
 	caller: caller,
-	name: string,
-	description: string,
-	token_prefix: string,
+	name: String,
+	description: String,
+	token_prefix: String,
 ) -> Result<(
 	T::CrossAccountId,
 	CollectionName,
@@ -89,10 +89,10 @@
 fn create_collection_internal<T: Config>(
 	caller: caller,
 	value: value,
-	name: string,
+	name: String,
 	collection_mode: CollectionMode,
-	description: string,
-	token_prefix: string,
+	description: String,
+	token_prefix: String,
 ) -> Result<Address> {
 	let (caller, name, description, token_prefix) =
 		convert_data::<T>(caller, name, description, token_prefix)?;
@@ -151,9 +151,9 @@
 		&mut self,
 		caller: caller,
 		value: value,
-		name: string,
-		description: string,
-		token_prefix: string,
+		name: String,
+		description: String,
+		token_prefix: String,
 	) -> Result<Address> {
 		let (caller, name, description, token_prefix) =
 			convert_data::<T>(caller, name, description, token_prefix)?;
@@ -190,9 +190,9 @@
 		&mut self,
 		caller: caller,
 		value: value,
-		name: string,
-		description: string,
-		token_prefix: string,
+		name: String,
+		description: String,
+		token_prefix: String,
 	) -> Result<Address> {
 		create_collection_internal::<T>(
 			caller,
@@ -210,9 +210,9 @@
 		&mut self,
 		caller: caller,
 		value: value,
-		name: string,
-		description: string,
-		token_prefix: string,
+		name: String,
+		description: String,
+		token_prefix: String,
 	) -> Result<Address> {
 		create_collection_internal::<T>(
 			caller,
@@ -230,10 +230,10 @@
 		&mut self,
 		caller: caller,
 		value: value,
-		name: string,
+		name: String,
 		decimals: u8,
-		description: string,
-		token_prefix: string,
+		description: String,
+		token_prefix: String,
 	) -> Result<Address> {
 		create_collection_internal::<T>(
 			caller,
@@ -250,7 +250,7 @@
 		&mut self,
 		caller: caller,
 		collection: Address,
-		base_uri: string,
+		base_uri: String,
 	) -> Result<()> {
 		let caller = T::CrossAccountId::from_eth(caller);
 		let collection =

--- a/runtime/tests/src/tests.rs
+++ b/runtime/tests/src/tests.rs
@@ -323,7 +323,7 @@
 				.map(|d| { d.into() })
 				.collect()
 		));
-		for (index, data) in items_data.into_iter().enumerate() {
+		for (index, _data) in items_data.into_iter().enumerate() {
 			let balance = <pallet_refungible::Balance<Test>>::get((
 				CollectionId(1),
 				TokenId((index + 1) as u32),