git.delta.rocks / unique-network / refs/commits / 13a972624863

40use frame_support::{40use frame_support::{
41	assert_ok,41	assert_ok,
42	codec::Decode,42	codec::Decode,
43	traits::{Currency, EnsureOrigin, Get},43	traits::{
44		EnsureOrigin,
45		fungible::{Inspect, Mutate},
46		Get,
47	},
44};48};
45use frame_system::{EventRecord, RawOrigin};49use frame_system::{EventRecord, RawOrigin};
78) -> T::AccountId {82) -> T::AccountId {
79	let user = account(string, n, SEED);83	let user = account(string, n, SEED);
80	let balance = balance_unit::<T>() * balance_factor.into();84	let balance = balance_unit::<T>() * balance_factor.into();
81	let _ = T::Currency::make_free_balance_be(&user, balance);85	let _ = T::Currency::set_balance(&user, balance);
82	user86	user
83}87}
8488
137	);141	);
138142
139	for who in candidates {143	for who in candidates {
140		T::Currency::make_free_balance_be(&who, <LicenseBond<T>>::get() * 2u32.into());144		T::Currency::set_balance(&who, <LicenseBond<T>>::get() * 2u32.into());
141		<CollatorSelection<T>>::get_license(RawOrigin::Signed(who.clone()).into()).unwrap();145		<CollatorSelection<T>>::get_license(RawOrigin::Signed(who.clone()).into()).unwrap();
142		<CollatorSelection<T>>::onboard(RawOrigin::Signed(who).into()).unwrap();146		<CollatorSelection<T>>::onboard(RawOrigin::Signed(who).into()).unwrap();
143	}147	}
153	);157	);
154158
155	for who in candidates {159	for who in candidates {
156		T::Currency::make_free_balance_be(&who, <LicenseBond<T>>::get() * 2u32.into());160		T::Currency::set_balance(&who, <LicenseBond<T>>::get() * 2u32.into());
157		<CollatorSelection<T>>::get_license(RawOrigin::Signed(who.clone()).into()).unwrap();161		<CollatorSelection<T>>::get_license(RawOrigin::Signed(who.clone()).into()).unwrap();
158	}162	}
159}163}
160164
161/// `Currency::minimum_balance` was used originally, but in unique-chain, we have165/// `Currency::minimum_balance` was used originally, but in unique-chain, we have
162/// zero existential deposit, thus triggering zero bond assertion.166/// zero existential deposit, thus triggering zero bond assertion.
163fn balance_unit<T: Config>() -> <T::Currency as Currency<T::AccountId>>::Balance {167fn balance_unit<T: Config>() -> BalanceOf<T> {
164	200u32.into()168	200u32.into()
165}169}
166170
182188
183		let new_invulnerable: T::AccountId = whitelisted_caller();189		let new_invulnerable: T::AccountId = whitelisted_caller();
184		let bond: BalanceOf<T> = balance_unit::<T>() * 2u32.into();190		let bond: BalanceOf<T> = balance_unit::<T>() * 2u32.into();
185		T::Currency::make_free_balance_be(&new_invulnerable, bond.clone());191		T::Currency::set_balance(&new_invulnerable, bond.clone());
186192
187		<session::Pallet<T>>::set_keys(193		<session::Pallet<T>>::set_keys(
188			RawOrigin::Signed(new_invulnerable.clone()).into(),194			RawOrigin::Signed(new_invulnerable.clone()).into(),
227233
228		let caller: T::AccountId = whitelisted_caller();234		let caller: T::AccountId = whitelisted_caller();
229		let bond: BalanceOf<T> = balance_unit::<T>() * 2u32.into();235		let bond: BalanceOf<T> = balance_unit::<T>() * 2u32.into();
230		T::Currency::make_free_balance_be(&caller, bond.clone());236		T::Currency::set_balance(&caller, bond.clone());
231237
232		<session::Pallet<T>>::set_keys(238		<session::Pallet<T>>::set_keys(
233			RawOrigin::Signed(caller.clone()).into(),239			RawOrigin::Signed(caller.clone()).into(),
253259
254		let caller: T::AccountId = whitelisted_caller();260		let caller: T::AccountId = whitelisted_caller();
255		let bond: BalanceOf<T> = balance_unit::<T>() * 2u32.into();261		let bond: BalanceOf<T> = balance_unit::<T>() * 2u32.into();
256		T::Currency::make_free_balance_be(&caller, bond.clone());262		T::Currency::set_balance(&caller, bond.clone());
257263
258		let origin = RawOrigin::Signed(caller.clone());264		let origin = RawOrigin::Signed(caller.clone());
259265
329	// worst case is paying a non-existing candidate account.335	// worst case is paying a non-existing candidate account.
330	note_author {336	note_author {
331		<LicenseBond<T>>::put(balance_unit::<T>());337		<LicenseBond<T>>::put(balance_unit::<T>());
332		T::Currency::make_free_balance_be(338		T::Currency::set_balance(
333			&<CollatorSelection<T>>::account_id(),339			&<CollatorSelection<T>>::account_id(),
334			balance_unit::<T>() * 4u32.into(),340			balance_unit::<T>() * 4u32.into(),
335		);341		);
336		let author = account("author", 0, SEED);342		let author = account("author", 0, SEED);
337		let new_block: T::BlockNumber = 10u32.into();343		let new_block: T::BlockNumber = 10u32.into();
338344
339		frame_system::Pallet::<T>::set_block_number(new_block);345		frame_system::Pallet::<T>::set_block_number(new_block);
340		assert!(T::Currency::free_balance(&author) == 0u32.into());346		assert!(T::Currency::balance(&author) == 0u32.into());
341	}: {347	}: {
342		<CollatorSelection<T> as EventHandler<_, _>>::note_author(author.clone())348		<CollatorSelection<T> as EventHandler<_, _>>::note_author(author.clone())
343	} verify {349	} verify {
344		assert!(T::Currency::free_balance(&author) > 0u32.into());350		assert!(T::Currency::balance(&author) > 0u32.into());
345		assert_eq!(frame_system::Pallet::<T>::block_number(), new_block);351		assert_eq!(frame_system::Pallet::<T>::block_number(), new_block);
346	}352	}
347353

--- a/pallets/collator-selection/src/lib.rs
+++ b/pallets/collator-selection/src/lib.rs
@@ -92,6 +92,7 @@
 
 #[frame_support::pallet]
 pub mod pallet {
+	use super::*;
 	pub use crate::weights::WeightInfo;
 	use core::ops::Div;
 	use frame_support::{
@@ -100,8 +101,10 @@
 		pallet_prelude::*,
 		sp_runtime::traits::{AccountIdConversion, CheckedSub, Saturating, Zero},
 		traits::{
-			Currency, EnsureOrigin, ExistenceRequirement::KeepAlive, ReservableCurrency,
+			EnsureOrigin,
+			fungible::{Balanced, BalancedHold, Inspect, InspectHold, Mutate, MutateHold},
 			ValidatorRegistration,
+			tokens::{Precision, Preservation},
 		},
 		BoundedVec, PalletId,
 	};
@@ -158,6 +161,9 @@
 
 		/// The weight information of this pallet.
 		type WeightInfo: WeightInfo;
+
+		#[pallet::constant]
+		type LicenceBondIdentifier: Get<<<Self as pallet_configuration::Config>::Currency as InspectHold<Self::AccountId>>::Reason>;
 	}
 
 	#[pallet::pallet]
@@ -361,7 +367,7 @@
 
 			let deposit = <LicenseBond<T>>::get();
 
-			T::Currency::reserve(&who, deposit)?;
+			T::Currency::hold(&T::LicenceBondIdentifier::get(), &who, deposit)?;
 			LicenseDepositOf::<T>::insert(who.clone(), deposit);
 
 			Self::deposit_event(Event::LicenseObtained {
@@ -523,17 +529,24 @@
 						let slashed = T::SlashRatio::get() * deposit;
 						let remaining = deposit - slashed;
 
-						let (imbalance, _) = T::Currency::slash_reserved(who, slashed);
+						let (imbalance, _) =
+							T::Currency::slash(&T::LicenceBondIdentifier::get(), who, slashed);
 						//T::Currency::unreserve(who, remaining);
 						deposit_returned = remaining;
 
-						T::Currency::resolve_creating(&T::TreasuryAccountId::get(), imbalance);
+						T::Currency::resolve(&T::TreasuryAccountId::get(), imbalance)
+							.map_err(|_| DispatchError::Other("Failed to deposit imbalance"))?;
 					} else {
 						//T::Currency::unreserve(who, deposit);
 						deposit_returned = deposit;
 					}
 
-					T::Currency::unreserve(who, deposit_returned);
+					T::Currency::release(
+						&T::LicenceBondIdentifier::get(),
+						who,
+						deposit_returned,
+						Precision::Exact,
+					)?;
 					Ok(())
 				} else {
 					Err(Error::<T>::NoLicense.into())
@@ -594,12 +607,12 @@
 		fn note_author(author: T::AccountId) {
 			let pot = Self::account_id();
 			// assumes an ED will be sent to pot.
-			let reward = T::Currency::free_balance(&pot)
+			let reward = T::Currency::balance(&pot)
 				.checked_sub(&T::Currency::minimum_balance())
 				.unwrap_or_else(Zero::zero)
 				.div(2u32.into());
 			// `reward` is half of pot account minus ED, this should never fail.
-			let _success = T::Currency::transfer(&pot, &author, reward, KeepAlive);
+			let _success = T::Currency::transfer(&pot, &author, reward, Preservation::Preserve);
 			debug_assert!(_success.is_ok());
 			<LastAuthoredBlock<T>>::insert(author, frame_system::Pallet::<T>::block_number());
 

--- a/pallets/collator-selection/src/tests.rs
+++ b/pallets/collator-selection/src/tests.rs
@@ -417,7 +417,10 @@
 fn authorship_event_handler() {
 	new_test_ext().execute_with(|| {
 		// put 100 in the pot + 5 for ED
-		Balances::make_free_balance_be(&CollatorSelection::account_id(), 105);
+		<pallet_balances::Pallet<T> as fungible::Mutate<T::AccountId>>::set_balance(
+			&CollatorSelection::account_id(),
+			105,
+		);
 
 		// 4 is the default author.
 		assert_eq!(Balances::free_balance(4), 100);
@@ -441,7 +444,10 @@
 		// Nothing panics, no reward when no ED in balance
 		Authorship::on_initialize(1);
 		// put some money into the pot at ED
-		Balances::make_free_balance_be(&CollatorSelection::account_id(), 5);
+		<pallet_balances::Pallet<T> as fungible::Mutate<T::AccountId>>::set_balance(
+			&CollatorSelection::account_id(),
+			5,
+		);
 		// 4 is the default author.
 		assert_eq!(Balances::free_balance(4), 100);
 		get_license_and_onboard(4);

--- a/pallets/common/src/benchmarking.rs
+++ b/pallets/common/src/benchmarking.rs
@@ -27,12 +27,12 @@
 	MAX_PROPERTIES_PER_ITEM,
 };
 use frame_support::{
-	traits::{Currency, Get},
+	traits::{Get, fungible::Balanced, Imbalance, tokens::Precision},
 	pallet_prelude::ConstU32,
 	BoundedVec,
 };
 use core::convert::TryInto;
-use sp_runtime::DispatchError;
+use sp_runtime::{DispatchError, traits::Zero};
 
 const SEED: u32 = 1;
 
@@ -85,7 +85,12 @@
 	) -> Result<CollectionId, DispatchError>,
 	cast: impl FnOnce(CollectionHandle<T>) -> R,
 ) -> Result<R, DispatchError> {
-	<T as Config>::Currency::deposit_creating(&owner.as_sub(), T::CollectionCreationPrice::get());
+	let imbalance = <T as Config>::Currency::deposit(
+		&owner.as_sub(),
+		T::CollectionCreationPrice::get(),
+		Precision::Exact,
+	)?;
+	debug_assert!(imbalance.peek().is_zero());
 	let name = create_u16_data::<MAX_COLLECTION_NAME_LENGTH>();
 	let description = create_u16_data::<MAX_COLLECTION_DESCRIPTION_LENGTH>();
 	let token_prefix = create_data::<MAX_TOKEN_PREFIX_LENGTH>();

--- a/pallets/common/src/lib.rs
+++ b/pallets/common/src/lib.rs
@@ -64,7 +64,11 @@
 use frame_support::{
 	dispatch::{DispatchErrorWithPostInfo, DispatchResultWithPostInfo, Weight, PostDispatchInfo},
 	ensure,
-	traits::{Imbalance, Get, Currency, WithdrawReasons, ExistenceRequirement},
+	traits::{
+		Get,
+		fungible::{Balanced, Debt, Inspect},
+		tokens::{Imbalance, Precision, Preservation},
+	},
 	dispatch::Pays,
 	transactional, fail,
 };
@@ -85,7 +89,7 @@
 
 pub use pallet::*;
 use sp_core::H160;
-use sp_runtime::{ArithmeticError, DispatchError, DispatchResult};
+use sp_runtime::{ArithmeticError, DispatchError, DispatchResult, traits::Zero};
 
 #[cfg(feature = "runtime-benchmarks")]
 pub mod benchmarking;
@@ -424,7 +428,6 @@
 	use super::*;
 	use dispatch::CollectionDispatch;
 	use frame_support::{Blake2_128Concat, pallet_prelude::*, storage::Key, traits::StorageVersion};
-	use frame_support::traits::Currency;
 	use up_data_structs::{TokenId, mapping::TokenAddressMapping};
 	use scale_info::TypeInfo;
 	use weights::WeightInfo;
@@ -440,12 +443,12 @@
 		type RuntimeEvent: IsType<<Self as frame_system::Config>::RuntimeEvent> + From<Event<Self>>;
 
 		/// Handler of accounts and payment.
-		type Currency: Currency<Self::AccountId>;
+		type Currency: Balanced<Self::AccountId> + Inspect<Self::AccountId>;
 
 		/// Set price to create a collection.
 		#[pallet::constant]
 		type CollectionCreationPrice: Get<
-			<<Self as Config>::Currency as Currency<Self::AccountId>>::Balance,
+			<<Self as Config>::Currency as Inspect<Self::AccountId>>::Balance,
 		>;
 
 		/// Dispatcher of operations on collections.
@@ -1112,21 +1115,17 @@
 
 		// Take a (non-refundable) deposit of collection creation
 		{
-			let mut imbalance =
-				<<<T as Config>::Currency as Currency<T::AccountId>>::PositiveImbalance>::zero();
-			imbalance.subsume(
-				<<T as Config>::Currency as Currency<T::AccountId>>::deposit_creating(
-					&T::TreasuryAccountId::get(),
-					T::CollectionCreationPrice::get(),
-				),
-			);
-			<T as Config>::Currency::settle(
-				payer.as_sub(),
-				imbalance,
-				WithdrawReasons::TRANSFER,
-				ExistenceRequirement::KeepAlive,
-			)
-			.map_err(|_| Error::<T>::NotSufficientFounds)?;
+			let mut imbalance = <Debt<T::AccountId, <T as Config>::Currency>>::zero();
+			imbalance.subsume(<T as Config>::Currency::deposit(
+				&T::TreasuryAccountId::get(),
+				T::CollectionCreationPrice::get(),
+				Precision::Exact,
+			)?);
+			let credit =
+				<T as Config>::Currency::settle(payer.as_sub(), imbalance, Preservation::Preserve)
+					.map_err(|_| Error::<T>::NotSufficientFounds)?;
+
+			debug_assert!(credit.peek().is_zero())
 		}
 
 		<CreatedCollectionCount<T>>::put(created_count);

--- a/pallets/configuration/src/benchmarking.rs
+++ b/pallets/configuration/src/benchmarking.rs
@@ -19,7 +19,7 @@
 use super::*;
 use frame_benchmarking::benchmarks;
 use frame_system::{EventRecord, RawOrigin};
-use frame_support::{assert_ok, traits::Currency};
+use frame_support::{assert_ok, traits::fungible::Inspect};
 
 fn assert_last_event<T: Config>(generic_event: <T as Config>::RuntimeEvent) {
 	let events = frame_system::Pallet::<T>::events();
@@ -68,7 +68,7 @@
 	}
 
 	set_collator_selection_license_bond {
-		let bond_cost: Option<BalanceOf<T>> = Some(T::Currency::minimum_balance() * 10u32.into());
+		let bond_cost: Option<BalanceOf<T>> = Some(T::Balances::minimum_balance() * 10u32.into());
 	}: {
 		assert_ok!(
 			<Pallet<T>>::set_collator_selection_license_bond(RawOrigin::Root.into(), bond_cost.clone())

--- a/pallets/configuration/src/lib.rs
+++ b/pallets/configuration/src/lib.rs
@@ -42,7 +42,7 @@
 mod pallet {
 	use super::*;
 	use frame_support::{
-		traits::{Get, ReservableCurrency, Currency},
+		traits::{fungible, Get, ReservableCurrency, Currency},
 		pallet_prelude::{StorageValue, ValueQuery, DispatchResult, IsType},
 		log,
 	};
@@ -50,15 +50,19 @@
 
 	pub use crate::weights::WeightInfo;
 	pub type BalanceOf<T> =
-		<<T as Config>::Currency as Currency<<T as SystemConfig>::AccountId>>::Balance;
+		<<T as Config>::Currency as fungible::Inspect<<T as SystemConfig>::AccountId>>::Balance;
 
 	#[pallet::config]
 	pub trait Config: frame_system::Config {
 		/// Overarching event type.
 		type RuntimeEvent: From<Event<Self>> + IsType<<Self as frame_system::Config>::RuntimeEvent>;
 
-		/// The currency mechanism.
-		type Currency: ReservableCurrency<Self::AccountId>;
+		type Currency: fungible::Inspect<Self::AccountId>
+			+ fungible::Mutate<Self::AccountId>
+			+ fungible::MutateFreeze<Self::AccountId>
+			+ fungible::InspectHold<Self::AccountId>
+			+ fungible::MutateHold<Self::AccountId>
+			+ fungible::BalancedHold<Self::AccountId>;
 
 		#[pallet::constant]
 		type DefaultWeightToFeeCoefficient: Get<u64>;