Fix Pools 6->7 migration

prdoc/pr_2796.prdoc

@@ -1,4 +1,4 @@
-title: Rococo and Westend Asset-Hub: XCM Transfers with Pallet-Uniques
+title: "Rococo and Westend Asset-Hub: XCM Transfers with Pallet-Uniques"
 
 doc:
  - audience: Runtime User

prdoc/pr_2920.prdoc

@@ -1,11 +1,12 @@
 # Schema: Polkadot SDK PRDoc Schema (prdoc) v1.0.0
 # See doc at https://raw.githubusercontent.com/paritytech/polkadot-sdk/master/prdoc/schema_user.json
 
-title: Contracts: Stabilize sr25519_verify host function
+title: "Contracts: Stabilize sr25519_verify host function"
 
 doc:
  - audience: Runtime Dev
  description: |
  Removed the `#[unstable]` attrribute on `sr25519_verify` host function.
 
-crates: ["pallet-contracts"]
+crates:
+ - name: "pallet-contracts"

prdoc/pr_2942.prdoc

@@ -0,0 +1,13 @@
+# Schema: Polkadot SDK PRDoc Schema (prdoc) v1.0.0
+# See doc at https://raw.githubusercontent.com/paritytech/polkadot-sdk/master/prdoc/schema_user.json
+
+title: Fix pallet-nomination-pools v6 to v7 migration
+
+doc:
+ - audience: Node Dev
+ description: |
+ Restores the behaviour of the nomination pools `V6ToV7` migration so that it still works when
+ the pallet will be upgraded to V8 afterwards.
+
+crates:
+ - name: "pallet-nomination-pools"

substrate/frame/nomination-pools/src/migration.rs

@@ -59,22 +59,7 @@ pub mod versioned {
 pub mod v8 {
  use super::*;
 
- #[derive(Decode)]
- pub struct OldCommission<T: Config> {
- pub current: Option<(Perbill, T::AccountId)>,
- pub max: Option<Perbill>,
- pub change_rate: Option<CommissionChangeRate<BlockNumberFor<T>>>,
- pub throttle_from: Option<BlockNumberFor<T>>,
- }
-
- #[derive(Decode)]
- pub struct OldBondedPoolInner<T: Config> {
- pub commission: OldCommission<T>,
- pub member_counter: u32,
- pub points: BalanceOf<T>,
- pub roles: PoolRoles<T::AccountId>,
- pub state: PoolState,
- }
+ use super::v7::BondedPoolInner as OldBondedPoolInner;
 
  impl<T: Config> OldBondedPoolInner<T> {
  fn migrate_to_v8(self) -> BondedPoolInner<T> {
@@ -128,9 +113,50 @@ pub mod v8 {
 ///
 /// WARNING: This migration works under the assumption that the [`BondedPools`] cannot be inflated
 /// arbitrarily. Otherwise this migration could fail due to too high weight.
-mod v7 {
+pub(crate) mod v7 {
  use super::*;
 
+ #[derive(Encode, Decode, MaxEncodedLen, TypeInfo, DebugNoBound, PartialEq, Clone)]
+ #[codec(mel_bound(T: Config))]
+ #[scale_info(skip_type_params(T))]
+ pub struct Commission<T: Config> {
+ pub current: Option<(Perbill, T::AccountId)>,
+ pub max: Option<Perbill>,
+ pub change_rate: Option<CommissionChangeRate<BlockNumberFor<T>>>,
+ pub throttle_from: Option<BlockNumberFor<T>>,
+ }
+
+ #[derive(Encode, Decode, MaxEncodedLen, TypeInfo, DebugNoBound, PartialEq, Clone)]
+ #[codec(mel_bound(T: Config))]
+ #[scale_info(skip_type_params(T))]
+ pub struct BondedPoolInner<T: Config> {
+ pub commission: Commission<T>,
+ pub member_counter: u32,
+ pub points: BalanceOf<T>,
+ pub roles: PoolRoles<T::AccountId>,
+ pub state: PoolState,
+ }
+
+ #[allow(dead_code)]
+ #[derive(RuntimeDebugNoBound)]
+ #[cfg_attr(feature = "std", derive(Clone, PartialEq))]
+ pub struct BondedPool<T: Config> {
+ /// The identifier of the pool.
+ id: PoolId,
+ /// The inner fields.
+ inner: BondedPoolInner<T>,
+ }
+
+ impl<T: Config> BondedPool<T> {
+ fn bonded_account(&self) -> T::AccountId {
+ Pallet::<T>::create_bonded_account(self.id)
+ }
+ }
+
+ #[frame_support::storage_alias]
+ pub type BondedPools<T: Config> =
+ CountedStorageMap<Pallet<T>, Twox64Concat, PoolId, BondedPoolInner<T>>;
+
  pub struct VersionUncheckedMigrateV6ToV7<T>(sp_std::marker::PhantomData<T>);
  impl<T: Config> VersionUncheckedMigrateV6ToV7<T> {
  fn calculate_tvl_by_total_stake() -> BalanceOf<T> {