rm migration

frame/nomination-pools/src/migration.rs

@@ -448,97 +448,4 @@ pub mod v3 {
 			Ok(())
 		}
 	}
-}
-
-pub mod v4 {
-	use super::*;
-
-	#[derive(Decode)]
-	pub struct OldBondedPoolInner<T: Config> {
-		pub points: BalanceOf<T>,
-		pub state: PoolState,
-		pub member_counter: u32,
-		pub roles: OldPoolRoles<T::AccountId>,
-	}
-
-	#[derive(Decode)]
-	pub struct OldPoolRoles<AccountId> {
-		pub depositor: AccountId,
-		pub root: Option<AccountId>,
-		pub nominator: Option<AccountId>,
-		pub state_toggler: Option<AccountId>,
-	}
-
-	impl<T: Config> OldBondedPoolInner<T> {
-		fn migrate_to_v4(self) -> BondedPoolInner<T> {
-			BondedPoolInner {
-				member_counter: self.member_counter,
-				points: self.points,
-				state: self.state,
-				roles: PoolRoles {
-					depositor: self.roles.depositor,
-					root: self.roles.root,
-					nominator: self.roles.nominator,
-					bouncer: self.roles.state_toggler,
-				},
-			}
-		}
-	}
-
-	/// This migration changes `PoolRoles` `state_toggler` property to `bouncer`.
-	pub struct MigrateToV4<T>(sp_std::marker::PhantomData<T>);
-	impl<T: Config> OnRuntimeUpgrade for MigrateToV4<T> {
-		fn on_runtime_upgrade() -> Weight {
-			let current = Pallet::<T>::current_storage_version();
-			let onchain = Pallet::<T>::on_chain_storage_version();
-
-			log!(
-				info,
-				"Running migration with current storage version {:?} / onchain {:?}",
-				current,
-				onchain
-			);
-
-			if current == 4 && onchain == 3 {
-				let mut translated = 0u64;
-				BondedPools::<T>::translate::<OldBondedPoolInner<T>, _>(|_key, old_value| {
-					translated.saturating_inc();
-					Some(old_value.migrate_to_v4())
-				});
-
-				current.put::<Pallet<T>>();
-				log!(info, "Upgraded {} pools, storage to version {:?}", translated, current);
-
-				// reads: translated + onchain version.
-				// writes: translated + current.put
-				T::DbWeight::get().reads_writes(translated + 1, translated + 1)
-			} else {
-				log!(info, "Migration did not execute. This probably should be removed");
-				T::DbWeight::get().reads(1)
-			}
-		}
-
-		#[cfg(feature = "try-runtime")]
-		fn pre_upgrade() -> Result<Vec<u8>, &'static str> {
-			ensure!(
-				Pallet::<T>::current_storage_version() > Pallet::<T>::on_chain_storage_version(),
-				"the on_chain version is equal or more than the current one"
-			);
-			Ok(Vec::new())
-		}
-
-		#[cfg(feature = "try-runtime")]
-		fn post_upgrade(_: Vec<u8>) -> Result<(), &'static str> {
-			// ensure all BondedPools items now contain an `inner.commission: Commission` field.
-			ensure!(
-				BondedPools::<T>::iter().all(|(_, inner)| inner.commission.current.is_none() &&
-					inner.commission.max.is_none() &&
-					inner.commission.change_rate.is_none() &&
-					inner.commission.throttle_from.is_none()),
-				"a commission value has been incorrectly set"
-			);
-			ensure!(Pallet::<T>::on_chain_storage_version() == 4, "wrong storage version");
-			Ok(())
-		}
-	}
-}
+}