Update VARIANT_COUNT for HoldIdentifier

The value is used as the max number of hold an account can create, it is set to 100
for now to workaround some issues with this limit, should revist the value later.

Signed-off-by: linning <[email protected]>

Author: NingLin-P

crates/pallet-domains/src/tests.rs

@@ -8,7 +8,6 @@ use crate::{
     Operators, ReceiptHashFor,
 };
 use codec::{Decode, Encode, MaxEncodedLen};
-use core::mem;
 use domain_runtime_primitives::opaque::Header as DomainHeader;
 use domain_runtime_primitives::BlockNumber as DomainBlockNumber;
 use frame_support::dispatch::{DispatchInfo, RawOrigin};
@@ -133,11 +132,10 @@ impl pallet_domains::HoldIdentifier<Test> for HoldIdentifier {
 }
 
 impl VariantCount for HoldIdentifier {
-    const VARIANT_COUNT: u32 = mem::variant_count::<Self>() as u32;
+    const VARIANT_COUNT: u32 = 10;
 }
 
 parameter_types! {
-    pub const MaxHolds: u32 = 10;
     pub const ExistentialDeposit: Balance = 1;
 }
 

crates/subspace-runtime/src/lib.rs

@@ -32,7 +32,6 @@ use crate::fees::{OnChargeTransaction, TransactionByteFee};
 use crate::object_mapping::extract_block_object_mapping;
 pub use crate::signed_extensions::{CheckStorageAccess, DisablePallets};
 use codec::{Decode, Encode, MaxEncodedLen};
-use core::mem;
 use core::num::NonZeroU64;
 use domain_runtime_primitives::opaque::Header as DomainHeader;
 use domain_runtime_primitives::{
@@ -408,12 +407,11 @@ impl pallet_messenger::HoldIdentifier<Runtime> for HoldIdentifier {
 }
 
 impl VariantCount for HoldIdentifier {
-    const VARIANT_COUNT: u32 = mem::variant_count::<Self>() as u32;
-}
-
-parameter_types! {
-    // TODO: revisit this
-    pub const MaxHolds: u32 = 100;
+    // TODO: revist this value, it is used as the max number of hold an account can
+    // create. Currently, nomination an operator will create 2 holds and opening an
+    // XDM channel will create 1 hold, so this value also used as the limit of how
+    // many operator/channel an account can nominate/open.
+    const VARIANT_COUNT: u32 = 100;
 }
 
 impl pallet_balances::Config for Runtime {

domains/runtime/auto-id/src/lib.rs

@@ -13,7 +13,6 @@ extern crate alloc;
 #[cfg(not(feature = "std"))]
 use alloc::format;
 use codec::{Decode, Encode, MaxEncodedLen};
-use core::mem;
 use domain_runtime_primitives::opaque::Header;
 pub use domain_runtime_primitives::{
     block_weights, maximum_block_length, opaque, Balance, BlockNumber, Hash, Nonce,
@@ -201,10 +200,6 @@ impl OnUnbalanced<Credit<AccountId, Balances>> for DustRemovalHandler {
     }
 }
 
-parameter_types! {
-    pub const MaxHolds: u32 = 100;
-}
-
 impl pallet_balances::Config for Runtime {
     type RuntimeFreezeReason = RuntimeFreezeReason;
     type MaxLocks = MaxLocks;
@@ -354,7 +349,10 @@ pub enum HoldIdentifier {
 }
 
 impl VariantCount for HoldIdentifier {
-    const VARIANT_COUNT: u32 = mem::variant_count::<Self>() as u32;
+    // TODO: revist this value, it is used as the max number of hold an account can
+    // create. Currently, opening an XDM channel will create 1 hold, so this value
+    // also used as the limit of how many channel an account can open.
+    const VARIANT_COUNT: u32 = 100;
 }
 
 impl pallet_messenger::HoldIdentifier<Runtime> for HoldIdentifier {

domains/runtime/evm/src/lib.rs

@@ -15,7 +15,6 @@ extern crate alloc;
 #[cfg(not(feature = "std"))]
 use alloc::format;
 use codec::{Decode, Encode, MaxEncodedLen};
-use core::mem;
 use domain_runtime_primitives::opaque::Header;
 pub use domain_runtime_primitives::{
     block_weights, maximum_block_length, opaque, Balance, BlockNumber, Hash, Nonce,
@@ -318,10 +317,6 @@ impl OnUnbalanced<Credit<AccountId, Balances>> for DustRemovalHandler {
     }
 }
 
-parameter_types! {
-    pub const MaxHolds: u32 = 100;
-}
-
 impl pallet_balances::Config for Runtime {
     type RuntimeFreezeReason = RuntimeFreezeReason;
     type MaxLocks = MaxLocks;
@@ -468,7 +463,10 @@ pub enum HoldIdentifier {
 }
 
 impl VariantCount for HoldIdentifier {
-    const VARIANT_COUNT: u32 = mem::variant_count::<Self>() as u32;
+    // TODO: revist this value, it is used as the max number of hold an account can
+    // create. Currently, opening an XDM channel will create 1 hold, so this value
+    // also used as the limit of how many channel an account can open.
+    const VARIANT_COUNT: u32 = 100;
 }
 
 impl pallet_messenger::HoldIdentifier<Runtime> for HoldIdentifier {

domains/test/runtime/evm/src/lib.rs

@@ -15,7 +15,6 @@ extern crate alloc;
 #[cfg(not(feature = "std"))]
 use alloc::format;
 use codec::{Decode, Encode, MaxEncodedLen};
-use core::mem;
 pub use domain_runtime_primitives::opaque::Header;
 use domain_runtime_primitives::{
     block_weights, maximum_block_length, EXISTENTIAL_DEPOSIT, MAXIMUM_BLOCK_WEIGHT, SLOT_DURATION,
@@ -306,10 +305,6 @@ parameter_types! {
     pub const MaxReserves: u32 = 50;
 }
 
-parameter_types! {
-    pub const MaxHolds: u32 = 100;
-}
-
 impl pallet_balances::Config for Runtime {
     type RuntimeFreezeReason = RuntimeFreezeReason;
     type MaxLocks = MaxLocks;
@@ -432,7 +427,7 @@ pub enum HoldIdentifier {
 }
 
 impl VariantCount for HoldIdentifier {
-    const VARIANT_COUNT: u32 = mem::variant_count::<Self>() as u32;
+    const VARIANT_COUNT: u32 = 10;
 }
 
 impl pallet_messenger::HoldIdentifier<Runtime> for HoldIdentifier {

test/subspace-test-runtime/src/lib.rs

@@ -24,7 +24,6 @@
 include!(concat!(env!("OUT_DIR"), "/wasm_binary.rs"));
 
 use codec::{Compact, CompactLen, Decode, Encode, MaxEncodedLen};
-use core::mem;
 use core::num::NonZeroU64;
 use domain_runtime_primitives::opaque::Header as DomainHeader;
 use domain_runtime_primitives::{
@@ -344,11 +343,7 @@ impl pallet_messenger::HoldIdentifier<Runtime> for HoldIdentifier {
 }
 
 impl VariantCount for HoldIdentifier {
-    const VARIANT_COUNT: u32 = mem::variant_count::<Self>() as u32;
-}
-
-parameter_types! {
-    pub const MaxHolds: u32 = 10;
+    const VARIANT_COUNT: u32 = 10;
 }
 
 impl pallet_balances::Config for Runtime {