git.delta.rocks / unique-network / refs/commits / ebc4cdb8a3c4

difftreelog

add force methods for collection in `Unique` , add events Stake, Unstake, SetAdmin

PraetorP2022-09-05parent: #9704432.patch.diff
in: master

8 files changed

modifiedCargo.lockdiffbeforeafterboth
--- a/Cargo.lock
+++ b/Cargo.lock
@@ -6473,7 +6473,7 @@
 
 [[package]]
 name = "pallet-unique"
-version = "0.1.3"
+version = "0.1.4"
 dependencies = [
  "ethereum",
  "evm-coder",
modifiedpallets/app-promotion/src/benchmarking.rsdiffbeforeafterboth
--- a/pallets/app-promotion/src/benchmarking.rs
+++ b/pallets/app-promotion/src/benchmarking.rs
@@ -54,7 +54,7 @@
 
 	payout_stakers{
 		let pallet_admin = account::<T::AccountId>("admin", 0, SEED);
-		let share = Perbill::from_rational(1u32, 10);
+		let share = Perbill::from_rational(1u32, 100);
 		PromototionPallet::<T>::set_admin_address(RawOrigin::Root.into(), T::CrossAccountId::from_sub(pallet_admin.clone()))?;
 		let _ = <T as Config>::Currency::make_free_balance_be(&pallet_admin,  Perbill::from_rational(1u32, 2) * BalanceOf::<T>::max_value());
 		let staker: T::AccountId = account("caller", 0, SEED);
modifiedpallets/app-promotion/src/lib.rsdiffbeforeafterboth
--- a/pallets/app-promotion/src/lib.rs
+++ b/pallets/app-promotion/src/lib.rs
@@ -70,16 +70,11 @@
 };
 
 pub const LOCK_IDENTIFIER: [u8; 8] = *b"appstake";
+
 const PENDING_LIMIT_PER_BLOCK: u32 = 3;
 
 type BalanceOf<T> =
 	<<T as Config>::Currency as Currency<<T as frame_system::Config>::AccountId>>::Balance;
-
-// const SECONDS_TO_BLOCK: u32 = 6;
-// const DAY: u32 = 60 * 60 * 24 / SECONDS_TO_BLOCK;
-// const WEEK: u32 = 7 * DAY;
-// const TWO_WEEK: u32 = 2 * WEEK;
-// const YEAR: u32 = DAY * 365;
 
 #[frame_support::pallet]
 pub mod pallet {
@@ -115,9 +110,9 @@
 		#[pallet::constant]
 		type PendingInterval: Get<Self::BlockNumber>;
 
-		/// In chain blocks.
-		#[pallet::constant]
-		type Day: Get<Self::BlockNumber>; // useless
+		// /// In chain blocks.
+		// #[pallet::constant]
+		// type Day: Get<Self::BlockNumber>; // useless
 
 		#[pallet::constant]
 		type Nominal: Get<BalanceOf<Self>>;
@@ -150,6 +145,9 @@
 			/// Amount of accrued interest
 			BalanceOf<T>,
 		),
+		Stake(T::AccountId, BalanceOf<T>),
+		Unstake(T::AccountId, BalanceOf<T>),
+		SetAdmin(T::AccountId),
 	}
 
 	#[pallet::error]
@@ -205,9 +203,9 @@
 		ValueQuery,
 	>;
 
-	/// A block when app-promotion has started .I think this is redundant, because we only need `NextInterestBlock`.
-	#[pallet::storage]
-	pub type StartBlock<T: Config> = StorageValue<Value = T::BlockNumber, QueryKind = ValueQuery>;
+	// /// A block when app-promotion has started .I think this is redundant, because we only need `NextInterestBlock`.
+	// #[pallet::storage]
+	// pub type StartBlock<T: Config> = StorageValue<Value = T::BlockNumber, QueryKind = ValueQuery>;
 
 	// /// Next target block when interest is recalculated
 	// #[pallet::storage]
@@ -257,50 +255,14 @@
 		#[pallet::weight(T::WeightInfo::set_admin_address())]
 		pub fn set_admin_address(origin: OriginFor<T>, admin: T::CrossAccountId) -> DispatchResult {
 			ensure_root(origin)?;
+
 			<Admin<T>>::set(Some(admin.as_sub().to_owned()));
 
+			Self::deposit_event(Event::SetAdmin(admin.as_sub().to_owned()));
+
 			Ok(())
 		}
