difftreelog
Add change_property_permissions
in: master
10 files changed
pallets/common/src/lib.rsdiffbeforeafterboth--- a/pallets/common/src/lib.rs
+++ b/pallets/common/src/lib.rs
@@ -36,7 +36,7 @@
CUSTOM_DATA_LIMIT, CollectionLimits, CustomDataLimit, CreateCollectionData, SponsorshipState,
CreateItemExData, SponsoringRateLimit, budget::Budget, COLLECTION_FIELD_LIMIT, CollectionField,
PhantomType, Property, Properties, PropertiesPermissionMap, PropertyKey, PropertyPermission,
- PropertiesError,
+ PropertiesError, PropertyKeyPermission,
};
pub use pallet::*;
use sp_core::H160;
@@ -293,6 +293,8 @@
CollectionPropertySet(CollectionId, Property),
TokenPropertySet(CollectionId, TokenId, Property),
+
+ PropertyPermissionSet(CollectionId, PropertyKeyPermission),
}
#[pallet::error]
@@ -741,7 +743,7 @@
|properties| properties.try_change_property(property.clone())
)?;
- <Pallet<T>>::deposit_event(Event::CollectionPropertySet(collection.id, property));
+ Self::deposit_event(Event::CollectionPropertySet(collection.id, property));
Ok(())
}
@@ -761,19 +763,39 @@
pub fn change_property_permission(
collection: &CollectionHandle<T>,
sender: &T::CrossAccountId,
- property_key: PropertyKey,
- permission: PropertyPermission,
+ property_permission: PropertyKeyPermission
) -> DispatchResult {
collection.check_is_owner_or_admin(sender)?;
+ let all_permissions = CollectionPropertyPermissions::<T>::get(collection.id);
+ let current_permission = all_permissions.get(&property_permission.key);
+ if matches![current_permission, Some(PropertyPermission::AdminConst | PropertyPermission::ItemOwnerConst)] {
+ return Err(<Error<T>>::NoPermission.into());
+ }
+
CollectionPropertyPermissions::<T>::try_mutate(collection.id, |permissions| {
- permissions.try_insert(property_key, permission)
+ let property_permission = property_permission.clone();
+ permissions.try_insert(property_permission.key, property_permission.permission)
})
.map_err(|_| PropertiesError::PropertyLimitReached)?;
+ Self::deposit_event(Event::PropertyPermissionSet(collection.id, property_permission));
+
Ok(())
}
+ pub fn change_property_permissions(
+ collection: &CollectionHandle<T>,
+ sender: &T::CrossAccountId,
+ property_permissions: Vec<PropertyKeyPermission>
+ ) -> DispatchResult {
+ for prop_pemission in property_permissions {
+ Self::change_property_permission(collection, sender, prop_pemission)?;
+ }
+
+ Ok(())
+ }
+
fn set_field_raw(
collection_id: CollectionId,
field: CollectionField,
@@ -928,6 +950,7 @@
fn burn_item() -> Weight;
fn change_collection_properties(amount: u32) -> Weight;
fn change_token_properties(amount: u32) -> Weight;
+ fn change_property_permissions(amount: u32) -> Weight;
fn transfer() -> Weight;
fn approve() -> Weight;
fn transfer_from() -> Weight;
@@ -962,20 +985,22 @@
token: TokenId,
amount: u128,
) -> DispatchResultWithPostInfo;
-
fn change_collection_properties(
&self,
sender: T::CrossAccountId,
properties: Vec<Property>,
) -> DispatchResultWithPostInfo;
-
fn change_token_properties(
&self,
sender: T::CrossAccountId,
token_id: TokenId,
property: Vec<Property>,
) -> DispatchResultWithPostInfo;
-
+ fn change_property_permissions(
+ &self,
+ sender: &T::CrossAccountId,
+ property_permissions: Vec<PropertyKeyPermission>,
+ ) -> DispatchResultWithPostInfo;
fn transfer(
&self,
sender: T::CrossAccountId,
pallets/fungible/src/common.rsdiffbeforeafterboth--- a/pallets/fungible/src/common.rs
+++ b/pallets/fungible/src/common.rs
@@ -21,7 +21,7 @@
use pallet_common::{CommonCollectionOperations, CommonWeightInfo, with_weight};
use sp_runtime::ArithmeticError;
use sp_std::{vec::Vec, vec};
-use up_data_structs::{CustomDataLimit, Property};
+use up_data_structs::{CustomDataLimit, Property, PropertyKeyPermission,};
use crate::{
Allowance, Balance, Config, Error, FungibleHandle, Pallet, SelfWeightOf, weights::WeightInfo,
@@ -58,6 +58,10 @@
<SelfWeightOf<T>>::change_token_properties(amount)
}
+ fn change_property_permissions(amount: u32) -> Weight {
+ <SelfWeightOf<T>>::change_property_permissions(amount)
+ }
+
fn transfer() -> Weight {
<SelfWeightOf<T>>::transfer()
}
@@ -250,6 +254,14 @@
fail!(<Error<T>>::PropertiesNotAllowed)
}
+ fn change_property_permissions(
+ &self,
+ _sender: &T::CrossAccountId,
+ _property_permissions: Vec<PropertyKeyPermission>,
+ ) -> DispatchResultWithPostInfo {
+ fail!(<Error<T>>::PropertiesNotAllowed)
+ }
+
fn set_variable_metadata(
&self,
_sender: T::CrossAccountId,
pallets/fungible/src/weights.rsdiffbeforeafterboth--- a/pallets/fungible/src/weights.rs
+++ b/pallets/fungible/src/weights.rs
@@ -37,6 +37,7 @@
fn burn_item() -> Weight;
fn change_collection_properties(amount: u32) -> Weight;
fn change_token_properties(amount: u32) -> Weight;
+ fn change_property_permissions(amount: u32) -> Weight;
fn transfer() -> Weight;
fn approve() -> Weight;
fn transfer_from() -> Weight;
@@ -82,6 +83,11 @@
0
}
+ fn change_property_permissions(amount: u32) -> Weight {
+ // Error
+ 0
+ }
+
// Storage: Fungible Balance (r:2 w:2)
fn transfer() -> Weight {
(17_713_000 as Weight)
@@ -150,6 +156,11 @@
0
}
+ fn change_property_permissions(amount: u32) -> Weight {
+ // Error
+ 0
+ }
+
// Storage: Fungible Balance (r:2 w:2)
fn transfer() -> Weight {
(17_713_000 as Weight)
pallets/nonfungible/src/common.rsdiffbeforeafterboth--- a/pallets/nonfungible/src/common.rs
+++ b/pallets/nonfungible/src/common.rs
@@ -18,7 +18,7 @@
use frame_support::{dispatch::DispatchResultWithPostInfo, ensure, fail, weights::Weight, BoundedVec};
use up_data_structs::{
- TokenId, CustomDataLimit, CreateItemExData, CollectionId, budget::Budget, Property,
+ TokenId, CustomDataLimit, CreateItemExData, CollectionId, budget::Budget, Property, PropertyKeyPermission,
};
use pallet_common::{CommonCollectionOperations, CommonWeightInfo, with_weight};
use sp_runtime::DispatchError;
@@ -58,6 +58,10 @@
<SelfWeightOf<T>>::change_token_properties(amount)
}
+ fn change_property_permissions(amount: u32) -> Weight {
+ <SelfWeightOf<T>>::change_property_permissions(amount)
+ }
+
fn transfer() -> Weight {
<SelfWeightOf<T>>::transfer()
}
@@ -176,6 +180,19 @@
)
}
+ fn change_property_permissions(
+ &self,
+ sender: &T::CrossAccountId,
+ property_permissions: Vec<PropertyKeyPermission>,
+ ) -> DispatchResultWithPostInfo {
+ let weight = <CommonWeights<T>>::change_property_permissions(property_permissions.len() as u32);
+
+ with_weight(
+ <Pallet<T>>::change_property_permissions(self, sender, property_permissions),
+ weight
+ )
+ }
+
fn burn_item(
&self,
sender: T::CrossAccountId,
pallets/nonfungible/src/lib.rsdiffbeforeafterboth1// Copyright 2019-2022 Unique Network (Gibraltar) Ltd.2// This file is part of Unique Network.34// Unique Network is free software: you can redistribute it and/or modify5// it under the terms of the GNU General Public License as published by6// the Free Software Foundation, either version 3 of the License, or7// (at your option) any later version.89// Unique Network is distributed in the hope that it will be useful,10// but WITHOUT ANY WARRANTY; without even the implied warranty of11// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the12// GNU General Public License for more details.1314// You should have received a copy of the GNU General Public License15// along with Unique Network. If not, see <http://www.gnu.org/licenses/>.1617#![cfg_attr(not(feature = "std"), no_std)]1819use erc::ERC721Events;20use frame_support::{BoundedVec, ensure, fail};21use up_data_structs::{22 AccessMode, CollectionId, CustomDataLimit, TokenId, CreateCollectionData, CreateNftExData,23 mapping::TokenAddressMapping, NestingRule, budget::Budget, Property, PropertyPermission,24};25use pallet_evm::account::CrossAccountId;26use pallet_common::{27 Error as CommonError, Pallet as PalletCommon, Event as CommonEvent, CollectionHandle,28 dispatch::CollectionDispatch,29};30use pallet_structure::Pallet as PalletStructure;31use pallet_evm_coder_substrate::{SubstrateRecorder, WithRecorder};32use sp_core::H160;33use sp_runtime::{ArithmeticError, DispatchError, DispatchResult};34use sp_std::{vec::Vec, vec};35use core::ops::Deref;36use sp_std::collections::btree_map::BTreeMap;37use codec::{Encode, Decode, MaxEncodedLen};38use scale_info::TypeInfo;3940pub use pallet::*;41#[cfg(feature = "runtime-benchmarks")]42pub mod benchmarking;43pub mod common;44pub mod erc;45pub mod weights;4647pub type CreateItemData<T> = CreateNftExData<<T as pallet_evm::account::Config>::CrossAccountId>;48pub(crate) type SelfWeightOf<T> = <T as Config>::WeightInfo;4950#[derive(Encode, Decode, TypeInfo, MaxEncodedLen)]51pub struct ItemData<CrossAccountId> {52 pub const_data: BoundedVec<u8, CustomDataLimit>,53 pub variable_data: BoundedVec<u8, CustomDataLimit>,54 pub owner: CrossAccountId,55}5657#[frame_support::pallet]58pub mod pallet {59 use super::*;60 use frame_support::{Blake2_128Concat, Twox64Concat, pallet_prelude::*, storage::Key};61 use up_data_structs::{CollectionId, TokenId};62 use super::weights::WeightInfo;6364 #[pallet::error]65 pub enum Error<T> {66 /// Not Nonfungible item data used to mint in Nonfungible collection.67 NotNonfungibleDataUsedToMintFungibleCollectionToken,68 /// Used amount > 1 with NFT69 NonfungibleItemsHaveNoAmount,70 }7172 #[pallet::config]73 pub trait Config:74 frame_system::Config + pallet_common::Config + pallet_structure::Config75 {76 type WeightInfo: WeightInfo;77 }7879 #[pallet::pallet]80 #[pallet::generate_store(pub(super) trait Store)]81 pub struct Pallet<T>(_);8283 #[pallet::storage]84 pub type TokensMinted<T: Config> =85 StorageMap<Hasher = Twox64Concat, Key = CollectionId, Value = u32, QueryKind = ValueQuery>;86 #[pallet::storage]87 pub type TokensBurnt<T: Config> =88 StorageMap<Hasher = Twox64Concat, Key = CollectionId, Value = u32, QueryKind = ValueQuery>;8990 #[pallet::storage]91 pub type TokenData<T: Config> = StorageNMap<92 Key = (Key<Twox64Concat, CollectionId>, Key<Twox64Concat, TokenId>),93 Value = ItemData<T::CrossAccountId>,94 QueryKind = OptionQuery,95 >;9697 #[pallet::storage]98 pub type TokenProperties<T: Config> = StorageNMap<99 Key = (Key<Twox64Concat, CollectionId>, Key<Twox64Concat, TokenId>),100 Value = up_data_structs::Properties,101 QueryKind = ValueQuery,102 OnEmpty = up_data_structs::TokenProperties,103 >;104105 /// Used to enumerate tokens owned by account106 #[pallet::storage]107 pub type Owned<T: Config> = StorageNMap<108 Key = (109 Key<Twox64Concat, CollectionId>,110 Key<Blake2_128Concat, T::CrossAccountId>,111 Key<Twox64Concat, TokenId>,112 ),113 Value = bool,114 QueryKind = ValueQuery,115 >;116117 #[pallet::storage]118 pub type AccountBalance<T: Config> = StorageNMap<119 Key = (120 Key<Twox64Concat, CollectionId>,121 Key<Blake2_128Concat, T::CrossAccountId>,122 ),123 Value = u32,124 QueryKind = ValueQuery,125 >;126127 #[pallet::storage]128 pub type Allowance<T: Config> = StorageNMap<129 Key = (Key<Twox64Concat, CollectionId>, Key<Twox64Concat, TokenId>),130 Value = T::CrossAccountId,131 QueryKind = OptionQuery,132 >;133}134135pub struct NonfungibleHandle<T: Config>(pallet_common::CollectionHandle<T>);136impl<T: Config> NonfungibleHandle<T> {137 pub fn cast(inner: pallet_common::CollectionHandle<T>) -> Self {138 Self(inner)139 }140 pub fn into_inner(self) -> pallet_common::CollectionHandle<T> {141 self.0142 }143}144impl<T: Config> WithRecorder<T> for NonfungibleHandle<T> {145 fn recorder(&self) -> &SubstrateRecorder<T> {146 self.0.recorder()147 }148 fn into_recorder(self) -> SubstrateRecorder<T> {149 self.0.into_recorder()150 }151}152impl<T: Config> Deref for NonfungibleHandle<T> {153 type Target = pallet_common::CollectionHandle<T>;154155 fn deref(&self) -> &Self::Target {156 &self.0157 }158}159160impl<T: Config> Pallet<T> {161 pub fn total_supply(collection: &NonfungibleHandle<T>) -> u32 {162 <TokensMinted<T>>::get(collection.id) - <TokensBurnt<T>>::get(collection.id)163 }164 pub fn token_exists(collection: &NonfungibleHandle<T>, token: TokenId) -> bool {165 <TokenData<T>>::contains_key((collection.id, token))166 }167}168169// unchecked calls skips any permission checks170impl<T: Config> Pallet<T> {171 pub fn init_collection(172 owner: T::AccountId,173 data: CreateCollectionData<T::AccountId>,174 ) -> Result<CollectionId, DispatchError> {175 <PalletCommon<T>>::init_collection(owner, data)176 }177 pub fn destroy_collection(178 collection: NonfungibleHandle<T>,179 sender: &T::CrossAccountId,180 ) -> DispatchResult {181 let id = collection.id;182183 // =========184185 PalletCommon::destroy_collection(collection.0, sender)?;186187 <TokenData<T>>::remove_prefix((id,), None);188 <Owned<T>>::remove_prefix((id,), None);189 <TokensMinted<T>>::remove(id);190 <TokensBurnt<T>>::remove(id);191 <Allowance<T>>::remove_prefix((id,), None);192 <AccountBalance<T>>::remove_prefix((id,), None);193 Ok(())194 }195196 pub fn burn(197 collection: &NonfungibleHandle<T>,198 sender: &T::CrossAccountId,199 token: TokenId,200 ) -> DispatchResult {201 let token_data =202 <TokenData<T>>::get((collection.id, token)).ok_or(<CommonError<T>>::TokenNotFound)?;203 ensure!(204 &token_data.owner == sender205 || (collection.limits.owner_can_transfer() && collection.is_owner_or_admin(sender)),206 <CommonError<T>>::NoPermission207 );208209 if collection.access == AccessMode::AllowList {210 collection.check_allowlist(sender)?;211 }212213 let burnt = <TokensBurnt<T>>::get(collection.id)214 .checked_add(1)215 .ok_or(ArithmeticError::Overflow)?;216217 let balance = <AccountBalance<T>>::get((collection.id, token_data.owner.clone()))218 .checked_sub(1)219 .ok_or(ArithmeticError::Overflow)?;220221 if balance == 0 {222 <AccountBalance<T>>::remove((collection.id, token_data.owner.clone()));223 } else {224 <AccountBalance<T>>::insert((collection.id, token_data.owner.clone()), balance);225 }226 // =========227228 <Owned<T>>::remove((collection.id, &token_data.owner, token));229 <TokensBurnt<T>>::insert(collection.id, burnt);230 <TokenData<T>>::remove((collection.id, token));231 let old_spender = <Allowance<T>>::take((collection.id, token));232233 if let Some(old_spender) = old_spender {234 <PalletCommon<T>>::deposit_event(CommonEvent::Approved(235 collection.id,236 token,237 sender.clone(),238 old_spender,239 0,240 ));241 }242243 collection.log_mirrored(ERC721Events::Transfer {244 from: *token_data.owner.as_eth(),245 to: H160::default(),246 token_id: token.into(),247 });248 <PalletCommon<T>>::deposit_event(CommonEvent::ItemDestroyed(249 collection.id,250 token,251 token_data.owner,252 1,253 ));254 Ok(())255 }256257 pub fn change_token_property(258 collection: &NonfungibleHandle<T>,259 sender: &T::CrossAccountId,260 token_id: TokenId,261 property: Property,262 ) -> DispatchResult {263 let permission = <PalletCommon<T>>::property_permission(collection.id)264 .get(&property.key)265 .map(|p| p.clone())266 .unwrap_or(PropertyPermission::None);267268 let token_data = <TokenData<T>>::get((collection.id, token_id))269 .ok_or(<CommonError<T>>::TokenNotFound)?;270271 let check_token_owner = || -> DispatchResult {272 ensure!(&token_data.owner == sender, <CommonError<T>>::NoPermission);273 Ok(())274 };275276 let is_property_exists = TokenProperties::<T>::get((collection.id, token_id))277 .get_property(&property.key)278 .is_some();279280 match (permission, is_property_exists) {281 (PropertyPermission::AdminConst, false) => {282 collection.check_is_owner_or_admin(sender)?283 }284 (PropertyPermission::Admin, _) => collection.check_is_owner_or_admin(sender)?,285 (PropertyPermission::ItemOwnerConst, false) => check_token_owner()?,286 (PropertyPermission::ItemOwner, _) => check_token_owner()?,287 (PropertyPermission::ItemOwnerOrAdmin, _) => {288 check_token_owner().or(collection.check_is_owner_or_admin(sender))?;289 }290 _ => return Err(<CommonError<T>>::NoPermission.into()),291 }292293 <TokenProperties<T>>::try_mutate((collection.id, token_id), |properties| {294 properties.try_change_property(property.clone())295 })?;296297 <PalletCommon<T>>::deposit_event(CommonEvent::TokenPropertySet(298 collection.id,299 token_id,300 property,301 ));302303 Ok(())304 }305306 pub fn change_token_properties(307 collection: &NonfungibleHandle<T>,308 sender: &T::CrossAccountId,309 token_id: TokenId,310 properties: Vec<Property>,311 ) -> DispatchResult {312 for property in properties {313 Self::change_token_property(collection, sender, token_id, property)?;314 }315316 Ok(())317 }318319 pub fn change_collection_properties(320 collection: &NonfungibleHandle<T>,321 sender: &T::CrossAccountId,322 properties: Vec<Property>,323 ) -> DispatchResult {324 <PalletCommon<T>>::change_collection_properties(collection, sender, properties)325 }326327 pub fn transfer(328 collection: &NonfungibleHandle<T>,329 from: &T::CrossAccountId,330 to: &T::CrossAccountId,331 token: TokenId,332 nesting_budget: &dyn Budget,333 ) -> DispatchResult {334 ensure!(335 collection.limits.transfers_enabled(),336 <CommonError<T>>::TransferNotAllowed337 );338339 let token_data =340 <TokenData<T>>::get((collection.id, token)).ok_or(<CommonError<T>>::TokenNotFound)?;341 // TODO: require sender to be token, owner, require admins to go through transfer_from342 ensure!(343 &token_data.owner == from344 || (collection.limits.owner_can_transfer() && collection.is_owner_or_admin(from)),345 <CommonError<T>>::NoPermission346 );347348 if collection.access == AccessMode::AllowList {349 collection.check_allowlist(from)?;350 collection.check_allowlist(to)?;351 }352 <PalletCommon<T>>::ensure_correct_receiver(to)?;353354 let balance_from = <AccountBalance<T>>::get((collection.id, from))355 .checked_sub(1)356 .ok_or(<CommonError<T>>::TokenValueTooLow)?;357 let balance_to = if from != to {358 let balance_to = <AccountBalance<T>>::get((collection.id, to))359 .checked_add(1)360 .ok_or(ArithmeticError::Overflow)?;361362 ensure!(363 balance_to < collection.limits.account_token_ownership_limit(),364 <CommonError<T>>::AccountTokenLimitExceeded,365 );366367 Some(balance_to)368 } else {369 None370 };371372 if let Some(target) = T::CrossTokenAddressMapping::address_to_token(to) {373 let handle = <CollectionHandle<T>>::try_get(target.0)?;374 let dispatch = T::CollectionDispatch::dispatch(handle);375 let dispatch = dispatch.as_dyn();376377 dispatch.check_nesting(378 from.clone(),379 (collection.id, token),380 target.1,381 nesting_budget,382 )?;383 }384385 // =========386387 <TokenData<T>>::insert(388 (collection.id, token),389 ItemData {390 owner: to.clone(),391 ..token_data392 },393 );394395 if let Some(balance_to) = balance_to {396 // from != to397 if balance_from == 0 {398 <AccountBalance<T>>::remove((collection.id, from));399 } else {400 <AccountBalance<T>>::insert((collection.id, from), balance_from);401 }402 <AccountBalance<T>>::insert((collection.id, to), balance_to);403 <Owned<T>>::remove((collection.id, from, token));404 <Owned<T>>::insert((collection.id, to, token), true);405 }406 Self::set_allowance_unchecked(collection, from, token, None, true);407408 collection.log_mirrored(ERC721Events::Transfer {409 from: *from.as_eth(),410 to: *to.as_eth(),411 token_id: token.into(),412 });413 <PalletCommon<T>>::deposit_event(CommonEvent::Transfer(414 collection.id,415 token,416 from.clone(),417 to.clone(),418 1,419 ));420 Ok(())421 }422423 pub fn create_multiple_items(424 collection: &NonfungibleHandle<T>,425 sender: &T::CrossAccountId,426 data: Vec<CreateItemData<T>>,427 nesting_budget: &dyn Budget,428 ) -> DispatchResult {429 if !collection.is_owner_or_admin(sender) {430 ensure!(431 collection.mint_mode,432 <CommonError<T>>::PublicMintingNotAllowed433 );434 collection.check_allowlist(sender)?;435436 for item in data.iter() {437 collection.check_allowlist(&item.owner)?;438 }439 }440441 for data in data.iter() {442 <PalletCommon<T>>::ensure_correct_receiver(&data.owner)?;443 }444445 let first_token = <TokensMinted<T>>::get(collection.id);446 let tokens_minted = first_token447 .checked_add(data.len() as u32)448 .ok_or(ArithmeticError::Overflow)?;449 ensure!(450 tokens_minted <= collection.limits.token_limit(),451 <CommonError<T>>::CollectionTokenLimitExceeded452 );453454 let mut balances = BTreeMap::new();455 for data in &data {456 let balance = balances457 .entry(&data.owner)458 .or_insert_with(|| <AccountBalance<T>>::get((collection.id, &data.owner)));459 *balance = balance.checked_add(1).ok_or(ArithmeticError::Overflow)?;460461 ensure!(462 *balance <= collection.limits.account_token_ownership_limit(),463 <CommonError<T>>::AccountTokenLimitExceeded,464 );465 }466467 for (i, data) in data.iter().enumerate() {468 let token = TokenId(first_token + i as u32 + 1);469 if let Some(target) = T::CrossTokenAddressMapping::address_to_token(&data.owner) {470 let handle = <CollectionHandle<T>>::try_get(target.0)?;471 let dispatch = T::CollectionDispatch::dispatch(handle);472 let dispatch = dispatch.as_dyn();473 dispatch.check_nesting(474 sender.clone(),475 (collection.id, token),476 target.1,477 nesting_budget,478 )?;479 }480 }481482 // =========483484 <TokensMinted<T>>::insert(collection.id, tokens_minted);485 for (account, balance) in balances {486 <AccountBalance<T>>::insert((collection.id, account), balance);487 }488 for (i, data) in data.into_iter().enumerate() {489 let token = first_token + i as u32 + 1;490491 <TokenData<T>>::insert(492 (collection.id, token),493 ItemData {494 const_data: data.const_data,495 variable_data: data.variable_data,496 owner: data.owner.clone(),497 },498 );499 <Owned<T>>::insert((collection.id, &data.owner, token), true);500501 collection.log_mirrored(ERC721Events::Transfer {502 from: H160::default(),503 to: *data.owner.as_eth(),504 token_id: token.into(),505 });506 <PalletCommon<T>>::deposit_event(CommonEvent::ItemCreated(507 collection.id,508 TokenId(token),509 data.owner.clone(),510 1,511 ));512 }513 Ok(())514 }515516 pub fn set_allowance_unchecked(517 collection: &NonfungibleHandle<T>,518 sender: &T::CrossAccountId,519 token: TokenId,520 spender: Option<&T::CrossAccountId>,521 assume_implicit_eth: bool,522 ) {523 if let Some(spender) = spender {524 let old_spender = <Allowance<T>>::get((collection.id, token));525 <Allowance<T>>::insert((collection.id, token), spender);526 // In ERC721 there is only one possible approved user of token, so we set527 // approved user to spender528 collection.log_mirrored(ERC721Events::Approval {529 owner: *sender.as_eth(),530 approved: *spender.as_eth(),531 token_id: token.into(),532 });533 // In Unique chain, any token can have any amount of approved users, so we need to534 // set allowance of old owner to 0, and allowance of new owner to 1535 if old_spender.as_ref() != Some(spender) {536 if let Some(old_owner) = old_spender {537 <PalletCommon<T>>::deposit_event(CommonEvent::Approved(538 collection.id,539 token,540 sender.clone(),541 old_owner,542 0,543 ));544 }545 <PalletCommon<T>>::deposit_event(CommonEvent::Approved(546 collection.id,547 token,548 sender.clone(),549 spender.clone(),550 1,551 ));552 }553 } else {554 let old_spender = <Allowance<T>>::take((collection.id, token));555 if !assume_implicit_eth {556 // In ERC721 there is only one possible approved user of token, so we set557 // approved user to zero address558 collection.log_mirrored(ERC721Events::Approval {559 owner: *sender.as_eth(),560 approved: H160::default(),561 token_id: token.into(),562 });563 }564 // In Unique chain, any token can have any amount of approved users, so we need to565 // set allowance of old owner to 0566 if let Some(old_spender) = old_spender {567 <PalletCommon<T>>::deposit_event(CommonEvent::Approved(568 collection.id,569 token,570 sender.clone(),571 old_spender,572 0,573 ));574 }575 }576 }577578 pub fn set_allowance(579 collection: &NonfungibleHandle<T>,580 sender: &T::CrossAccountId,581 token: TokenId,582 spender: Option<&T::CrossAccountId>,583 ) -> DispatchResult {584 if collection.access == AccessMode::AllowList {585 collection.check_allowlist(sender)?;586 if let Some(spender) = spender {587 collection.check_allowlist(spender)?;588 }589 }590591 if let Some(spender) = spender {592 <PalletCommon<T>>::ensure_correct_receiver(spender)?;593 }594 let token_data =595 <TokenData<T>>::get((collection.id, token)).ok_or(<CommonError<T>>::TokenNotFound)?;596 if &token_data.owner != sender {597 ensure!(598 collection.ignores_owned_amount(sender),599 <CommonError<T>>::CantApproveMoreThanOwned600 );601 }602603 // =========604605 Self::set_allowance_unchecked(collection, sender, token, spender, false);606 Ok(())607 }608609 fn check_allowed(610 collection: &NonfungibleHandle<T>,611 spender: &T::CrossAccountId,612 from: &T::CrossAccountId,613 token: TokenId,614 nesting_budget: &dyn Budget,615 ) -> DispatchResult {616 if spender.conv_eq(from) {617 return Ok(());618 }619 if collection.access == AccessMode::AllowList {620 // `from`, `to` checked in [`transfer`]621 collection.check_allowlist(spender)?;622 }623 if let Some(source) = T::CrossTokenAddressMapping::address_to_token(from) {624 // TODO: should collection owner be allowed to perform this transfer?625 ensure!(626 <PalletStructure<T>>::check_indirectly_owned(627 spender.clone(),628 source.0,629 source.1,630 None,631 nesting_budget632 )?,633 <CommonError<T>>::ApprovedValueTooLow,634 );635 return Ok(());636 }637 if <Allowance<T>>::get((collection.id, token)).as_ref() == Some(spender) {638 return Ok(());639 }640 ensure!(641 collection.ignores_allowance(spender),642 <CommonError<T>>::ApprovedValueTooLow643 );644 Ok(())645 }646647 pub fn transfer_from(648 collection: &NonfungibleHandle<T>,649 spender: &T::CrossAccountId,650 from: &T::CrossAccountId,651 to: &T::CrossAccountId,652 token: TokenId,653 nesting_budget: &dyn Budget,654 ) -> DispatchResult {655 Self::check_allowed(collection, spender, from, token, nesting_budget)?;656657 // =========658659 // Allowance is reset in [`transfer`]660 Self::transfer(collection, from, to, token, nesting_budget)661 }662663 pub fn burn_from(664 collection: &NonfungibleHandle<T>,665 spender: &T::CrossAccountId,666 from: &T::CrossAccountId,667 token: TokenId,668 nesting_budget: &dyn Budget,669 ) -> DispatchResult {670 Self::check_allowed(collection, spender, from, token, nesting_budget)?;671672 // =========673674 Self::burn(collection, from, token)675 }676677 pub fn set_variable_metadata(678 collection: &NonfungibleHandle<T>,679 sender: &T::CrossAccountId,680 token: TokenId,681 data: BoundedVec<u8, CustomDataLimit>,682 ) -> DispatchResult {683 let token_data =684 <TokenData<T>>::get((collection.id, token)).ok_or(<CommonError<T>>::TokenNotFound)?;685 collection.check_can_update_meta(sender, &token_data.owner)?;686687 // =========688689 <TokenData<T>>::insert(690 (collection.id, token),691 ItemData {692 variable_data: data,693 ..token_data694 },695 );696 Ok(())697 }698699 pub fn check_nesting(700 handle: &NonfungibleHandle<T>,701 sender: T::CrossAccountId,702 from: (CollectionId, TokenId),703 under: TokenId,704 nesting_budget: &dyn Budget,705 ) -> DispatchResult {706 fn ensure_sender_allowed<T: Config>(707 collection: CollectionId,708 token: TokenId,709 for_nest: (CollectionId, TokenId),710 sender: T::CrossAccountId,711 budget: &dyn Budget,712 ) -> DispatchResult {713 ensure!(714 <PalletStructure<T>>::check_indirectly_owned(715 sender,716 collection,717 token,718 Some(for_nest),719 budget720 )?,721 <CommonError<T>>::OnlyOwnerAllowedToNest,722 );723 Ok(())724 }725 match handle.limits.nesting_rule() {726 NestingRule::Disabled => fail!(<CommonError<T>>::NestingIsDisabled),727 NestingRule::Owner => {728 ensure_sender_allowed::<T>(handle.id, under, from, sender, nesting_budget)?729 }730 NestingRule::OwnerRestricted(whitelist) => {731 ensure!(732 whitelist.contains(&from.0),733 <CommonError<T>>::SourceCollectionIsNotAllowedToNest734 );735 ensure_sender_allowed::<T>(handle.id, under, from, sender, nesting_budget)?736 }737 }738 Ok(())739 }740741 /// Delegated to `create_multiple_items`742 pub fn create_item(743 collection: &NonfungibleHandle<T>,744 sender: &T::CrossAccountId,745 data: CreateItemData<T>,746 nesting_budget: &dyn Budget,747 ) -> DispatchResult {748 Self::create_multiple_items(collection, sender, vec![data], nesting_budget)749 }750}pallets/nonfungible/src/weights.rsdiffbeforeafterboth--- a/pallets/nonfungible/src/weights.rs
+++ b/pallets/nonfungible/src/weights.rs
@@ -38,6 +38,7 @@
fn burn_item() -> Weight;
fn change_collection_properties(amount: u32) -> Weight;
fn change_token_properties(amount: u32) -> Weight;
+ fn change_property_permissions(amount: u32) -> Weight;
fn transfer() -> Weight;
fn approve() -> Weight;
fn transfer_from() -> Weight;
@@ -103,6 +104,11 @@
(50_000_000 as Weight).saturating_mul(amount as Weight)
}
+ fn change_property_permissions(amount: u32) -> Weight {
+ // TODO calculate appropriate weight
+ (50_000_000 as Weight).saturating_mul(amount as Weight)
+ }
+
// Storage: Nonfungible TokenData (r:1 w:1)
// Storage: Nonfungible AccountBalance (r:2 w:2)
// Storage: Nonfungible Allowance (r:1 w:0)
@@ -203,6 +209,11 @@
(50_000_000 as Weight).saturating_mul(amount as Weight)
}
+ fn change_property_permissions(amount: u32) -> Weight {
+ // TODO calculate appropriate weight
+ (50_000_000 as Weight).saturating_mul(amount as Weight)
+ }
+
// Storage: Nonfungible TokenData (r:1 w:1)
// Storage: Nonfungible AccountBalance (r:2 w:2)
// Storage: Nonfungible Allowance (r:1 w:0)
pallets/refungible/src/common.rsdiffbeforeafterboth--- a/pallets/refungible/src/common.rs
+++ b/pallets/refungible/src/common.rs
@@ -20,7 +20,7 @@
use frame_support::{dispatch::DispatchResultWithPostInfo, fail, weights::Weight, BoundedVec};
use up_data_structs::{
CollectionId, TokenId, CustomDataLimit, CreateItemExData, CreateRefungibleExData,
- budget::Budget, Property,
+ budget::Budget, Property, PropertyKeyPermission,
};
use pallet_common::{CommonCollectionOperations, CommonWeightInfo, with_weight};
use sp_runtime::DispatchError;
@@ -74,6 +74,10 @@
<SelfWeightOf<T>>::change_token_properties(amount)
}
+ fn change_property_permissions(amount: u32) -> Weight {
+ <SelfWeightOf<T>>::change_property_permissions(amount)
+ }
+
fn transfer() -> Weight {
max_weight_of!(
transfer_normal(),
@@ -269,6 +273,14 @@
fail!(<Error<T>>::PropertiesNotAllowed)
}
+ fn change_property_permissions(
+ &self,
+ _sender: &T::CrossAccountId,
+ _property_permissions: Vec<PropertyKeyPermission>,
+ ) -> DispatchResultWithPostInfo {
+ fail!(<Error<T>>::PropertiesNotAllowed)
+ }
+
fn set_variable_metadata(
&self,
sender: T::CrossAccountId,
pallets/refungible/src/weights.rsdiffbeforeafterboth--- a/pallets/refungible/src/weights.rs
+++ b/pallets/refungible/src/weights.rs
@@ -40,6 +40,7 @@
fn burn_item_fully() -> Weight;
fn change_collection_properties(amount: u32) -> Weight;
fn change_token_properties(amount: u32) -> Weight;
+ fn change_property_permissions(amount: u32) -> Weight;
fn transfer_normal() -> Weight;
fn transfer_creating() -> Weight;
fn transfer_removing() -> Weight;
@@ -142,6 +143,11 @@
0
}
+ fn change_property_permissions(amount: u32) -> Weight {
+ // Error
+ 0
+ }
+
// Storage: Refungible Balance (r:2 w:2)
fn transfer_normal() -> Weight {
(19_766_000 as Weight)
@@ -321,6 +327,11 @@
0
}
+ fn change_property_permissions(amount: u32) -> Weight {
+ // Error
+ 0
+ }
+
// Storage: Refungible Balance (r:2 w:2)
fn transfer_normal() -> Weight {
(19_766_000 as Weight)
pallets/unique/src/lib.rsdiffbeforeafterboth--- a/pallets/unique/src/lib.rs
+++ b/pallets/unique/src/lib.rs
@@ -39,7 +39,7 @@
MAX_COLLECTION_NAME_LENGTH, MAX_COLLECTION_DESCRIPTION_LENGTH, MAX_TOKEN_PREFIX_LENGTH,
AccessMode, CreateItemData, CollectionLimits, CollectionId, CollectionMode, TokenId,
SchemaVersion, SponsorshipState, MetaUpdatePermission, CreateCollectionData, CustomDataLimit,
- CreateItemExData, budget, CollectionField, Property,
+ CreateItemExData, budget, CollectionField, Property, PropertyKeyPermission,
};
use pallet_evm::account::CrossAccountId;
use pallet_common::{
@@ -723,6 +723,20 @@
dispatch_call::<T, _>(collection_id, |d| d.change_token_properties(sender, token_id, properties))
}
+ #[weight = T::CommonWeightInfo::change_property_permissions(property_permissions.len() as u32)]
+ #[transactional]
+ pub fn change_property_permissions(
+ origin,
+ collection_id: CollectionId,
+ property_permissions: Vec<PropertyKeyPermission>,
+ ) -> DispatchResultWithPostInfo {
+ ensure!(!property_permissions.is_empty(), Error::<T>::EmptyArgument);
+
+ let sender = T::CrossAccountId::from_sub(ensure_signed(origin)?);
+
+ dispatch_call::<T, _>(collection_id, |d| d.change_property_permissions(&sender, property_permissions))
+ }
+
#[weight = T::CommonWeightInfo::create_multiple_items_ex(&data)]
#[transactional]
pub fn create_multiple_items_ex(origin, collection_id: CollectionId, data: CreateItemExData<T::CrossAccountId>) -> DispatchResultWithPostInfo {
runtime/common/src/weights.rsdiffbeforeafterboth--- a/runtime/common/src/weights.rs
+++ b/runtime/common/src/weights.rs
@@ -62,6 +62,10 @@
dispatch_weight::<T>() + max_weight_of!(change_token_properties(amount))
}
+ fn change_property_permissions(amount: u32) -> Weight {
+ dispatch_weight::<T>() + max_weight_of!(change_property_permissions(amount))
+ }
+
fn transfer() -> Weight {
dispatch_weight::<T>() + max_weight_of!(transfer())
}