difftreelog
fix scheduler rebase
in: master
7 files changed
pallets/scheduler/src/lib.rsdiffbeforeafterboth--- a/pallets/scheduler/src/lib.rs
+++ b/pallets/scheduler/src/lib.rs
@@ -86,7 +86,7 @@
use sp_std::{borrow::Borrow, cmp::Ordering, marker::PhantomData, prelude::*};
use frame_support::{
- dispatch::{DispatchError, DispatchResult, Dispatchable, UnfilteredDispatchable, Parameter},
+ dispatch::{DispatchError, DispatchResult, Dispatchable, UnfilteredDispatchable, Parameter, GetDispatchInfo},
traits::{
schedule::{self, DispatchTime, MaybeHashed},
NamedReservableCurrency, EnsureOrigin, Get, IsType, OriginTrait, PrivilegeCmp,
@@ -101,7 +101,7 @@
pub type TaskAddress<BlockNumber> = (BlockNumber, u32);
pub const MAX_TASK_ID_LENGTH_IN_BYTES: u8 = 16;
-type ScheduledId = [u8; MAX_TASK_ID_LENGTH_IN_BYTES as usize];
+pub type ScheduledId = [u8; MAX_TASK_ID_LENGTH_IN_BYTES as usize];
pub type CallOrHashOf<T> =
MaybeHashed<<T as Config>::RuntimeCall, <T as frame_system::Config>::Hash>;
@@ -231,10 +231,10 @@
/// The aggregated call type.
type RuntimeCall: Parameter
- + Dispatchable<Origin = <Self as Config>::RuntimeOrigin, PostInfo = PostDispatchInfo>
- + UnfilteredDispatchable<Origin = <Self as system::Config>::RuntimeOrigin>
+ + Dispatchable<RuntimeOrigin = <Self as Config>::RuntimeOrigin, PostInfo = PostDispatchInfo>
+ + UnfilteredDispatchable<RuntimeOrigin = <Self as system::Config>::RuntimeOrigin>
+ GetDispatchInfo
- + From<system::RuntimeCall<Self>>;
+ + From<system::Call<Self>>;
/// The maximum weight that may be scheduled per block for any dispatchables of less
/// priority than `schedule::HARD_DEADLINE`.
@@ -611,7 +611,7 @@
priority: schedule::Priority,
) -> DispatchResult {
T::PrioritySetOrigin::ensure_origin(origin.clone())?;
- let origin = <T as Config>::Origin::from(origin);
+ let origin = <T as Config>::RuntimeOrigin::from(origin);
Self::do_change_named_priority(origin.caller().clone(), id, priority)
}
}
runtime/common/config/pallets/scheduler.rsdiffbeforeafterboth--- a/runtime/common/config/pallets/scheduler.rs
+++ b/runtime/common/config/pallets/scheduler.rs
@@ -25,7 +25,7 @@
use codec::Decode;
use crate::{
runtime_common::{scheduler::SchedulerPaymentExecutor, config::substrate::RuntimeBlockWeights},
- Runtime, Call, Event, Origin, OriginCaller, Balances,
+ Runtime, RuntimeCall, RuntimeEvent, RuntimeOrigin, OriginCaller, Balances,
};
use pallet_unique_scheduler::ScheduledEnsureOriginSuccess;
use up_common::types::AccountId;
runtime/common/config/test_pallets.rsdiffbeforeafterboth--- a/runtime/common/config/test_pallets.rs
+++ b/runtime/common/config/test_pallets.rs
@@ -14,8 +14,8 @@
// You should have received a copy of the GNU General Public License
// along with Unique Network. If not, see <http://www.gnu.org/licenses/>.
-use crate::{Runtime, Event};
+use crate::{Runtime, RuntimeEvent};
impl pallet_test_utils::Config for Runtime {
- type Event = Event;
+ type RuntimeEvent = RuntimeEvent;
}
runtime/common/construct_runtime/mod.rsdiffbeforeafterboth--- a/runtime/common/construct_runtime/mod.rs
+++ b/runtime/common/construct_runtime/mod.rs
@@ -57,8 +57,8 @@
Inflation: pallet_inflation::{Pallet, Call, Storage} = 60,
Unique: pallet_unique::{Pallet, Call, Storage, Event<T>} = 61,
- // #[runtimes(opal)]
- // Scheduler: pallet_unique_scheduler::{Pallet, Call, Storage, Event<T>} = 62,
+ #[runtimes(opal)]
+ Scheduler: pallet_unique_scheduler::{Pallet, Call, Storage, Event<T>} = 62,
Configuration: pallet_configuration::{Pallet, Call, Storage} = 63,
runtime/common/scheduler.rsdiffbeforeafterboth--- a/runtime/common/scheduler.rs
+++ b/runtime/common/scheduler.rs
@@ -16,7 +16,7 @@
use frame_support::{
traits::NamedReservableCurrency,
- weights::{GetDispatchInfo, PostDispatchInfo, DispatchInfo},
+ dispatch::{GetDispatchInfo, PostDispatchInfo, DispatchInfo},
};
use sp_runtime::{
traits::{Dispatchable, Applyable, Member},
@@ -25,7 +25,7 @@
DispatchErrorWithPostInfo, DispatchError,
};
use codec::Encode;
-use crate::{Runtime, Call, Origin, Balances};
+use crate::{Runtime, RuntimeCall, RuntimeOrigin, Balances};
use up_common::types::{AccountId, Balance};
use fp_self_contained::SelfContainedCall;
use pallet_unique_scheduler::DispatchCall;
@@ -62,20 +62,20 @@
impl<T: frame_system::Config + pallet_unique_scheduler::Config, SelfContainedSignedInfo>
DispatchCall<T, SelfContainedSignedInfo> for SchedulerPaymentExecutor
where
- <T as frame_system::Config>::Call: Member
- + Dispatchable<Origin = Origin, Info = DispatchInfo>
+ <T as frame_system::Config>::RuntimeCall: Member
+ + Dispatchable<RuntimeOrigin = RuntimeOrigin, Info = DispatchInfo>
+ SelfContainedCall<SignedInfo = SelfContainedSignedInfo>
+ GetDispatchInfo
+ From<frame_system::Call<Runtime>>,
SelfContainedSignedInfo: Send + Sync + 'static,
- Call: From<<T as frame_system::Config>::Call>
- + From<<T as pallet_unique_scheduler::Config>::Call>
+ RuntimeCall: From<<T as frame_system::Config>::RuntimeCall>
+ + From<<T as pallet_unique_scheduler::Config>::RuntimeCall>
+ SelfContainedCall<SignedInfo = SelfContainedSignedInfo>,
sp_runtime::AccountId32: From<<T as frame_system::Config>::AccountId>,
{
fn dispatch_call(
signer: Option<<T as frame_system::Config>::AccountId>,
- call: <T as pallet_unique_scheduler::Config>::Call,
+ call: <T as pallet_unique_scheduler::Config>::RuntimeCall,
) -> Result<
Result<PostDispatchInfo, DispatchErrorWithPostInfo<PostDispatchInfo>>,
TransactionValidityError,
@@ -93,7 +93,7 @@
let extrinsic = fp_self_contained::CheckedExtrinsic::<
AccountId,
- Call,
+ RuntimeCall,
SignedExtraScheduler,
SelfContainedSignedInfo,
> {
@@ -107,7 +107,7 @@
fn reserve_balance(
id: [u8; 16],
sponsor: <T as frame_system::Config>::AccountId,
- call: <T as pallet_unique_scheduler::Config>::Call,
+ call: <T as pallet_unique_scheduler::Config>::RuntimeCall,
count: u32,
) -> Result<(), DispatchError> {
let dispatch_info = call.get_dispatch_info();
@@ -125,7 +125,7 @@
fn pay_for_call(
id: [u8; 16],
sponsor: <T as frame_system::Config>::AccountId,
- call: <T as pallet_unique_scheduler::Config>::Call,
+ call: <T as pallet_unique_scheduler::Config>::RuntimeCall,
) -> Result<u128, DispatchError> {
let dispatch_info = call.get_dispatch_info();
let weight: Balance =
test-pallets/utils/Cargo.tomldiffbeforeafterboth--- a/test-pallets/utils/Cargo.toml
+++ b/test-pallets/utils/Cargo.toml
@@ -8,8 +8,8 @@
[dependencies]
codec = { package = "parity-scale-codec", version = "3.0.0", default-features = false, features = ["derive"] }
scale-info = { version = "2.1.1", default-features = false, features = ["derive"] }
-frame-support = { default-features = false, git = "https://github.com/paritytech/substrate", branch = "polkadot-v0.9.29" }
-frame-system = { default-features = false, git = "https://github.com/paritytech/substrate", branch = "polkadot-v0.9.29" }
+frame-support = { default-features = false, git = "https://github.com/paritytech/substrate", branch = "polkadot-v0.9.30" }
+frame-system = { default-features = false, git = "https://github.com/paritytech/substrate", branch = "polkadot-v0.9.30" }
pallet-unique-scheduler = { path = '../../pallets/scheduler', default-features = false }
[features]
test-pallets/utils/src/lib.rsdiffbeforeafterboth1// Copyright 2019-2022 Unique Network (Gibraltar) Ltd.2// This file is part of Unique Network.34// Unique Network is free software: you can redistribute it and/or modify5// it under the terms of the GNU General Public License as published by6// the Free Software Foundation, either version 3 of the License, or7// (at your option) any later version.89// Unique Network is distributed in the hope that it will be useful,10// but WITHOUT ANY WARRANTY; without even the implied warranty of11// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the12// GNU General Public License for more details.1314// You should have received a copy of the GNU General Public License15// along with Unique Network. If not, see <http://www.gnu.org/licenses/>.1617#![cfg_attr(not(feature = "std"), no_std)]1819pub use pallet::*;20use frame_support::pallet_prelude::*;21use frame_system::pallet_prelude::*;2223#[frame_support::pallet]24pub mod pallet {25 use frame_support::pallet_prelude::*;26 use frame_system::pallet_prelude::*;27 use pallet_unique_scheduler::{ScheduledId, Pallet as SchedulerPallet};2829 #[pallet::config]30 pub trait Config: frame_system::Config + pallet_unique_scheduler::Config {31 type Event: From<Event<Self>> + IsType<<Self as frame_system::Config>::Event>;32 }3334 #[pallet::event]35 #[pallet::generate_deposit(pub(super) fn deposit_event)]36 pub enum Event<T: Config> {37 ValueIsSet,38 ShouldRollback,39 }4041 #[pallet::pallet]42 #[pallet::generate_store(pub(super) trait Store)]43 pub struct Pallet<T>(_);4445 #[pallet::storage]46 #[pallet::getter(fn is_enabled)]47 pub type Enabled<T> = StorageValue<_, bool, ValueQuery>;4849 #[pallet::storage]50 #[pallet::getter(fn test_value)]51 pub type TestValue<T> = StorageValue<_, u32, ValueQuery>;5253 #[pallet::error]54 pub enum Error<T> {55 TestPalletDisabled,56 TriggerRollback,57 }5859 #[pallet::call]60 impl<T: Config> Pallet<T> {61 #[pallet::weight(10_000)]62 pub fn enable(origin: OriginFor<T>) -> DispatchResult {63 ensure_root(origin)?;64 <Enabled<T>>::set(true);6566 Ok(())67 }6869 #[pallet::weight(10_000)]70 pub fn set_test_value(origin: OriginFor<T>, value: u32) -> DispatchResult {71 Self::ensure_origin_and_enabled(origin)?;7273 <TestValue<T>>::put(value);7475 Self::deposit_event(Event::ValueIsSet);7677 Ok(())78 }7980 #[pallet::weight(10_000)]81 pub fn set_test_value_and_rollback(origin: OriginFor<T>, value: u32) -> DispatchResult {82 Self::set_test_value(origin, value)?;8384 Self::deposit_event(Event::ShouldRollback);8586 Err(<Error<T>>::TriggerRollback.into())87 }8889 #[pallet::weight(10_000)]90 pub fn inc_test_value(origin: OriginFor<T>) -> DispatchResult {91 Self::set_test_value(origin, <TestValue<T>>::get() + 1)92 }9394 #[pallet::weight(10_000)]95 pub fn self_canceling_inc(96 origin: OriginFor<T>,97 id: ScheduledId,98 max_test_value: u32,99 ) -> DispatchResult {100 Self::ensure_origin_and_enabled(origin.clone())?;101102 if <TestValue<T>>::get() < max_test_value {103 Self::inc_test_value(origin)?;104 } else {105 SchedulerPallet::<T>::cancel_named(origin, id)?;106 }107108 Ok(())109 }110111 #[pallet::weight(100_000_000)]112 pub fn just_take_fee(origin: OriginFor<T>) -> DispatchResult {113 Self::ensure_origin_and_enabled(origin)?;114 Ok(())115 }116 }117}118119impl<T: Config> Pallet<T> {120 fn ensure_origin_and_enabled(origin: OriginFor<T>) -> DispatchResult {121 ensure_signed(origin)?;122 <Enabled<T>>::get()123 .then(|| ())124 .ok_or(<Error<T>>::TestPalletDisabled.into())125 }126}