-
-		// #[pallet::weight(T::WeightInfo::start_app_promotion())]
-		// pub fn start_app_promotion(
-		// 	origin: OriginFor<T>,
-		// 	promotion_start_relay_block: Option<T::BlockNumber>,
-		// ) -> DispatchResult
-		// where
-		// 	<T as frame_system::Config>::BlockNumber: From<u32>,
-		// {
-		// 	ensure_root(origin)?;
-
-		// 	// Start app-promotion mechanics if it has not been yet initialized
-		// 	if <StartBlock<T>>::get() == 0u32.into() {
-		// 		let start_block = promotion_start_relay_block
-		// 			.unwrap_or(T::RelayBlockNumberProvider::current_block_number());
-
-		// 		// Set promotion global start block
-		// 		<StartBlock<T>>::set(start_block);
-
-		// 		<NextInterestBlock<T>>::set(start_block + T::RecalculationInterval::get());
-		// 	}
-
-		// 	Ok(())
-		// }
 
-		// #[pallet::weight(T::WeightInfo::stop_app_promotion())]
-		// pub fn stop_app_promotion(origin: OriginFor<T>) -> DispatchResult
-		// where
-		// 	<T as frame_system::Config>::BlockNumber: From<u32>,
-		// {
-		// 	ensure_root(origin)?;
-
-		// 	if <StartBlock<T>>::get() != 0u32.into() {
-		// 		<StartBlock<T>>::set(T::BlockNumber::default());
-		// 		<NextInterestBlock<T>>::set(T::BlockNumber::default());
-		// 	}
-
-		// 	Ok(())
-		// }
-
 		#[pallet::weight(T::WeightInfo::stake())]
 		pub fn stake(staker: OriginFor<T>, amount: BalanceOf<T>) -> DispatchResult {
 			let staker_id = ensure_signed(staker)?;
@@ -350,6 +312,9 @@
 			);
 
 			StakesPerAccount::<T>::mutate(&staker_id, |stakes| *stakes += 1);
+
+			Self::deposit_event(Event::Stake(staker_id, amount));
+
 			Ok(())
 		}
 
@@ -392,6 +357,8 @@
 
 			StakesPerAccount::<T>::remove(&staker_id);
 
+			Self::deposit_event(Event::Unstake(staker_id, total_staked));
+
 			Ok(None.into())
 		}
 
@@ -527,13 +494,13 @@
 			// 		}
 			// 	}
 			// }
