Skip to content

Commit d2ab18f

Browse files
ggwpezbkontur
andauthored
Apply suggestions from code review
Co-authored-by: Branislav Kontur <bkontur@gmail.com>
1 parent e80ad48 commit d2ab18f

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

system-parachains/asset-hubs/asset-hub-kusama/src/xcm_config.rs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -513,9 +513,9 @@ impl Contains<RuntimeCall> for SafeCallFilter {
513513
>::report_bridge_status {
514514
..
515515
}) => true,
516-
RuntimeCall::Utility(pallet_utility::Call::as_derivative { .. }) => true,
517-
RuntimeCall::Utility(pallet_utility::Call::batch_all { .. }) => true,
518-
RuntimeCall::Utility(pallet_utility::Call::force_batch { .. }) => true,
516+
RuntimeCall::Utility(pallet_utility::Call::as_derivative { call, .. }) => Self::contains(call),
517+
RuntimeCall::Utility(pallet_utility::Call::batch_all { calls }) => calls.iter().all(Self::contains),
518+
RuntimeCall::Utility(pallet_utility::Call::force_batch { calls }) => calls.iter().all(Self::contains),
519519
RuntimeCall::Whitelist(pallet_whitelist::Call::whitelist_call { .. }) => true,
520520
_ => false,
521521
}

system-parachains/asset-hubs/asset-hub-polkadot/src/xcm_config.rs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -592,9 +592,9 @@ impl Contains<RuntimeCall> for SafeCallFilter {
592592
>::report_bridge_status {
593593
..
594594
}) => true,
595-
RuntimeCall::Utility(pallet_utility::Call::as_derivative { .. }) => true,
596-
RuntimeCall::Utility(pallet_utility::Call::batch_all { .. }) => true,
597-
RuntimeCall::Utility(pallet_utility::Call::force_batch { .. }) => true,
595+
RuntimeCall::Utility(pallet_utility::Call::as_derivative { call, .. }) => Self::contains(call),
596+
RuntimeCall::Utility(pallet_utility::Call::batch_all { calls }) => calls.iter().all(Self::contains),
597+
RuntimeCall::Utility(pallet_utility::Call::force_batch { calls }) => calls.iter().all(Self::contains),
598598
RuntimeCall::Whitelist(pallet_whitelist::Call::whitelist_call { .. }) => true,
599599
_ => false,
600600
}

0 commit comments

Comments
 (0)