git.delta.rocks / unique-network / refs/commits / 10c46cb7be24

--- a/pallets/nft/src/lib.rs
+++ b/pallets/nft/src/lib.rs
@@ -1122,6 +1122,7 @@
 			let sender = T::CrossAccountId::from_sub(ensure_signed(origin)?);
 
 			let collection = Self::get_collection(collection_id)?;
+			Self::meta_update_check(&sender, &collection, item_id)?;
 
 			Self::set_variable_meta_data_internal(&sender, &collection, item_id, data)?;
 

1789		let data = default_nft_data();1789		let data = default_nft_data();
1790		assert_noop!(1790		assert_noop!(
1791			TemplateModule::create_item(origin1, 1, account(1), data.into()),1791			TemplateModule::create_item(origin1, 1, account(1), data.into()),
1792			Error::<Test>::AddressOwnershipLimitExceeded1792			Error::<Test>::AccountTokenLimitExceeded
1793		);1793		);
1794	});1794	});
1795}1795}
2016		let data = default_nft_data();2016		let data = default_nft_data();
2017		create_test_item(1, &data.into());2017		create_test_item(1, &data.into());
20182018
2019		TemplateModule::set_meta_update_permission_flag(2019		assert_ok!(TemplateModule::set_meta_update_permission_flag(
2020			origin1.clone(),2020			origin1.clone(),
2021			collection_id,2021			collection_id,
2022			MetaUpdatePermission::ItemOwner,2022			MetaUpdatePermission::ItemOwner,
2023		);2023		));
20242024
2025		let variable_data = b"ten chars.".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(
2040}2040}
20412041
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		let collection_id = create_test_collection_for_owner(&CollectionMode::NFT, 1, 1);
20462046
2047		let collection_id = create_test_collection_for_owner(&CollectionMode::NFT, 2, 1);2047		let origin1 = Origin::signed(1);
20482048
2049		let origin1 = Origin::signed(1);2049		assert_ok!(TemplateModule::set_mint_permission(
2050		let origin2 = Origin::signed(2);2050			origin1.clone(),
20512051			collection_id,
2052		assert_ok!(TemplateModule::set_mint_permission(2052			true
2053			origin2.clone(),2053		));
2054			collection_id,2054		assert_ok!(TemplateModule::add_to_white_list(
2055			true2055			origin1.clone(),
2056		));2056			collection_id,
2057		assert_ok!(TemplateModule::add_to_white_list(2057			account(1)
2058			origin2.clone(),2058		));
2059			collection_id,2059
2060			account(1)2060		let data = default_nft_data();
2061		));2061		create_test_item(1, &data.into());
20622062
2063		let data = default_nft_data();2063		assert_ok!(TemplateModule::set_meta_update_permission_flag(
2064		create_test_item(1, &data.into());2064			origin1.clone(),
20652065			collection_id,
2066		assert_ok!(TemplateModule::set_meta_update_permission_flag(2066			MetaUpdatePermission::ItemOwner,
2067			origin2.clone(),2067		));
2068			collection_id,2068
2069			MetaUpdatePermission::ItemOwner,2069		let variable_data = b"1234567890123".to_vec();
2070		));2070		assert_noop!(
20712071			TemplateModule::set_variable_meta_data(
2072		let variable_data = b"ten chars.++".to_vec();2072				origin1,
2073		assert_noop!(2073				collection_id,
2074			TemplateModule::set_variable_meta_data(2074				1,
2075				origin2,2075				variable_data.clone()
2076				collection_id,2076			),
2077				1,2077			Error::<Test>::TokenVariableDataLimitExceeded
2078				variable_data.clone()2078		);
2079			),2079	})
2080			Error::<Test>::TokenVariableDataLimitExceeded2080}
2081		);
20822081
2083		#[test]2082#[test]
2084		fn collection_transfer_flag_works() {2083fn collection_transfer_flag_works() {
2141					MetaUpdatePermission::Admin,2140			MetaUpdatePermission::Admin,
2142				));2141		));
21432142
2144				let variable_data = b"test set_variable_meta_data method.".to_vec();2143		let variable_data = b"test.".to_vec();
2145				assert_ok!(TemplateModule::set_variable_meta_data(2144		assert_ok!(TemplateModule::set_variable_meta_data(
2146					origin1,2145			origin1,
2147					collection_id,2146			collection_id,
2188					MetaUpdatePermission::Admin,2187			MetaUpdatePermission::Admin,
2189				));2188		));
21902189
2191				let variable_data = b"test set_variable_meta_data method.".to_vec();2190		let variable_data = b"test.".to_vec();
2192				assert_noop!(2191		assert_noop!(
2193					TemplateModule::set_variable_meta_data(2192			TemplateModule::set_variable_meta_data(
2194						origin1,2193				origin1,
2243					MetaUpdatePermission::None,2242			MetaUpdatePermission::None,
2244				));2243		));
22452244
2246				let variable_data = b"test set_variable_meta_data method.".to_vec();2245		let variable_data = b"test.".to_vec();
2247				assert_noop!(2246		assert_noop!(
2248					TemplateModule::set_variable_meta_data(2247			TemplateModule::set_variable_meta_data(
2249						origin1.clone(),2248				origin1.clone(),
2285				assert_eq!(TemplateModule::address_tokens(1, 1), [1]);2284		assert_eq!(TemplateModule::address_tokens(1, 1), [1]);
2286			});2285	});
2287		}2286}
2288	});
2289}
22902287