diff --git a/CHANGELOG.md b/CHANGELOG.md index a88fe5bc0..93881d34b 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -14,7 +14,6 @@ - [\#1234](https://github.com/Manta-Network/Manta/pull/1234) Serialize/deserialize vfr key [MACA] - [\#1191](https://github.com/Manta-Network/Manta/pull/1191) Fix metadata diff checker workflow [MACA] - [\#1243](https://github.com/Manta-Network/Manta/pull/1243) Add manta docker image to publish -- [\#1236](https://github.com/Manta-Network/Manta/pull/1236) Update Asset metadata call to not have dummy parameters [MACA] - [\#1253](https://github.com/Manta-Network/Manta/pull/1253) Fix benchmark [CA] ## v4.4.0 diff --git a/pallets/asset-manager/src/benchmarking.rs b/pallets/asset-manager/src/benchmarking.rs index c4f15ab41..37107fe22 100644 --- a/pallets/asset-manager/src/benchmarking.rs +++ b/pallets/asset-manager/src/benchmarking.rs @@ -96,7 +96,7 @@ benchmarks! { let metadata = AssetRegistryMetadata::::testing_default(); Pallet::::register_asset(RawOrigin::Root.into(), location, metadata.clone())?; let some_valid_asset_id = ::AssetId::from(assets_count); - }: _(RawOrigin::Root, some_valid_asset_id, metadata.metadata.clone()) + }: _(RawOrigin::Root, some_valid_asset_id, metadata.clone()) verify { assert_last_event::(crate::Event::AssetMetadataUpdated { asset_id: some_valid_asset_id, metadata }.into()); } diff --git a/pallets/asset-manager/src/lib.rs b/pallets/asset-manager/src/lib.rs index f9ea76f32..4554a726a 100644 --- a/pallets/asset-manager/src/lib.rs +++ b/pallets/asset-manager/src/lib.rs @@ -541,15 +541,14 @@ pub mod pallet { /// /// * `origin`: Caller of this extrinsic, the access control is specified by `ForceOrigin`. /// * `asset_id`: AssetId to be updated. - /// * `metadata`: new `metadata` to be associated with `asset_id`, note `is_frozen` - /// flag in metadata will have no effect and and cannot be changed. + /// * `metadata`: new `metadata` to be associated with `asset_id`. #[pallet::call_index(2)] #[pallet::weight(T::WeightInfo::update_asset_metadata())] #[transactional] pub fn update_asset_metadata( origin: OriginFor, asset_id: T::AssetId, - metadata: AssetStorageMetadata, + metadata: AssetRegistryMetadata, ) -> DispatchResult { T::ModifierOrigin::ensure_origin(origin)?; ensure!( @@ -562,25 +561,11 @@ pub mod pallet { ); >::AssetRegistry::update_asset_metadata( &asset_id, - metadata.clone(), + metadata.clone().into(), )?; - let mut registered_metadata = - AssetIdMetadata::::get(asset_id).ok_or(Error::::UpdateNonExistentAsset)?; - let new_metadata = AssetStorageMetadata { - name: metadata.name, - symbol: metadata.symbol, - decimals: metadata.decimals, - // is frozen flag doesn't do anything in metadata - is_frozen: registered_metadata.metadata.is_frozen, - }; - registered_metadata.metadata = new_metadata; - - AssetIdMetadata::::insert(asset_id, ®istered_metadata); - Self::deposit_event(Event::::AssetMetadataUpdated { - asset_id, - metadata: registered_metadata, - }); + AssetIdMetadata::::insert(asset_id, &metadata); + Self::deposit_event(Event::::AssetMetadataUpdated { asset_id, metadata }); Ok(()) } diff --git a/pallets/asset-manager/src/tests.rs b/pallets/asset-manager/src/tests.rs index 5515a528f..11812a2c3 100644 --- a/pallets/asset-manager/src/tests.rs +++ b/pallets/asset-manager/src/tests.rs @@ -82,7 +82,7 @@ fn wrong_modifier_origin_should_not_work() { AssetManager::update_asset_metadata( RuntimeOrigin::signed([3u8; 32].into()), 0, - asset_metadata.metadata + asset_metadata ), BadOrigin ); @@ -174,14 +174,14 @@ fn update_asset() { AssetManager::update_asset_metadata( RuntimeOrigin::root(), native_asset_id, - new_metadata.metadata.clone(), + new_metadata.clone(), ), Error::::CannotUpdateNativeAssetMetadata ); assert_ok!(AssetManager::update_asset_metadata( RuntimeOrigin::root(), asset_id, - new_metadata.metadata.clone(), + new_metadata.clone(), ),); assert_eq!(Assets::name(&asset_id), new_name); assert_eq!(Assets::symbol(&asset_id), new_symbol); @@ -213,7 +213,7 @@ fn update_asset() { AssetManager::update_asset_metadata( RuntimeOrigin::root(), next_asset_id, - new_metadata.metadata.clone() + new_metadata.clone(), ), Error::::UpdateNonExistentAsset ); diff --git a/runtime/calamari/src/diff_tx_fees.rs b/runtime/calamari/src/diff_tx_fees.rs index 86cc09aeb..ecfd8794d 100644 --- a/runtime/calamari/src/diff_tx_fees.rs +++ b/runtime/calamari/src/diff_tx_fees.rs @@ -1211,7 +1211,7 @@ fn calculate_all_current_extrinsic_tx_fee() -> ( let call = crate::RuntimeCall::AssetManager(pallet_asset_manager::Call::update_asset_metadata { asset_id: 1, - metadata: AssetRegistryMetadata::testing_default().into(), + metadata: AssetRegistryMetadata::testing_default(), }); let (dispatch_info, call_len) = get_call_details(&call); calamari_runtime_calls.push(( diff --git a/runtime/manta/src/diff_tx_fees.rs b/runtime/manta/src/diff_tx_fees.rs index 9a97ebbc7..dcf596c0a 100644 --- a/runtime/manta/src/diff_tx_fees.rs +++ b/runtime/manta/src/diff_tx_fees.rs @@ -1198,7 +1198,7 @@ fn calculate_all_current_extrinsic_tx_fee() -> ( let call = crate::RuntimeCall::AssetManager(pallet_asset_manager::Call::update_asset_metadata { asset_id: 1, - metadata: AssetRegistryMetadata::testing_default().into(), + metadata: AssetRegistryMetadata::testing_default(), }); let (dispatch_info, call_len) = get_call_details(&call); calamari_runtime_calls.push((