difftreelog
fix(rmrk) sending nft
in: master
5 files changed
pallets/proxy-rmrk-core/src/lib.rsdiffbeforeafterboth--- a/pallets/proxy-rmrk-core/src/lib.rs
+++ b/pallets/proxy-rmrk-core/src/lib.rs
@@ -25,7 +25,7 @@
Pallet as PalletCommon, Error as CommonError, CollectionHandle, CommonCollectionOperations,
};
use pallet_nonfungible::{Pallet as PalletNft, NonfungibleHandle, TokenData};
-use pallet_structure::Pallet as PalletStructure;
+use pallet_structure::{Pallet as PalletStructure, Error as StructureError};
use pallet_evm::account::CrossAccountId;
use core::convert::AsRef;
@@ -128,8 +128,10 @@
NoAvailableNftId,
CollectionUnknown,
NoPermission,
+ NonTransferable,
CollectionFullOrLocked,
ResourceDoesntExist,
+ CannotSendToDescendentOrSelf,
}
#[pallet::call]
@@ -207,7 +209,7 @@
);
<PalletNft<T>>::destroy_collection(collection, &cross_sender)
- .map_err(Self::map_common_err_to_proxy)?;
+ .map_err(Self::map_unique_err_to_proxy)?;
Self::deposit_event(Event::CollectionDestroyed {
issuer: sender,
@@ -283,6 +285,7 @@
recipient: Option<T::AccountId>,
royalty_amount: Option<Permill>,
metadata: RmrkString,
+ transferable: bool,
) -> DispatchResult {
let sender = ensure_signed(origin)?;
let sender = T::CrossAccountId::from_sub(sender);
@@ -304,6 +307,8 @@
&collection,
[
Self::rmrk_property(TokenType, &NftType::Regular)?,
+ Self::rmrk_property(Transferable, &transferable)?,
+ Self::rmrk_property(PendingNftAccept, &false)?,
Self::rmrk_property(RoyaltyInfo, &royalty_info)?,
Self::rmrk_property(Metadata, &metadata)?,
Self::rmrk_property(Equipped, &false)?,
@@ -327,7 +332,7 @@
)
.map_err(|err| match err {
DispatchError::Arithmetic(_) => <Error<T>>::NoAvailableNftId.into(),
- err => Self::map_common_err_to_proxy(err),
+ err => Self::map_unique_err_to_proxy(err),
})?;
Self::deposit_event(Event::NftMinted {
@@ -373,7 +378,7 @@
new_owner: RmrkAccountIdOrCollectionNftTuple<T::AccountId>,
) -> DispatchResult {
let sender = ensure_signed(origin.clone())?;
- let cross_sender = T::CrossAccountId::from_sub(sender.clone());
+ let cross_sender = T::CrossAccountId::from_sub(sender);
let collection_id = Self::unique_collection_id(rmrk_collection_id)?;
let nft_id = rmrk_nft_id.into();
@@ -388,8 +393,14 @@
misc::CollectionType::Regular,
)?;
- let budget = budget::Value::new(NESTING_BUDGET);
+ if !Self::get_nft_property_decoded(collection_id, nft_id, RmrkProperty::Transferable)? {
+ return Err(<Error<T>>::NonTransferable.into());
+ }
+ if Self::get_nft_property_decoded(collection_id, nft_id, RmrkProperty::PendingNftAccept)? {
+ return Err(<Error<T>>::NoPermission.into());
+ }
+
let target_owner;
match new_owner {
@@ -399,43 +410,45 @@
RmrkAccountIdOrCollectionNftTuple::CollectionAndNftTuple(target_collection_id, target_nft_id) => {
let target_collection_id = Self::unique_collection_id(target_collection_id)?;
- target_owner = T::CrossTokenAddressMapping::token_to_address(
- target_collection_id,
- target_nft_id.into(),
- );
+ let target_nft_budget = budget::Value::new(NESTING_BUDGET);
- let spender = <PalletStructure<T>>::get_indirect_owner(
+ let target_nft_owner = <PalletStructure<T>>::get_checked_indirect_owner(
target_collection_id,
target_nft_id.into(),
Some((collection_id, nft_id)),
- &budget,
- )?;
+ &target_nft_budget,
+ ).map_err(Self::map_unique_err_to_proxy)?;
- let is_approval_required = cross_sender != spender;
+ let is_approval_required = cross_sender != target_nft_owner;
if is_approval_required {
- // FIXME
- // <PalletNft<T>>::set_allowance(
- // &collection,
- // &cross_sender,
- // nft_id,
- // Some(&spender),
- // &budget
- // ).map_err(Self::map_common_err_to_proxy)?;
+ target_owner = target_nft_owner;
- return Ok(());
+ <PalletNft<T>>::set_scoped_token_property(
+ collection.id,
+ nft_id,
+ PropertyScope::Rmrk,
+ Self::rmrk_property(PendingNftAccept, &is_approval_required)?,
+ )?;
+ } else {
+ target_owner = T::CrossTokenAddressMapping::token_to_address(
+ target_collection_id,
+ target_nft_id.into(),
+ );
}
}
}
+ let src_nft_budget = budget::Value::new(NESTING_BUDGET);
+
<PalletNft<T>>::transfer_from(
&collection,
&cross_sender,
&from,
&target_owner,
nft_id,
- &budget
- ).map_err(Self::map_common_err_to_proxy)?;
+ &src_nft_budget
+ ).map_err(Self::map_unique_err_to_proxy)?;
Ok(())
}
@@ -719,7 +732,7 @@
let collection = Self::get_typed_nft_collection(collection_id, collection_type)?;
<PalletNft<T>>::burn(&collection, &sender, token_id)
- .map_err(Self::map_common_err_to_proxy)?;
+ .map_err(Self::map_unique_err_to_proxy)?;
Ok(())
}
@@ -762,7 +775,7 @@
)
.map_err(|err| match err {
DispatchError::Arithmetic(_) => <Error<T>>::NoAvailableNftId.into(),
- err => Self::map_common_err_to_proxy(err),
+ err => Self::map_unique_err_to_proxy(err),
})?;
Ok(resource_id.0)
@@ -794,7 +807,7 @@
let sender = T::CrossAccountId::from_sub(sender);
if topmost_owner == sender {
<PalletNft<T>>::burn(&resource_collection, &sender, resource_id)
- .map_err(Self::map_common_err_to_proxy)?;
+ .map_err(Self::map_unique_err_to_proxy)?;
} else {
<PalletNft<T>>::set_scoped_token_property(
resource_collection_id,
@@ -828,7 +841,7 @@
) -> DispatchResult {
collection
.check_is_owner(account)
- .map_err(Self::map_common_err_to_proxy)
+ .map_err(Self::map_unique_err_to_proxy)
}
pub fn last_collection_idx() -> RmrkCollectionId {
@@ -1064,14 +1077,16 @@
Ok(properties)
}
- fn map_common_err_to_proxy(err: DispatchError) -> DispatchError {
- map_common_err_to_proxy! {
+ fn map_unique_err_to_proxy(err: DispatchError) -> DispatchError {
+ map_unique_err_to_proxy! {
match err {
- NoPermission => NoPermission,
- CollectionTokenLimitExceeded => CollectionFullOrLocked,
- PublicMintingNotAllowed => NoPermission,
- TokenNotFound => NoAvailableNftId,
- ApprovedValueTooLow => NoPermission
+ CommonError::NoPermission => NoPermission,
+ CommonError::CollectionTokenLimitExceeded => CollectionFullOrLocked,
+ CommonError::PublicMintingNotAllowed => NoPermission,
+ CommonError::TokenNotFound => NoAvailableNftId,
+ CommonError::ApprovedValueTooLow => NoPermission,
+ StructureError::TokenNotFound => NoAvailableNftId,
+ StructureError::OuroborosDetected => CannotSendToDescendentOrSelf
}
}
}
pallets/proxy-rmrk-core/src/misc.rsdiffbeforeafterboth--- a/pallets/proxy-rmrk-core/src/misc.rs
+++ b/pallets/proxy-rmrk-core/src/misc.rs
@@ -2,10 +2,10 @@
use codec::{Encode, Decode, Error};
#[macro_export]
-macro_rules! map_common_err_to_proxy {
- (match $err:ident { $($common_err:ident => $proxy_err:ident),+ }) => {
+macro_rules! map_unique_err_to_proxy {
+ (match $err:ident { $($unique_err_ty:ident :: $unique_err:ident => $proxy_err:ident),+ }) => {
$(
- if $err == <CommonError<T>>::$common_err.into() {
+ if $err == <$unique_err_ty<T>>::$unique_err.into() {
return <Error<T>>::$proxy_err.into()
} else
)+ {
pallets/proxy-rmrk-core/src/property.rsdiffbeforeafterboth--- a/pallets/proxy-rmrk-core/src/property.rs
+++ b/pallets/proxy-rmrk-core/src/property.rs
@@ -5,11 +5,13 @@
Metadata,
CollectionType,
TokenType,
+ Transferable,
RoyaltyInfo,
Equipped,
ResourceCollection,
ResourcePriorities,
ResourceType,
+ PendingNftAccept,
PendingResourceAccept,
PendingResourceRemoval,
Parts,
@@ -49,13 +51,15 @@
Self::Metadata => key!("metadata"),
Self::CollectionType => key!("collection-type"),
Self::TokenType => key!("token-type"),
+ Self::Transferable => key!("transferable"),
Self::RoyaltyInfo => key!("royalty-info"),
Self::Equipped => key!("equipped"),
Self::ResourceCollection => key!("resource-collection"),
Self::ResourcePriorities => key!("resource-priorities"),
Self::ResourceType => key!("resource-type"),
- Self::PendingResourceAccept => key!("pending-accept"),
- Self::PendingResourceRemoval => key!("pending-removal"),
+ Self::PendingNftAccept => key!("pending-nft-accept"),
+ Self::PendingResourceAccept => key!("pending-resource-accept"),
+ Self::PendingResourceRemoval => key!("pending-resource-removal"),
Self::Parts => key!("parts"),
Self::Base => key!("base"),
Self::Src => key!("src"),
pallets/structure/src/lib.rsdiffbeforeafterboth--- a/pallets/structure/src/lib.rs
+++ b/pallets/structure/src/lib.rs
@@ -149,7 +149,7 @@
})
}
- pub fn get_indirect_owner(
+ pub fn get_checked_indirect_owner(
collection: CollectionId,
token: TokenId,
for_nest: Option<(CollectionId, TokenId)>,
@@ -203,7 +203,7 @@
None => user,
};
- Self::get_indirect_owner(
+ Self::get_checked_indirect_owner(
collection,
token,
for_nest,
runtime/common/src/runtime_apis.rsdiffbeforeafterboth188 None => return Ok(None)188 None => return Ok(None)189 };189 };190191 let allowance = pallet_nonfungible::Allowance::<Runtime>::get((collection_id, nft_id));192190193 Ok(Some(RmrkInstanceInfo {191 Ok(Some(RmrkInstanceInfo {194 owner: owner,192 owner: owner,195 royalty: RmrkCore::get_nft_property_decoded(collection_id, nft_id, RmrkProperty::RoyaltyInfo)?,193 royalty: RmrkCore::get_nft_property_decoded(collection_id, nft_id, RmrkProperty::RoyaltyInfo)?,196 metadata: RmrkCore::get_nft_property_decoded(collection_id, nft_id, RmrkProperty::Metadata)?,194 metadata: RmrkCore::get_nft_property_decoded(collection_id, nft_id, RmrkProperty::Metadata)?,197 equipped: RmrkCore::get_nft_property_decoded(collection_id, nft_id, RmrkProperty::Equipped)?,195 equipped: RmrkCore::get_nft_property_decoded(collection_id, nft_id, RmrkProperty::Equipped)?,198 pending: allowance.is_some(),196 pending: RmrkCore::get_nft_property_decoded(collection_id, nft_id, RmrkProperty::PendingNftAccept)?,199 }))197 }))200 }198 }201199202 fn account_tokens(account_id: AccountId, collection_id: RmrkCollectionId) -> Result<Vec<RmrkNftId>, DispatchError> {200 fn account_tokens(account_id: AccountId, collection_id: RmrkCollectionId) -> Result<Vec<RmrkNftId>, DispatchError> {203 use pallet_proxy_rmrk_core::misc::CollectionType;201 use pallet_proxy_rmrk_core::{RmrkProperty, misc::CollectionType};204 use pallet_common::CommonCollectionOperations;202 use pallet_common::CommonCollectionOperations;205203206 let cross_account_id = CrossAccountId::from_sub(account_id);204 let cross_account_id = CrossAccountId::from_sub(account_id);210 Err(_) => return Ok(Vec::new()),208 Err(_) => return Ok(Vec::new()),211 };209 };212210213 Ok(214 collection.account_tokens(cross_account_id)211 let tokens = collection.account_tokens(cross_account_id)215 .into_iter()212 .into_iter()213 .filter(|token| {214 let is_pending = RmrkCore::get_nft_property_decoded(215 collection_id,216 *token,217 RmrkProperty::PendingNftAccept218 ).unwrap_or(true);219220 !is_pending221 })216 .map(|token| token.0)222 .map(|token| token.0)217 .collect()223 .collect();218 )224225 Ok(tokens)219 }226 }220227221 fn nft_children(collection_id: RmrkCollectionId, nft_id: RmrkNftId) -> Result<Vec<RmrkNftChild>, DispatchError> {228 fn nft_children(collection_id: RmrkCollectionId, nft_id: RmrkNftId) -> Result<Vec<RmrkNftChild>, DispatchError> {229 use pallet_proxy_rmrk_core::RmrkProperty;230222 let collection_id = match RmrkCore::unique_collection_id(collection_id) {231 let collection_id = match RmrkCore::unique_collection_id(collection_id) {223 Ok(id) => id,232 Ok(id) => id,229 Ok(238 Ok(230 pallet_nonfungible::TokenChildren::<Runtime>::iter_prefix((collection_id, nft_id))239 pallet_nonfungible::TokenChildren::<Runtime>::iter_prefix((collection_id, nft_id))231 .filter_map(|((child_collection, child_token), _)| {240 .filter_map(|((child_collection, child_token), _)| {241 let is_pending = RmrkCore::get_nft_property_decoded(242 child_collection,243 child_token,244 RmrkProperty::PendingNftAccept245 ).ok()?;246247 if is_pending {248 return None;249 }250232 let rmrk_child_collection = RmrkCore::rmrk_collection_id(251 let rmrk_child_collection = RmrkCore::rmrk_collection_id(233 child_collection252 child_collection