-			
+
 			{
 				let mut stakers_number = stakers_number.unwrap_or(20);
-				let  last_id = RefCell::new(None);
+				let last_id = RefCell::new(None);
 				let income_acc = RefCell::new(BalanceOf::<T>::default());
 				let amount_acc = RefCell::new(BalanceOf::<T>::default());
-				
+
 				let flush_stake = || -> DispatchResult {
 					if let Some(last_id) = &*last_id.borrow() {
 						if !income_acc.borrow().is_zero() {
@@ -573,7 +540,7 @@
 					};
 					*last_id.borrow_mut() = Some(current_id.clone());
 					if current_recalc_block >= next_recalc_block_for_stake {
-						*amount_acc.borrow_mut() += amount;  
+						*amount_acc.borrow_mut() += amount;
 						Self::recalculate_and_insert_stake(
 							&current_id,
 							staked_block,
@@ -588,7 +555,6 @@
 				}
 				flush_stake()?;
 			}
-
 
 			Ok(())
 		}
modifiedpallets/app-promotion/src/types.rsdiffbeforeafterboth
--- a/pallets/app-promotion/src/types.rs
+++ b/pallets/app-promotion/src/types.rs
@@ -5,7 +5,7 @@
 
 use pallet_balances::{BalanceLock, Config as BalancesConfig, Pallet as PalletBalances};
 use pallet_common::CollectionHandle;
-use pallet_unique::{Event as UniqueEvent, Error as UniqueError};
+
 use sp_runtime::DispatchError;
 use up_data_structs::{CollectionId, SponsorshipState};
 use sp_std::borrow::ToOwned;
@@ -53,36 +53,11 @@
 		sponsor_id: Self::AccountId,
 		collection_id: Self::CollectionId,
 	) -> DispatchResult {
-		let mut target_collection = <CollectionHandle<T>>::try_get(collection_id)?;
-		target_collection.check_is_internal()?;
-		target_collection.set_sponsor(sponsor_id.clone())?;
-
-		Self::deposit_event(UniqueEvent::<T>::CollectionSponsorSet(
-			collection_id,
-			sponsor_id.clone(),
-		));
-
-		ensure!(
-			target_collection.confirm_sponsorship(&sponsor_id)?,
-			UniqueError::<T>::ConfirmUnsetSponsorFail
-		);
-
-		Self::deposit_event(UniqueEvent::<T>::SponsorshipConfirmed(
-			collection_id,
-			sponsor_id,
-		));
-
-		target_collection.save()
+		Self::force_set_sponsor(sponsor_id, collection_id)
 	}
 
 	fn remove_collection_sponsor(collection_id: Self::CollectionId) -> DispatchResult {
-		let mut target_collection = <CollectionHandle<T>>::try_get(collection_id)?;
-		target_collection.check_is_internal()?;
-		target_collection.sponsorship = SponsorshipState::Disabled;
-
-		Self::deposit_event(UniqueEvent::<T>::CollectionSponsorRemoved(collection_id));
-
-		target_collection.save()
+		Self::force_remove_collection_sponsor(collection_id)
 	}
 
 	fn get_sponsor(
modifiedpallets/unique/CHANGELOG.mddiffbeforeafterboth
--- a/pallets/unique/CHANGELOG.md
+++ b/pallets/unique/CHANGELOG.md
@@ -3,22 +3,27 @@
 All notable changes to this project will be documented in this file.
 
 <!-- bureaucrate goes here -->
+
+## [v0.1.4] 2022-09-5
+
+### Added
+
 ## [v0.1.3] 2022-08-16
 
 ### Other changes
 
-- build: Upgrade polkadot to v0.9.27 2c498572636f2b34d53b1c51b7283a761a7dc90a
+-   build: Upgrade polkadot to v0.9.27 2c498572636f2b34d53b1c51b7283a761a7dc90a
 
-- build: Upgrade polkadot to v0.9.26 85515e54c4ca1b82a2630034e55dcc804c643bf8
+-   build: Upgrade polkadot to v0.9.26 85515e54c4ca1b82a2630034e55dcc804c643bf8
 
-- refactor: Remove `#[transactional]` from extrinsics 7fd36cea2f6e00c02c67ccc1de9649ae404efd31
+-   refactor: Remove `#[transactional]` from extrinsics 7fd36cea2f6e00c02c67ccc1de9649ae404efd31
 
 Every extrinsic now runs in transaction implicitly, and
 `#[transactional]` on pallet dispatchable is now meaningless
 
 Upstream-Change: https://github.com/paritytech/substrate/issues/10806
 
-- refactor: Switch to new prefix removal methods 26734e9567589d75cdd99e404eabf11d5a97d975
+-   refactor: Switch to new prefix removal methods 26734e9567589d75cdd99e404eabf11d5a97d975
 
 New methods allows to call `remove_prefix` with limit multiple times
 in the same block
@@ -27,10 +32,12 @@
 
 Upstream-Change: https://github.com/paritytech/substrate/pull/11490
 
-- build: Upgrade polkadot to v0.9.25 cdfb9bdc7b205ff1b5134f034ef9973d769e5e6b
+-   build: Upgrade polkadot to v0.9.25 cdfb9bdc7b205ff1b5134f034ef9973d769e5e6b
 
 ## [v0.1.1] - 2022-07-25
+
 ### Added
-- Method for creating `ERC721Metadata` compatible NFT collection.
-- Method for creating `ERC721Metadata` compatible ReFungible collection.
-- Method for creating ReFungible collection.
+
+-   Method for creating `ERC721Metadata` compatible NFT collection.
+-   Method for creating `ERC721Metadata` compatible ReFungible collection.
+-   Method for creating ReFungible collection.
modifiedpallets/unique/Cargo.tomldiffbeforeafterboth
--- a/pallets/unique/Cargo.toml
+++ b/pallets/unique/Cargo.toml
@@ -9,7 +9,7 @@
 license = 'GPLv3'
 name = 'pallet-unique'
 repository = 'https://github.com/UniqueNetwork/unique-chain'
-version = "0.1.3"
+version = "0.1.4"
 
 [package.metadata.docs.rs]
 targets = ['x86_64-unknown-linux-gnu']
modifiedpallets/unique/src/lib.rsdiffbeforeafterboth
--- a/pallets/unique/src/lib.rs
+++ b/pallets/unique/src/lib.rs
@@ -1103,3 +1103,35 @@
 		}
 	}
 }
