`7791279df554`

2 files changed

modifiedpallets/nft/src/lib.rsdiff before after both

43	OFFCHAIN_SCHEMA_LIMIT, MAX_TOKEN_PREFIX_LENGTH, MAX_COLLECTION_NAME_LENGTH,43	OFFCHAIN_SCHEMA_LIMIT, MAX_TOKEN_PREFIX_LENGTH, MAX_COLLECTION_NAME_LENGTH,
44	MAX_COLLECTION_DESCRIPTION_LENGTH, AccessMode, Collection, CreateItemData, CollectionLimits,44	MAX_COLLECTION_DESCRIPTION_LENGTH, AccessMode, Collection, CreateItemData, CollectionLimits,
45	CollectionId, CollectionMode, TokenId, SchemaVersion, SponsorshipState, Ownership, NftItemType,45	CollectionId, CollectionMode, TokenId, SchemaVersion, SponsorshipState, Ownership, NftItemType,
46	FungibleItemType, ReFungibleItemType,46	MetaUpdatePermission, FungibleItemType, ReFungibleItemType,
47};47};
4848
49#[cfg(test)]49#[cfg(test)]
971			Self::check_owner_permissions(&target_collection, &sender)?;971			Self::check_owner_permissions(&target_collection, &sender)?;
972972
973			target_collection.meta_update_permission = value;973			target_collection.meta_update_permission = value;
974			Self::save_collection(target_collection);974			<CollectionById<T>>::insert(target_collection.id, target_collection.collection);
975975
976			Ok(())976			Ok(())
977		}977		}
1546		item_id: TokenId,1546		item_id: TokenId,
1547	) -> DispatchResult {1547	) -> DispatchResult {
1548		match collection.meta_update_permission {1548		match collection.meta_update_permission {
1549			MetaUpdatePermission::ItemOwner => {1549			MetaUpdatePermission::ItemOwner => ensure!(
1550				ensure!(
1551					Self::is_item_owner(sender, collection, item_id),1550				Self::is_item_owner(sender, collection, item_id)?,
1552					Error::<T>::NoPermission1551				Error::<T>::NoPermission
1553				);1552			),
1554			}
1555			MetaUpdatePermission::Admin => {1553			MetaUpdatePermission::Admin => ensure!(
1556				ensure!(
1557					Self::is_owner_or_admin_permissions(collection, sender),1554				Self::is_owner_or_admin_permissions(collection, sender)?,
1558					Error::<T>::NoPermission1555				Error::<T>::NoPermission
1559				);1556			),
1560			}
1561			MetaUpdatePermission::None => {1557			MetaUpdatePermission::None => fail!(Error::<T>::MetadataUpdateDenied),
1562				fail!(Error::<T>::MetadataUpdateDenied);
1563			}
1564		}1558		}
15651559
1566		Ok(())1560		Ok(())

modifiedpallets/nft/src/tests.rsdiff before after both

2007#[test]2007#[test]
2008fn set_variable_meta_data_on_nft_with_item_owner_permission_flag() {2008fn set_variable_meta_data_on_nft_with_item_owner_permission_flag() {
2009	new_test_ext().execute_with(|| {2009	new_test_ext().execute_with(|| {
2010		default_limits();2010		//default_limits();
20112011
2012		let collection_id = create_test_collection(&CollectionMode::NFT, 1);2012		let collection_id = create_test_collection(&CollectionMode::NFT, 1);
20132013
2022			MetaUpdatePermission::ItemOwner,2022			MetaUpdatePermission::ItemOwner,
2023		);2023		);
20242024
2025		let variable_data = b"test set_variable_meta_data method.".to_vec();2025		let variable_data = b"ten chars.".to_vec();
2026		assert_ok!(TemplateModule::set_variable_meta_data(2026		assert_ok!(TemplateModule::set_variable_meta_data(
2027			origin1,2027			origin1,
2028			collection_id,2028			collection_id,
2042#[test]2042#[test]
2043fn set_variable_meta_data_on_nft_with_item_owner_permission_flag_neg() {2043fn set_variable_meta_data_on_nft_with_item_owner_permission_flag_neg() {
2044	new_test_ext().execute_with(|| {2044	new_test_ext().execute_with(|| {
2045		default_limits();2045		// default_limits();
20462046
2047		let collection_id = create_test_collection_for_owner(&CollectionMode::NFT, 2, 1);2047		let collection_id = create_test_collection_for_owner(&CollectionMode::NFT, 2, 1);
20482048
2063		let data = default_nft_data();2063		let data = default_nft_data();
2064		create_test_item(1, &data.into());2064		create_test_item(1, &data.into());
20652065
2066		TemplateModule::set_meta_update_permission_flag(2066		assert_ok!(TemplateModule::set_meta_update_permission_flag(
2067			origin2.clone(),2067			origin2.clone(),
2068			collection_id,2068			collection_id,
2069			MetaUpdatePermission::ItemOwner,2069			MetaUpdatePermission::ItemOwner,
2070		);2070		));
20712071
2072		let variable_data = b"test set_variable_meta_data method.".to_vec();2072		let variable_data = b"ten chars.++".to_vec();
2073		assert_noop!(2073		assert_noop!(
2074			TemplateModule::set_variable_meta_data(2074			TemplateModule::set_variable_meta_data(
2075				origin2,2075				origin2,
2076				collection_id,2076				collection_id,
2077				1,2077				1,
2078				variable_data.clone()2078				variable_data.clone()
2079			),2079			),
2080			Error::<Test>::NoPermission2080			Error::<Test>::TokenVariableDataLimitExceeded
2081		);2081		);
20822082
2083		#[test]2083		#[test]
2108		#[test]2108		#[test]
2109		fn set_variable_meta_data_on_nft_with_admin_flag() {2109		fn set_variable_meta_data_on_nft_with_admin_flag() {
2110			new_test_ext().execute_with(|| {2110			new_test_ext().execute_with(|| {
2111				default_limits();2111				// default_limits();
21122112
2113				let collection_id = create_test_collection_for_owner(&CollectionMode::NFT, 2, 1);2113				let collection_id = create_test_collection_for_owner(&CollectionMode::NFT, 2, 1);
21142114
2135				let data = default_nft_data();2135				let data = default_nft_data();
2136				create_test_item(1, &data.into());2136				create_test_item(1, &data.into());
21372137
2138				TemplateModule::set_meta_update_permission_flag(2138				assert_ok!(TemplateModule::set_meta_update_permission_flag(
2139					origin2.clone(),2139					origin2.clone(),
2140					collection_id,2140					collection_id,
2141					MetaUpdatePermission::Admin,2141					MetaUpdatePermission::Admin,
2142				);2142				));
21432143
2144				let variable_data = b"test set_variable_meta_data method.".to_vec();2144				let variable_data = b"test set_variable_meta_data method.".to_vec();
2145				assert_ok!(TemplateModule::set_variable_meta_data(2145				assert_ok!(TemplateModule::set_variable_meta_data(
2161		#[test]2161		#[test]
2162		fn set_variable_meta_data_on_nft_with_admin_flag_neg() {2162		fn set_variable_meta_data_on_nft_with_admin_flag_neg() {
2163			new_test_ext().execute_with(|| {2163			new_test_ext().execute_with(|| {
2164				default_limits();2164				// default_limits();
21652165
2166				let collection_id = create_test_collection_for_owner(&CollectionMode::NFT, 2, 1);2166				let collection_id = create_test_collection_for_owner(&CollectionMode::NFT, 2, 1);
21672167
2182				let data = default_nft_data();2182				let data = default_nft_data();
2183				create_test_item(1, &data.into());2183				create_test_item(1, &data.into());
21842184
2185				TemplateModule::set_meta_update_permission_flag(2185				assert_ok!(TemplateModule::set_meta_update_permission_flag(
2186					origin2.clone(),2186					origin2.clone(),
2187					collection_id,2187					collection_id,
2188					MetaUpdatePermission::Admin,2188					MetaUpdatePermission::Admin,
2189				);2189				));
21902190
2191				let variable_data = b"test set_variable_meta_data method.".to_vec();2191				let variable_data = b"test set_variable_meta_data method.".to_vec();
2192				assert_noop!(2192				assert_noop!(
2204		#[test]2204		#[test]
2205		fn set_variable_meta_flag_after_freeze() {2205		fn set_variable_meta_flag_after_freeze() {
2206			new_test_ext().execute_with(|| {2206			new_test_ext().execute_with(|| {
2207				default_limits();2207				// default_limits();
22082208
2209				let collection_id = create_test_collection_for_owner(&CollectionMode::NFT, 2, 1);2209				let collection_id = create_test_collection_for_owner(&CollectionMode::NFT, 2, 1);
22102210
2211				let origin2 = Origin::signed(2);2211				let origin2 = Origin::signed(2);
22122212
2213				TemplateModule::set_meta_update_permission_flag(2213				assert_ok!(TemplateModule::set_meta_update_permission_flag(
2214					origin2.clone(),2214					origin2.clone(),
2215					collection_id,2215					collection_id,
2216					MetaUpdatePermission::None,2216					MetaUpdatePermission::None,
2217				);2217				));
2218				assert_noop!(2218				assert_noop!(
2219					TemplateModule::set_meta_update_permission_flag(2219					TemplateModule::set_meta_update_permission_flag(
2220						origin2.clone(),2220						origin2.clone(),
2229		#[test]2229		#[test]
2230		fn set_variable_meta_data_on_nft_with_none_flag_neg() {2230		fn set_variable_meta_data_on_nft_with_none_flag_neg() {
2231			new_test_ext().execute_with(|| {2231			new_test_ext().execute_with(|| {
2232				default_limits();2232				// default_limits();
22332233
2234				let collection_id = create_test_collection_for_owner(&CollectionMode::NFT, 1, 1);2234				let collection_id = create_test_collection_for_owner(&CollectionMode::NFT, 1, 1);
2235				let origin1 = Origin::signed(1);2235				let origin1 = Origin::signed(1);
22362236
2237				let data = default_nft_data();2237				let data = default_nft_data();
2238				create_test_item(1, &data.into());2238				create_test_item(1, &data.into());
22392239
2240				TemplateModule::set_meta_update_permission_flag(2240				assert_ok!(TemplateModule::set_meta_update_permission_flag(
2241					origin1.clone(),2241					origin1.clone(),
2242					collection_id,2242					collection_id,
2243					MetaUpdatePermission::None,2243					MetaUpdatePermission::None,
2244				);2244				));
22452245
2246				let variable_data = b"test set_variable_meta_data method.".to_vec();2246				let variable_data = b"test set_variable_meta_data method.".to_vec();
2247				assert_noop!(2247				assert_noop!(