--- a/pallets/nft/src/sponsorship.rs +++ b/pallets/nft/src/sponsorship.rs @@ -21,17 +21,16 @@ who: &T::AccountId, item_id: &TokenId, ) -> Option<()> { - // preliminary sponsoring correctness check - if !((pallet_nonfungible::TokenData::::get((collection.id, item_id))?.owner).as_sub() - == who) || (pallet_refungible::Owned::::get(( - collection.id, - T::CrossAccountId::from_sub(who.clone()), - item_id, - ))) { + if !((pallet_nonfungible::TokenData::::get((collection.id, item_id))?.owner).as_sub() == who) + || (pallet_refungible::Owned::::get(( + collection.id, + T::CrossAccountId::from_sub(who.clone()), + item_id, + ))) { return None; } - + // sponsor timeout let block_number = >::block_number() as T::BlockNumber; let limit = collection @@ -107,14 +106,13 @@ item_id: &TokenId, data: &[u8], ) -> Option<()> { - // preliminary sponsoring correctness check - if !((pallet_nonfungible::TokenData::::get((collection.id, item_id))?.owner).as_sub() - == who) || (pallet_refungible::Owned::::get(( - collection.id, - T::CrossAccountId::from_sub(who.clone()), - item_id, - ))) { + if !((pallet_nonfungible::TokenData::::get((collection.id, item_id))?.owner).as_sub() == who) + || (pallet_refungible::Owned::::get(( + collection.id, + T::CrossAccountId::from_sub(who.clone()), + item_id, + ))) { return None; } @@ -228,7 +226,8 @@ data, } => { let (sponsor, collection) = load(*collection_id)?; - withdraw_set_variable_meta_data::(&who, &collection, item_id, data).map(|()| sponsor) + withdraw_set_variable_meta_data::(&who, &collection, item_id, data) + .map(|()| sponsor) } _ => None, }