+
+impl<T: Config> Pallet<T> {
+	pub fn force_set_sponsor(sponsor: T::AccountId, collection_id: CollectionId) -> DispatchResult {
+		let mut target_collection = <CollectionHandle<T>>::try_get(collection_id)?;
+		target_collection.check_is_internal()?;
+		target_collection.set_sponsor(sponsor.clone())?;
+
+		Self::deposit_event(Event::<T>::CollectionSponsorSet(
+			collection_id,
+			sponsor.clone(),
+		));
+
+		ensure!(
+			target_collection.confirm_sponsorship(&sponsor)?,
+			Error::<T>::ConfirmUnsetSponsorFail
+		);
+
+		Self::deposit_event(Event::<T>::SponsorshipConfirmed(collection_id, sponsor));
+
+		target_collection.save()
+	}
+
+	pub fn force_remove_collection_sponsor(collection_id: CollectionId) -> DispatchResult {
+		let mut target_collection = <CollectionHandle<T>>::try_get(collection_id)?;
+		target_collection.check_is_internal()?;
+		target_collection.sponsorship = SponsorshipState::Disabled;
+
+		Self::deposit_event(Event::<T>::CollectionSponsorRemoved(collection_id));
+
+		target_collection.save()
+	}
+}
modifiedruntime/common/config/pallets/app_promotion.rsdiffbeforeafterboth
22use frame_support::{parameter_types, PalletId};22use frame_support::{parameter_types, PalletId};
23use sp_arithmetic::Perbill;23use sp_arithmetic::Perbill;
24use up_common::{24use up_common::{
25 constants::{DAYS, UNIQUE, RELAY_DAYS},25 constants::{ UNIQUE, RELAY_DAYS},
26 types::Balance,26 types::Balance,
27};27};
2828
32 pub const RecalculationInterval: BlockNumber = 20;32 pub const RecalculationInterval: BlockNumber = 20;
33 pub const PendingInterval: BlockNumber = 10;33 pub const PendingInterval: BlockNumber = 10;
34 pub const Nominal: Balance = UNIQUE;34 pub const Nominal: Balance = UNIQUE;
35 pub const Day: BlockNumber = DAYS;35 // pub const Day: BlockNumber = DAYS;
36 pub IntervalIncome: Perbill = Perbill::from_rational(RecalculationInterval::get(), RELAY_DAYS) * Perbill::from_rational(5u32, 10_000);36 pub IntervalIncome: Perbill = Perbill::from_rational(RecalculationInterval::get(), RELAY_DAYS) * Perbill::from_rational(5u32, 10_000);
37}37}
3838
42 pub const RecalculationInterval: BlockNumber = RELAY_DAYS;42 pub const RecalculationInterval: BlockNumber = RELAY_DAYS;
43 pub const PendingInterval: BlockNumber = 7 * RELAY_DAYS;43 pub const PendingInterval: BlockNumber = 7 * RELAY_DAYS;
44 pub const Nominal: Balance = UNIQUE;44 pub const Nominal: Balance = UNIQUE;
45 pub const Day: BlockNumber = RELAY_DAYS;45 // pub const Day: BlockNumber = RELAY_DAYS;
46 pub IntervalIncome: Perbill = Perbill::from_rational(5u32, 10_000);46 pub IntervalIncome: Perbill = Perbill::from_rational(5u32, 10_000);
47}47}
4848
56 type RelayBlockNumberProvider = RelayChainBlockNumberProvider<Runtime>;56 type RelayBlockNumberProvider = RelayChainBlockNumberProvider<Runtime>;
57 type RecalculationInterval = RecalculationInterval;57 type RecalculationInterval = RecalculationInterval;
58 type PendingInterval = PendingInterval;58 type PendingInterval = PendingInterval;
59 type Day = Day;59 // type Day = Day;
60 type Nominal = Nominal;60 type Nominal = Nominal;
61 type IntervalIncome = IntervalIncome;61 type IntervalIncome = IntervalIncome;
62 type Event = Event;62 type Event = Event;