feat: reset team on start_refund

Author: rflechtner

pallets/pallet-bonded-coins/src/lib.rs

@@ -393,6 +393,7 @@ pub mod pallet {
 
 			let pool_details = Pools::<T>::get(&pool_id).ok_or(Error::<T>::PoolUnknown)?;
 
+			ensure!(pool_details.state.is_live(), Error::<T>::PoolNotLive);
 			ensure!(pool_details.is_manager(&who), Error::<T>::NoPermission);
 
 			let asset_id = pool_details
@@ -958,8 +959,10 @@ pub mod pallet {
 				Error::<T>::NothingToRefund
 			);
 
-			let has_holders = pool_details
-				.bonded_currencies
+			// cloning here lets us avoid cloning the pool details later
+			let bonded_currencies = pool_details.bonded_currencies.clone();
+
+			let has_holders = bonded_currencies
 				.iter()
 				.any(|asset_id| T::Fungibles::total_issuance(asset_id.clone()) > FungiblesBalanceOf::<T>::zero());
 			// no token holders to refund
@@ -970,6 +973,18 @@ pub mod pallet {
 			new_pool_details.state.start_refund();
 			Pools::<T>::set(&pool_id, Some(new_pool_details));
 
+			// reset team on currencies to avoid unexpected burns etc.
+			let pool_account = pool_id.clone().into();
+			for asset_id in bonded_currencies {
+				T::Fungibles::reset_team(
+					asset_id,
+					pool_account.clone(),
+					pool_account.clone(),
+					pool_account.clone(),
+					pool_account.clone(),
+				)?;
+			}
+
 			Self::deposit_event(Event::RefundingStarted { id: pool_id });
 
 			Ok(n_currencies)